Bug 12309

Summary: Add more fields to batch patron modification tool
Product: Koha Reporter: Owen Leonard <oleonard>
Component: ToolsAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: jonathan.druart, pierre.lorgue, sandboxes, tomascohen
Version: Main   
Hardware: All   
OS: All   
See Also: http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15332
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15607
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:
Attachments: Bug 12309 - Add more fields to batch patron modification tool
Bug 12309 - Add more fields to batch patron modification tool
Bug 12309: Adapt zipcode wording to match the other patron pages
Bug 12309 - Add more fields to batch patron modification tool
Bug 12309: Adapt zipcode wording to match the other patron pages

Description Owen Leonard 2014-05-22 13:48:36 UTC
I propose to add city, state, zip, and country to the list of fields which can be modified with the batch patron modification tool.
Comment 1 Owen Leonard 2014-05-22 14:03:41 UTC Comment hidden (obsolete)
Comment 2 pierre.lorgue 2014-06-03 08:00:57 UTC
The batch patron modification tool is correct.
The fields city, state, zip and country have been added.
The functionality is effective with a lot of patrons.
Comment 3 Biblibre Sandboxes 2014-06-03 08:03:50 UTC
Patch tested with a sandbox, by cachan <pierre.lorgue@ens-cachan.fr>
Comment 4 Biblibre Sandboxes 2014-06-03 08:04:09 UTC Comment hidden (obsolete)
Comment 5 Jonathan Druart 2014-06-06 14:38:42 UTC Comment hidden (obsolete)
Comment 6 Jonathan Druart 2014-06-06 14:39:04 UTC
Created attachment 28711 [details] [review]
Bug 12309 - Add more fields to batch patron modification tool

It would be helpful to be able to use the batch patron modification tool
to update city, state, zipcode, and country. This patch adds this
functionality.

This patch corrects an error in the batch modification script which
would have incorrectly marked first name as required if surname was
required.

This patch also converts the table sorting configuration to use table
header class names instead of index-based sorting for columns containing
dates.

To test, apply the patch and submit a batch of patrons to the batch
patron modification tool (Tools -> Batch patron modification).

In the list of patrons which will be modified you should see columns for
city, state, zipcode, and country. Table sorting should work correctly
without errors, including the registration and expiry date columns.

In the list of fields which can be batch modified you should see inputs
for city, state zipcode, and country. These fields should correctly
reflect mandatory/not mandatory based on the BorrowerMandatoryField
system preference.

Submit batch changes to a variety of fields and confirm that changes are
applied correctly. On the page of results the table should correctly
show all fields and be sorted correctly.

Signed-off-by: cachan <pierre.lorgue@ens-cachan.fr>

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Comment 7 Jonathan Druart 2014-06-06 14:39:09 UTC
Created attachment 28712 [details] [review]
Bug 12309: Adapt zipcode wording to match the other patron pages

Replace "Zip code" with "Zip/Postal code" which is used in
members/memberentry.pl and members/moremember.pl.

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Comment 8 Tomás Cohen Arazi (tcohen) 2014-09-01 14:07:49 UTC
Patches pushed to master.

Thanks Owen!