Bug 31282 - Broken characters in patron_emailer.pl verbose mode
Summary: Broken characters in patron_emailer.pl verbose mode
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Command-line Utilities (show other bugs)
Version: unspecified
Hardware: All All
: P5 - low normal (vote)
Assignee: Tomás Cohen Arazi
QA Contact: Kyle M Hall
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-08-02 12:22 UTC by Tomás Cohen Arazi
Modified: 2023-06-08 22:26 UTC (History)
5 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Trivial patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
This fixes the patron_emailer.pl script (misc/cronjobs/patron_emailer.pl) so that non-ASCII characters in notices display correctly.
Version(s) released in:
22.11.00,22.05.06,21.11.12


Attachments
Screenshot (57.25 KB, image/png)
2022-08-02 12:23 UTC, Tomás Cohen Arazi
Details
Bug 31282: Fix broken characters in patron_emailer.pl verbose mode (1.28 KB, patch)
2022-08-02 12:28 UTC, Tomás Cohen Arazi
Details | Diff | Splinter Review
Bug 31282: Fix broken characters in patron_emailer.pl verbose mode (1.33 KB, patch)
2022-08-18 23:02 UTC, David Nind
Details | Diff | Splinter Review
Bug 31282: Fix broken characters in patron_emailer.pl verbose mode (1.38 KB, patch)
2022-08-19 12:14 UTC, Kyle M Hall
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Tomás Cohen Arazi 2022-08-02 12:22:07 UTC
If you have non-ASCII characters in the printed notice, and run it with --verbose or without --commit, you will get broken characters.
Comment 1 Tomás Cohen Arazi 2022-08-02 12:23:46 UTC
Created attachment 138502 [details]
Screenshot
Comment 2 Tomás Cohen Arazi 2022-08-02 12:28:25 UTC
Created attachment 138504 [details] [review]
Bug 31282: Fix broken characters in patron_emailer.pl verbose mode

To test:
1. Have a report containing:
   SELECT tomascohen@theke.io;
2. Have a members notice containing ¡ and other non-ASCII characters.
3. Run (changing the report number and notice code accordingly):
   $ kshell
  k$ perl misc/cronjobs/patron_emailer.pl --report 4 \
           --notice BIRTHDAY --from tomascohen@theke.io --module members
=> FAIL: non-ASCII characters are broken
4. Apply this patch
5. Repeat 3
=> SUCCESS: Things print correctly!
6. Sign off :-D

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Comment 3 David Nind 2022-08-18 23:02:19 UTC
Created attachment 139439 [details] [review]
Bug 31282: Fix broken characters in patron_emailer.pl verbose mode

To test:
1. Have a report containing:
   SELECT tomascohen@theke.io;
2. Have a members notice containing ¡ and other non-ASCII characters.
3. Run (changing the report number and notice code accordingly):
   $ kshell
  k$ perl misc/cronjobs/patron_emailer.pl --report 4 \
           --notice BIRTHDAY --from tomascohen@theke.io --module members
=> FAIL: non-ASCII characters are broken
4. Apply this patch
5. Repeat 3
=> SUCCESS: Things print correctly!
6. Sign off :-D

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>

Signed-off-by: David Nind <david@davidnind.com>
Comment 4 David Nind 2022-08-18 23:12:48 UTC
Testing notes (using koha-testing-docker):

- I couldn't get step 1 to work for me, so I ended up:
  . Creating a new report: SELECT borrowernumber, firstname, surname, address, email, address2, city, zipcode, dateofbirth FROM borrowers WHERE MONTH(dateofbirth) = 08
 . Edited details of patrons in the results so that they have an email address.
 . Created a notice with a code called BIRTHDAY and added some text to the Email notice with non-ASCII characters.
 ==> step 3 worked as expected
Comment 5 Kyle M Hall 2022-08-19 12:14:08 UTC
Created attachment 139486 [details] [review]
Bug 31282: Fix broken characters in patron_emailer.pl verbose mode

To test:
1. Have a report containing:
   SELECT tomascohen@theke.io;
2. Have a members notice containing ¡ and other non-ASCII characters.
3. Run (changing the report number and notice code accordingly):
   $ kshell
  k$ perl misc/cronjobs/patron_emailer.pl --report 4 \
           --notice BIRTHDAY --from tomascohen@theke.io --module members
=> FAIL: non-ASCII characters are broken
4. Apply this patch
5. Repeat 3
=> SUCCESS: Things print correctly!
6. Sign off :-D

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>

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

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 6 Tomás Cohen Arazi 2022-08-19 12:58:11 UTC
Pushed to master for 22.11.

Nice work everyone, thanks!
Comment 7 Lucas Gass 2022-10-14 19:18:49 UTC
Backported to 22.05.x for 22.05.06 release
Comment 8 Victor Grousset/tuxayo 2022-10-21 22:16:31 UTC
Not backported to oldoldstable (21.05.x). Feel free to ask if it's needed.

Nothing to document it seems, marking resolved.
Comment 9 Arthur Suzuki 2022-10-22 22:09:39 UTC
Thanks!

Pushed to 21.11 for 21.11.12