Change in ovirt-engine[ovirt-engine-4.0]: webadmin: Fix multi selection deletion in UI

tnisan at redhat.com tnisan at redhat.com
Wed Aug 31 07:52:35 UTC 2016


Tal Nisan has submitted this change and it was merged.

Change subject: webadmin: Fix multi selection deletion in UI
......................................................................


webadmin: Fix multi selection deletion in UI

Deletion of multiple selected items from a table view in UI
resulted in an uncaught UI exception (out of bounds).
This could have been observed in multiple disk profiles deletion
multiple CPU profiles and network interfaces deletion.

Multiple items deletion from a table view (using SHIFT + multiple
selection) will not result in an exception.

Change-Id: Ib15be1a6474b9504053da0940825c23224d6fd34
Bug-Url: https://bugzilla.redhat.com/1359483
Signed-off-by: yquinn <yquinn at redhat.com>
(cherry picked from commit bc96835)
Signed-off-by: yquinn <yquinn at redhat.com>
---
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/table/OrderedMultiSelectionModel.java
1 file changed, 4 insertions(+), 0 deletions(-)

Approvals:
  Martin Sivák: Looks good to me, but someone else must approve
  Jenkins CI: Passed CI tests
  Roy Golan: Looks good to me, approved
  Yanir Quinn: Verified



-- 
To view, visit https://gerrit.ovirt.org/62724
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib15be1a6474b9504053da0940825c23224d6fd34
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.0
Gerrit-Owner: Yanir Quinn <yquinn at redhat.com>
Gerrit-Reviewer: Alexander Wels <awels at redhat.com>
Gerrit-Reviewer: Greg Sheremeta <gshereme at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Yanir Quinn <yquinn at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list