SF.net SVN: ledger-smb:[3620] trunk/bin/am.pl

[..hidden..]

Revision: 3620
          http://ledger-smb.svn.sourceforge.net/ledger-smb/?rev=3620&view=rev
Author:   einhverfr
Date:     2011-08-03 20:57:19 +0000 (Wed, 03 Aug 2011)

Log Message:
-----------
Comments explaining issues with recurring transactions

Modified Paths:
--------------
    trunk/bin/am.pl

Modified: trunk/bin/am.pl
===================================================================
--- trunk/bin/am.pl	2011-08-03 20:56:46 UTC (rev 3619)
+++ trunk/bin/am.pl	2011-08-03 20:57:19 UTC (rev 3620)
@@ -2452,6 +2452,13 @@
             $header = $form->{header};
 
             # reset $form
+            # XXX THIS IS A BUG FACTORY. PLEASE READ:
+            # This is old code from SL, and it basically forces a reset of the
+            # LedgerSMB::Form object by deleting all keys and then copying a few
+            # back in.  This is error prone and buggy.  If you have issues with
+            # recurring transactions, the first thing to do is to see if 
+            # something is not being copied back that needs to be.  Looking 
+            # forward to removing this code. --CT
             for ( keys %$form ) { delete $form->{$_}; }
             for (qw(dbversion company dbh login path sessionid stylesheet timeout)) {
                 $form->{$_} = $pt->{$_};


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