Bug 24664

Summary: Add missing *-messages-js.po
Product: Koha Reporter: Bernardo Gonzalez Kriegel <bgkriegel>
Component: I18N/L10NAssignee: Bernardo Gonzalez Kriegel <bgkriegel>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: 1joynelson, f.demians, hayleypelham, julian.maurice, lucas
Version: Main   
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24734
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
20.05.00, 19.11.04, 19.05.09
Bug Depends on:    
Bug Blocks: 24662    
Attachments: Bug 24664: Add missing *-messages-js.po
Bug 24664: Add missing *-messages-js.po
Bug 24664: Add missing *-messages-js.po

Description Bernardo Gonzalez Kriegel 2020-02-14 14:33:28 UTC
We need to add missing [lang]-messages-js.po
Comment 1 Bernardo Gonzalez Kriegel 2020-02-14 14:41:00 UTC
Created attachment 98948 [details] [review]
Bug 24664: Add missing *-messages-js.po

If you try to update some language translation files
translate script will complain of missing *-messajes-js.po
file.

This patch adds those missing files

To test:
1) Go to misc translation
2) Update some language, eg- fr-FR
Check error
/usr/bin/msgmerge: error while opening "/.../misc/translator/po/es-ES-messages-js.po" for reading: No such file or directory
3) Apply this patch
4) Repeat 2), no error message
Comment 2 Julian Maurice 2020-02-14 14:50:27 UTC
Created attachment 98956 [details] [review]
Bug 24664: Add missing *-messages-js.po

If you try to update some language translation files
translate script will complain of missing *-messajes-js.po
file.

This patch adds those missing files

To test:
1) Go to misc translation
2) Update some language, eg- fr-FR
Check error
/usr/bin/msgmerge: error while opening "/.../misc/translator/po/es-ES-messages-js.po" for reading: No such file or directory
3) Apply this patch
4) Repeat 2), no error message

Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
Comment 3 Katrin Fischer 2020-02-29 21:23:49 UTC
Created attachment 99834 [details] [review]
Bug 24664: Add missing *-messages-js.po

If you try to update some language translation files
translate script will complain of missing *-messajes-js.po
file.

This patch adds those missing files

To test:
1) Go to misc translation
2) Update some language, eg- fr-FR
Check error
/usr/bin/msgmerge: error while opening "/.../misc/translator/po/es-ES-messages-js.po" for reading: No such file or directory
3) Apply this patch
4) Repeat 2), no error message

Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 4 Katrin Fischer 2020-02-29 21:24:33 UTC
Wondering if we can improve the license text a bit here - I assume it's from Pootle or po standard?
Comment 5 Bernardo Gonzalez Kriegel 2020-02-29 23:22:39 UTC
(In reply to Katrin Fischer from comment #4)
> Wondering if we can improve the license text a bit here - I assume it's from
> Pootle or po standard?

It's from 'xgettext' command, used to create the files.
It can be changed of course, after the file is created.
Other po files does not contain initial comments, an option is to eliminate them.
But I suppose that all files must contain an initial (correct) notice.

In other bug :)
Comment 6 Martin Renvoize 2020-03-02 10:55:07 UTC
Nice work everyone!

Pushed to master for 20.05
Comment 7 Joy Nelson 2020-03-12 23:46:31 UTC
Pushed to 19.11.x branch for 19.11.04
Comment 8 Lucas Gass 2020-03-18 16:37:02 UTC
backported to 19.05.x for 19.05.09
Comment 9 Hayley Pelham 2020-03-19 02:39:37 UTC
I cannot replicate the error message, so not backporting.
If you think this needs to be backported to 18.11.x please let me know and provide steps to replicate