Bug 19673: Allow to set patron attributes to 0 with batch patron modification
authorJonathan Druart <jonathan.druart@bugs.koha-community.org>
Thu, 21 Dec 2017 00:35:47 +0000 (21:35 -0300)
committerNick Clemens <nick@bywatersolutions.com>
Mon, 14 May 2018 13:43:19 +0000 (13:43 +0000)
commit16dfb0309b778725fdb76cadee9943a0344285ed
treebfa6d280624f3a0a7d6fc4a262dfacc87d590b7e
parent4fcddf80d559e2d7feeaa97c4bb114c8d2876ebd
Bug 19673: Allow to set patron attributes to 0 with batch patron modification

This patch will have to be tested deeply to make sure it will not
introduce regression!

The idea is to display an empty option in the patron attributes select
and ignore it. That way we can deal with false values 0 and "" which
were skipped before.

Test plan:
Add several patron attributes
Use the batch patron modification tool to add/update/remove them
Play with empty "" and 0 values, as well as other values
Modify several attributes in a row

Signed-off-by: Charles Farmer <charles.farmer@inLibro.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Conflicts:
koha-tmpl/intranet-tmpl/prog/en/modules/tools/modborrowers.tt
koha-tmpl/intranet-tmpl/prog/en/modules/tools/modborrowers.tt
tools/modborrowers.pl