Bug 14631 - Change to "Patron" in patroncard toolbar
Summary: Change to "Patron" in patroncard toolbar
Status: CLOSED INVALID
Alias: None
Product: Koha
Classification: Unclassified
Component: Label/patron card printing (show other bugs)
Version: master
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Indranil Das Gupta
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-08-03 00:35 UTC by Indranil Das Gupta
Modified: 2016-12-05 21:23 UTC (History)
6 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Trivial patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
patron(s) described as item(s) (27.77 KB, image/png)
2015-08-03 00:39 UTC, Indranil Das Gupta
Details
Bug 14631 - Change item(s) to patron(s) in toolbar (1.56 KB, patch)
2015-08-03 01:15 UTC, Indranil Das Gupta
Details | Diff | Splinter Review
Bug 14631: More changes from 'items' to 'patrons' (4.53 KB, patch)
2015-12-21 20:58 UTC, Aleisha Amohia
Details | Diff | Splinter Review
Bug 14631: Translatability fix (1.06 KB, patch)
2015-12-22 19:49 UTC, Aleisha Amohia
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Indranil Das Gupta 2015-08-03 00:35:14 UTC
The toolbar for "Manage patron card batches" template shows "item(s)" instead of "patron(s)". This patch proposes to fix it.
Comment 1 Indranil Das Gupta 2015-08-03 00:39:12 UTC
Created attachment 41305 [details]
patron(s) described as item(s)
Comment 2 Indranil Das Gupta 2015-08-03 01:15:38 UTC
Created attachment 41306 [details] [review]
Bug 14631 - Change item(s) to patron(s) in toolbar

The toolbar for "Manage patron card batches" template shows "item(s)"
instead of "patron(s)". This patch proposes to fix it.

Test plan
=========

1/ Go to Patron card creator > Manage patron card batches notice.
2/ Select a batch to edit, you should see the toolbar as described
   in attachment 41305 [details]
3/ Apply patch and refresh page. The toolbar should read correctly
   as 'patron(s)'
4/ Go back to step #1 above and select "New batch" and let the page
   automatically refresh. The toolbar button presented should read
   "Add patron(s)"
Comment 3 Aleisha Amohia 2015-08-03 04:38:47 UTC
The patch works fine, but just wanted to add a few things which still should be changed from item(s) to patron(s):

- title of 'Items in Batch number x' (which can be seen in your screenshot)
- on the main Manage batches page where it gives a list of Currently Available Batches, there is a column title 'Item Count'
- when adding a new batch, there is a message saying "There are no items in batch 0 yet, Use the toolbar above to add items."

So in my opinion, four more instances where it should say patrons. Will set to Failed QA so you can submit a new patch :)
Comment 4 Liz Rea 2015-08-14 01:56:07 UTC
I noticed this while working on bug 14676, and was going to tidy it up as part of that. I'll use your stuff if you beat me to it though. :)
Comment 5 Marc Véron 2015-09-10 13:20:51 UTC
I hope it's OK to change module to Label/patroncard printing
Comment 6 Tom Misilo 2015-09-16 18:54:52 UTC
Would this make sense to change the text in the file instead of using javascript to do it? If so I can try and make a new patch that does that.
Comment 7 Liz Rea 2015-09-16 21:08:15 UTC
Yeah, I think we should not do this with javascript.

The right place would be to change the labels where they are generated, or have a look at jonathan's translation fixes for the label creator, which makes things more translatable: http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=41625&action=diff
Comment 8 Liz Rea 2015-09-16 21:09:39 UTC
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=41626&action=diff sorry I meant this one.
Comment 9 Bernardo Gonzalez Kriegel 2015-12-21 15:00:53 UTC
On current master this has been fixed.
We have "Add patron(s)  Remove selected patrons ...  Export selected card(s)  Export card batch"

There are other strings that need attention:

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"));

<strong>[% duplicate_count %] duplicate item(s) found</strong> and removed from batch [% batch_id %].

<h2>Items in batch number [% batch_id %]</h2>
<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>
Comment 10 Aleisha Amohia 2015-12-21 20:58:40 UTC
Created attachment 45889 [details] [review]
Bug 14631: More changes from 'items' to 'patrons'

Some of the changes Bernardo noticed were missed
Comment 11 Marc Véron 2015-12-22 07:48:37 UTC
A small translatability woe (sentence splitting by tags):

<strong>[% duplicate_count %] duplicate patron(s) found</strong> and removed from batch [% batch_id %].

Maybe something like:

<strong>Duplicate patron(s) removed from batch [% batch_id %]:</strong> [% duplicate_count %]

or:
<strong>Duplicates removed from batch [% batch_id %]:</strong> [% duplicate_count %] patron(s)

or even:
<strong>Duplicates removed from batch [% batch_id %]:</strong> [% duplicate_count %]
Comment 12 Aleisha Amohia 2015-12-22 19:49:47 UTC
Created attachment 45919 [details] [review]
Bug 14631: Translatability fix
Comment 13 Danielle Elder 2016-01-13 22:06:18 UTC
Comment on attachment 41306 [details] [review]
Bug 14631 - Change item(s) to patron(s) in toolbar

 Did not apply.
Comment 14 Liz Rea 2016-01-13 22:13:30 UTC
Hi, 

I'm not sure most of this is even applicable, as the patron card creator has had a UI overhaul since this bug was started.

It would pay to go through and make sure we didn't miss any items -> patrons conversion as part of that work, but this may not be relevant anymore.

Liz
Comment 15 Liz Rea 2016-01-13 22:14:09 UTC
Bug 14676 is the one that did most of that.
Comment 16 Liz Rea 2016-01-13 23:14:25 UTC
Aleisha has opened bug 15573 for the remaining things to be fixed here. As the UI is totally changed, nothing here is of importance anymore.

Liz