ndarshan has uploaded a new change for review.

Change subject: gluster: fix for bug:user with permissions to manipulate 
volumes cannot delete a volume(#1015478)
......................................................................

gluster: fix for bug:user with permissions to manipulate volumes cannot delete 
a volume(#1015478)

This patch adds permission to delete a volume to a role whenever permission
to manipulate volume is given to that role.

Change-Id: I576c940c628febf849442b7f52ecb498145ea1f9
Bug-url: https://bugzilla.redhat.com/show_bug.cgi?id=1015478
Signed-off-by: ndarshan <dnara...@redhat.com>
---
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/configure/roles_ui/RoleListModel.java
1 file changed, 5 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/90/20290/1

diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/configure/roles_ui/RoleListModel.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/configure/roles_ui/RoleListModel.java
index 823683f..8b9ae8e 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/configure/roles_ui/RoleListModel.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/configure/roles_ui/RoleListModel.java
@@ -515,6 +515,11 @@
                             }
                             actionDistinctSet.put(actionGroup, actionGroup);
                             actions.add(actionGroup);
+                            if 
(smGrandChild.getTitle().equals("MANIPULATE_GLUSTER_VOLUME")){//$NON-NLS-1$
+                                actionGroup = 
ActionGroup.valueOf("DELETE_GLUSTER_VOLUME");//$NON-NLS-1$
+                                actionDistinctSet.put(actionGroup, 
actionGroup);
+                                actions.add(actionGroup);
+                            }
                         }
                     }
                 }


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

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

Reply via email to