Summary: | Can't enqueue letter HASH(0x55edf1806850) at /usr/share/koha/Koha/ArticleRequest.pm line 123. | ||
---|---|---|---|
Product: | Koha | Reporter: | Marcel de Rooy <m.de.rooy> |
Component: | Notices | Assignee: | Marcel de Rooy <m.de.rooy> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | minor | ||
Priority: | P5 - low | CC: | fridolin.somers, kyle, martin.renvoize, victor |
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: |
21.11.00,21.05.02,20.11.08
|
|
Circulation function: | |||
Attachments: |
Bug 28582: Fix hashref in a warning message
Bug 28582: Fix hashref in a warning message |
Description
Marcel de Rooy
2021-06-17 09:33:12 UTC
Created attachment 122058 [details] [review] Bug 28582: Fix hashref in a warning message Can't enqueue letter HASH(0x55edf1806850) Test plan: If you cancel an article request when there is no email address (no branch email, no KohaAdminEmailAddress), Koha warns. Verify that the warning now contains the letter code. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Self signoff: trivial++ Created attachment 122060 [details] [review] Bug 28582: Fix hashref in a warning message Can't enqueue letter HASH(0x55edf1806850) Test plan: If you cancel an article request when there is no email address (no branch email, no KohaAdminEmailAddress), Koha warns. Verify that the warning now contains the letter code. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Works perfectly :). Passing QA I feel like there's a pattern here begging to be adopted for the rest of the 'EnqueueLetter' calls.. but that can wait for another bug. Pushed to master for 21.11, thanks to everybody involved! Pushed to 21.05.x for 21.05.02 Pushed to 20.11.x for 20.11.08 Not backported to oldoldstable (20.05.x). Feel free to ask if it's needed. |