From 58cf7fe4372ff86f0f3eb1217c7d0db152aa4e2c Mon Sep 17 00:00:00 2001 From: Lari Taskula Date: Wed, 2 Nov 2016 14:51:23 +0200 Subject: [PATCH] Bug 17539: Fix Undefined subroutine &C4::Circulation::GetItem error perl t/db_dependent/Reserves.t 1..72 ok 1 - use C4::Reserves; ok 2 - CheckReserves Test 1 ok 3 - CheckReserves() include reserve_id in its response ok 4 - CheckReserves Test 2 ok 5 - CheckReserves Test 3 ok 6 - GetReservesControlBranch returns item home branch when set to ItemHomeLibrary ok 7 - GetReservesControlBranch returns patron home branch when set to PatronLibrary ok 8 - Item is correctly waiting ok 9 - Item is correctly priority 1 ok 10 - Item is correctly priority 2 ok 11 - GetWaiting got only the waiting reserve ok 12 - GetWaiting got the reserve for the correct borrower Undefined subroutine &C4::Circulation::GetItem called at /home/ubuntu/kohaclone/C4/Circulation.pm line 1853. This patch fixes this error. The solution was to use C4::Reserves before C4::Items. Also due to this edit, change use_ok to require_ok --- t/db_dependent/Reserves.t | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/t/db_dependent/Reserves.t b/t/db_dependent/Reserves.t index 39ee8c5..1e9a067 100755 --- a/t/db_dependent/Reserves.t +++ b/t/db_dependent/Reserves.t @@ -25,6 +25,7 @@ use MARC::Record; use DateTime::Duration; use C4::Biblio; +use C4::Reserves; use C4::Items; use C4::Members; use C4::Circulation; @@ -37,7 +38,7 @@ use Koha::Patron::Categories; use Data::Dumper; BEGIN { - use_ok('C4::Reserves'); + require_ok('C4::Reserves'); } # a very minimal mack of userenv for use by the test of DelItemCheck -- 1.9.1