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@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@redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Daniel Erez <derez@redhat.com> Gerrit-Reviewer: Idan Shaby <ishaby@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan@redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek@redhat.com> Gerrit-Reviewer: automation@ovirt.org