Arik Hadas has uploaded a new change for review.

Change subject: webadmin: edit 'instance type' on double click
......................................................................

webadmin: edit 'instance type' on double click

The method getEditCommand in a model must return the command that opens
the edit dialog in order to open the edit dialog on double click on one
of its items.

The model for instance-types didn't override this method, thus it
returned null and the edit dialog wasn't opened on double click on
instance type. The solution is to override the getEditCommand method and
delegate to the getEditInstanceTypeCommand method.

Change-Id: Ic15aff5ee200b9e5a9f04a397a6571f362b90c38
Bug-Url: https://bugzilla.redhat.com/1174655
Signed-off-by: Arik Hadas <aha...@redhat.com>
---
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/configure/instancetypes/InstanceTypeListModel.java
1 file changed, 5 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/98/36398/1

diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/configure/instancetypes/InstanceTypeListModel.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/configure/instancetypes/InstanceTypeListModel.java
index 38d9ae0..7bdf852 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/configure/instancetypes/InstanceTypeListModel.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/configure/instancetypes/InstanceTypeListModel.java
@@ -416,6 +416,11 @@
         }
     }
 
+    @Override
+    public UICommand getEditCommand() {
+        return getEditInstanceTypeCommand();
+    }
+
     public UICommand getNewInstanceTypeCommand() {
         return newInstanceTypeCommand;
     }


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

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

Reply via email to