Change in ovirt-engine[master]: webadmin: fix LUN ID checkbox in new iscsi domain window

derez at redhat.com derez at redhat.com
Wed Aug 19 15:02:10 UTC 2015


Daniel Erez has submitted this change and it was merged.

Change subject: webadmin: fix LUN ID checkbox in new iscsi domain window
......................................................................


webadmin: fix LUN ID 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 LUN ID 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/45011
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I9a43be0d9e33149666408cfcd92f625fe5ff9838
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
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