Bug 19655: Make To.json escape doesn't escape newlines to create valid JSON
authorKyle M Hall <kyle@bywatersolutions.com>
Mon, 20 Nov 2017 16:05:35 +0000 (11:05 -0500)
committerJonathan Druart <jonathan.druart@bugs.koha-community.org>
Sun, 26 Nov 2017 15:59:04 +0000 (12:59 -0300)
commit9cd9240c362336e390eed01acf3630f33e73825f
tree037313fddd2ac9c6f2c93ab668999a207b765b62
parent8d7dd0c69d82d0b53b352a1f8b5450402993ed41
Bug 19655: Make To.json escape doesn't escape newlines  to create valid JSON

JSON does not allow real line-breaks. If a field contains them and they are not escaped, a JSON parser will be unable to convert the stringified JSON back into an object.

This is clearly exemplified by the guarantor search, where a multi-line note on the guarantor's record will break the ability to select that guarantor.

Test Plan:
1) Create Patron A with a "Circulation note" that has multiple lines in it
2) Create Patron B
3) Attempt to set Patron A to be the guarantor for Patron B
4) Note selecting the patron does nothing
5) Apply this patch
6) Repeat step 3
7) Selecting the guarantor now works!

Signed-off-by: Simon Pouchol <simon.pouchol@biblibre.com>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Koha/Template/Plugin/To.pm