Summary: | Descriptions of acquisition permissions are unclear | ||
---|---|---|---|
Product: | Koha | Reporter: | Nicole C. Engard <nengard> |
Component: | Acquisitions | Assignee: | Katrin Fischer <katrin.fischer> |
Status: | CLOSED FIXED | QA Contact: | Bugs List <koha-bugs> |
Severity: | normal | ||
Priority: | P5 - low | CC: | cunha, jonathan.druart, katrin.fischer, martin.renvoize, nick |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | Small patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Bug Depends on: | |||
Bug Blocks: | 7651, 21966 | ||
Attachments: |
screenshot of acq permissions
Bug 3849: Improve descriptions of granular acquisition permissions Bug 3849: Improve descriptions of granular acquisition permissions Bug 3849: (follow-up) Add database update to change descriptions Bug 3849: Improve descriptions of granular acquisition permissions Bug 3849: (follow-up) Add database update to change descriptions |
Description
Chris Cormack
2010-05-21 01:15:44 UTC
wow... Nicole, is this entry still relevant ? Do you still have a question ? Yes, an answer would be nice. I think you're right, and CAN_user_acquisition_budget_add_del is not used anywhere except in budget.pl, to check that the user has this permission before doing any action (otherwise, prevent adding/modifying) I think/suspect this line can be updated to use budget_manage and we could get rid of add_dell Created attachment 6099 [details] screenshot of acq permissions Ok, to me acq permissions are still very confusing. Please can someone work out which permission is used where and update the permission texts accordingly? budget_add_del Add and delete budgets (but cant modify budgets) budget_manage Manage budgets budget_modify Modify budget (can't create lines, but can modify existing ones) >Still confusing. contracts_manage Manage contracts group_manage Manage orders & basketgroups order_manage Manage orders & basket order_receive Manage orders & basket >Looks all the same? period_manage Manage periods >What periods? planning_manage Manage budget plannings vendors_manage Manage vendors (In reply to Katrin Fischer from comment #4) > order_manage Manage orders & basket > order_receive Manage orders & basket > >Looks all the same? I just got a report of this from a library - they were confused - this is still an issue. Nicole *** Bug 12751 has been marked as a duplicate of this bug. *** What is the issue here? The permissions don't work (in this case, what are the issues?) or the descriptions need to be updated? We don't know what the permissions are supposed to do because the descriptions are unclear so we can't tell you if they work :) So we need clearer permission descriptions and then we'll tell you if they work. Nicole I would say group_manage is for basket groups, order_receive for receiving orders and order_manage for "managing" orders :) By managing I suppose it's any actions != receive. We still need to fix the descriptions: group_manage Manage orders & basketgroups order_manage Manage orders & basket order_manage_all Manage all orders and baskets, regardless of restrictions on them order_receive Manage orders & basket It should not all say 'manage orders...', but be more specific. Are orders the same as baskets? Or does it means orders and order lines here? Created attachment 75450 [details] [review] Bug 3849: Improve descriptions of granular acquisition permissions The permissions for acquisitions were not very clearly described. This patch changes the descriptions in the include file (GUI) and in the default SQL. To test: - Go to any patron account in staff - Go to more > permissions - Verify that the meaning of each acquisition permission is clear from the description Created attachment 75473 [details] [review] Bug 3849: Improve descriptions of granular acquisition permissions The permissions for acquisitions were not very clearly described. This patch changes the descriptions in the include file (GUI) and in the default SQL. To test: - Go to any patron account in staff - Go to more > permissions - Verify that the meaning of each acquisition permission is clear from the description Signed-off-by: Owen Leonard <oleonard@myacpl.org> Should not we add an update DB entry? (In reply to Jonathan Druart from comment #13) > Should not we add an update DB entry? ... or remove the "description" column on a separate bug report? I can add a db update and file a separate report for removing it - then you can pick :) Created attachment 75618 [details] [review] Bug 3849: (follow-up) Add database update to change descriptions We still have the descriptions on system preferences too... chose the database update here for now. Created attachment 75673 [details] [review] Bug 3849: Improve descriptions of granular acquisition permissions The permissions for acquisitions were not very clearly described. This patch changes the descriptions in the include file (GUI) and in the default SQL. To test: - Go to any patron account in staff - Go to more > permissions - Verify that the meaning of each acquisition permission is clear from the description Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Created attachment 75674 [details] [review] Bug 3849: (follow-up) Add database update to change descriptions Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Pushed to master for 18.11, awesome work everybody! Not pushing to 18.05.x as it could surprise unsuspecting librarians during a minor update. Nice fix for 18.11 though, marking resolved. Some of this got undone by a later patch, filed a fresh bug to fix it: bug 21966 |