]> git.koha-community.org Git - koha.git/commit
Bug 10861: Add a check on cardnumber length
authorJonathan Druart <jonathan.druart@biblibre.com>
Fri, 19 Apr 2013 07:58:02 +0000 (09:58 +0200)
committerGalen Charlton <gmc@esilibrary.com>
Wed, 12 Mar 2014 02:16:18 +0000 (02:16 +0000)
commit834abff631a4ea9bc779424d2096551d63dd13bb
tree4890795d85c54d445621d909d72a4786b948ce35
parentfd773f8e798f328292f49969b103892eb7dc6721
Bug 10861: Add a check on cardnumber length

Some libraries would like to add a check on the cardnumber length.
This patch adds the ability to restrict the cardnumber to a specific
length (strictly equal to XX, or length > XX or min < length < max).
This restriction is checked on inserting/updating a patron or on importing
patrons.

This patch adds:
- 1 new syspref CardnumberLength. 2 formats: a number or a range
  (xx,yy).

- 1 new unit test file t/Members/checkcardnumber.t for the
C4::Members::checkcardnumber routine.

Test plan:
1/ Fill the pref CardnumberLength with '5,8'
2/ Create a new patron with an invalid cardnumber (123456789)
3/ Check that you cannot save
4/ With Firebug, replace the pattern attribute value (for the cardnumber
input) with ".{5,10}"
5/ You are allowed to save but an error occurred.
6/ Try the same steps for update.
7/ Go to the import borrowers tool.
8/ Play with the import borrowers tool. We must test add/update patrons
and the "record matching" field (cardnumber or a uniq patron attribute)

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Tested adding, updating; importing and ran unit test.
Preliminary QA comments on Bugzilla

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
C4/Members.pm
installer/data/mysql/updatedatabase.pl
koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/patrons.pref
koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt
koha-tmpl/intranet-tmpl/prog/en/modules/tools/import_borrowers.tt
members/memberentry.pl
t/Members/checkcardnumber.t [new file with mode: 0644]
tools/import_borrowers.pl