Koha-community/Koha
13
7
Fork 0
Code Issues Releases Wiki Activity

Bug 36529: manage_additional_fields permission for more than acquisitions and serials

Browse source 
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>
(cherry picked from commit e251212ff5)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
(cherry picked from commit fd982c027d)
Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
This commit is contained in:
Caroline Cyr La Rose 2024-04-12 05:53:25 -04:00 committed by Lucas Gass
parent d310549dae
commit 22061c248f
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
koha-tmpl/intranet-tmpl/prog/en/includes/permissions.inc
View file

@ -261,7 +261,7 @@
<span class="permissioncode">([% name | html %])</span>
[%- CASE 'manage_additional_fields' -%]
<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 class="permissioncode">([% name | html %])</span>
[%- CASE 'manage_sms_providers' -%]