From aa9ad782ced801981242504d417348b31fa9f8d8 Mon Sep 17 00:00:00 2001 From: Nick Clemens Date: Wed, 29 Jan 2020 13:48:09 +0000 Subject: [PATCH] Bug 24526: (follow-up) clean up verbose conditional Signed-off-by: Andrew Fuerste-Henry Signed-off-by: Jonathan Druart Signed-off-by: Martin Renvoize --- misc/cronjobs/automatic_renewals.pl | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/misc/cronjobs/automatic_renewals.pl b/misc/cronjobs/automatic_renewals.pl index 547100afb8..a5667e5f39 100755 --- a/misc/cronjobs/automatic_renewals.pl +++ b/misc/cronjobs/automatic_renewals.pl @@ -93,8 +93,10 @@ while ( my $auto_renew = $auto_renews->next ) { my $date_due = AddRenewal( $auto_renew->borrowernumber, $auto_renew->itemnumber, $auto_renew->branchcode ); $auto_renew->auto_renew_error(undef)->store; } - $verbose && print "Issue id: " . $auto_renew->issue_id . " for borrower: " . $auto_renew->borrowernumber . " and item: " . $auto_renew->itemnumber; - $verbose && $commit ? print " will be renewed.\n" : print " would be renewed.\n"; + if ( $verbose ){ + print "Issue id: " . $auto_renew->issue_id . " for borrower: " . $auto_renew->borrowernumber . " and item: " . $auto_renew->itemnumber; + $commit ? print " will be renewed.\n" : print " would be renewed.\n"; + } push @{ $report{ $auto_renew->borrowernumber } }, $auto_renew; } elsif ( $error eq 'too_many' or $error eq 'on_reserve' -- 2.39.5