Gilad Chaplik has uploaded a new change for review. Change subject: webadmin: fix permission button for cpu profiles ......................................................................
webadmin: fix permission button for cpu profiles Change-Id: Ib5e104c059b122cf21bfbbed00fab66162cdc5f6 Bug-Url: https://bugzilla.redhat.com/1143869 Signed-off-by: Gilad Chaplik <[email protected]> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/presenter/tab/cluster/SubTabClusterCpuProfilePresenter.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/cluster/CpuProfilesListModelTable.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/cluster/SubTabClusterCpuProfileView.java 3 files changed, 30 insertions(+), 0 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/77/37077/1 diff --git a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/presenter/tab/cluster/SubTabClusterCpuProfilePresenter.java b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/presenter/tab/cluster/SubTabClusterCpuProfilePresenter.java index 9797d6c..7678f7e 100644 --- a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/presenter/tab/cluster/SubTabClusterCpuProfilePresenter.java +++ b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/presenter/tab/cluster/SubTabClusterCpuProfilePresenter.java @@ -32,6 +32,8 @@ } public interface ViewDef extends AbstractSubTabPresenter.ViewDef<VDSGroup> { + + void addModelListeners(); } @TabInfo(container = ClusterSubTabPanelPresenter.class) @@ -58,4 +60,10 @@ updateMainTabSelection(event.getSelectedItems()); } + @Override + public void initializeHandlers() { + super.initializeHandlers(); + + getView().addModelListeners(); + } } diff --git a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/cluster/CpuProfilesListModelTable.java b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/cluster/CpuProfilesListModelTable.java index d26e2a5..b0561e0 100644 --- a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/cluster/CpuProfilesListModelTable.java +++ b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/cluster/CpuProfilesListModelTable.java @@ -1,11 +1,13 @@ package org.ovirt.engine.ui.webadmin.section.main.view.tab.cluster; +import org.ovirt.engine.core.common.businessentities.Permissions; import org.ovirt.engine.core.common.businessentities.profiles.CpuProfile; import org.ovirt.engine.core.common.businessentities.qos.CpuQos; import org.ovirt.engine.ui.common.CommonApplicationConstants; import org.ovirt.engine.ui.common.idhandler.ElementIdHandler; import org.ovirt.engine.ui.common.system.ClientStorage; import org.ovirt.engine.ui.common.uicommon.model.SearchableTableModelProvider; +import org.ovirt.engine.ui.common.widget.table.SimpleActionTable; import org.ovirt.engine.ui.common.widget.table.column.TextColumnWithTooltip; import org.ovirt.engine.ui.common.widget.uicommon.AbstractModelBoundTableWidget; import org.ovirt.engine.ui.common.widget.uicommon.permissions.PermissionWithInheritedPermissionListModelTable; @@ -27,6 +29,7 @@ import com.google.gwt.user.client.ui.SimplePanel; import com.google.gwt.user.client.ui.SplitLayoutPanel; import com.google.gwt.user.client.ui.Widget; +import com.google.gwt.view.client.SelectionChangeEvent; public class CpuProfilesListModelTable extends AbstractModelBoundTableWidget<CpuProfile, CpuProfileListModel> { @@ -50,12 +53,15 @@ private boolean permissionPanelVisible = false; + private final CpuProfilePermissionModelProvider cpuProfilePermissionModelProvider; + public CpuProfilesListModelTable(SearchableTableModelProvider<CpuProfile, CpuProfileListModel> modelProvider, CpuProfilePermissionModelProvider cpuProfilePermissionModelProvider, EventBus eventBus, ClientStorage clientStorage, CommonApplicationConstants constants) { super(modelProvider, eventBus, clientStorage, false); + this.cpuProfilePermissionModelProvider = cpuProfilePermissionModelProvider; ViewIdHandler.idHandler.generateAndSetIds(this); // Create cpu profile table tableContainer.add(getTable()); @@ -169,4 +175,15 @@ } }); } + + @Override + public void addModelListeners() { + final SimpleActionTable<Permissions> table = permissionListModelTable.getTable(); + table.getSelectionModel().addSelectionChangeHandler(new SelectionChangeEvent.Handler() { + @Override + public void onSelectionChange(SelectionChangeEvent event) { + cpuProfilePermissionModelProvider.setSelectedItems(table.getSelectedItems()); + } + }); + } } diff --git a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/cluster/SubTabClusterCpuProfileView.java b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/cluster/SubTabClusterCpuProfileView.java index 4b938c9..158d89d 100644 --- a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/cluster/SubTabClusterCpuProfileView.java +++ b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/cluster/SubTabClusterCpuProfileView.java @@ -42,4 +42,9 @@ initTable(constants); initWidget(getModelBoundTableWidget()); } + + @Override + public void addModelListeners() { + getModelBoundTableWidget().addModelListeners(); + } } -- To view, visit http://gerrit.ovirt.org/37077 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ib5e104c059b122cf21bfbbed00fab66162cdc5f6 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Gilad Chaplik <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
