Change in ovirt-engine[master]: webadmin: fixed remove brick actions disabled issue in volum...

kmayilsa at redhat.com kmayilsa at redhat.com
Wed Nov 13 07:16:32 UTC 2013


Kanagaraj M has submitted this change and it was merged.

Change subject: webadmin: fixed remove brick actions disabled issue in volumes tab
......................................................................


webadmin: fixed remove brick actions disabled issue in volumes tab

 When user move between sub tabs, entity of the previous tabs (all in
active tabs) being null. Because of this, menu items in volumes
tab are not getting enabled. So moved the volume entity as part of
VolumeBricksListModel and removed the usage of getEntity() in
all remove brick related commands.

Change-Id: I005592a1cda7d65cc069ad88f2c46d9f68b1f368
Bug-Url: https://bugzilla.redhat.com/1022585
Signed-off-by: Ramesh Nachimuthu <rnachimu at redhat.com>
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeBrickListModel.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/volumes/VolumeListModel.java
2 files changed, 19 insertions(+), 9 deletions(-)

Approvals:
  Kanagaraj M: Looks good to me, approved
  Ramesh N: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I005592a1cda7d65cc069ad88f2c46d9f68b1f368
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ramesh N <rnachimu at redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik at redhat.com>
Gerrit-Reviewer: Kanagaraj M <kmayilsa at redhat.com>
Gerrit-Reviewer: Ramesh N <rnachimu at redhat.com>
Gerrit-Reviewer: anmolbabu <anbabu at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list