Bug 15010: Import patron tool creates 'duplicate' restrictions ( debarments )
authorKyle M Hall <kyle@bywatersolutions.com>
Wed, 14 Oct 2015 12:48:30 +0000 (08:48 -0400)
committerTomas Cohen Arazi <tomascohen@theke.io>
Fri, 30 Oct 2015 13:00:20 +0000 (10:00 -0300)
commitfd123903651435c56ca14d88897e9ce4d7a38bb5
treefc69d81860c391ec2421af532d13420e74e925ba
parentb44ac38ce65e1c8a3bf016acf014ed3bd2cae4bb
Bug 15010: Import patron tool creates 'duplicate' restrictions ( debarments )

If a patron has restrictions and a CSV file of patrons to be overwritten
is uploaded but has no debarred / debarredcomment columns, the a new
restriction will be added that combines all the existing restrictions (
i.e. a new debarment based on the existing borrowers.debarred
and borrowers.debarredcomment fields ).

Test Plan:
1) Create a patron with one or more restrictions
2) Import a CSV file that updates that patron, but with no debarred columns
3) Note a new restrictions is created
4) Delete the new restriction
5) Apply this patch
6) Re-import the CSV file again
7) Note that no new restriction is created
8) Add debarred and debarredcomment columns to the CSV file with a test
   date and description
9) Re-import the CSV file again
10) Note the new restriction is added to the patron

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Daniel Grobani <dgrobani@samuelmerritt.edu>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
tools/import_borrowers.pl