Resolving bug 169959

git-svn-id: https://ledger-smb.svn.sourceforge.net/svnroot/ledger-smb/branches/1.2@1083 4979c152-3d1c-0410-bac9-87ea11338e46

2 files changed, 8 insertions, 1 deletions

diff --git a/Changelog b/Changelog
index 6c3a7f37..acedf2c1 100644
--- a/Changelog
+++ b/Changelog
@@ -1,5 +1,8 @@
 Changelog for 1.2.4
 * Fixed internal functions avgcost() and lastcost() to not use float (Joshua D)
+* Fixed error posting vendor invoice with fraction costs.  (Chris M)
+* Fixed sales tax display issue on invoice/order entry screen (Chris T)
+
 
 Changelog for 1.2.3
 * Corrected per-user locale selection (Seneca)
diff --git a/LedgerSMB/Form.pm b/LedgerSMB/Form.pm
index 38322350..7f0c6ec1 100755
--- a/LedgerSMB/Form.pm
+++ b/LedgerSMB/Form.pm
@@ -214,7 +214,7 @@ sub error {
         delete $self->{pre};
 
         if ( !$self->{header} ) {
-            $self->eader;
+            $self->header;
         }
 
         print
@@ -2726,6 +2726,10 @@ sub save_recurring {
             $s{print},     $s{email},     $s{message}
         ) = split /,/, $self->{recurring};
 
+	if ($s{howmany} == 0){
+		$self->error("Cannot set to recur 0 times");
+	}
+
         for (qw(reference message)) { $s{$_} = $self->unescape( $s{$_} ) }
         for (qw(repeat howmany payment)) { $s{$_} *= 1 }