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@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@redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Darshan N <dnarayan@redhat.com> Gerrit-Reviewer: Eli Mesika <emesika@redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel@redhat.com> Gerrit-Reviewer: Sahina Bose <sabose@redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat@redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server