From 2dc6ef96312c9c2649719289c8e71466441a8da4 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Wed, 5 Feb 2020 16:02:25 +0100 Subject: [PATCH] Bug 24526: Use sprintf for readability Signed-off-by: Jonathan Druart Signed-off-by: Martin Renvoize --- misc/cronjobs/automatic_renewals.pl | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/misc/cronjobs/automatic_renewals.pl b/misc/cronjobs/automatic_renewals.pl index a5667e5f39..76f43d882a 100755 --- a/misc/cronjobs/automatic_renewals.pl +++ b/misc/cronjobs/automatic_renewals.pl @@ -89,14 +89,14 @@ while ( my $auto_renew = $auto_renews->next ) { # CanBookBeRenewed returns 'auto_renew' when the renewal should be done by this script my ( $ok, $error ) = CanBookBeRenewed( $auto_renew->borrowernumber, $auto_renew->itemnumber ); if ( $error eq 'auto_renew' ) { + if ($verbose) { + say sprintf "Issue id: %s for borrower: %s and item: %s ". ( $commit ? 'will' : 'would') . " be renewed.", + $auto_renew->issue_id, $auto_renew->borrowernumber, $auto_renew->itemnumber; + } if ($commit){ my $date_due = AddRenewal( $auto_renew->borrowernumber, $auto_renew->itemnumber, $auto_renew->branchcode ); $auto_renew->auto_renew_error(undef)->store; } - 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' @@ -108,9 +108,8 @@ while ( my $auto_renew = $auto_renews->next ) { or $error eq 'auto_too_soon' or $error eq 'item_denied_renewal' ) { if ( $verbose ) { - print "Issue id: " . $auto_renew->issue_id . " for borrower: " . $auto_renew->borrowernumber . " and item: " . $auto_renew->itemnumber; - $commit ? print " will not be renewed " : print " would not be renewed "; - print "($error)\n"; + say sprintf "Issue id: %s for borrower: %s and item: %s ". ( $commit ? 'will' : 'would') . " not be renewed. (%s)", + $auto_renew->issue_id, $auto_renew->borrowernumber, $auto_renew->itemnumber, $error; } if ( not $auto_renew->auto_renew_error or $error ne $auto_renew->auto_renew_error ) { $auto_renew->auto_renew_error($error)->store if $commit; -- 2.39.5