From 3f287ef4b623db0b544f822449f0d6d5b0f312df Mon Sep 17 00:00:00 2001 From: Jason Stephenson Date: Mon, 22 Oct 2012 10:05:32 -0400 Subject: [PATCH] Fix handling of ranges in OpenILS::SIP::Patron::fine_items(). When the client specifies a begin and end range for SIP2 fine items, only the first fine item is returned. This is because of a typo on line 646. The @fines array should be dereferenced there as @fines and not $fines. Signed-off-by: Jason Stephenson --- Open-ILS/src/perlmods/lib/OpenILS/SIP/Patron.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Open-ILS/src/perlmods/lib/OpenILS/SIP/Patron.pm b/Open-ILS/src/perlmods/lib/OpenILS/SIP/Patron.pm index 4a77bd0962..9299738ed9 100644 --- a/Open-ILS/src/perlmods/lib/OpenILS/SIP/Patron.pm +++ b/Open-ILS/src/perlmods/lib/OpenILS/SIP/Patron.pm @@ -643,7 +643,7 @@ sub fine_items { my $log_status = $@ ? 'ERROR: ' . $@ : 'OK'; syslog('LOG_DEBUG', 'OILS: Patron->fine_items() ' . $log_status); return (defined $start and defined $end) ? - [ $fines[($start-1)..($end-1)] ] : \@fines; + [ @fines[($start-1)..($end-1)] ] : \@fines; } # not currently supported -- 2.11.0