]> 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)
committerJulian Maurice <julian.maurice@biblibre.com>
Mon, 9 May 2016 08:35:49 +0000 (10:35 +0200)
commitb53075b58df01e65371e13dee0b6848d12a181f2
tree7b7394bad37b13e65b145f1a72afcd35429a6125
parentea45e3f39ebdd2a33b7ea00730ef278ba0f461a7
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>
C4/Members.pm
koha-tmpl/intranet-tmpl/prog/en/modules/tools/import_borrowers.tt
tools/import_borrowers.pl