Summary: | manage_additional_fields permission for more than acquisitions and serials | ||
---|---|---|---|
Product: | Koha | Reporter: | Caroline Cyr La Rose <caroline.cyr-la-rose> |
Component: | Patrons | Assignee: | Caroline Cyr La Rose <caroline.cyr-la-rose> |
Status: | RESOLVED FIXED | QA Contact: | Testopia <testopia> |
Severity: | trivial | ||
Priority: | P5 - low | CC: | fridolin.somers, gmcharlt, kyle.m.hall, lucas, pedro.amorim, wainuiwitikapark |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | String patch | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: |
24.05.00,23.11.06,23.05.12
|
Circulation function: | |
Attachments: |
Bug 36529: manage_additional_fields permission for more than acquisitions and serials
Bug 36529: manage_additional_fields permission for more than acquisitions and serials Bug 36529: manage_additional_fields permission for more than acquisitions and serials |
Description
Caroline Cyr La Rose
2024-04-04 21:04:58 UTC
Created attachment 164821 [details] [review] 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 Created attachment 166777 [details] [review] 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> I wonder if we should consider something more generic like: "Manage additional fields for tables that support it (may require additional specific permissions for each table)" The above wording could be improved but my point is that I can see this falling out of sync again in the future as new tables start adopting additional fields and this label is forgotten. Just my 2 cents, definitely not a blocker. Created attachment 166781 [details] [review] 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> Switching to a trivial bug - string change and it makes sense with the changed functionality. Pushed for 24.05! Well done everyone, thank you! Pushed to 23.11.x for 23.11.06 Backported to 23.05.x for upcoming 23.05.12 Not backporting to 22.11 |