ofri masad has posted comments on this change.
Change subject: webadmin: Add cluster policy to edit cluster
......................................................................
Patch Set 2: (4 inline comments)
....................................................
File
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterGeneralModel.java
Line 140: {
Line 141: super.setEntity(value);
Line 142: }
Line 143:
Line 144: public ClusterGeneralModel(ClusterListModel clusterListModel)
Done.
Had to workaround the fact that the detailed models are created before the
commands.
Line 145: {
Line 146: this.clusterListModel = clusterListModel;
Line 147:
setTitle(ConstantsManager.getInstance().getConstants().generalTitle());
Line 148: setHashName("general"); //$NON-NLS-1$
....................................................
File
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/gin/uicommon/ClusterModule.java
Line 93:
Line 94: @Provides
Line 95: @Singleton
Line 96: public DetailModelProvider<ClusterListModel, ClusterGeneralModel>
getClusterGeneralProvider(ClientGinjector ginjector,
Line 97: final Provider<ClusterPopupPresenterWidget>
policyPopupProvider,
Done
Line 98: final Provider<MultipleHostsPopupPresenterWidget>
multipleHostsProvider,
Line 99: final Provider<DetachGlusterHostsPopupPresenterWidget>
detachHostsProvider) {
Line 100: return new DetailTabModelProvider<ClusterListModel,
ClusterGeneralModel>(ginjector,
Line 101: ClusterListModel.class,
Line 103: @Override
Line 104: public AbstractModelBoundPopupPresenterWidget<? extends
Model, ?> getModelPopup(ClusterGeneralModel source,
Line 105: UICommand lastExecutedCommand, Model windowModel)
{
Line 106: if (lastExecutedCommand ==
getModel().getEditPolicyCommand()) {
Line 107: return policyPopupProvider.get();
Done
Line 108: } else if (lastExecutedCommand ==
getModel().getImportNewGlusterHostsCommand()) {
Line 109: return multipleHostsProvider.get();
Line 110: } else if (lastExecutedCommand ==
getModel().getDetachNewGlusterHostsCommand()) {
Line 111: return detachHostsProvider.get();
....................................................
File
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/ClusterPopupView.java
Line 425: if (ApplicationModeHelper.getUiMode() ==
ApplicationMode.GlusterOnly)
Line 426: {
Line 427: optimizationTab.setVisible(false);
Line 428: resiliencePolicyTab.setVisible(false);
Line 429: clusterPolicyTab.setVisible(true);
Done
Line 430:
dataCenterPanel.addStyleName(style.generalTabTopDecoratorEmpty());
Line 431: }
Line 432: }
Line 433:
--
To view, visit http://gerrit.ovirt.org/10573
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I195d9ec1c7901a31e6e656221964d82bc90c9695
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: ofri masad <[email protected]>
Gerrit-Reviewer: Daniel Erez <[email protected]>
Gerrit-Reviewer: Doron Fediuck <[email protected]>
Gerrit-Reviewer: Einav Cohen <[email protected]>
Gerrit-Reviewer: Gilad Chaplik <[email protected]>
Gerrit-Reviewer: Kanagaraj M <[email protected]>
Gerrit-Reviewer: ofri masad <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
