Strings defined for translation in admin/sms_providers.tt can now be wrapped in the new translation function and put in sms_providers.js.
Created attachment 108532 [details] [review] Bug 26240: Move translatable strings out of sms_providers.tt and into sms_providers.js 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 you must have the SMSSendDriver preference populated ("Email" is fine). - Apply the patch and go to Administration -> SMS cellular providers. - Click "New SMS provider." - The legend on the form's fieldset should read "Add an SMS cellular provider." - Add an SMS provider. - Edit an SMS provider. - The legend on the form's fieldset should read "Edit provider <provider name>" - If necessary, edit a patron's SMS settings to use one of your existing SMS providers. - From the list of SMS providers, click to delete the provider which is in use. - The error message should read "Are you sure you want to delete <provider name>? <number> patron(s) are using it!" - Click to delete a provider which isn't in use. The error message should read "Are you sure you want to delete <provider>?" 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/sms_providers.js for translation, e.g.: msgid "Add an SMS cellular provider" 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.
Created attachment 110781 [details] [review] Bug 26240: Move translatable strings out of sms_providers.tt and into sms_providers.js 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 you must have the SMSSendDriver preference populated ("Email" is fine). - Apply the patch and go to Administration -> SMS cellular providers. - Click "New SMS provider." - The legend on the form's fieldset should read "Add an SMS cellular provider." - Add an SMS provider. - Edit an SMS provider. - The legend on the form's fieldset should read "Edit provider <provider name>" - If necessary, edit a patron's SMS settings to use one of your existing SMS providers. - From the list of SMS providers, click to delete the provider which is in use. - The error message should read "Are you sure you want to delete <provider name>? <number> patron(s) are using it!" - Click to delete a provider which isn't in use. The error message should read "Are you sure you want to delete <provider>?" 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/sms_providers.js for translation, e.g.: msgid "Add an SMS cellular provider" 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>
Note that if you: 1. Add an SMS celluar provider (shows 0 patrons). 2. Open another window and add a patron to the SMS provider. 3. Delete an SMS celluar provider in the window used for step 1 without refreshing the window (it shows 0 patrons). 4. The delete message doesn't add '<number> patron(s) are using it!' to the delete provider message. Probably an edge case... (happens both before and after patch applied).
Created attachment 110787 [details] [review] Bug 26240: Move translatable strings out of sms_providers.tt and into sms_providers.js 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 you must have the SMSSendDriver preference populated ("Email" is fine). - Apply the patch and go to Administration -> SMS cellular providers. - Click "New SMS provider." - The legend on the form's fieldset should read "Add an SMS cellular provider." - Add an SMS provider. - Edit an SMS provider. - The legend on the form's fieldset should read "Edit provider <provider name>" - If necessary, edit a patron's SMS settings to use one of your existing SMS providers. - From the list of SMS providers, click to delete the provider which is in use. - The error message should read "Are you sure you want to delete <provider name>? <number> patron(s) are using it!" - Click to delete a provider which isn't in use. The error message should read "Are you sure you want to delete <provider>?" 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/sms_providers.js for translation, e.g.: msgid "Add an SMS cellular provider" 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>
Pushed to master for 20.11, thanks to everybody involved!
backported to 20.05.x for 20.05.05
enhancement, not backported to 19.11.x