Bug 36529: manage_additional_fields permission for more than acquisitions and serials
This patch renames the manage_additional_fields permission to remove the specificity of acquisitions orders and serial subscriptions. Since each additional field requires a corresponding permission, I wrote each of them next to it. It makes it very long, but right now there isn't really a way to link permissions other than the permission description. To test: 1. Apply patch 2. Go to a patron account 3. Click More > Set permissions 4. Go to the Administration panel permissions and open the sub-permissions 5. Check that the manage_additional_fields permission description makes sense, check grammar and spelling, try the various combinations Signed-off-by: Pedro Amorim <pedro.amorim@ptfs-europe.com> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
This commit is contained in:
parent
ce7844c430
commit
e251212ff5
1 changed files with 1 additions and 1 deletions
|
@ -269,7 +269,7 @@
|
||||||
<span class="permissioncode">([% name | html %])</span>
|
<span class="permissioncode">([% name | html %])</span>
|
||||||
[%- CASE 'manage_additional_fields' -%]
|
[%- CASE 'manage_additional_fields' -%]
|
||||||
<span class="sub_permission manage_aditional_fields_subpermission">
|
<span class="sub_permission manage_aditional_fields_subpermission">
|
||||||
Manage additional fields for baskets or subscriptions (requires edit_subscription or order_manage permissions)
|
Manage additional fields (requires the corresponding permission, one of edit_subscription, order_manage, edit_invoices, or remaining_permissions under updatecharges)
|
||||||
</span>
|
</span>
|
||||||
<span class="permissioncode">([% name | html %])</span>
|
<span class="permissioncode">([% name | html %])</span>
|
||||||
[%- CASE 'manage_sms_providers' -%]
|
[%- CASE 'manage_sms_providers' -%]
|
||||||
|
|
Loading…
Reference in a new issue