Bug 15228: Patron card batches - Improve translatability
authorKatrin Fischer <Katrin.Fischer.83@web.de>
Sun, 22 Nov 2015 01:04:12 +0000 (02:04 +0100)
committerBrendan A Gallagher <brendan@bywatersolutions.com>
Wed, 30 Dec 2015 02:20:42 +0000 (02:20 +0000)
This patch removes the \n\n from a translatable string
as those don't translate correctly to new lines.

It also switches 'item' for 'patron' to avoid confusion
with catalog items.

To test:
1) Trigger changed error message
- Create a new batch with 1 patron entry
- Select patron in list using the checkbox
- Click 'Remove selected patrons' in toolbar
- Verify alert displays correctly

2) Verify that the page no longer uses "items" but
   patrons instead.

Signed-off-by: Hector Castro <hector.hecaxmmx@gmail.com>
Works as advertised

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Brendan A Gallagher <brendan@bywatersolutions.com>
koha-tmpl/intranet-tmpl/prog/en/modules/patroncards/edit-batch.tt

index ab5bbec..c45dabc 100644 (file)
@@ -32,7 +32,7 @@
                     item_msg = item_num.join(", ");
                     var msg = _("Are you sure you want to remove card number(s): %s from this batch?").format(item_msg);
                 } else if (document.items.action.checked) {
-                    alert(_("Deletion of item from a batch with only one item will delete the batch.\n\nIf this is what you want, select 'Delete batch' option from toolbar"));
+                    alert(_("Deletion of patron from a batch with only one patron will delete the batch.") + "\n\n" + _("If this is what you want, select the 'Delete batch' option from the toolbar"));
                     return; // no deletion for single item batch
                 }
                 else {
 
                 [% IF ( duplicate_message ) %]
                     <div class="dialog message">
-                        <strong>[% duplicate_count %] duplicate item(s) found</strong> and removed from batch [% batch_id %].
+                        <strong>[% duplicate_count %] duplicate patron(s) found</strong> and removed from batch [% batch_id %].
                     </div>
                 [% END %]
 
                                 <fieldset class="rows" style="border-bottom: 0px; border: 0px;">
                                 <ol><li>
                                     <div class="dialog message">
-                                        <h4>There are no items in this batch yet</h4>
-                                        <p>Add items by borrowernumber using the text area above or leave empty to add via patron search.</p>
+                                        <h4>There are no patrons in this batch yet</h4>
+                                        <p>Add patrons by borrowernumber using the text area above or leave empty to add via patron search.</p>
                                     </div>
                                 </li></ol>
                                 </fieldset>