[Koha-bugs] [Bug 34728] HTML notices should not be pre-formatted

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Wed Sep 6 13:33:06 CEST 2023


https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34728

--- Comment #2 from Matt Blenkinsop <matt.blenkinsop at ptfs-europe.com> ---
Created attachment 155265
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=155265&action=edit
Bug 34728: Make pre tag conditional

This patch makes the pre-formatting of notice templates conditional on the
content type.

Test plan:
1) Navigate to tools > Notices and slips
2) Edit the notice with the code of HOLD
3) Under print, paste the following
<p>Please collect within 10 days from the date of this letter and don't forget
to bring your library card with you.</p>
<p>Be aware that opening hours vary at different library branches so if in
doubt double check the times before you visit.</p>
<p>Should you no longer require this item please contact us so we can make it
available for another customer.</p>

<p>Thank you</p>
4) Check the box that says HTML message and save the notice
5) In Patrons, select a patron and in Patron messaging preferences, check the
box in the Email column for Hold filled
6) Find a record in the catalogue
7) Copy the barcode for one of the items
8) Place a hold on the item for the patron you selected
9) Navigate to Circulation > Check-in
10) Paste the barcode in the Check-in box and click Check-in
11) In the pop-up box, click Confirm hold
12) In the command line, run perl misc/cronjobs/gather_print_notices.pl
--letter_code=HOLD --html /tmp
13) cd /tmp
14) There should be a file in there called 'notices_HOLD-date.html
15) cat this file
16) The html you pasted in the template will be wrapped in a <pre> tag
17) Apply patch and restart_all
18) rm the notices file that was created
19) cd /kohadevbox/koha
20) Select a new record from the catalogue and repeat steps 7-15 for this
record, using the same patron
21) This time, the html file should not contain the <pre> tag
22) Sign off!

-- 
You are receiving this mail because:
You are watching all bug changes.


More information about the Koha-bugs mailing list