| Summary: | Fix selecting fund users requiring budget_modify permission | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Mathieu Saby <mathsabypro> |
| Component: | Acquisitions | Assignee: | Bugs List <koha-bugs> |
| Status: | NEW --- | QA Contact: | Testopia <testopia> |
| Severity: | minor | ||
| Priority: | P5 - low | ||
| Version: | Main | ||
| Hardware: | All | ||
| OS: | All | ||
| GIT URL: | Initiative type: | --- | |
| Sponsorship status: | --- | Comma delimited list of Sponsors: | |
| Crowdfunding goal: | 0 | Patch complexity: | --- |
| Documentation contact: | Documentation submission: | ||
| Text to go in the release notes: | Version(s) released in: | ||
| Circulation function: | |||
|
Description
Mathieu Saby
2024-09-10 08:52:07 UTC
I don't think the permission description is wrong, the behavior of the form configuring fund users is wrong. Fund users are implied to be users that can order from a fund. If you are a fund user and have receive_order etc., you can use that fund for ordering. If you are a fund user and have budget_manage, you should probably be able to edit the fund. But when adding fund users the list of patrons is limited by default to those having budget_manage and that is wrong. It should probably include users with any acq permission or another select list of multiple (receive_order, order_manage, etc. I'm not sure of the intended behavior of the preference. It is used in C4::Budgets::CanUserModifyBudget, which is called in admin/aqbudgets.pl If I create a staff user with just this only permission in Acquisitions permission, - he can display admin/aqbudgets.pl - he can only see the funds whose access permission is "Restrict access to: None" - he cannot modify the funds he can see (he can not open /admin/aqbudgets.pl page) As an admin I can search him from a fund page, with "Select owner" or "Add users", and give him permission over a "restricted" fund. Now - he can see this new fund in admin/aqbudgets.pl - but he cannot modify it So I stil don't understand the terminology "Modify fund"... Note that "Modify fund" is used in template of /admin/aqbudgets.pl in breadcrumbs. |