This is probably what we really want for NSF... two transactions. One for backing out the bank deposit, and the other for adjusting A/R and all the associated statement entries. This is working reasonably well, even seemingly fine for the collected rents report. I do see that there is another bug though, that is causing statement entries to have the opposite account_id than intended (A/R instead of Cash) when entering new receipts. Not sure when I broke that, but it must have been when working on deposits.

git-svn-id: file:///svn-source/pmgr/branches/yafr_20090716@427 97e9348a-65ac-dc4b-aefc-98561f571b83

site/models/transaction.php

@@ -549,8 +549,6 @@ class Transaction extends AppModel {
 	     array('checkpoint' => 'Pre Transaction Save')
 	     + compact('transaction')));
 
-    pr(compact('data'));
-
     // Save transaction to the database
     $this->create();
     if (!$this->save($transaction))
@@ -567,7 +565,6 @@ class Transaction extends AppModel {
       $le1_tender = null;
       $le2 = null;
       $se = null;
-      pr(compact('entry'));
       extract($entry);
 
       if (!empty($le1) && !empty($le2)) {
@@ -591,7 +588,6 @@ class Transaction extends AppModel {
 	  continue;
 	}
       }
-      pr(compact('ret'));
     }
 
 /*     if (($transaction['type'] == 'INVOICE' || */