Eli Mesika has submitted this change and it was merged.
Change subject: engine: Selecting permissions based on app mode
......................................................................
engine: Selecting permissions based on app mode
Updated procedures getAllForEntity and getTreeForEntity to
list permissions based on application mode as well.
Earlier list of permissions applicable for all the application modes
were getting listed
Change-Id: I7d2d7f1c3d37ecfb66d0965bb0701f7ced6af84d
https://bugzilla.redhat.com/show_bug.cgi?id=1031899
Signed-off-by: ndarshan <dnarayan(a)redhat.com>
---
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/PermissionDAO.java
M
backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/PermissionDAODbFacadeImpl.java
M
backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/PermissionDAOTest.java
M packaging/dbscripts/multi_level_administration_sp.sql
4 files changed, 41 insertions(+), 9 deletions(-)
Approvals:
Eli Mesika: Verified; Looks good to me, approved
--
To view, visit
http://gerrit.ovirt.org/21532
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I7d2d7f1c3d37ecfb66d0965bb0701f7ced6af84d
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Darshan N <dnarayan(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Darshan N <dnarayan(a)redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com>
Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server