Bug 9224: Make acqui/finishreceive.pl Plack-compatible
authorJacek Ablewicz <abl@biblos.pk.edu.pl>
Wed, 16 Oct 2013 15:31:04 +0000 (17:31 +0200)
committerGalen Charlton <gmc@esilibrary.com>
Fri, 13 Dec 2013 18:23:26 +0000 (18:23 +0000)
commit6dcc34c1b44f72a8602c4ee95540836e6cd1e7bd
tree5bc0e47b86c9d3b0f65726ab802a440b34f9b443
parent4927140966844b88ef54dfc4d8e5f39d164ac8f3
Bug 9224: Make acqui/finishreceive.pl Plack-compatible

Under Plack/mod_perl wrapping, sub update_item() will become a closure,
so after the 1st run it will retain its own private instances of the
following variables: $booksellerid, $datereceived, $unitprice, $rrp,
$biblionumber.

I.e., in case update_item() gets invoked 2nd+ time (inside
the same process, but for different-subsequent receives) it may
incorrectly flag the (old, wrong) biblionumber for Zebra reindexing,
and erronously modify the current item[s] with the previously
used (wrong) values.

This simple patch should make acqui/finishreceive.pl Plack-compatible.

Test plan:
Test patched acqui/finishreceive.pl script (create and receive some
orders w/ items, etc.). Ensure items are gettting added and/or modified
correctly during receiving process.

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Passes koha-qa.pl, works as advertised, no regressions found.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
acqui/finishreceive.pl