| Summary: | Vendor module permissions are ignored | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Matt Blenkinsop <matt.blenkinsop> |
| Component: | Acquisitions | Assignee: | Matt Blenkinsop <matt.blenkinsop> |
| Status: | Needs documenting --- | QA Contact: | Laura Escamilla <Laura.escamilla> |
| Severity: | normal | ||
| Priority: | P5 - low | CC: | baptiste.wojtkowski, blawlor, Laura.escamilla, lucas, michaela.sieber, mspinney |
| Version: | Main | Keywords: | rel_25_05_candidate |
| 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: |
25.11.00,25.05.05
|
|
| Circulation function: | |||
| Bug Depends on: | 40684 | ||
| Bug Blocks: | |||
| Attachments: |
Bug 40868: Restore full list of permissions to the vendors app
Bug 40868: Restore full list of permissions to the vendors app Bug 40868: Restore full list of permissions to the vendors app |
||
|
Description
Matt Blenkinsop
2025-09-24 09:11:34 UTC
Created attachment 186859 [details] [review] Bug 40868: Restore full list of permissions to the vendors app This patch adjusts the logic to consider all permissions and not just the top level flags Test plan: 1) Create or edit a user to have all acquisitions permissions except one (not relevant which one, any will do) 2) Log into the staff interface and go to acquisitions > vendors and click the vendor in the table 3) The left menu will be missing everything except invoices 4) Apply patch and restart_all 5) Refresh the page, the menu should now be populated Created attachment 186919 [details] [review] Bug 40868: Restore full list of permissions to the vendors app This patch adjusts the logic to consider all permissions and not just the top level flags Test plan: 1) Create or edit a user to have all acquisitions permissions except one (not relevant which one, any will do) 2) Log into the staff interface and go to acquisitions > vendors and click the vendor in the table 3) The left menu will be missing everything except invoices 4) Apply patch and restart_all 5) Refresh the page, the menu should now be populated Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu> Created attachment 186996 [details] [review] Bug 40868: Restore full list of permissions to the vendors app This patch adjusts the logic to consider all permissions and not just the top level flags Test plan: 1) Create or edit a user to have all acquisitions permissions except one (not relevant which one, any will do) 2) Log into the staff interface and go to acquisitions > vendors and click the vendor in the table 3) The left menu will be missing everything except invoices 4) Apply patch and restart_all 5) Refresh the page, the menu should now be populated Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu> Signed-off-by: Laura_Escamilla <laura.escamilla@bywatersolutions.com> Nice work everyone! Pushed to main for 25.11 *** Bug 40912 has been marked as a duplicate of this bug. *** Nice work everyone! Pushed to 25.05.x |