Daniel Erez has uploaded a new change for review.

Change subject: webadmin: remove storage - gray out hosts list
......................................................................

webadmin: remove storage - gray out hosts list

Host list is irrelevant for ISO/Export domains when 'format'
is unchecked (since no operation is performed by the host).
Hence, graying out the select-box.

Change-Id: I27496ae060d0255d2e8237fde3a80737ac9ff6bd
Bug-Url: https://bugzilla.redhat.com/1003241
Signed-off-by: Daniel Erez <de...@redhat.com>
---
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/RemoveStorageModel.java
1 file changed, 18 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/51/18751/1

diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/RemoveStorageModel.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/RemoveStorageModel.java
index f043834..c149c36 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/RemoveStorageModel.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/RemoveStorageModel.java
@@ -3,6 +3,8 @@
 import org.ovirt.engine.ui.uicommonweb.models.EntityModel;
 import org.ovirt.engine.ui.uicommonweb.models.ListModel;
 import org.ovirt.engine.ui.uicommonweb.models.Model;
+import org.ovirt.engine.ui.uicompat.Event;
+import org.ovirt.engine.ui.uicompat.EventArgs;
 
 @SuppressWarnings("unused")
 public class RemoveStorageModel extends Model
@@ -37,9 +39,25 @@
         setHostList(new ListModel());
 
         setFormat(new EntityModel());
+        getFormat().getEntityChangedEvent().addListener(this);
+        getFormat().getPropertyChangedEvent().addListener(this);
         getFormat().setEntity(false);
     }
 
+    @Override
+    public void eventRaised(Event ev, Object sender, EventArgs args)
+    {
+        super.eventRaised(ev, sender, args);
+
+        if (sender == getFormat()) {
+            format_Changed(sender, args);
+        }
+    }
+
+    private void format_Changed(Object sender, EventArgs args) {
+        getHostList().setIsChangable(!getFormat().getIsAvailable() || 
Boolean.TRUE.equals(getFormat().getEntity()));
+    }
+
     public boolean validate()
     {
         getHostList().setIsValid(getHostList().getSelectedItem() != null);


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I27496ae060d0255d2e8237fde3a80737ac9ff6bd
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Daniel Erez <de...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to