From 5782da035f736765dceced4b2db9699b30e22017 Mon Sep 17 00:00:00 2001
From: Katrin Fischer <Katrin.Fischer.83@web.de>
Date: Sun, 22 Nov 2015 02:04:12 +0100
Subject: [PATCH]  Bug 15228: Patron card batches - Improve translatability

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>
---
 .../prog/en/modules/patroncards/edit-batch.tt             | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/patroncards/edit-batch.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/patroncards/edit-batch.tt
index ab5bbecb35..c45dabc8da 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/patroncards/edit-batch.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/patroncards/edit-batch.tt
@@ -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 {
@@ -179,7 +179,7 @@
 
                 [% 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 %]
 
@@ -256,8 +256,8 @@
                                 <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>
-- 
2.39.5