Change in ovirt-engine[master]: webadmin: reverted removal of selectedItemsChanged method.

lvernia at redhat.com lvernia at redhat.com
Wed Jan 21 13:25:38 UTC 2015


Lior Vernia has submitted this change and it was merged.

Change subject: webadmin: reverted removal of selectedItemsChanged method.
......................................................................


webadmin: reverted removal of selectedItemsChanged method.

This method was mistakingly removed as a part of one big refactor.
Apparently it was removed because it seems that this method does the
same thing as overriden one, but that's not true, both calls their
private equally named method. This could be fixed by making that
method protected, but after some research this is not easily doable,
so I just returned removed method back and place warning comment to
method being overridden.

Change-Id: I473566cf0755519eb7fc922c435ae8a2ee545779
Bug-Url: https://bugzilla.redhat.com/1180897
Signed-off-by: Martin Mucha <mmucha at redhat.com>
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/SearchableListModel.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/macpool/SharedMacPoolListModel.java
2 files changed, 11 insertions(+), 0 deletions(-)

Approvals:
  Lior Vernia: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I473566cf0755519eb7fc922c435ae8a2ee545779
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Martin Mucha <mmucha at redhat.com>
Gerrit-Reviewer: Alexander Wels <awels at redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan at redhat.com>
Gerrit-Reviewer: Lior Vernia <lvernia at redhat.com>
Gerrit-Reviewer: Martin Mucha <mmucha at redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list