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

rgolan at redhat.com rgolan at redhat.com
Tue Aug 23 15:43:04 UTC 2016


Roy Golan 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
and multiple CPU profiles 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>
---
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:
  Alexander Wels: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Greg Sheremeta: Looks good to me, approved
  Yanir Quinn: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib15be1a6474b9504053da0940825c23224d6fd34
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yanir Quinn <yquinn at redhat.com>
Gerrit-Reviewer: Alexander Wels <awels at redhat.com>
Gerrit-Reviewer: Andrej Krejcir <akrejcir at redhat.com>
Gerrit-Reviewer: Greg Sheremeta <gshereme at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Jenny Tokar <jtokar at redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Phillip Bailey <phbailey at redhat.com>
Gerrit-Reviewer: Roman Mohr <rmohr at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan 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