Summary: | Can't delete other library's messages to the patron with AllowAllMessageDeletion on | ||
---|---|---|---|
Product: | Koha | Reporter: | Owen Leonard <oleonard> |
Component: | Circulation | Assignee: | Owen Leonard <oleonard> |
Status: | CLOSED FIXED | QA Contact: | Bugs List <koha-bugs> |
Severity: | normal | ||
Priority: | PATCH-Sent (DO NOT USE) | CC: | chris, gmcharlt, koha.sekjal, nengard |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | --- | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: | Circulation function: | ||
Attachments: |
Proposed fix
[SIGNED-OFF] Fix for Bug 7024 - Can't delete other library's messages with AllowAllMessageDeletion on |
Description
Owen Leonard
2011-10-13 13:55:51 UTC
Created attachment 5875 [details] [review] Proposed fix The display of messages "to the patron" on the checkout screen lacked the same logic controlling messages "to other librarians," affecting conditions under which the [Delete] link would be shown. This match makes the two categories consistent with each other: A delete link will be shown if the logged in branch matches the branch where the message was set OR if AllowAllMessageDeletion is turned on. Created attachment 5892 [details] [review] [SIGNED-OFF] Fix for Bug 7024 - Can't delete other library's messages with AllowAllMessageDeletion on The display of messages "to the patron" on the checkout screen lacked the same logic controlling messages "to other librarians," affecting conditions under which the [Delete] link would be shown. This match makes the two categories consistent with each other: A delete link will be shown if the logged in branch matches the branch where the message was set OR if AllowAllMessageDeletion is turned on. Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com> Template only change, easy to read. While this does introduce a string, it's the same string "[Delete]" that already exists in conditions of the code, so no new translation is required. Marking as Passed QA Pushed please test |