Ravi Nori has uploaded a new change for review.

Change subject: webadmin: Failed add a host via “Guide Me” on Local Data Center
......................................................................

webadmin: Failed add a host via “Guide Me” on Local Data Center

Failed add a host via “Guide Me” on Local Data Center

Change-Id: Ie9bcb4c4824b58da345d30c70d0fe0c6ecc99a1a
Bug-Url: https://bugzilla.redhat.com/998386
Signed-off-by: Ravi Nori <rn...@redhat.com>
---
M 
frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/editor/EntityModelCellTable.java
M 
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/guide/MoveHostPopupView.java
2 files changed, 16 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/50/18450/1

diff --git 
a/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/editor/EntityModelCellTable.java
 
b/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/editor/EntityModelCellTable.java
index 05703ee..0e5948f 100644
--- 
a/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/editor/EntityModelCellTable.java
+++ 
b/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/editor/EntityModelCellTable.java
@@ -186,6 +186,12 @@
             break;
         }
 
+        setSelectionChangeHandler();
+        buildAddCheckBox(hideCheckbox, showSelectAllCheckbox);
+
+    }
+
+    public void setSelectionChangeHandler() {
         // Handle selection
         getSelectionModel().addSelectionChangeHandler(new Handler() {
             @Override
@@ -219,6 +225,13 @@
                 }
             }
         });
+    }
+
+    public void removeCheckBoxColumn() {
+        this.removeColumn(0);
+    }
+
+    private void buildAddCheckBox(boolean hideCheckbox, boolean 
showSelectAllCheckbox) {
 
         if (!hideCheckbox) {
             // Add selection column
diff --git 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/guide/MoveHostPopupView.java
 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/guide/MoveHostPopupView.java
index 20110b2..7c812d7 100644
--- 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/guide/MoveHostPopupView.java
+++ 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/guide/MoveHostPopupView.java
@@ -105,8 +105,10 @@
 
     @Override
     public void edit(MoveHost object) {
-        if (!object.isMultiSelection())
+        if (!object.isMultiSelection()) {
             table.setSelectionModel(new SingleSelectionModel<EntityModel>());
+            table.setSelectionChangeHandler();
+        }
 
         driver.edit(object);
         table.asEditor().edit(object);


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

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

Reply via email to