diff --git a/C4/Reserves.pm b/C4/Reserves.pm index 7273802d56..c6dc9770a6 100644 --- a/C4/Reserves.pm +++ b/C4/Reserves.pm @@ -1804,7 +1804,8 @@ sub _Findgroupreserve { reserves.priority AS priority, reserves.timestamp AS timestamp, biblioitems.biblioitemnumber AS biblioitemnumber, - reserves.itemnumber AS itemnumber + reserves.itemnumber AS itemnumber, + reserves.reserve_id AS reserve_id FROM reserves JOIN biblioitems USING (biblionumber) JOIN hold_fill_targets USING (biblionumber, borrowernumber) @@ -1835,7 +1836,8 @@ sub _Findgroupreserve { reserves.priority AS priority, reserves.timestamp AS timestamp, reserveconstraints.biblioitemnumber AS biblioitemnumber, - reserves.itemnumber AS itemnumber + reserves.itemnumber AS itemnumber, + reserves.reserve_id AS reserve_id FROM reserves LEFT JOIN reserveconstraints ON reserves.biblionumber = reserveconstraints.biblionumber WHERE reserves.biblionumber = ? diff --git a/t/db_dependent/Reserves.t b/t/db_dependent/Reserves.t index f511c41ec6..cd0c831289 100755 --- a/t/db_dependent/Reserves.t +++ b/t/db_dependent/Reserves.t @@ -2,7 +2,8 @@ use Modern::Perl; -use Test::More tests => 8; +use Test::More tests => 9; + use MARC::Record; use C4::Branch; @@ -72,6 +73,8 @@ my ($status, $reserve, $all_reserves) = CheckReserves($itemnumber, $barcode); is($status, "Reserved", "CheckReserves Test 1"); +ok(exists($reserve->{reserve_id}), 'CheckReserves() include reserve_id in its response'); + ($status, $reserve, $all_reserves) = CheckReserves($itemnumber); is($status, "Reserved", "CheckReserves Test 2");