Arik Hadas has uploaded a new change for review.

Change subject: webadmin: hide memory checkbox from create snapshot popup on 
error
......................................................................

webadmin: hide memory checkbox from create snapshot popup on error

The checkbox that allows the user to select whether to save the memory
as part of the created snapshot or not, was visible also when the create
snapshot popup presented an error message. This patch hides the memory
checkbox from the create snapshot dialog in case it presents an error.

Change-Id: I4b57dc687e19fecbadbec331627f08128b1d6ca0
Bug-Url: https://bugzilla.redhat.com/993645
Signed-off-by: Arik Hadas <[email protected]>
---
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/SnapshotModel.java
1 file changed, 1 insertion(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/62/18662/1

diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/SnapshotModel.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/SnapshotModel.java
index d5c6cd7..ef78c0f 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/SnapshotModel.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/SnapshotModel.java
@@ -264,6 +264,7 @@
         for (Snapshot snapshot : snapshots) {
             if (!validateNewSnapshotByStatus(snapshot.getStatus()) || 
!validateNewSnapshotByType(snapshot.getType())) {
                 getDescription().setIsAvailable(false);
+                getMemory().setIsAvailable(false);
                 return true;
             }
         }


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

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

Reply via email to