Bug 11973

Summary: Fix Armenian language description
Product: Koha Reporter: Bernardo Gonzalez Kriegel <bgkriegel>
Component: I18N/L10NAssignee: Bugs List <koha-bugs>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: minor    
Priority: P5 - low CC: f.demians, fridolin.somers, gmcharlt, h.meissner.82, katrin.fischer
Version: Main   
Hardware: All   
OS: All   
See Also: http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9056
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11568
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11393
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 11973 - Fix Armenian language description
Bug 11973 - Fix Armenian language description
Bug 11973 - Fix Armenian language description
[PASSED QA] Bug 11973 - Fix Armenian language description

Description Bernardo Gonzalez Kriegel 2014-03-20 11:42:42 UTC
Armenian language description (in armenian) 
should be Հայերեն instead of Հայերէն  (sixth letter)
Comment 1 Bernardo Gonzalez Kriegel 2014-03-20 12:37:40 UTC Comment hidden (obsolete)
Comment 2 Bernardo Gonzalez Kriegel 2014-03-20 19:50:35 UTC Comment hidden (obsolete)
Comment 3 Holger Meißner 2014-03-21 10:42:08 UTC Comment hidden (obsolete)
Comment 4 Holger Meißner 2014-03-21 10:48:30 UTC
Those letters look so similar. I increased the font size and did a screenshot before and after, to be sure. :D
Comment 5 Katrin Fischer 2014-03-22 08:48:27 UTC
This patch fixes the English sql file, but the wrong string still appears in the translated SQL files:

installer/data/mysql/de-DE/mandatory/subtag_registry.sql	
86 VALUES ( 'hy', 'language', 'hy', '&#1344;&#1377;&#1397;&#1381;&#1408;&#1383;&#1398;');
installer/data/mysql/es-ES/mandatory/subtag_registry.sql	
74 VALUES ( 'hy', 'language', 'hy', '&#1344;&#1377;&#1397;&#1381;&#1408;&#1383;&#1398;');
installer/data/mysql/it-IT/necessari/subtag_registry.sql	
76 VALUES ( 'hy', 'language', 'hy', '&#1344;&#1377;&#1397;&#1381;&#1408;&#1383;&#1398;');
installer/data/mysql/nb-NO/1-Obligatorisk/subtag_registry.sql	
95 VALUES ( 'hy', 'language', 'hy', '&#1344;&#1377;&#1397;&#1381;&#1408;&#1383;&#1398;');
installer/data/mysql/pl-PL/mandatory/subtag_registry.sql	
74 VALUES ( 'hy', 'language', 'hy', '&#1344;&#1377;&#1397;&#1381;&#1408;&#1383;&#1398;');
installer/data/mysql/ru-RU/mandatory/subtag_registry.sql	
74 VALUES ( 'hy', 'language', 'hy', '&#1344;&#1377;&#1397;&#1381;&#1408;&#1383;&#1398;');
installer/data/mysql/uk-UA/mandatory/subtag_registry.sql	
74 VALUES ( 'hy', 'language', 'hy', '&#1344;&#1377;&#1397;&#1381;&#1408;&#1383;&#1398;');


We really should only maintain the language files once, as they contain all the translations.
Comment 6 Katrin Fischer 2014-03-22 08:58:32 UTC
Created attachment 26517 [details] [review]
[PASSED QA] Bug 11973 - Fix Armenian language description

This patch fix Armenian language description

To test:
1) Do a fake translation to Armenian,
   mkdir koha-tmpl/intranet-tmpl/prog/hy-Armn
2) Go to Administration > System preferences > I18N/L10N
3) Armenian description looks like the second string in Bug description
4) Apply the patch
5) Run updatedatabase.pl
6) Reload page, now description look as first string in
Bug description (it's the sixth letter, similar to 't')

Ammended patch, wrong column.

Signed-off-by: Holger Meißner <h.meissner.82@web.de>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Patch fixes the English sample file for translations and includes
a database update to correct the string in existing installations.
Passes all tests and QA script.
Comment 7 Katrin Fischer 2014-03-22 09:01:07 UTC
I have added some other related bugs. My suggestion would be, that we first fix the English file as reference and then either 
- copy the corrected English file for all translated installers
- or better: move it out of the language specific installer directories (like syspref.sql)
Comment 8 Galen Charlton 2014-03-22 12:58:49 UTC
Pushed to master.  Thanks, Bernardo!
Comment 9 Bernardo Gonzalez Kriegel 2014-03-31 00:09:31 UTC
(In reply to Katrin Fischer from comment #7)
> I have added some other related bugs. My suggestion would be, that we first
> fix the English file as reference and then either 
> - copy the corrected English file for all translated installers
> - or better: move it out of the language specific installer directories
> (like syspref.sql)

I did a patch for your last suggestion, changed Installer.pm 
and did a install to test it. It worked

Later I changed my mind and did Bug 12017. I think it's
better to have languages descriptions out of database, 
at last for translations.
Comment 10 Katrin Fischer 2014-03-31 15:54:47 UTC
Hi Bernardo, thx a lot for looking into this. I think one of the reasons it wasn't moved to the templates was that we don't want to translate, but show the native description for languages - not sure how to do that with the translation system. Also in advanced search we show the native and the translated description together.
Comment 11 Bernardo Gonzalez Kriegel 2014-04-02 04:46:39 UTC
Katrin, I did a new version. Now all is native, and advanced search shows translations. All in templates.
Comment 12 Fridolin Somers 2014-06-12 09:22:39 UTC
Pushed to 3.14.x, will be in 3.14.08