From c93e5b12f89507f317b202208a765dba4f555425 Mon Sep 17 00:00:00 2001 From: Kenza Zaki Date: Mon, 12 Aug 2013 14:40:50 +0200 Subject: [PATCH] Bug 10711: make GetOfflineOperation() return $sth->fetchrow_hashref instead of $result To test : prove t/db_dependent/Circulation_OfflineOperation.t t/db_dependent/Circulation_OfflineOperation.t .. ok All tests successful. Files=1, Tests=7, 19 wallclock secs ( 0.02 usr 0.01 sys + 0.24 cusr 0.02 csys = 0.29 CPU) Result: PASS Signed-off-by: Bernardo Gonzalez Kriegel Comment: applied 10681 and 10710 before 10711 prove t/db_dependent/Circulation_OfflineOperation.t run without errors koha-qa reports only 2 tabulations, fixed in a follow-up Signed-off-by: Chris Cormack Signed-off-by: Galen Charlton --- C4/Circulation.pm | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/C4/Circulation.pm b/C4/Circulation.pm index 39d9ed23fc..80b9e742b7 100644 --- a/C4/Circulation.pm +++ b/C4/Circulation.pm @@ -3317,11 +3317,12 @@ sub GetOfflineOperations { } sub GetOfflineOperation { + my $operationid = shift; + return unless $operationid; my $dbh = C4::Context->dbh; my $sth = $dbh->prepare("SELECT * FROM pending_offline_operations WHERE operationid=?"); - $sth->execute( shift ); - my $result = $sth->fetchrow_hashref; - return $result; + $sth->execute( $operationid ); + return $sth->fetchrow_hashref; } sub AddOfflineOperation { -- 2.39.5