From 3f24a07a7ee83859aed10a0d874438a6c6fad2a2 Mon Sep 17 00:00:00 2001 From: Marcel de Rooy Date: Tue, 3 Jul 2018 15:52:05 +0200 Subject: [PATCH] Bug 21033: Remove few warns in acqui/basket.pl Resolve (line numbers based on 16.11.x): Use of uninitialized value in hash element at acqui/basket.pl line 337. Use of uninitialized value in hash element at acqui/basket.pl line 338. Use of uninitialized value in hash element at acqui/basket.pl line 340. Use of uninitialized value in hash element at acqui/basket.pl line 342. Use of uninitialized value in hash element at acqui/basket.pl line 344. Argument "" isn't numeric in multiplication (*) at koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basket.tt line 486. Argument "" isn't numeric in multiplication (*) at koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basket.tt line 591. Test plan: If you have older acq data, you may have records in aqorders with field tax_rate_on_ordering is NULL. These orders will trigger the above warns. If you do not have, you could simulate by setting this field to NULL. Signed-off-by: Katrin Fischer Signed-off-by: Jonathan Druart Signed-off-by: Nick Clemens (cherry picked from commit af82a1d59709d02b767613539146b59fa10c3ab3) Signed-off-by: Martin Renvoize (cherry picked from commit 9a6ed1c852c00644349597e59a369bef5a19d9a1) Signed-off-by: Fridolin Somers --- acqui/basket.pl | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/acqui/basket.pl b/acqui/basket.pl index 92fae60a0a..9f2ffdf7e1 100755 --- a/acqui/basket.pl +++ b/acqui/basket.pl @@ -340,8 +340,8 @@ if ( $op eq 'list' ) { $template->param( uncertainprices => 1 ); } - $line->{tax_rate} = $line->{tax_rate_on_ordering}; - $line->{tax_value} = $line->{tax_value_on_ordering}; + $line->{tax_rate} = $line->{tax_rate_on_ordering} // 0; + $line->{tax_value} = $line->{tax_value_on_ordering} // 0; push @books_loop, $line; -- 2.39.5