The following notices relating to recalls all need to be updated to use Template Toolkit syntax: RETURN_RECALLED_ITEM PICKUP_RECALLED_ITEM RECALL_REQUESTER_DET
Created attachment 166784 [details] [review] Bug 36020: Port default recall notices to Template Toolkit The following notices relating to recalls all need to be updated to use Template Toolkit syntax: RETURN_RECALLED_ITEM PICKUP_RECALLED_ITEM RECALL_REQUESTER_DET To test: 1) Observe the default notice contains <<>> tags 2) Apply the patch 3) reset_all 4) Check that the notice is now all in template toolkit [% %] tags 5) Enable recalls 6) Check an item out to a patron 7) Make sure the patron has a primary email 8) Make sure a secondary patron has a primary email 9) Generate a recall on the checked out item for the second patron 10) Check the first patron's notices tab to confirm the return recalled item notice generated properly. 11) Check the item back in and generate the recall slip and confirm it generated correctly. 12) Check the second patron's notices tab to confirm the pickup recalled item notice generated properly.
Created attachment 166934 [details] [review] Bug 36020: Port default recall notices to Template Toolkit The following notices relating to recalls all need to be updated to use Template Toolkit syntax: RETURN_RECALLED_ITEM PICKUP_RECALLED_ITEM RECALL_REQUESTER_DET To test: 1) Observe the default notice contains <<>> tags 2) Apply the patch 3) reset_all 4) Check that the notice is now all in template toolkit [% %] tags 5) Enable recalls 6) Check an item out to a patron 7) Make sure the patron has a primary email 8) Make sure a secondary patron has a primary email 9) Generate a recall on the checked out item for the second patron 10) Check the first patron's notices tab to confirm the return recalled item notice generated properly. 11) Check the item back in and generate the recall slip and confirm it generated correctly. 12) Check the second patron's notices tab to confirm the pickup recalled item notice generated properly. Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Created attachment 166935 [details] [review] Bug 36020: (follow-up) Use title includes This patch updates the recall sample notices to use the patron-title and biblio-title includes for patron and biblio title displays.
'biblio-title.inc' produces the text "No bibliographic record" for me with this applied. 'biblio-title.inc' biblio=title' Shouldn't this pass the biblionumber?
Created attachment 170885 [details] [review] Bug 36020: (follow-up) Use title includes This patch updates the recall sample notices to use the patron-title and biblio-title includes for patron and biblio title displays.
Created attachment 170886 [details] [review] Bug 36020: (follow-up) Fix title includes This patch updates the recall notices to fill the titles when using the title include. Author: Lisette Scheer <lisette@bywatersolutions.com>
Created attachment 170887 [details] [review] Bug 36020: (follow-up) Fix title includes This patch updates the recall notices to fill the titles when using the title include. Author: Lisette Scheer <lisette@bywatersolutions.com>
Created attachment 170889 [details] [review] Bug 36020: (follow-up) Changed biblio=title to biblio=biblio Note: fixed a typo
A couple of notices have errors: RECALL_REQUESTER_DET - The commas in the address are included even if a patron does not have address info stored in the account - HTML isn't rendered under the ITEMS RECALLED heading PICKUP_RECALLED_ITEM - Recall pickup location is not present - Recall expiration date is not present
Created attachment 172134 [details] [review] Bug 36020: (follow-up) Changed biblio=title to biblio=biblio Note: fixed a typo
Created attachment 172135 [details] [review] Bug: 36020: (follow-up) fixed no_html and recalls to recall Fixed the errors from the last comment. To test: follow the test plan and confirm it works now.
Created attachment 172147 [details] [review] Bug: 36020: (follow-up) fixed no_html and recalls to recall
Created attachment 172148 [details] [review] Bug 36020: (follow-up) Add TT support to recall notices
RECALL_REQUESTER_DET - This slip is still displaying unrendered HTML: Date: 09/27/2024 Recall for pickup at Centerville koha (42) , , example2@example.com ITEM RECALLED <span class="biblio-title">Seesaw</span> by Hart, Beth Barcode: 39999000013733 Callnumber: Waiting since: 2024-09-27 18:37:03 Notes: - It would be good to omit the extra commas in the address line if there's no data - borrower.state is missing between borrower.city and borrower.zipcode RETURN_RECALLED_ITEM - Still not showing the issue.date_due PICKUP_RECALLED_ITEM - It would be better to use the branch's name rather than its id in the [% recall.pickup_library_id %] area
Created attachment 176478 [details] [review] Bug 36020: (follow-up) Fix formatting and dates RECALL_REQUESTER_DET will need to be marked as HTML. I fixed all the issues from Comment 14.
Created attachment 176479 [details] [review] Bug 36020: (follow-up) fixed no_html and recalls to recall Fixed a typo in the commit title.