]> git.koha-community.org Git - koha.git/commit
Bug 16426: follow-up of bug 15840 - correctly manage userid while inserting patrons
authorJonathan Druart <jonathan.druart@bugs.koha-community.org>
Tue, 3 May 2016 07:58:33 +0000 (08:58 +0100)
committerChris Cormack <chrisc@catalyst.net.nz>
Sun, 19 Jun 2016 20:55:37 +0000 (08:55 +1200)
commit9aea6d98ab95b0b5d7b8d97f5d726f30737a6483
tree445652588beba5720cdf6b968a5851e219333faf
parentc7a9da21172e2cb1af90a914109870c85515c1b4
Bug 16426: follow-up of bug 15840 - correctly manage userid while inserting patrons

Bug 15840 tried to fix a bug but makes things more complicated than it
was before.
If an userid is not provided for 1 or more rows of the csv file, it
should not be updated. However, if a userid is provided and it already
used by an other patron, the import should fail for this row (but not
crash!).

Test plan:
0/ Create a patron with a userid=your_userid
1/ Use the import patron tool to update this userid
=> userid should have been updated
2/ Update another data and do not provide the userid
=> data should have been updated and not the userid
3/ Update another data and provide the userid, but set it to an empty
string, or '0'
=> data should have been updated and not the userid
4/ Update another patron, and set userid=your_userid
=> Update should fail and an error whouls be displayed ("already used by
another patron")

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Brendan Gallagher <brendan@bywatersolutions.com>
(cherry picked from commit 7b76b24fad305b0253eb1d779f074d265087ca73)
Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
(cherry picked from commit b53075b58df01e65371e13dee0b6848d12a181f2)
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
C4/Members.pm
koha-tmpl/intranet-tmpl/prog/en/modules/tools/import_borrowers.tt
tools/import_borrowers.pl