From 8bf7c342bfaf7425f17427f6e82167f26c2311f8 Mon Sep 17 00:00:00 2001 From: Owen Leonard Date: Fri, 4 Sep 2020 19:10:50 +0000 Subject: [PATCH] Bug 26395: Move translatable strings out of letter.tt into letter.js This patch moves strings defined for translation in letter.tt into letter.js for translation using the new double-underscore i81n function. Note, the "MSG_NO_NOTICE_FOUND" usage has been removed because it didn't work. To test, apply the patch and go to Tools -> Notices & Slips. - Use the "Copy" dropdown to copy an existing notice to a specific library. - Save the copied notice. - Return to the "All libraries" view on the Notices & Slips page. - Repeat the same process above: Copy the same notice to the same library. - When you click save on the "Add notice" page you should get an error message, "A letter with the code 'ACCTDETAILS' already exists for " - From the "All libraries" view on the Notices & Slips page, copy the text in the "Code" column of an existing notice. - Click "New notice" and choose the module of the notice you copied from. - On the "Add notice" page, paste the code into the "Code" field and enter a name in the "Name" field. - Click "Save" without making any other changes. You should get an error message, "Please fill at least one template." - Enter some data into one of the message body fields but not message subject, and click "Save" again. - You should get an error, "Please specify title and content for " - Enter data in the message subject field and click "Save" again. - You should get an error, "A default letter with the code already exists." 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/letter.js for translation, e.g.: msgid "Please fill at least one template." 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 Signed-off-by: Katrin Fischer Signed-off-by: Jonathan Druart --- .../intranet-tmpl/prog/en/modules/tools/letter.tt | 6 ------ koha-tmpl/intranet-tmpl/prog/js/letter.js | 13 +++++-------- 2 files changed, 5 insertions(+), 14 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/letter.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/letter.tt index 29e89152cc..e51e719aba 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/letter.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/letter.tt @@ -552,12 +552,6 @@ var code = '[% code | html %]'; var new_lettercode = '[% new_lettercode | html %]'; var new_branchcode = '[% new_branchcode | html %]'; - var MSG_CODE_EXISTS = _("A default letter with the code '%s' already exists."); - var MSG_CODE_EXISTS_FOR_LIBRARY = _("A letter with the code '%s' already exists for '%s'."); - var MSG_EMPTY_TITLE_AND_CONTENT = _("Please specify title and content for %s"); - var MSG_EMPTY_TEMPLATES = _("Please fill at least one template."); - var MSG_LOADING = _("Loading"); - var MSG_NO_NOTICE_FOUND = _("No matching notices found"); var columns_settings = [% TablesSettings.GetColumns( 'tools', 'notices', 'lettert', 'json' ) | $raw %]; [% Asset.js("js/letter.js") | $raw %] diff --git a/koha-tmpl/intranet-tmpl/prog/js/letter.js b/koha-tmpl/intranet-tmpl/prog/js/letter.js index 7f4cb2ef6d..c2e353bfe8 100644 --- a/koha-tmpl/intranet-tmpl/prog/js/letter.js +++ b/koha-tmpl/intranet-tmpl/prog/js/letter.js @@ -1,5 +1,5 @@ /* Variables defined in letter.tt: */ -/* global _ module add_form copy_form no_op_set MSG_EMPTY_TITLE_AND_CONTENT MSG_EMPTY_TEMPLATES code MSG_CODE_EXISTS MSG_CODE_EXISTS_FOR_LIBRARY MSG_NO_NOTICE_FOUND interface theme KohaTable columns_settings */ +/* global _ module add_form copy_form no_op_set code interface theme KohaTable columns_settings */ var modal_loading = "
\"\" "+ __('Loading...') +"
"; @@ -18,9 +18,9 @@ function checkCodes( new_lettercode, new_branchcode ){ success: function (data) { if ( data.letters.length > 0 ) { if( new_branchcode === '' ) { - alert( MSG_CODE_EXISTS.format(new_lettercode)); + alert(__("A default letter with the code '%s' already exists.").format(new_lettercode)); } else { - alert( MSG_CODE_EXISTS_FOR_LIBRARY.format(new_lettercode, new_branchcode) ); + alert(__("A letter with the code '%s' already exists for '%s'.").format(new_lettercode, new_branchcode)); } $(".spinner").hide(); } else { @@ -49,9 +49,6 @@ $(document).ready(function() { { "bSortable": false, "aTargets": [ "nosort" ] }, { "sType": "title-string", "aTargets" : [ "title-string"] } ], - "oLanguage": { - "sZeroRecords": MSG_NO_NOTICE_FOUND - }, "exportColumns": [0,1,2,3], }, columns_settings); @@ -92,7 +89,7 @@ $(document).ready(function() { ) { var mtt = $(this).find('input[name="message_transport_type"]').val(); at_least_one_exists = 1; // Only one template has to be filled in for form to be valid - alert( MSG_EMPTY_TITLE_AND_CONTENT.format( mtt ) ); + alert(__("Please specify title and content for %s").format(mtt)); are_valid = 0; } else if ( title.length > 0 && content.length > 0 ) { at_least_one_exists = 1; @@ -101,7 +98,7 @@ $(document).ready(function() { if ( ! at_least_one_exists ) { // No templates were filled out - alert( MSG_EMPTY_TEMPLATES ); + alert( __("Please fill at least one template.") ); return false; } -- 2.39.5