Bug 13861

Summary: \n in strings don't translate well
Product: Koha Reporter: Katrin Fischer <katrin.fischer>
Component: I18N/L10NAssignee: Katrin Fischer <katrin.fischer>
Status: CLOSED FIXED QA Contact: Marcel de Rooy <m.de.rooy>
Severity: normal    
Priority: P5 - low CC: chris, f.demians, m.de.rooy, tomascohen
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:
Attachments: Bug 13861: \n in confirmation message on closing a budget
Bug 13861: \n in confirmation message on closing a budget

Description Katrin Fischer 2015-03-18 00:45:11 UTC
Line breaks done with \n in Javascript strings will cause problems in translation as Pootle escapes them automatically. The result is that in a translated template \n is shown instead of a line break.
Comment 1 Katrin Fischer 2015-03-18 00:49:24 UTC Comment hidden (obsolete)
Comment 2 Marcel de Rooy 2015-03-19 10:35:10 UTC
Created attachment 37004 [details] [review]
Bug 13861: \n in confirmation message on closing a budget

\n in strings doesn't translate well, so we move it outside
the translated string.

To test:
- create at least 2 budgets
- close one of the budgets (moving orders from one budget to the other)
- after selecting the budget to move your orders to, save
- the confirmation message shown should read:

You have chosen to move all unreceived orders from 'budget 1' to 'budget 2'.
This action cannot be reversed. Do you wish to continue?

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 3 Marcel de Rooy 2015-03-19 10:36:05 UTC
Very trivial: Passed QA in one run.
Comment 4 Tomás Cohen Arazi (tcohen) 2015-03-19 17:11:41 UTC
Patch pushed to master.

Thanks Katrina!
Comment 5 Chris Cormack 2015-03-27 23:54:28 UTC
Pushed to 3.18.x will be in 3.18.6