Daniel Erez has uploaded a new change for review.

Change subject: webadmin: custom preview dialog - close on success
......................................................................

webadmin: custom preview dialog - close on success

Custom preview dialog:
close dialog only on return value success;
i.e. keep dialog open after canDoAction popup
(to allow selection amend).

Change-Id: I7c35c03bd1fdd7b8b2a46c7ff7596d344ab729e0
Related-To: https://bugzilla.redhat.com/1096884
Signed-off-by: Daniel Erez <de...@redhat.com>
---
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmSnapshotListModel.java
1 file changed, 3 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/03/28303/1

diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmSnapshotListModel.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmSnapshotListModel.java
index cf2ce7f..a9436d6 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmSnapshotListModel.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmSnapshotListModel.java
@@ -487,7 +487,9 @@
                 @Override
                 public void executed(FrontendActionAsyncResult result) {
                     model.stopProgress();
-                    cancel();
+                    if (result.getReturnValue().getSucceeded()) {
+                        cancel();
+                    }
                 }
             });
     }


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I7c35c03bd1fdd7b8b2a46c7ff7596d344ab729e0
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.4
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