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