From ab1aaba6ef06cf3bf0d124b2e658690d7a309c3e Mon Sep 17 00:00:00 2001 From: Kyle M Hall Date: Mon, 23 May 2016 17:34:29 +0000 Subject: [PATCH] Bug 16499 [QA Followup] - Improve readability Signed-off-by: Kyle M Hall (cherry picked from commit 59c0d44792b0ce4f88c028be97a32a62fb4a8b36) Signed-off-by: Julian Maurice --- circ/circulation.pl | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/circ/circulation.pl b/circ/circulation.pl index 69fdfc328d..c4210850d5 100755 --- a/circ/circulation.pl +++ b/circ/circulation.pl @@ -73,7 +73,7 @@ if (!C4::Context->userenv){ my $barcodes = []; my $barcode = $query->param('barcode'); # Barcode given by user could be '0' -if ( $barcode or defined $barcode and $barcode eq '0' ) { +if ( $barcode || ( defined($barcode) && $barcode eq '0' ) ) { $barcodes = [ $barcode ]; } else { my $filefh = $query->upload('uploadfile'); @@ -159,7 +159,7 @@ my $duedatespec = $query->param('duedatespec') || $session->param('stickydu $duedatespec = eval { output_pref( { dt => dt_from_string( $duedatespec ), dateformat => 'iso' }); } if ( $duedatespec ); my $restoreduedatespec = $query->param('restoreduedatespec') || $session->param('stickyduedate') || $duedatespec; -if ($restoreduedatespec and $restoreduedatespec eq "highholds_empty") { +if ( $restoreduedatespec && $restoreduedatespec eq "highholds_empty" ) { undef $restoreduedatespec; } my $issueconfirmed = $query->param('issueconfirmed');