Task #756
closed
Add permission check for button "Change permission"
100%
Description
Please add check permision to role_request to button "Change permission". After click this button show some message with information about permission to this action.
Thank you.
Updated by Radek Tomiška over 6 years ago
- Status changed from New to Needs feedback
- Assignee changed from Radek Tomiška to Ondřej Kopr
- % Done changed from 0 to 90
I've has to add new permission 'CHANGEPERMISSION' to identity - it works the same way as 'PASSWORDCHANGE' and is evaluated on identity roles page. Now is possible to show button for change identity permissions, only when logged identity has this permission and split 'UPDATE' and 'CHANGEPERMISSION' permission on identity - it's possible to create role request (changing identity's permission) for read only identities. RoleRequestByIdentityEvaluator behavior was updated - see doc. Tests are included.
Documentation:
https://wiki.czechidm.com/devel/dev/security/authorization#identity
https://wiki.czechidm.com/devel/dev/security/authorization#selfrolerequestevaluator
https://wiki.czechidm.com/devel/dev/security/authorization#rolerequestbyidentityevaluator
https://wiki.czechidm.com/devel/dev/security/authorization#default_settings_of_permissions_for_an_identity_profile
https://github.com/bcvsolutions/CzechIdMng/blob/develop/CHANGELOG.md#identity
Commit:
https://github.com/bcvsolutions/CzechIdMng/commit/96e60e7aba7f07cd456ffe97c9221eec8e661468
Could you do a review pls?
Updated by Ondřej Kopr over 6 years ago
- Status changed from Needs feedback to Resolved
- Assignee changed from Ondřej Kopr to Radek Tomiška
- % Done changed from 90 to 100
I did review change permission works perfectly i only create new task for some minor issue (#773). Thanks for that.
Updated by Radek Tomiška over 6 years ago
- Status changed from Resolved to Closed