Bug 15840: Catch errors if userid already exists when importing patrons
commit57d231f9cbc9f2c02c0d85289cf0878d608ed31c
authorJonathan Druart <jonathan.druart@bugs.koha-community.org>
Wed, 17 Feb 2016 16:21:50 +0000 (17 16:21 +0000)
committerFrédéric Demians <f.demians@tamil.fr>
Wed, 23 Mar 2016 12:47:42 +0000 (23 13:47 +0100)
treea149f01723b6587c5f5a8b5112eb14c0eed4eec6
parentf9a7f31eb0152db9cb6d06286fd99d82cc503a15
Bug 15840: Catch errors if userid already exists when importing patrons

Import borrowers tool explodes if userid already exists. This error
should be caught to avoid the script to explodes.

TEST PLAN
---------
1) back up DB
2) drop DB, create empty DB
3) Apply patch 15842
4) web install all the sample data
5) login as system user and import patron data
   -- sample provided to trigger error.
   -- you will get a DBIx::class exception
      "Duplicate entry ' for key 'userid'"
6) Apply this patch
7) Repeat steps 2-5
   -- patrons won't be inserted but an error
      will be nicely displayed in the error list.
8) run koha qa test tools

Note that it would be easy to insert the patrons anyway and generate a
userid based on the surname and firstname.

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Brendan Gallagher brendan@bywatersolutions.com
(cherry picked from commit f1e156453a1bc636914bc5c969e1665d7108cb95)
Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
(cherry picked from commit 8d0a57f5241a911d7e95a24ddb01acd824451c99)
Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
koha-tmpl/intranet-tmpl/prog/en/modules/tools/import_borrowers.tt
tools/import_borrowers.pl