Kanagaraj M has uploaded a new change for review.

Change subject: webadmin: fix volume bricks tab refresh issue
......................................................................

webadmin: fix volume bricks tab refresh issue

Bricks sub-tab was not showing the data  immediately
when the user is navigating in the volume sub-tabs.
This is now fixed.

Change-Id: If91de652fe7e88df063a871948c0abbda7647698
Bug-Url: https://bugzilla.redhat.com/1012871
Signed-off-by: Kanagaraj M <kmayi...@redhat.com>
---
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeBrickListModel.java
1 file changed, 6 insertions(+), 10 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/70/21070/1

diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeBrickListModel.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeBrickListModel.java
index 376de86..05b4040 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeBrickListModel.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeBrickListModel.java
@@ -50,7 +50,6 @@
     public VolumeBrickListModel() {
         setTitle(ConstantsManager.getInstance().getConstants().bricksTitle());
         setHashName("bricks"); //$NON-NLS-1$
-        setIsTimerDisabled(false);
         setAddBricksCommand(new UICommand("Add Bricks", this)); //$NON-NLS-1$
         setRemoveBricksCommand(new UICommand("Remove Bricks", this)); 
//$NON-NLS-1$
         setStopRemoveBricksCommand(new UICommand("StopRemoveBricks", this)); 
//$NON-NLS-1$
@@ -246,16 +245,13 @@
     @Override
     protected void onEntityChanged() {
         super.onEntityChanged();
-        getSearchCommand().execute();
-    }
-
-    @Override
-    protected void syncSearch() {
-        if (getEntity() == null) {
-            return;
+        if (getEntity() != null) {
+            GlusterVolumeEntity glusterVolumeEntity = (GlusterVolumeEntity) 
getEntity();
+            setItems(glusterVolumeEntity.getBricks());
         }
-        GlusterVolumeEntity glusterVolumeEntity = (GlusterVolumeEntity) 
getEntity();
-        setItems(glusterVolumeEntity.getBricks());
+        else {
+            setItems(null);
+        }
     }
 
     private void checkUpServerAndAddBricks() {


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

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

Reply via email to