Add letter template variables for accountlines (where amountoutstanding > 0). Use case ======== Your fines: <accountlines> <<accountlines.amountoutstanding>> <<accountlines.description>> </accounlines> Totalt: <<accountlines.total>>
Hi Martin, I think a lot of people will like this - which letters are you thinking about?
Hi Katrin, I'm thinking of the "Notices & Slips"-letters
That's very broad :) You might have to exclude some of them, like the claims letters - but I think it could work for all the letters that are intended to be sent to the patrons.
:D thanks, I'll try to narrow it down to reasonable proportions :)
Created attachment 45768 [details] [review] Bug 15364: accountlines variable in letters Add accountlines variable to letter templates for advance_notices and overdue_notices. Test plan ========= 1. Edit the overdue notice template to look like this: <accountline><<accountlines.amountoutstanding>> - <<accountlines.description>></accountline> Total: <<accountlines.total>> 2. Make sure at least one patron has at least one accountline, if not, create one by checking out an item with a past due date. 3. run bin/cronjobs/overdue_notices.pl and check the generated mail content 4. run bin/cronjobs/advance_notices.pl and check the generated mail content 5. generated mail content should look something like this: $5 - Fine $10 - Rent Total: $15
CONFLICT (content): Merge conflict in misc/cronjobs/overdue_notices.pl I think it is a simple conflict, but not sure, if you could take a look I'll try to sign off again tomorrow
I think we can now do this with TT and would probably want to not introduce new 'hungry alligator' patterns. Closing, please reopen if you disagree.