Bug 14015 - Checkout: Fix software error if barcode '0' is given
authorMarc Véron <veron@veron.ch>
Mon, 28 Dec 2015 11:15:18 +0000 (12:15 +0100)
committerBrendan A Gallagher <brendan@bywatersolutions.com>
Wed, 27 Jan 2016 02:20:41 +0000 (02:20 +0000)
commit9afc8e96f3d173a3998cbec2a4ad7ceaf506e49a
tree754f9ec510e35fefc6e3a6532e09f94ffa6fb752
parent581759e985c170db0edb4a895cda641930e5ac11
Bug 14015 - Checkout: Fix software error if barcode '0' is given

To reproduce the bug:
- Go to Home > Circulation > Checkouts > [a patron]
- Enter 0 (zero) in barcode field and hit Enter

Result:
Software error
  Can't use string ("1") as a HASH ref while "strict refs" in use
  at /usr/share/kohaclone/circ/circulation.pl line 473

To test:
- Apply patch
- Repeat steps above
- Verify that the sofware error is gone and that you get a
  message as with other invalid barcodes.
- Test with other values: '00', '000', existing barcode,
  and a barcode that does not exist
- Test with empty barcode (for printing slip)
- Search for regressions

This patch implements Jonathan's solution (see comment #15)

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
circ/circulation.pl