Change in ovirt-engine[master]: webadmin: snapshots sub-tab - ensure model listeners

derez at redhat.com derez at redhat.com
Wed Mar 12 15:59:27 UTC 2014


Daniel Erez has submitted this change and it was merged.

Change subject: webadmin: snapshots sub-tab - ensure model listeners
......................................................................


webadmin: snapshots sub-tab - ensure model listeners

VMs -> Snapshots sub-sub:
Ensure model listeners existence after sign out.
Re-adding the listeners in presenter's onUiCommonInit method
as VmSnapshotListModel is being initialized on sign in
(along with the other list models:  VmListModel -> initDetailModels).

Change-Id: I006023de74150754d8c79f95b79eb9ad0158b055
Bug-Url: https://bugzilla.redhat.com/1058618
Signed-off-by: Daniel Erez <derez at redhat.com>
---
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/AbstractModelBoundTableWidget.java
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/vm/VmSnapshotListModelTable.java
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/presenter/tab/virtualMachine/SubTabVirtualMachineSnapshotPresenter.java
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/virtualMachine/SubTabVirtualMachineSnapshotView.java
4 files changed, 41 insertions(+), 12 deletions(-)

Approvals:
  Daniel Erez: Verified; Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/25536
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I006023de74150754d8c79f95b79eb9ad0158b055
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list