Bug 10664 [QA Followup] - fix error in overdue_notices.pl if there is no active currency
authorKyle M Hall <kyle@bywatersolutions.com>
Fri, 16 Aug 2013 11:21:10 +0000 (07:21 -0400)
committerChris Hall <followingthepath@gmail.com>
Sun, 20 Oct 2013 06:58:52 +0000 (19:58 +1300)
Fixes the following koha-qa.pl error:
 FAIL   misc/cronjobs/overdue_notices.pl
    OK     pod
    OK     forbidden patterns
    OK     valid
    FAIL   critic
               # Variables::ProhibitConditionalDeclarations:

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
(cherry picked from commit e476378323eb8dd31e67e4741d271d88de61b59e)
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
(cherry picked from commit 22f96c6ebeb77da5efe7285d1980ce705e22726e)
Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>
(cherry picked from commit 22f96c6ebeb77da5efe7285d1980ce705e22726e)

misc/cronjobs/overdue_notices.pl

index 9cb8c88..9bd02ad 100755 (executable)
@@ -659,7 +659,8 @@ sub parse_letter {
     }
 
     my $currencies = GetCurrency();
-    my $currency_format = $currencies->{currency} if defined($currencies);
+    my $currency_format;
+    $currency_format = $currencies->{currency} if defined($currencies);
 
     my @item_tables;
     if ( my $i = $params->{'items'} ) {