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