bugfix suppress warnings coming from acquisitions homepage
authorColin Campbell <colin.campbell@ptfs-europe.com>
Thu, 11 Feb 2010 14:40:44 +0000 (14:40 +0000)
committerGalen Charlton <gmcharlt@gmail.com>
Fri, 12 Feb 2010 00:39:44 +0000 (19:39 -0500)
Before formatting and doing arithmetic with them we should
ensure undefined numeric elements are numbers
(This should be enforced in the db)

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
acqui/acqui-home.pl

index 0193492..8ce7d00 100755 (executable)
@@ -149,6 +149,16 @@ foreach my $result (@results) {
 
     my $member      = GetMember( borrowernumber => $result->{budget_owner_id} );
     my $member_full = $member->{'firstname'} . ' ' . $member->{'surname'} if $member;
+    # as we are going to treat these as numbers let's ensure they are not NULL
+    if (!defined $result->{budget_amount} ) {
+        $result->{budget_amount} = 0;
+    }
+    if (!defined $result->{budget_spent} ) {
+        $result->{budget_spent} = 0;
+    }
+    if (!defined $result->{budget_ordered} ) {
+        $result->{budget_ordered} = 0;
+    }
 
     $result->{'budget_owner'}   = $member_full;
     $result->{'budget_ordered'} = GetBudgetOrdered( $result->{'budget_id'} );