Bug 18791: (follow up) Fix translation of copy messages
authorOwen Leonard <oleonard@myacpl.org>
Tue, 27 Mar 2018 15:09:08 +0000 (15:09 +0000)
committerJonathan Druart <jonathan.druart@bugs.koha-community.org>
Fri, 13 Apr 2018 16:55:22 +0000 (13:55 -0300)
This patch updates DataTables configuration so that more strings are
translatable:

https://datatables.net/extensions/buttons/examples/html5/copyi18n.html

To test, apply the patch and clear your browser cache.

- Open the account tab for a patron who has only one fine.
- Confirm that the new DataTables buttons appear.
- Click the 'Copy' button and confirm that a "Copied 1 row to clipboard"
  message appears.
- Test again with a patron who has multiple fines, confirm that the
  message reads "Copied X rows to clipboard."

To test the translation:

- Confirm that "Copied 1 row to clipboard" is missing from the language
  file you're testing with (misc/translator/po/xx-YY-staff-prog.po).
- Run 'translate update xx-YY'
- Check that the clipboard strings are now in the po file.
- Add translations for those strings.
- Run 'translate update xx-YY' and 'translate install xx-YY'
- Switch to the translated language and clear your browser cache.
- Test the 'Copy' button again. Your translated text should appear.

Signed-off-by: Claire Gravely <claire.gravely@bsz-bw.de>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
koha-tmpl/intranet-tmpl/prog/en/includes/datatables.inc
koha-tmpl/intranet-tmpl/prog/js/datatables.js

index 480cb01..b6cd884 100644 (file)
     var MSG_DT_ZERO_RECORDS = _("No matching records found");
     var MSG_DT_ALL = _("All");
     var CONFIG_EXCLUDE_ARTICLES_FROM_SORT = _("a an the");
+    var MSG_DT_COPY_TITLE = _("Copy to clipboard");
+    var MSG_DT_COPY_KEYS = _("Press ctrl or ⌘ + C to copy the table data<br>to your system clipboard.<br><br>To cancel, click this message or press escape.");
+    var MSG_DT_COPY_SUCCESS_ONE = _("Copied one row to clipboard");
+    var MSG_DT_COPY_SUCCESS_X = _("Copied %d rows to clipboard");
 //]]>
 </script>
 [% Asset.js("js/datatables.js") %]
index 99aec83..9c3caf1 100644 (file)
@@ -21,7 +21,15 @@ var dataTablesDefaults = {
         "sLoadingRecords"   : window.MSG_DT_LOADING_RECORDS || "Loading...",
         "sProcessing"       : window.MSG_DT_PROCESSING || "Processing...",
         "sSearch"           : window.MSG_DT_SEARCH || "Search:",
-        "sZeroRecords"      : window.MSG_DT_ZERO_RECORDS || "No matching records found"
+        "sZeroRecords"      : window.MSG_DT_ZERO_RECORDS || "No matching records found",
+        buttons: {
+            "copyTitle"     : window.MSG_DT_COPY_TITLE || "Copy to clipboard",
+            "copyKeys"      : window.MSG_DT_COPY_KEYS || "Press <i>ctrl</i> or <i>⌘</i> + <i>C</i> to copy the table data<br>to your system clipboard.<br><br>To cancel, click this message or press escape.",
+            "copySuccess": {
+                _: window.MSG_DT_COPY_SUCCESS_X || "Copied %d rows to clipboard",
+                1: window.MSG_DT_COPY_SUCCESS_ONE || "Copied one row to clipboard"
+            }
+        }
     },
     "dom": '<"top pager"ilpfB>tr<"bottom pager"ip>',
     "buttons": [],