Bug 30667

Summary: Holds reminder cronjob (holds_reminder.pl) never uses default letter template
Product: Koha Reporter: Kyle M Hall <kyle>
Component: Command-line UtilitiesAssignee: Kyle M Hall <kyle>
Status: CLOSED FIXED QA Contact: Marcel de Rooy <m.de.rooy>
Severity: normal    
Priority: P5 - low CC: andrewfh, arthur.suzuki, evelyn, m.de.rooy, robin, victor
Version: master   
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:
22.05.00, 21.11.11
Bug Depends on: 28514    
Bug Blocks: 30666    
Attachments: Bug 30667: Holds reminder cronjob (holds_reminder.pl) never uses default letter template
Bug 30667: Holds reminder cronjob (holds_reminder.pl) never uses default letter template
Bug 30667: Holds reminder cronjob (holds_reminder.pl) never uses default letter template

Description Kyle M Hall 2022-05-03 10:53:25 UTC
Bug 28514 changed the way holds_reminder.pl searches for templates, using a direct search for letters, but should be using find_effective_template instead. Now, if a branch specific template does not exist, it will skip that branch.
Comment 1 Kyle M Hall 2022-05-03 10:58:19 UTC
Created attachment 134504 [details] [review]
Bug 30667: Holds reminder cronjob (holds_reminder.pl) never uses default letter template

Bug 28514 changed the way holds_reminder.pl searches for templates, using a direct search for letters,
but should be using find_effective_template instead. Now, if a branch specific template does not exist,
it will skip that branch.

Test Plan:
1) Ensure you only have the default HOLD_REMINDER template
2) Become the koha user using koha-shell
3) Run misc/cronjobs/holds/holds_reminder.pl --days 7 -v
4) Note that the script skips every branch
5) Apply this patch
6) Run the command in step 3 again
7) Note the script doesn't skip over any branches
Comment 2 David Nind 2022-05-04 02:45:11 UTC
Created attachment 134564 [details] [review]
Bug 30667: Holds reminder cronjob (holds_reminder.pl) never uses default letter template

Bug 28514 changed the way holds_reminder.pl searches for templates, using a direct search for letters,
but should be using find_effective_template instead. Now, if a branch specific template does not exist,
it will skip that branch.

Test Plan:
1) Ensure you only have the default HOLD_REMINDER template
2) Become the koha user using koha-shell
3) Run misc/cronjobs/holds/holds_reminder.pl --days 7 -v
4) Note that the script skips every branch
5) Apply this patch
6) Run the command in step 3 again
7) Note the script doesn't skip over any branches

Signed-off-by: David Nind <david@davidnind.com>
Comment 3 Marcel de Rooy 2022-05-13 09:44:28 UTC
Looking here
Comment 4 Marcel de Rooy 2022-05-13 09:46:58 UTC
Created attachment 134958 [details] [review]
Bug 30667: Holds reminder cronjob (holds_reminder.pl) never uses default letter template

Bug 28514 changed the way holds_reminder.pl searches for templates, using a direct search for letters,
but should be using find_effective_template instead. Now, if a branch specific template does not exist,
it will skip that branch.

Test Plan:
1) Ensure you only have the default HOLD_REMINDER template
2) Become the koha user using koha-shell
3) Run misc/cronjobs/holds/holds_reminder.pl --days 7 -v
4) Note that the script skips every branch
5) Apply this patch
6) Run the command in step 3 again
7) Note the script doesn't skip over any branches

Signed-off-by: David Nind <david@davidnind.com>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 5 Fridolin Somers 2022-05-14 01:43:01 UTC
Pushed to master for 22.05, thanks to everybody involved 🦄
Comment 6 Evelyn Hartline 2022-07-08 14:17:20 UTC
Could this be backported to 21.11?
Comment 7 Katrin Fischer 2022-07-08 15:32:52 UTC
(In reply to evelyn from comment #6)
> Could this be backported to 21.11?

I've added the RMaint to CC
Comment 8 Liz Rea 2022-08-16 15:15:52 UTC
*** Bug 28797 has been marked as a duplicate of this bug. ***
Comment 9 Arthur Suzuki 2022-08-21 08:44:42 UTC
pushed to 21.11.x for 21.11.11
Comment 10 Victor Grousset/tuxayo 2022-08-22 23:58:45 UTC
Not backported to oldoldstable (21.05.x). Feel free to ask if it's needed.

Nothing to document, marking resolved.