Bug 14817

Summary: Encoding issues in columns.def
Product: Koha Reporter: Jonathan Druart <jonathan.druart>
Component: ToolsAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: josef.moravec, tomascohen
Version: Main   
Hardware: All   
OS: All   
See Also: http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14808
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 11944    
Bug Blocks:    
Attachments: Bug 14817: Fix encoding issues in columns.def
Bug 14817: (follow-up) Fix encoding issues in columns.def
Bug 14817: Fix encoding issues in columns.def
Bug 14817: (follow-up) Fix encoding issues in columns.def
[PASSED QA] Bug 14817: Fix encoding issues in columns.def
[PASSED QA] Bug 14817: (follow-up) Fix encoding issues in columns.def

Description Jonathan Druart 2015-09-14 14:02:40 UTC
The columns.def file is not opened using the utf-8 encoding.
Comment 1 Jonathan Druart 2015-09-14 14:04:07 UTC Comment hidden (obsolete)
Comment 2 Josef Moravec 2015-09-16 06:36:58 UTC
The patch works as expected for me, but there is another place to fix: 

https://github.com/Koha-Community/Koha/blob/master/C4/Reports/Guided.pm#L875

And there is used this routine from Reports::Guided

https://github.com/Koha-Community/Koha/blob/master/tools/inventory.pl#L37
https://github.com/Koha-Community/Koha/blob/master/tools/inventory.pl#L374
Comment 3 Jonathan Druart 2015-09-16 08:27:15 UTC Comment hidden (obsolete)
Comment 4 Jonathan Druart 2015-09-16 08:28:00 UTC
Thanks for the catch Josef, I haven't checked this area I thought it called the same subroutine.
Comment 5 Josef Moravec 2015-09-16 10:32:54 UTC Comment hidden (obsolete)
Comment 6 Josef Moravec 2015-09-16 10:32:59 UTC Comment hidden (obsolete)
Comment 7 Katrin Fischer 2015-09-19 06:17:55 UTC
Created attachment 42713 [details] [review]
[PASSED QA] Bug 14817: Fix encoding issues in columns.def

Test plan:
1/ Use a translated template (fr-FR or ar-Arab)
2/ Go on the tools/import_patrons.pl page
3/ The field names in the "default values" block should be correctly
encoded.

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 8 Katrin Fischer 2015-09-19 06:18:16 UTC
Created attachment 42714 [details] [review]
[PASSED QA] Bug 14817: (follow-up) Fix encoding issues in columns.def

Follow-up for reports.

Test plan:
1/ Use a translated template (fr-FR or ar-Arab)
2/ Go on the report guided page, step 3
3/ The field names should be correctly encoded.

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

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 9 Tomás Cohen Arazi 2015-09-25 14:56:17 UTC
Patches pushed to master.

Thanks Jonathan!