rattail/tailbone
1
0
Fork 0
Code Issues Pull requests Actions Packages Projects Releases Wiki Activity

Only show granted permissions when viewing role details

Browse source 
Hopefully a little easier on the eyes..still could use more improvements
though I'm sure.
This commit is contained in:
Lance Edgar 2016-08-11 22:29:55 -05:00
parent 2b1a493706
commit 41ca3859f6
2 changed files with 18 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
tailbone/static/css/perms.css
View file

@ -3,6 +3,10 @@
* Permission Lists * Permission Lists
******************************/ ******************************/
div.field-wrapper.permissions {
overflow: visible;
}
div.field-wrapper.permissions div.field div.group { div.field-wrapper.permissions div.field div.group {
margin-bottom: 10px; margin-bottom: 10px;
} }

6
tailbone/views/roles.py
View file

@ -74,10 +74,12 @@ def PermissionsFieldRenderer(permissions, *args, **kwargs):
for groupkey in sorted(permissions, key=lambda k: permissions[k]['label'].lower()): for groupkey in sorted(permissions, key=lambda k: permissions[k]['label'].lower()):
inner = HTML.tag('p', c=permissions[groupkey]['label']) inner = HTML.tag('p', c=permissions[groupkey]['label'])
perms = permissions[groupkey]['perms'] perms = permissions[groupkey]['perms']
rendered = False
for key in sorted(perms, key=lambda p: perms[p]['label'].lower()): for key in sorted(perms, key=lambda p: perms[p]['label'].lower()):
checked = has_permission(Session(), role, key, checked = has_permission(Session(), role, key,
include_guest=False, include_guest=False,
include_authenticated=False) include_authenticated=False)
if checked or not readonly:
label = perms[key]['label'] label = perms[key]['label']
if readonly: if readonly:
span = HTML.tag('span', c="[X]" if checked else "[ ]") span = HTML.tag('span', c="[X]" if checked else "[ ]")
@ -85,7 +87,11 @@ def PermissionsFieldRenderer(permissions, *args, **kwargs):
else: else:
inner += tags.checkbox(self.name + '-' + key, inner += tags.checkbox(self.name + '-' + key,
checked=checked, label=label) checked=checked, label=label)
rendered = True
if rendered:
html += HTML.tag('div', class_='group', c=inner) html += HTML.tag('div', class_='group', c=inner)
if not html:
return "(none granted)"
return html return html
def render(self, **kwargs): def render(self, **kwargs):