darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-05-05 00:20:43 -04:00
Code Issues Packages Projects Releases Wiki Activity

Fix the permission check for users without permissions.

Browse Source 
A users without permissions should be redirected to choose-templates page when signing in.
This commit is contained in:
Rebecca Law
2018-11-19 15:26:43 +00:00
parent 754ae743ca
commit dd22fa06a4
6 changed files with 34 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
app/models/user.py
View File

@@ -119,7 +119,6 @@ class User(UserMixin):
def has_permissions(self, *permissions, restrict_admin_usage=False):
unknown_permissions = set(permissions) - all_permissions
if unknown_permissions:
raise TypeError('{} are not valid permissions'.format(list(unknown_permissions)))
@@ -139,7 +138,7 @@ class User(UserMixin):
if org_id:
return org_id in self.organisations
if not permissions:
return service_id in self._permissions
return service_id in self.services
if service_id:
return any(x in self._permissions.get(service_id, []) for x in permissions)