From bc62ff53869e0f1cb07b4fb238d2ba50dc3feea7 Mon Sep 17 00:00:00 2001 From: Fridolyn SOMERS Date: Tue, 11 Dec 2012 17:33:11 +0100 Subject: [PATCH] Bug 9263: Trim barcodes in offline circulation Signed-off-by: Kyle M Hall --- offline_circ/service.pl | 34 ++++++++++++++++++++++------------ 1 files changed, 22 insertions(+), 12 deletions(-) diff --git a/offline_circ/service.pl b/offline_circ/service.pl index ce0ab4a..30789f3 100755 --- a/offline_circ/service.pl +++ b/offline_circ/service.pl @@ -33,24 +33,34 @@ my ($status, $cookie, $sessionId) = C4::Auth::check_api_auth($cgi, undef); my $result; if ($status eq 'ok') { # if authentication is ok + + $userid = $cgi->param('userid') || ''; + $branchcode = $cgi->param('branchcode') || ''; + $timestamp = $cgi->param('timestamp') || ''; + $action = $cgi->param('action') || ''; + $barcode = $cgi->param('barcode') || ''; + $barcode =~ s/^\s+|\s+$//; + $cardnumber = $cgi->param('cardnumber') || ''; + $cardnumber =~ s/^\s+|\s+$//; + if ( $cgi->param('pending') eq 'true' ) { # if the 'pending' flag is true, we store the operation in the db instead of directly processing them $result = AddOfflineOperation( - $cgi->param('userid') || '', - $cgi->param('branchcode') || '', - $cgi->param('timestamp') || '', - $cgi->param('action') || '', - $cgi->param('barcode') || '', - $cgi->param('cardnumber') || '', + $userid, + $branchcode, + $timestamp, + $action, + $barcode, + $cardnumber, ); } else { $result = ProcessOfflineOperation( { - 'userid' => $cgi->param('userid'), - 'branchcode' => $cgi->param('branchcode'), - 'timestamp' => $cgi->param('timestamp'), - 'action' => $cgi->param('action'), - 'barcode' => $cgi->param('barcode'), - 'cardnumber' => $cgi->param('cardnumber'), + 'userid' => $userid, + 'branchcode' => $branchcode, + 'timestamp' => $timestamp, + 'action' => $action, + 'barcode' => $barcode, + 'cardnumber' => $cardnumber, } ); } -- 1.7.2.5