projects
/
koha.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
(parent:
3f37855
)
Bug 10664 [QA Followup] - fix error in overdue_notices.pl if there is no active currency
author
Kyle M Hall
<kyle@bywatersolutions.com>
Fri, 16 Aug 2013 11:21:10 +0000 (07:21 -0400)
committer
Chris 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
patch
|
blob
|
history
diff --git
a/misc/cronjobs/overdue_notices.pl
b/misc/cronjobs/overdue_notices.pl
index
9cb8c88
..
9bd02ad
100755
(executable)
--- a/
misc/cronjobs/overdue_notices.pl
+++ b/
misc/cronjobs/overdue_notices.pl
@@
-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'} ) {