[Bug 26237] New: Move translatable strings out of preferences.tt and into JavaScript files
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26237 Bug ID: 26237 Summary: Move translatable strings out of preferences.tt and into JavaScript files Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: ASSIGNED Severity: enhancement Priority: P5 - low Component: I18N/L10N Assignee: oleonard@myacpl.org Reporter: oleonard@myacpl.org QA Contact: testopia@bugs.koha-community.org CC: frederic@tamil.fr, testopia@bugs.koha-community.org Depends on: 21156 Strings defined for translation in admin/preferences.tt can now be wrapped in the new translation function and put in the corresponding JavaScript files, ajax.js or preferences.js. Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21156 [Bug 21156] Internationalization: plural forms, context, and more for JS files -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26237 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch complexity|--- |Small patch Status|ASSIGNED |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26237 --- Comment #1 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 108527 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=108527&action=edit Bug 26237: Move translatable strings out of preferences.tt and into JavaScript files This patch removes the definition of translatable strings out of templates and into the corresponding JavaScript file, using the new JS i81n function. To test (a non-comprehensive list): - Apply the patch, go to Administration -> System preference - Modify any system preference. "modified" should appear next to the preference name. - Click the "Save" button. You should get a message, "Saved preference <pref name>" - Hover your mouse over a section heading (e.g. "Features" under Accounting). You should see a tooltip, "Click to collapse this section." - Click to collapse the section. The tooltip should change to "Click to expand the section." - Open a new tab for the staff interface and log out. - In the original tab, modify and try to save a system preference. You should get a message, "Error; your data might not have been saved." TESTING TRANSLATABILITY - Update a translation, e.g. fr-FR:
cd misc/translator perl translate update fr-FR
- Open the corresponding .po file for JavaScript strings, e.g. misc/translator/po/fr-FR-messages-js.po - Locate strings pulled from koha-tmpl/intranet-tmpl/prog/js/pages/preferences.js for translation, e.g.: msgid "Saved preference %s" msgstr "" - Edit the "msgstr" string however you want (it's just for testing). - Install the updated translation:
perl translate install fr-FR
- Switch to your newly translated language in the staff client and repeat the test plan above. The translated strings should appear. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26237 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26237 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #108527|0 |1 is obsolete| | --- Comment #2 from David Nind <david@davidnind.com> --- Created attachment 110780 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=110780&action=edit Bug 26237: Move translatable strings out of preferences.tt and into JavaScript files This patch removes the definition of translatable strings out of templates and into the corresponding JavaScript file, using the new JS i81n function. To test (a non-comprehensive list): - Apply the patch, go to Administration -> System preference - Modify any system preference. "modified" should appear next to the preference name. - Click the "Save" button. You should get a message, "Saved preference <pref name>" - Hover your mouse over a section heading (e.g. "Features" under Accounting). You should see a tooltip, "Click to collapse this section." - Click to collapse the section. The tooltip should change to "Click to expand the section." - Open a new tab for the staff interface and log out. - In the original tab, modify and try to save a system preference. You should get a message, "Error; your data might not have been saved." TESTING TRANSLATABILITY - Update a translation, e.g. fr-FR:
cd misc/translator perl translate update fr-FR
- Open the corresponding .po file for JavaScript strings, e.g. misc/translator/po/fr-FR-messages-js.po - Locate strings pulled from koha-tmpl/intranet-tmpl/prog/js/pages/preferences.js for translation, e.g.: msgid "Saved preference %s" msgstr "" - Edit the "msgstr" string however you want (it's just for testing). - Install the updated translation:
perl translate install fr-FR
- Switch to your newly translated language in the staff client and repeat the test plan above. The translated strings should appear. Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26237 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26237 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #110780|0 |1 is obsolete| | --- Comment #3 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 110788 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=110788&action=edit Bug 26237: Move translatable strings out of preferences.tt and into JavaScript files This patch removes the definition of translatable strings out of templates and into the corresponding JavaScript file, using the new JS i81n function. To test (a non-comprehensive list): - Apply the patch, go to Administration -> System preference - Modify any system preference. "modified" should appear next to the preference name. - Click the "Save" button. You should get a message, "Saved preference <pref name>" - Hover your mouse over a section heading (e.g. "Features" under Accounting). You should see a tooltip, "Click to collapse this section." - Click to collapse the section. The tooltip should change to "Click to expand the section." - Open a new tab for the staff interface and log out. - In the original tab, modify and try to save a system preference. You should get a message, "Error; your data might not have been saved." TESTING TRANSLATABILITY - Update a translation, e.g. fr-FR:
cd misc/translator perl translate update fr-FR
- Open the corresponding .po file for JavaScript strings, e.g. misc/translator/po/fr-FR-messages-js.po - Locate strings pulled from koha-tmpl/intranet-tmpl/prog/js/pages/preferences.js for translation, e.g.: msgid "Saved preference %s" msgstr "" - Edit the "msgstr" string however you want (it's just for testing). - Install the updated translation:
perl translate install fr-FR
- Switch to your newly translated language in the staff client and repeat the test plan above. The translated strings should appear. Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26237 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to master Version(s)| |20.11.00 released in| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26237 --- Comment #4 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Pushed to master for 20.11, thanks to everybody involved! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26237 Lucas Gass <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |lucas@bywatersolutions.com --- Comment #5 from Lucas Gass <lucas@bywatersolutions.com> --- patch doesnt apply cleanly to 20.05.x, no backport please rebase if needed -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org