Bug 13763: Added check for filter syspref in renew.pl
authorHayley Mapley <hayleymapley@catalyst.net.nz>
Thu, 14 Mar 2019 04:58:31 +0000 (04:58 +0000)
committerLucas Gass <lucas@bywatersolutions.com>
Mon, 1 Apr 2019 15:35:18 +0000 (15:35 +0000)
commit66550615441e84a00d5252018697baca20c5bf11
tree504224e1d4e748e1559099e1e8404d8d62aa235c
parentc15a41bcddcd052d3e8f79af35cb2c4118e567a8
Bug 13763: Added check for filter syspref in renew.pl

In Circulation->Renew, the renew feature does not check the system
preference itemBarcodeFilter to see whether the barcode needs to be
converted. This patch fixes this, adding a check for the system pref and
applying it accordingly to convert the barcode if it is needed.

Test plan:
1) In Circulation->Renew, enter a barcode that you know exists in the
catalogue but add extra spaces, breaking up the barcode. Note that this
fails with a warning that says 'No item matches this barcode'

2) In system Preferences, set the itemBarcodeInputFilter pref to Remove
spaces and repeat step1. Note that the result is the same as in step 1

3) Apply the patch

4) Repeat step 1, and note that the filter removes the spaces and the
item is renewed (or gives an error that indicates it recognises the
barcode)

Sponsored-by: Catalyst IT
Signed-off-by: Bin Wen <bin.wen@inlibro.com>

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
(cherry picked from commit 48e9082043115416063e66217db60f0054b68088)
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
(cherry picked from commit 4505ad4dc52c5384866c157c08ba6a5a6c95a619)

Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
circ/renew.pl