diff options
| author | Lester Caine <lester@lsces.co.uk> | 2026-05-30 17:52:48 +0100 |
|---|---|---|
| committer | Lester Caine <lester@lsces.co.uk> | 2026-05-30 17:52:48 +0100 |
| commit | 72f9d187d8412a2502651646679d35c46ef70181 (patch) | |
| tree | fe8ebd7b9d85770f156f351c453b206e76591b3e | |
| parent | 7b4034e98282614c8e45e9c9d660b02e2cea9281 (diff) | |
| download | users-72f9d187d8412a2502651646679d35c46ef70181.tar.gz users-72f9d187d8412a2502651646679d35c46ef70181.tar.bz2 users-72f9d187d8412a2502651646679d35c46ef70181.zip | |
Fix role_permissions nav bar dots and package filter on save
Package filter links wrapped in <ul class="list-inline"> so <li> elements
are valid HTML — previously <li> inside <p> caused bullets to detach.
Hidden package field default changed from 'All' to '' so form submission
does not filter permissions to an empty set and silently save nothing.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
| -rwxr-xr-x | templates/admin_role_permissions.tpl | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/templates/admin_role_permissions.tpl b/templates/admin_role_permissions.tpl index f814237..6075438 100755 --- a/templates/admin_role_permissions.tpl +++ b/templates/admin_role_permissions.tpl @@ -8,16 +8,16 @@ {formfeedback hash=$feedback} {form} - <input type="hidden" name="package" value="{$smarty.request.package|default:'All'}" /> + <input type="hidden" name="package" value="{$smarty.request.package}" /> - <p> - {smartlink ititle=All package=$packageKey} + <ul class="list-inline"> + <li>{smartlink ititle=All}</li> {foreach from=$permPackages key=i item=packageKey} {if $gBitSystem->isPackageActive($packageKey)} - • {smartlink ititle=$gBitSystem->mPackages.$packageKey.name|default:$packageKey package=$packageKey} + <li>• {smartlink ititle=$gBitSystem->mPackages.$packageKey.name|default:$packageKey package=$packageKey}</li> {/if} {/foreach} - </p> + </ul> <table class="table data"> <caption>{tr}Available Permissions{/tr}</caption> |