Change in ovirt-engine[ovirt-engine-3.6]: webadmin: fix select all LUNs checkbox in new iscsi domain w...

tnisan at redhat.com tnisan at redhat.com
Thu Aug 20 12:32:16 UTC 2015


Tal Nisan has submitted this change and it was merged.

Change subject: webadmin: fix select all LUNs checkbox in new iscsi domain window
......................................................................


webadmin: fix select all LUNs checkbox in new iscsi domain window

This patch fixes a regression in the Luns to Targets view in the New
iscsi Domain window.
The regression was caused by the addition of an else statement in
lunModelSelectedItemListener which raised an event that canceled the
user's selection when clicking on the select all LUNs checkbox.

Change-Id: I9a43be0d9e33149666408cfcd92f625fe5ff9838
Bug-Url: https://bugzilla.redhat.com/1215416
Signed-off-by: Idan Shaby <ishaby at redhat.com>
---
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/storage/SanStorageLunToTargetList.java
1 file changed, 10 insertions(+), 9 deletions(-)

Approvals:
  Tomas Jelinek: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Idan Shaby: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I9a43be0d9e33149666408cfcd92f625fe5ff9838
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6
Gerrit-Owner: Idan Shaby <ishaby at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Idan Shaby <ishaby at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list