Daniel Erez has uploaded a new change for review.

Change subject: webadmin: DirectLUN FC - fix radio-buttons behavior
......................................................................

webadmin: DirectLUN FC - fix radio-buttons behavior

Fixed radio-button single-selection behavior for FC DirectLUN
(added single-selection listeners to FC LunModels for
clearing unselected radio-buttons).

Change-Id: I739adc6da16ca075acd4a3ae6d48e4ad2d8cac79
Bug-Url: https://bugzilla.redhat.com/918568
Signed-off-by: Daniel Erez <[email protected]>
---
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/SanStorageModel.java
1 file changed, 13 insertions(+), 6 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/99/12899/1

diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/SanStorageModel.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/SanStorageModel.java
index 2761c6f..abcac48 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/SanStorageModel.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/SanStorageModel.java
@@ -392,6 +392,19 @@
             }
 
             setItems(items);
+
+            if (!isMultiSelection() && newLuns != null) {
+                addLunModelSelectionEventListeners(newLuns);
+            }
+        }
+    }
+
+    private void addLunModelSelectionEventListeners(List<LunModel> luns) {
+        for (LunModel lun : luns) {
+            // Adding PropertyEventListener to LunModel if needed
+            if 
(!lun.getPropertyChangedEvent().getListeners().contains(lunModelEventListener)) 
{
+                
lun.getPropertyChangedEvent().addListener(lunModelEventListener);
+            }
         }
     }
 
@@ -425,12 +438,6 @@
                     currLun.setIsSelected(lun.getIsSelected());
                     currLun.setEntity(lun.getEntity());
                 }
-            }
-
-            // Adding PropertyEventListener to LunModel
-            if (!isMultiSelection()) {
-                
lun.getPropertyChangedEvent().removeListener(lunModelEventListener);
-                
lun.getPropertyChangedEvent().addListener(lunModelEventListener);
             }
         }
     }


--
To view, visit http://gerrit.ovirt.org/12899
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I739adc6da16ca075acd4a3ae6d48e4ad2d8cac79
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Daniel Erez <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to