[Koha-bugs] [Bug 9224] acqui/finishreceive.pl is not Plack-compatible
bugzilla-daemon at bugs.koha-community.org
bugzilla-daemon at bugs.koha-community.org
Fri Dec 13 19:13:58 CET 2013
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9224
Kyle M Hall <kyle at bywatersolutions.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #23216|0 |1
is obsolete| |
--- Comment #5 from Kyle M Hall <kyle at bywatersolutions.com> ---
Created attachment 23533
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=23533&action=edit
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 it's 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 at biblibre.com>
Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
Passes koha-qa.pl, works as advertised, no regressions found.
--
You are receiving this mail because:
You are watching all bug changes.
More information about the Koha-bugs
mailing list