Summary: | Overdue notices script does not care about borrower's language, always takes default template | ||
---|---|---|---|
Product: | Koha | Reporter: | Sophie MEYNIEUX <sophie.meynieux> |
Component: | Notices | Assignee: | Jonathan Druart <jonathan.druart> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | major | ||
Priority: | P5 - low | CC: | aleisha, andreas.jonsson, jonathan.druart, lucas, martin.renvoize, victor |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
See Also: | https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17762 | ||
Change sponsored?: | --- | Patch complexity: | Small patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: |
20.11.00, 20.05.05, 19.11.11, 19.05.16
|
|
Circulation function: | |||
Bug Depends on: | |||
Bug Blocks: | 28487 | ||
Attachments: |
Bug 26420: Use translated notices for overdue_notices.pl
Bug 26420: Use translated notices for overdue_notices.pl Bug 26420: Use translated notices for overdue_notices.pl |
Description
Sophie MEYNIEUX
2020-09-09 15:47:11 UTC
Created attachment 109837 [details] [review] Bug 26420: Use translated notices for overdue_notices.pl This script does not pass the patron's preferred lang for notices. Test plan: Enable TranslateNotices Create a patron with a preferred lang for notices != default Translate ODUE for this language Setup the overdue notices for this patron's category Check an item out for this patron (select the correct due date to trigger the notice) Run the script overdue_notices.pl Check the message_queue table and confirm that the notice enqueued is translated Created attachment 110036 [details] [review] Bug 26420: Use translated notices for overdue_notices.pl This script does not pass the patron's preferred lang for notices. Test plan: Enable TranslateNotices Create a patron with a preferred lang for notices != default Translate ODUE for this language Setup the overdue notices for this patron's category Check an item out for this patron (select the correct due date to trigger the notice) Run the script overdue_notices.pl Check the message_queue table and confirm that the notice enqueued is translated Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Works as expected, Signing off Created attachment 110123 [details] [review] Bug 26420: Use translated notices for overdue_notices.pl This script does not pass the patron's preferred lang for notices. Test plan: Enable TranslateNotices Create a patron with a preferred lang for notices != default Translate ODUE for this language Setup the overdue notices for this patron's category Check an item out for this patron (select the correct due date to trigger the notice) Run the script overdue_notices.pl Check the message_queue table and confirm that the notice enqueued is translated Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Pushed to master for 20.11, thanks to everybody involved! backported to 20.05.x for 20.05.05 backported to 19.11.x for 19.11.11 Backported to 19.05.x branch for 19.05.16 Shouldn't we still fall back to the default language if a tranlation is missing? I reported bug 28487. (In reply to Andreas Jonsson from comment #9) > Shouldn't we still fall back to the default language if a tranlation is > missing? I reported bug 28487. I think you are correct. |