Bug 21278

Summary: CANCEL_HOLD_ON_LOST notice contains different variables
Product: Koha Reporter: Caroline Cyr La Rose <caroline.cyr-la-rose>
Component: NoticesAssignee: Bugs List <koha-bugs>
Status: CLOSED INVALID QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: jonathan.druart
Version: Main   
Hardware: All   
OS: All   
GIT URL: Initiative type: ---
Sponsorship status: --- Comma delimited list of Sponsors:
Crowdfunding goal: 0 Crowdfunding committed: 0
Crowdfunding contact: Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:

Description Caroline Cyr La Rose 2018-08-27 14:49:42 UTC
The sample notice CANCEL_HOLD_ON_LOST contains variables with the syntax [% xxx %] instead of << xxx >>.

in installer/data/mysql/en/mandatory/sample_notices.sql
Comment 1 Katrin Fischer 2018-08-27 16:20:52 UTC
Hi Caroline, 

that's the new TT syntax - it should work. Have you tested?

The notice also uses an IF/ELSE construct to display the right reason, this would not be possible without TT (Template Toolkit)
Comment 2 Caroline Cyr La Rose 2018-08-27 16:25:27 UTC
Katrin,

I didn't test it, but I figured it might be confusing to end users if they see some notices have [% xxx %] and other notices have << xxx >>.

Either they should all be [% xxx %] or all << xxx >>, not a mix of the two.

If it is indeed changing, we have a lot of work to do in the manual and the wiki to change the instructions on how to customize the notices.
Comment 3 Katrin Fischer 2018-08-27 17:56:14 UTC
It's a process, I think starting with the new notices now. The account slip notices (payment/waive receipts) in 18.11 will also be TT from the start, as they make calculations and do formatting (look in master).

The old ones are supposed to be converted too, maybe this should be a goal for 18.11?

Look at the graph view of this bug:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15279

There are 2 examples for converted notices here:
https://wiki.koha-community.org/wiki/Notices_and_Slips_Library
Comment 4 Jonathan Druart 2018-08-29 20:32:07 UTC
All new notices should use the TT syntax.