Bug 18699

Summary: Get rid of %%] in translation for edi_accounts.tt
Product: Koha Reporter: Marc Véron <veron>
Component: I18N/L10NAssignee: Marc Véron <veron>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: trivial    
Priority: P5 - low CC: f.demians, fridolin.somers, katrin.fischer, mtj
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:
Attachments: Bug 18699: Get rid of %%] in translation for edi_accounts.tt
[SIGNED-OFF] Bug 18699: Get rid of %%] in translation for edi_accounts.tt
Bug 18699: Get rid of %%] in translation for edi_accounts.tt

Description Marc Véron 2017-05-30 13:55:13 UTC
Translation for edi_accounts.tt exposes:
[%% transport_types = [ 'FTP', 'SFTP', 'FILE' ] %%] 

Translators should not be confronted with such code internals.
Comment 1 Marc Véron 2017-05-30 14:13:29 UTC
Created attachment 63845 [details] [review]
Bug 18699: Get rid of %%] in translation for edi_accounts.tt

Translation for edi_accounts.tt exposes:
[%% transport_types = [ 'FTP', 'SFTP', 'FILE' ] %%]

Translators should not be confronted with such code internals.

To test:
- Review code change (it removes line breaks)
- Bonus test: Create a new translatin for "language" aa-AA and
  verify that the line no longer appears in aa-AA-staff-prog.po
Comment 2 Josef Moravec 2017-06-06 08:02:49 UTC
Created attachment 64030 [details] [review]
[SIGNED-OFF] Bug 18699: Get rid of %%] in translation for edi_accounts.tt

Translation for edi_accounts.tt exposes:
[%% transport_types = [ 'FTP', 'SFTP', 'FILE' ] %%]

Translators should not be confronted with such code internals.

To test:
- Review code change (it removes line breaks)
- Bonus test: Create a new translatin for "language" aa-AA and
  verify that the line no longer appears in aa-AA-staff-prog.po

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Comment 3 Brendan Gallagher 2017-06-16 18:39:32 UTC
Created attachment 64413 [details] [review]
Bug 18699: Get rid of %%] in translation for edi_accounts.tt

Translation for edi_accounts.tt exposes:
[%% transport_types = [ 'FTP', 'SFTP', 'FILE' ] %%]

Translators should not be confronted with such code internals.

To test:
- Review code change (it removes line breaks)
- Bonus test: Create a new translatin for "language" aa-AA and
  verify that the line no longer appears in aa-AA-staff-prog.po

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>

Signed-off-by: Brendan A Gallagher <brendan@bywatersolutions.com>
Comment 4 Jonathan Druart 2017-06-16 20:05:16 UTC
Pushed to master for 17.11, thanks to everybody involved!
Comment 5 Fridolin Somers 2017-06-27 14:34:24 UTC
Pushed to 17.05.x, will be in 17.05.02

Trivial enhancement and useful for translation on stable branches.
Comment 6 Katrin Fischer 2017-06-27 21:52:12 UTC
The other "Get rid of..." were classified as trivial bugs (see 16.11.09 release notes for examples). I am changing this one to match.

This patch has been pushed to 16.11.x and will be in 16.11.10.
Comment 7 Mason James 2017-07-31 11:30:17 UTC
Pushed to 16.05.x, for 16.05.15 release