SF.net SVN: ledger-smb:[4146] branches/1.3

[..hidden..]

Revision: 4146
          http://ledger-smb.svn.sourceforge.net/ledger-smb/?rev=4146&view=rev
Author:   einhverfr
Date:     2011-12-05 10:57:33 +0000 (Mon, 05 Dec 2011)
Log Message:
-----------
Fixed error:  ERROR: lower bound of FOR loop cannot be null

Modified Paths:
--------------
    branches/1.3/Changelog
    branches/1.3/scripts/payment.pl

Modified: branches/1.3/Changelog
===================================================================
--- branches/1.3/Changelog	2011-12-05 08:15:45 UTC (rev 4145)
+++ branches/1.3/Changelog	2011-12-05 10:57:33 UTC (rev 4146)
@@ -18,6 +18,7 @@
 * Fixed part account mappings broken after upgrade from 1.2 (Chris T)
 * Fixed quotation number increasing when saving existing quotation (Chris T)
 * Fixed invalid from address on backup routine (Chris T)
+* Fixed error "RROR: lower bound of FOR loop cannot be null" (Chris T)
 
 Changelog for LedgerSMB 1.3.7 (Categorized due to length)
 Database

Modified: branches/1.3/scripts/payment.pl
===================================================================
--- branches/1.3/scripts/payment.pl	2011-12-05 08:15:45 UTC (rev 4145)
+++ branches/1.3/scripts/payment.pl	2011-12-05 10:57:33 UTC (rev 4146)
@@ -1271,6 +1271,9 @@
      push @op_cash_account_id, $cashid;
      push @op_source, $request->{"overpayment_source1_$i"}.' '.$request->{"overpayment_source2_$i"};
      push @op_memo, $request->{"overpayment_memo_$i"};
+     if (!$id and $id ne "0"){
+         $request->error($request->{_locale}->text('No overpayment account selected.  Was one set up?'));
+     }
      push @op_account_id, $id;        
      } 
    }  

This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.