| Summary: | Wrong permissions check on item circulation alerts | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Nick Clemens (kidclamp) <nick> |
| Component: | Staff interface | Assignee: | Nick Clemens (kidclamp) <nick> |
| Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
| Severity: | normal | ||
| Priority: | P5 - low | CC: | gmcharlt, tomascohen |
| 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: |
22.11.00
|
|
| Circulation function: | |||
| Bug Depends on: | 14391 | ||
| Bug Blocks: | |||
| Attachments: |
Bug 32122: Correct permissions check on item circulation alerts
Bug 32122: Correct permissions check on item circulation alerts |
||
|
Description
Nick Clemens (kidclamp)
2022-11-07 13:56:08 UTC
Created attachment 143370 [details] [review] Bug 32122: Correct permissions check on item circulation alerts This patch changes the check from 'item_circ_alerts' to 'manage_item_circ_alerts' To test: 1 - Give an account catalogue and manage_item_circ_alerts permissions 2 - Log in to staff client iwht this account 3 - Browse to Admin->Item circulation alerts 4 - Error, you do not have permission to view this page 5 - Apply patch 6 - Restart all 7 - Browse to page agian, success! Straight to QA, minor and easy fix Created attachment 143372 [details] [review] Bug 32122: Correct permissions check on item circulation alerts This patch changes the check from 'item_circ_alerts' to 'manage_item_circ_alerts' To test: 1 - Give an account catalogue and manage_item_circ_alerts permissions 2 - Log in to staff client iwht this account 3 - Browse to Admin->Item circulation alerts 4 - Error, you do not have permission to view this page 5 - Apply patch 6 - Restart all 7 - Browse to page agian, success! Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Pushed to master for 22.11. Nice work everyone, thanks! |