Koha/members
emilyrose 34b12abe98 Bug 34870: Perform UTF8 encoding before redirection
Display special characters correctly when writing off an invoice.

The issue arises because pay.pl does not perform UTF-8 encoding on the “notes” parameter before redirecting the page. By using uri_escape_utf8, characters with a code above 255 are also UTF-8 encoded. Then, paycollect.pl can collect the information without any trouble.

This patch work with “;” and “:” characters.

TEST PLAN
1) Go to any patron profile > Accounting
2) Click “Create manual invoice”
3) Fill the fields and click the “Save” button
4) Repeat steps 2 and 3
5) Click “Make a payment”
6) Add a note with special characters for each one (e.g., éçö)
7) Select the created invoices and click the “Write off selected” button
8) In the note field, you should see “�” symbols where the special characters are supposed to be
9) Click the “Cancel” button
10) Now add a note with “:;” for one of the invoices
11) Select the invoices and click the “Write off selected” button
12) You should see a page with an error 500
13) Go back to the “Make a payment” page
14) Apply the patch
15) Add a note with special characters for each one (e.g., éçö)
16) Add “:;” to one of the note fields
17) Select the invoices and click the “Write off selected” button
18) Now the special characters are displayed correctly

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
(cherry picked from commit 1c01617634)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
(cherry picked from commit eea0310a62)
Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com>
2023-10-17 17:23:41 +00:00
..
accountline-details.pl Bug 22435: Update accountline-details page to show all history 2021-08-04 14:06:43 +02:00
apikeys.pl Bug 29859: Use iterator instead of as_list 2022-02-09 15:36:23 -10:00
boraccount.pl Bug 22990: Add CSRF protection to boraccount, pay and suggestion 2023-07-28 11:14:30 +00:00
cancel-charge.pl Bug 29457: Pass context borrowernumber 2021-12-21 20:44:15 -10:00
default_messageprefs.pl Bug 18403: Update permissions - borrowers => 1|* becomes borrowers => 'edit_borrowers' 2018-02-12 15:41:37 -03:00
deletemem.pl Bug 23991: Move SearchSuggestion to Koha::Suggestions 2022-06-27 12:30:28 -03:00
discharge.pl Bug 17600: Standardize our EXPORT_OK 2021-07-16 08:58:47 +02:00
discharges.pl Bug 17600: Standardize our EXPORT_OK 2021-07-16 08:58:47 +02:00
files.pl Bug 17600: Standardize our EXPORT_OK 2021-07-16 08:58:47 +02:00
holdshistory.pl Bug 17600: Standardize our EXPORT_OK 2021-07-16 08:58:47 +02:00
housebound.pl Bug 30718: Use flatpickr's altInput 2022-08-19 08:26:31 -03:00
ill-requests.pl Bug 17600: Standardize our EXPORT_OK 2021-07-16 08:58:47 +02:00
mancredit.pl Bug 29987: (follow-up) Fix missing payment type 2022-10-21 10:18:53 -03:00
maninvoice.pl Bug 31254: Add additional fields for accountlines 2022-09-21 18:58:28 -03:00
member-flags.pl Bug 17600: Standardize our EXPORT_OK 2021-07-16 08:58:47 +02:00
member-password.pl Bug 17600: Standardize our EXPORT_OK 2021-07-16 08:58:47 +02:00
member.pl Bug 34094: Use DefaultPatronSearchMethod consistently 2023-07-18 13:11:38 +00:00
memberentry.pl Bug 33803: Remove comment about tab width 2023-06-12 10:28:26 +00:00
members-home.pl Bug 30485: (bug 30063 follow-up) Display all patrons if not term passed 2022-04-13 15:55:39 +02:00
members-update-do.pl Bug 31893: Use checkauth and do not fetch about.tt template 2023-01-31 14:09:05 +00:00
members-update.pl Bug 29844: Fix ->search occurrences 2022-02-09 15:36:23 -10:00
merge-patrons.pl Bug 17600: Standardize our EXPORT_OK 2021-07-16 08:58:47 +02:00
mod_debarment.pl Bug 23681: Allow for selection of restriction type 2022-08-25 08:41:01 -03:00
moremember.pl Bug 31095: Remove GetDebarments from members/moremember.pl 2023-02-01 15:35:11 +00:00
notices.pl Bug 31739: Password recovery from staff fails if previous expired reset-entry exists. 2022-10-24 14:12:16 -03:00
patronimage.pl Bug 29541: Prevent users from another group to access patron's images 2022-02-02 21:05:29 -10:00
pay.pl Bug 34870: Perform UTF8 encoding before redirection 2023-10-17 17:23:41 +00:00
paycollect.pl Bug 31254: Add additional fields for accountlines 2022-09-21 18:58:28 -03:00
print_overdues.pl Bug 30420: Rename Koha::Patron->get_overdues with ->overdues 2022-06-14 08:46:05 -03:00
printfeercpt.pl Bug 26689: Prepend letter codes. 2022-07-22 15:01:36 -03:00
printinvoice.pl Bug 26689: Prepend letter codes. 2022-07-22 15:01:36 -03:00
printnotice.pl Bug 31714: Add Generic way to print patron slips 2022-10-21 10:09:03 -03:00
printslip.pl Bug 33010: Do not filter checkins with logged in patrons branch in printslip.pl 2023-06-08 16:51:19 +00:00
purchase-suggestions.pl Bug 23991: Move SearchSuggestion to Koha::Suggestions 2022-06-27 12:30:28 -03:00
readingrec.pl Bug 23838: Add renewals modal 2022-07-22 11:39:09 -03:00
recallshistory.pl Bug 30291: Changes to staff client files 2022-05-05 11:17:36 -10:00
routing-lists.pl Bug 17600: Standardize our EXPORT_OK 2021-07-16 08:58:47 +02:00
search.pl Bug 30093: Make patron search from request.pl use the REST API 2022-04-04 09:47:00 +02:00
setstatus.pl Bug 17600: Standardize our EXPORT_OK 2021-07-16 08:58:47 +02:00
statistics.pl Bug 17600: Standardize our EXPORT_OK 2021-07-16 08:58:47 +02:00
summary-print.pl Bug 17600: Standardize our EXPORT_OK 2021-07-16 08:58:47 +02:00
two_factor_auth.pl Bug 30588: (QA follow-up) Adjust flags for two_factor_auth.pl 2022-10-21 11:37:01 -03:00
update-child.pl Bug 29844: Fix ->search occurrences 2022-02-09 15:36:23 -10:00