Alexander Wels has uploaded a new change for review.

Change subject: webadmin: refresh system tree when removing last leaf
......................................................................

webadmin: refresh system tree when removing last leaf

- When removing the last element of a list the system
  tree would not refresh. For instance removing the last
  cluster from a data center. This patch fixes that issue.

Change-Id: If9e47bb6cbc5beb4da04b98c8f4b1c38c2932445
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1127600
Signed-off-by: Alexander Wels <aw...@redhat.com>
---
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/SystemTreeItemModel.java
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/65/32165/1

diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/SystemTreeItemModel.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/SystemTreeItemModel.java
index 7b2c374..7ad7015 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/SystemTreeItemModel.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/SystemTreeItemModel.java
@@ -126,7 +126,7 @@
         }
 
         // Compare item's children, if necessary
-        if (result && getChildren().size() > 0 && deepCompare) {
+        if (result && (!getChildren().isEmpty() || 
!otherModel.getChildren().isEmpty()) && deepCompare) {
             if (getChildren().size() != otherModel.getChildren().size()) {
                 result = false;
             } else {


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: If9e47bb6cbc5beb4da04b98c8f4b1c38c2932445
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5
Gerrit-Owner: Alexander Wels <aw...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to