Bug 2505 enabling warnings
authorChris Cormack <chris@bigballofwax.co.nz>
Sat, 12 Dec 2009 09:50:02 +0000 (22:50 +1300)
committerHenri-Damien LAURENT <henridamien.laurent@biblibre.com>
Sun, 20 Dec 2009 13:31:35 +0000 (14:31 +0100)
about.pl
acqui/acqui-home.pl
mainpage.pl

index 735138c..17316bf 100755 (executable)
--- a/about.pl
+++ b/about.pl
@@ -16,7 +16,7 @@
 # Suite 330, Boston, MA  02111-1307 USA
 
 use strict;
-
+use warnings;
 use C4::Output;    # contains gettemplate
 use C4::Auth;
 use C4::Context;
index 291383e..8c6fcc8 100755 (executable)
@@ -41,6 +41,7 @@ thus, it can be REJECTED, ACCEPTED, ORDERED, ASKED, AVAIBLE
 =cut
 
 use strict;
+use warnings;
 use Number::Format;
 
 use CGI;
@@ -146,14 +147,19 @@ foreach my $result (@results) {
     $result->{'budget_branchname'} = GetBranchName( $result->{'budget_branchcode'} );
 
     my $member      = GetMember( $result->{'budget_owner_id'} );
-    my $member_full = $member->{'firstname'} . ' ' . $member->{'surname'};
+    my $member_full = $member->{'firstname'} . ' ' . $member->{'surname'} if $member;
 
     $result->{'budget_owner'} = $member_full;
-    $result->{'budget_avail'} = $result->{'budget_amount'} - $result->{'budget_spent'};
+    if ($result->{'budget_spent'}){
+       $result->{'budget_avail'} = $result->{'budget_amount'} - $result->{'budget_spent'};
+    }
+    else {
+       $result->{'budget_avail'} = $result->{'budget_amount'};
+    }
     $result->{'budget_spent'} = GetBudgetSpent( $result->{'budget_id'} );
 
     $total    += $result->{'budget_amount'};
-    $totspent += $result->{'budget_spent'};
+    $totspent += $result->{'budget_spent'} if $result->{'budget_spent'};
     $totavail += $result->{'budget_avail'};
 
     $result->{'budget_amount'} = $num->format_price( $result->{'budget_amount'} );
@@ -164,7 +170,7 @@ foreach my $result (@results) {
     #        $result->{'budget_spent_percent'} = sprintf( "%00d", $spent_percent );
 
     my $borrower = &GetMember( $result->{budget_owner_id} );
-    $result->{budget_owner_name} = $borrower->{'firstname'} . ' ' . $borrower->{'surname'};
+    $result->{budget_owner_name} = $borrower->{'firstname'} . ' ' . $borrower->{'surname'} if $borrower;
 
     push( @loop_budget, { %{$result}, toggle => $toggle++ % 2, } );
 }
index 8116eeb..b5c0b1e 100755 (executable)
@@ -17,7 +17,7 @@
 #
 
 use strict;
-
+use warnings;
 use CGI;
 use C4::Output;
 use C4::Auth;