Summary: | on return with reserve or transfer the alerts are not shown | ||
---|---|---|---|
Product: | Koha | Reporter: | Fridolin Somers <fridolin.somers> |
Component: | Circulation | Assignee: | Fridolin Somers <fridolin.somers> |
Status: | CLOSED FIXED | QA Contact: | |
Severity: | normal | ||
Priority: | P5 - low | CC: | bgkriegel, chrish, gmcharlt, kyle.m.hall, srdjan, tomascohen |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | Trivial patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Attachments: |
Proposed patch
Bug 10362 - on return with reserve or transfer the alerts are not shown Bug 10362 - on return with reserve or transfer the alerts are not shown |
Description
Fridolin Somers
2013-05-28 13:42:30 UTC
Created attachment 18429 [details] [review] Proposed patch See commit message Created attachment 18536 [details] [review] Bug 10362 - on return with reserve or transfer the alerts are not shown When a return is performed on an item with a reserve or a transfer, the possible alert messages are not shown. These alerts are cases from errmsgloop like "NotIssued", "debarred" ... This patch changes the template to show both messages and alerts. Test plan : - Perform an checkout that will a checkin : * need a transfert (return in a different branch) * set user debarred (depends on issuing rules) - Perform check-in => You get tranfert message and debarment alert Signed-off-by: Srdjan <srdjan@catalyst.net.nz> Created attachment 19187 [details] [review] Bug 10362 - on return with reserve or transfer the alerts are not shown When a return is performed on an item with a reserve or a transfer, the possible alert messages are not shown. These alerts are cases from errmsgloop like "NotIssued", "debarred" ... This patch changes the template to show both messages and alerts. Test plan : - Perform an checkout that will a checkin : * need a transfert (return in a different branch) * set user debarred (depends on issuing rules) - Perform check-in => You get tranfert message and debarment alert Signed-off-by: Srdjan <srdjan@catalyst.net.nz> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Pushed to master. Thanks, Fridolyn! This patch has been pushed to 3.12.x, will be in 3.12.4. Thanks Fridolyn! Pushed to 3.10.x, will be in 3.10.11 Pushed to 3.8.x, will be in 3.8.19 |