Prevent grouping of ledger entries by transaction, at least for now.

git-svn-id: file:///svn-source/pmgr/branches/ledger_transactions_20090605/site@178 97e9348a-65ac-dc4b-aefc-98561f571b83

app_controller.php

@@ -305,13 +305,7 @@ class AppController extends Controller {
       $id = 'id';
       extract(array_intersect_key($fields, array_flip($special)));
       foreach ($records AS &$record) {
-	if (!isset($record[$table]))
-	  continue;
-
 	foreach (array_diff_key($fields, array_flip($special)) AS $field) {
-	  if (!isset($record[$table][$id]) || !isset($record[$table][$field]))
-	    continue;
-
 	  // DEBUG PURPOSES ONLY!
 	  //$params['linkrecord'][] = compact('table', 'field', 'id', 'controller', 'record');
 	  $record[$table][$field] =
@@ -344,38 +338,28 @@ class AppController extends Controller {
   }
 
   function jqGridDataOutputRecords(&$params, &$model, &$records) {
-    $id_field    = 'jqGrid_id';
+    $model_alias = $model->alias;
+    $id          = 'jqGrid_id';
+
     foreach ($records AS $record) {
-      $this->jqGridDataOutputRecord($params, $model, $record,
-				    $record[$id_field], $params['fields']);
-    }
-  }
+      echo " <row id='{$record[$id]}'>\n";
+      foreach ($params['fields'] AS $field) {
+	if (preg_match("/\./", $field)) {
+	  list($tbl, $col) = explode(".", $field);
+	  $data = $record[$tbl][$col];
+	}
+	else {
+	  $data = $record[$model_alias][$field];
+	}
 
-  function jqGridDataOutputRecord(&$params, &$model, &$record, $id, $fields) {
-    echo " <row id='$id'>\n";
-    foreach ($fields AS $field) {
-      $this->jqGridDataOutputRecordField($params, $model, $record, $field);
+	// be sure to put text data in CDATA
+	if (preg_match("/^\d*$/", $data))
+	  echo "  <cell>$data</cell>\n"; 
+	else
+	  echo "  <cell><![CDATA[$data]]></cell>\n"; 
+      }
+      echo " </row>\n";
     }
-    echo " </row>\n";
-  }
-
-  function jqGridDataOutputRecordField(&$params, &$model, &$record, $field) {
-    if (preg_match("/\./", $field)) {
-      list($tbl, $col) = explode(".", $field);
-      $data = $record[$tbl][$col];
-    }
-    else {
-      $data = $record[$model->alias][$field];
-    }
-    $this->jqGridDataOutputRecordCell($params, $model, $record, $field, $data);
-  }
-
-  function jqGridDataOutputRecordCell(&$params, &$model, &$record, $field, $data) {
-    // be sure to put text data in CDATA
-    if (preg_match("/^\d*$/", $data))
-      echo "  <cell>$data</cell>\n"; 
-    else
-      echo "  <cell><![CDATA[$data]]></cell>\n"; 
   }
 
   function jqGridDataFinalize(&$params) {

build.cmd

@@ -1,3 +0,0 @@
-@echo off
-%~dp0\scripts\sitelink2pmgr.pl %~dp0\db\schema.sql %~dp0db\vss.mdb > NUL
-echo Done!

controllers/leases_controller.php

@@ -71,7 +71,6 @@ class LeasesController extends AppController {
   function jqGridRecordLinks(&$params, &$model, &$records, $links) {
     $links['Lease'] = array('number');
     $links['Unit'] = array('name');
-    $links['Customer'] = array('name');
     return parent::jqGridRecordLinks($params, $model, $records, $links);
   }
 
@@ -108,6 +107,7 @@ class LeasesController extends AppController {
 	     array(// Models
 		   'LeaseType',
 		   'Unit',
+		   'Account' => array('CurrentLedger'),
 		   'Customer',
 		   ),
 	     'conditions' => array(array('Lease.id' => $id)),
@@ -115,15 +115,23 @@ class LeasesController extends AppController {
 	     )
        );
 
+    // Summarize each ledger
+    $this->Lease->statsMerge($lease, 
+			     $this->Lease->stats($lease['Lease']['id']));
+
     // Obtain the overall lease balance
     $this->Lease->statsMerge($lease['Lease'],
 			     array('stats' => $this->Lease->stats($id)));
-    $outstanding_balance = $lease['Lease']['stats']['balance'];
+    $outstanding_balance = $lease['Lease']['stats']['Account']['Ledger']['balance'];
 
     // Determine the lease security deposit
     $deposits = $this->Lease->findSecurityDeposits($lease['Lease']['id']);
     $outstanding_deposit = $deposits['summary']['balance'];
 
+    // Move the Leder stats into our alias 'CurrentLedger'
+    $lease['Account']['CurrentLedger'] += $lease['Account']['Ledger'];
+    unset($lease['Account']['Ledger']);
+
     // Prepare to render
     $title = 'Lease: #' . $lease['Lease']['id'];
     $this->set(compact('lease', 'title',

controllers/ledger_entries_controller.php

@@ -25,15 +25,7 @@ class LedgerEntriesController extends AppController {
    *    to jqGrid.
    */
 
-  function jqGridDataSetup(&$params) {
-    parent::jqGridDataSetup($params);
-    if (isset($params['custom']['ar_account'])) {
-      $params['custom']['account_id'] = 
-	$this->LedgerEntry->DebitLedger->Account->accountReceivableAccountID();
-    }
-  }
-
-  function jqGridDataCountTables(&$params, &$model) {
+  function jqGridDataTables(&$params, &$model) {
     $link =
       array(// Models
 	     'Transaction' =>
@@ -44,16 +36,6 @@ class LedgerEntriesController extends AppController {
 	     array('fields' => array('id', 'name'),
 		   ),
 
-	     'Customer' =>
-	     array('fields' => array('id', 'name'),
-		   ),
-
-	     'Lease' =>
-	     array('fields' => array('id', 'number'),
-		   'Unit' => 
-		   array('fields' => array('id', 'name'),
-			 ),
-		   ),
 	    );
 
     if (isset($params['custom']['account_ftype'])) {
@@ -95,86 +77,26 @@ class LedgerEntriesController extends AppController {
 	      );
     }
 
-    if (isset($params['custom']['account_id']) || isset($params['custom']['lease_id'])) {
-      if (isset($params['custom']['account_id']))
-	$account_id = $params['custom']['account_id'];
-      else
-	$account_id =
-	  $this->LedgerEntry->DebitLedger->Account->accountReceivableAccountID();
-
-      $link['Ledger'] =
-	array('fields' => array('id', 'sequence'),
-	      'conditions' => ("Ledger.id = IF(DebitLedger.account_id = $account_id," .
-			       "               LedgerEntry.credit_ledger_id," .
-			       "               LedgerEntry.debit_ledger_id)"),
-	      'Account' => array(
-				 'fields' => array('id', 'name'),
-				 ),
-	      );
-    }
-
-    if (isset($params['custom']['lease_id'])) {
-      $account_id = $params['custom']['lease_id'];
-      $link['Transaction']['ReconciledLedgerEntry'] =
-	array('fields' => array('id', 'Reconciliation.amount', 'Reconciliation.type'),
-	      );
-    }
-
     if (isset($params['custom']['reconcile_id'])) {
       $ftype = $params['custom']['account_ftype'];
       $ftype = $this->LedgerEntry->DebitLedger->Account->fundamentalOpposite($ftype);
-      $link['ReconcilingTransaction'] =
-	array('fields' => array('Reconciliation.amount'),
-	      'conditions' => array('Reconciliation.type' => $ftype),
-	      );
+      $ftype = ucfirst($ftype);
+      $link[$ftype.'ReconciliationLedgerEntry'] =
+	array('fields' => array('Reconciliation.amount'));
     }
 
     return array('link' => $link);
   }
 
-  function jqGridDataTables(&$params, &$model) {
-    return $this->jqGridDataCountTables($params, $model);
-  }
-
   function jqGridDataFields(&$params, &$model) {
-    if (isset($params['custom']['lease_id'])) {
-      $account_id =
-	$this->LedgerEntry->DebitLedger->Account->accountReceivableAccountID();
-
-      $fields = array('id', 'name', 'comment', 'amount');
-      $fields[] = ("IF(DebitLedger.account_id = $account_id," .
-		   "   SUM(IF(ReconciledLedgerEntry.amount IS NULL," .
-		   "          LedgerEntry.amount," .
-		   "          ReconciledLedgerEntry.amount))," .
-		   "   NULL) AS debit");
-      $fields[] = ("IF(CreditLedger.account_id = $account_id," .
-		   "   SUM(IF(ReconciledLedgerEntry.amount IS NULL," .
-		   "          LedgerEntry.amount," .
-		   "          ReconciledLedgerEntry.amount))," .
-		   "   NULL) AS credit");
-
-      $Account = new Account();
-      $account_ftype = ucfirst($Account->fundamentalType($account_id));
-      $fields[] = ("(IF({$account_ftype}Ledger.account_id = $account_id, 1, -1)" .
-		   " * SUM(IF(ReconciledLedgerEntry.amount IS NULL," .
-		   "          LedgerEntry.amount," .
-		   "          ReconciledLedgerEntry.amount)))" .
-		   " AS balance");
-
-      return $fields;
-    }
-
     $ledger_id = (isset($params['custom']['ledger_id'])
 		  ? $params['custom']['ledger_id']
 		  : null);
-    $account_id = (isset($params['custom']['account_id'])
-		     ? $params['custom']['account_id']
-		     : null);
     $account_type = (isset($params['custom']['account_type'])
 		     ? $params['custom']['account_type']
 		     : null);
 
-    return $model->ledgerContextFields2($ledger_id, $account_id, $account_type);
+    return $model->ledgerContextFields2($ledger_id, $account_type);
   }
 
   function jqGridDataConditions(&$params, &$model) {
@@ -187,44 +109,13 @@ class LedgerEntriesController extends AppController {
 
     $conditions = parent::jqGridDataConditions($params, $model);
 
-    if ($params['action'] === 'ledger' && isset($ledger_id)) {
+    if ($params['action'] === 'ledger') {
       $conditions[] = $model->ledgerContextConditions($ledger_id, $account_type);
     }
-
     if (isset($params['custom']['reconcile_id'])) {
       $ftype = $params['custom']['account_ftype'];
       //$ftype = $this->LedgerEntry->DebitLedger->Account->fundamentalOpposite($ftype);
-      //$link['ReconcilingTransaction']['conditions'][] = array('Reconciliation.type' => $ftype);
-      $conditions[] = array('Reconciliation.ledger_entry_id' => $params['custom']['reconcile_id']);
-	
-    }
-
-    if (isset($params['custom']['account_id'])) {
-      $conditions[] =
-	array('OR' =>
-	      array(array('CreditAccount.id' => $params['custom']['account_id']),
-		    array('DebitAccount.id' => $params['custom']['account_id'])));
-/*       $conditions[] = */
-/* 	array('Account.id' => $params['custom']['account_id']); */
-
-    }
-
-    if (isset($params['custom']['customer_id'])) {
-      $conditions[] = 
-	array('Customer.id' => $params['custom']['customer_id']);
-    }
-
-    if (isset($params['custom']['lease_id'])) {
-      $conditions[] = 
-	array('OR' => array
-	      (array('Lease.id' => $params['custom']['lease_id']),
-	       array('ReconciledLedgerEntry.lease_id' => $params['custom']['lease_id']),
-	       ));
-    }
-
-    if (isset($params['custom']['transaction_id'])) {
-      $conditions[] = 
-	array('Transaction.id' => $params['custom']['transaction_id']);
+      $conditions[] = array('Reconciliation.'.$ftype.'_ledger_entry_id' => $params['custom']['reconcile_id']);
     }
 
     return $conditions;
@@ -233,13 +124,14 @@ class LedgerEntriesController extends AppController {
   function jqGridRecordLinks(&$params, &$model, &$records, $links) {
     $links['Transaction']    = array('id');
     $links['LedgerEntry']    = array('id');
-    $links['Account']        = array('controller' => 'accounts', 'name');
-    $links['DebitAccount']   = array('controller' => 'accounts', 'name');
-    $links['CreditAccount']  = array('controller' => 'accounts', 'name');
+    if (isset($params['custom']['account_ftype']) || isset($params['custom']['ledger_id'])) {
+      $links['Account']   = array('controller' => 'accounts', 'name');
+    }
+    else {
+      $links['DebitAccount']   = array('controller' => 'accounts', 'name');
+      $links['CreditAccount']  = array('controller' => 'accounts', 'name');
+    }
     $links['MonetarySource'] = array('name');
-    $links['Customer']       = array('name');
-    $links['Lease']          = array('number');
-    $links['Unit']           = array('name');
     return parent::jqGridRecordLinks($params, $model, $records, $links);
   }
 
@@ -253,39 +145,8 @@ class LedgerEntriesController extends AppController {
   function jqGridDataOrder(&$params, &$model, $index, $direction) {
 /*     if ($index === 'balance') */
 /*       return ($index .' '. $direction); */
-    $order = parent::jqGridDataOrder($params, $model, $index, $direction);
 
-    if ($index === 'Transaction.stamp') {
-      $order[] = 'LedgerEntry.id ' . $direction;
-    }
-
-    return $order;
-  }
-
-  function jqGridRecordsPostProcess(&$params, &$model, &$records) {
-    parent::jqGridRecordsPostProcess($params, $model, $records);
-
-    $subtotal = 0;
-    foreach ($records AS &$record) {
-      $amount = (isset($record['LedgerEntry']['balance'])
-		 ? $record['LedgerEntry']['balance']
-		 : $record['LedgerEntry']['amount']);
-      $record['LedgerEntry']['subtotal'] = ($subtotal += $amount);
-
-      continue;
-      // Experiment to minimize columns by putting the monetary source
-      // as the Account, when available
-      if ($record['MonetarySource']['name'])
-	$record['Account']['name'] = $record['MonetarySource']['name'];
-    }
-  }
-
-  function jqGridDataOutputRecordCell(&$params, &$model, &$record, $field, $data) {
-/*     if ($field === 'CreditAccount.name') { */
-/*       $data .= '-OK'; */
-/*     } */
-
-    parent::jqGridDataOutputRecordCell($params, $model, $record, $field, $data);
+    return parent::jqGridDataOrder($params, $model, $index, $direction);
   }
 
 
@@ -306,7 +167,6 @@ class LedgerEntriesController extends AppController {
     $entry = $this->LedgerEntry->find
       ('first',
        array('contain' => array('MonetarySource.id',
-				'MonetarySource.name',
 				'MonetarySource.MonetaryType.id',
 				'Transaction.id',
 				'Transaction.stamp',
@@ -316,9 +176,6 @@ class LedgerEntriesController extends AppController {
 				'CreditLedger.id',
 				'CreditLedger.sequence',
 				'CreditLedger.account_id',
-				'Customer.id',
-				'Customer.name',
-				'Lease.id',
 				),
 
 	     'fields' => array('LedgerEntry.id',
@@ -327,7 +184,6 @@ class LedgerEntriesController extends AppController {
 
 	     'conditions' => array('LedgerEntry.id' => $id),
 	     ));
-    //pr($entry);
 
     // Because 'DebitLedger' and 'CreditLedger' both relate to 'Account',
     // CakePHP will not include them in the LedgerEntry->find (or so it
@@ -357,8 +213,8 @@ class LedgerEntriesController extends AppController {
       $stats['credit_amount_remaining'] = $entry['LedgerEntry']['amount'] - $stats['credit_amount_reconciled'];
     //pr($stats);
 
-    $reconciled = $this->LedgerEntry->findReconcilingTransactions($id);
-    //pr(compact('reconciled'));
+    $reconciled = $this->LedgerEntry->findReconciledLedgerEntries($id);
+    //pr($reconciled);
 
     // Prepare to render.
     $title = "Ledger Entry #{$entry['LedgerEntry']['id']}";

controllers/transactions_controller.php

@@ -41,43 +41,6 @@ class TransactionsController extends AppController {
    *    to jqGrid.
    */
 
-  function jqGridDataSetup(&$params) {
-    parent::jqGridDataSetup($params);
-  }
-
-  function jqGridDataTables(&$params, &$model) {
-    $link = array();
-
-    if (isset($params['custom']['reconcile_ledger_entry_id'])) {
-      $ftype = $params['custom']['reconcile_type'];
-      //$ftype = $this->Transaction->LedgerEntry->DebitLedger->Account->fundamentalOpposite($ftype);
-      $link['ReconciledLedgerEntry'] =
-	array('fields' => array('Reconciliation.amount'),
-	      'conditions' => array('Reconciliation.type' => $ftype)
-	      );
-    }
-
-    return array('link' => $link);
-  }
-
-  function jqGridDataFields(&$params, &$model) {
-    return parent::jqGridDataFields($params, $model);
-  }
-
-  function jqGridDataConditions(&$params, &$model) {
-    $conditions = parent::jqGridDataConditions($params, $model);
-
-    if (isset($params['custom']['reconcile_ledger_entry_id'])) {
-/*       $ftype = $params['custom']['reconcile_type']; */
-/*       $ftype = $this->LedgerEntry->DebitLedger->Account->fundamentalOpposite($ftype); */
-/*       $conditions[] = array('Reconciliation.type' => $ftype); */
-      $conditions[] = array('Reconciliation.ledger_entry_id'
-			    => $params['custom']['reconcile_ledger_entry_id']);
-    }
-
-    return $conditions;
-  }
-
   function jqGridRecordLinks(&$params, &$model, &$records, $links) {
     $links['Transaction'] = array('id');
     return parent::jqGridRecordLinks($params, $model, $records, $links);

controllers/units_controller.php

@@ -121,7 +121,7 @@ class UnitsController extends AppController {
     // Get the balance on each lease.
     foreach ($unit['Lease'] AS &$lease) {
       $stats = $this->Unit->Lease->stats($lease['id']);
-      $lease['balance'] = $stats['balance'];
+      $lease['balance'] = $stats['Account']['Ledger']['balance'];
     }
 
     $outstanding_balance = 0;
@@ -130,7 +130,7 @@ class UnitsController extends AppController {
       // Figure out the outstanding balance of the current lease.
       $stats = $this->Unit->stats($id);
       $outstanding_balance =
-	$stats['CurrentLease']['balance'];
+	$stats['CurrentLease']['Account']['Ledger']['balance'];
 
       // Figure out the total security deposit for the current lease.
       $deposits = $this->Unit->Lease->findSecurityDeposits($unit['CurrentLease']['id']);

models/account.php

@@ -82,26 +82,37 @@ class Account extends AppModel {
   /**************************************************************************
    **************************************************************************
    **************************************************************************
-   * function: accountNameToID
-   *  - Returns the ID of the named account
+   * function: securityDepositAccountID
+   *  - Returns the ID of the Security Deposit Account
    */
-  function accountNameToID($name) {
+  function securityDepositAccountID() {
     $this->cacheQueries = true;
     $account = $this->find('first', array
 			   ('recursive' => -1,
-			    'conditions' => compact('name'),
+			    'conditions' => array
+			    (array('name' => 'Security Deposit')),
 			    ));
     $this->cacheQueries = false;
     return $account['Account']['id'];
   }
 
-  function securityDepositAccountID()   { return $this->accountNameToID('Security Deposit'); }
-  function rentAccountID()              { return $this->accountNameToID('Rent'); }
-  function accountReceivableAccountID() { return $this->accountNameToID('A/R'); }
-  function invoiceAccountID()           { return $this->accountReceivableAccountID(); }
-  function receiptAccountID()           { return $this->accountReceivableAccountID(); }
-  //function invoiceAccountID()           { return $this->accountNameToID('Invoice'); }
-  //function receiptAccountID()           { return $this->accountNameToID('Receipt'); }
+
+  /**************************************************************************
+   **************************************************************************
+   **************************************************************************
+   * function:rentAccountID
+   *  - Returns the ID of the Rent Account
+   */
+  function rentAccountID() {
+    $this->cacheQueries = true;
+    $account = $this->find('first', array
+			   ('recursive' => -1,
+			    'conditions' => array
+			    (array('name' => 'Rent')),
+			    ));
+    $this->cacheQueries = false;
+    return $account['Account']['id'];
+  }
 
 
   /**************************************************************************
@@ -111,19 +122,22 @@ class Account extends AppModel {
    *  - Returns an array of ledger ids from the given account
    */
   function ledgers($id, $all = false) {
+    $cachekey = $all ? 'all' : 'current';
+    if (isset($this->cache[$id]['ledgers'][$cachekey])) {
+      return $this->cache[$id]['ledgers'][$cachekey];
+    }
+
     if ($all) {
       $contain = array('Ledger' => array('fields' => array('Ledger.id')));
     } else {
       $contain = array('CurrentLedger' => array('fields' => array('CurrentLedger.id')));
     }
 
-    $this->cacheQueries = true;
     $account = $this->find('first', array
 			   ('contain' => $contain,
 			    'fields' => array(),
 			    'conditions' => array(array('Account.id' => $id)),
 			    ));
-    $this->cacheQueries = false;
 
     if ($all) {
       $ledger_ids = array();
@@ -134,11 +148,14 @@ class Account extends AppModel {
       $ledger_ids = array($account['CurrentLedger']['id']);
     }
 
+    // Save the ledgers in our cache for future reference
+    $this->cache[$id]['ledgers'][$cachekey] = $ledger_ids;
+
 /*     pr(array('function' => 'Account::ledgers', */
 /* 	     'args' => compact('id', 'all'), */
-/* 	     'return' => $ledger_ids)); */
+/* 	     'return' => $this->cache[$id]['ledgers'][$cachekey])); */
 
-    return $ledger_ids;
+    return $this->cache[$id]['ledgers'][$cachekey];
   }
 
 
@@ -217,29 +234,25 @@ class Account extends AppModel {
    *    (such as charges not paid).
    */
 
-  function findUnreconciledLedgerEntries($id = null, $fundamental_type = null, $cond = null) {
-    if (!isset($cond))
-      $cond = array();
-    $cond[] = array('Account.id' => $id);
-
+  function findUnreconciledLedgerEntries($id = null, $fundamental_type = null) {
     foreach (($fundamental_type
 	      ? array($fundamental_type)
 	      : array('debit', 'credit')) AS $fund) {
       $ucfund = ucfirst($fund);
-      $unreconciled[$fund]['LedgerEntry'] = $this->find
+      $unreconciled[$fund]['entry'] = $this->find
 	('all', array
 	 ('link' => array
 	  ('Ledger' => array
 	   ('fields' => array(),
-	    'LedgerEntry' => array
+	    "LedgerEntry" => array
 	    ('class' => "{$ucfund}LedgerEntry",
 	     'fields' => array('id', 'amount'),
-	     'ReconcilingTransaction' => array
-	     ('fields' => array
+	     "ReconciliationLedgerEntry" => array
+	     ('class' => "{$ucfund}ReconciliationLedgerEntry",
+	      'fields' => array
 	      ("COALESCE(SUM(Reconciliation.amount),0) AS 'reconciled'",
 	       "LedgerEntry.amount - COALESCE(SUM(Reconciliation.amount),0) AS 'balance'",
 	       ),
-	      'conditions' => array('Reconciliation.type' => $fund),
 	      ),
 	     ),
 	    ),
@@ -247,11 +260,11 @@ class Account extends AppModel {
 	  'group' => ("LedgerEntry.id" .
 		      " HAVING LedgerEntry.amount" .
 		      " <> COALESCE(SUM(Reconciliation.amount),0)"),
-	  'conditions' => $cond,
+	  'conditions' => array('Account.id' => $id),
 	  'fields' => array(),
 	  ));
       $balance = 0;
-      foreach ($unreconciled[$fund]['LedgerEntry'] AS &$entry) {
+      foreach ($unreconciled[$fund]['entry'] AS &$entry) {
 	$entry = array_merge(array_diff_key($entry["LedgerEntry"], array(0=>true)),
 			     $entry[0]);
 	$balance += $entry['balance'];
@@ -277,24 +290,24 @@ class Account extends AppModel {
    *    whatever algorithm is simplest.
    */
 
-  function reconcileNewLedgerEntry($id, $fundamental_type, $amount, $cond = null) {
+  function reconcileNewLedgerEntry($id, $fundamental_type, $amount) {
     $ofund = $this->fundamentalOpposite($fundamental_type);
-    $unreconciled = array($ofund => array('LedgerEntry'=>array(), 'balance'=>0));
+    $unreconciled = array($ofund => array('entry'=>array(), 'balance'=>0));
     $applied = 0;
 
     // if there is no money in the entry, it can reconcile nothing
     // don't bother wasting time sifting ledger entries.
     if ($amount > 0) {
-      $unreconciled = $this->findUnreconciledLedgerEntries($id, $ofund, $cond);
+      $unreconciled = $this->findUnreconciledLedgerEntries($id, $ofund);
 
-      foreach ($unreconciled[$ofund]['LedgerEntry'] AS $i => &$entry) {
+      foreach ($unreconciled[$ofund]['entry'] AS $i => &$entry) {
 	// Determine if amount is sufficient to cover the entry
 	if ($amount > $entry['balance'])
 	  $apply = $entry['balance'];
 	elseif ($amount > 0)
 	  $apply = $amount;
 	else {
-	  unset($unreconciled[$ofund]['LedgerEntry'][$i]);
+	  unset($unreconciled[$ofund]['entry'][$i]);
 	  continue;
 	}
 

models/behaviors/linkable.php

@@ -85,28 +85,6 @@ class LinkableBehavior extends ModelBehavior {
 
 	protected $_defaults = array('type' => 'LEFT');
 	
-	function pr($lev, $mixed) {
-		if ($lev >= 5)
-			return;
-
-		pr($mixed);
-		return;
-
-		$trace = debug_backtrace(false);
-		//array_shift($trace);
-		$calls = array();
-		foreach ($trace AS $call) {
-			$call = array_intersect_key($call,
-						    array('file'=>1,
-							  'line'=>1,
-							  //'class'=>1,
-							  'function'=>1,
-							  ));
-			$calls[] = implode("; ", $call);
-		}
-		pr(array('debug' => $mixed, 'stack' => $calls));
-	}
-
 	/*
 	 * This is a function for made recursive str_replaces in an array
 	 * NOTE: The palacement of this function is terrible, but I don't
@@ -130,10 +108,7 @@ class LinkableBehavior extends ModelBehavior {
 	}
 
 	public function beforeFind(&$Model, $query) {
-		$this->pr(10,
-			  array('function' => 'Linkable::beforeFind',
-				'args' => array('Model->alias' => '$Model->alias') + compact('query'),
-				));
+/* 	  pr("Linkable::beforeFind() begin"); pr($query); */
 		if (isset($query[$this->_key])) {
 			$optionsDefaults = $this->_defaults + array('reference' =>
 			                                            array('class' => $Model->alias,
@@ -157,10 +132,7 @@ class LinkableBehavior extends ModelBehavior {
 					unset($iterator['defaults']);
 				}
 				$iterations = Set::normalize($iterator);
-				$this->pr(25,
-					  array('checkpoint' => 'Iterations',
-						compact('iterations'),
-						));
+/* 				pr(array('checkpoint' => 'Iterations', compact('iterations'))); */
 				foreach ($iterations as $alias => $options) {
 					if (is_null($options)) {
 						$options = array();
@@ -173,15 +145,7 @@ class LinkableBehavior extends ModelBehavior {
 					if (empty($options['class']))
 						$options['class'] = $alias;
 
-					if (!isset($options['conditions']))
-						$options['conditions'] = array();
-					elseif (!is_array($options['conditions']))
-						$options['conditions'] = array($options['conditions']);
-
-					$this->pr(20,
-						  array('checkpoint' => 'Begin Model Work',
-							compact('alias', 'options'),
-							));
+/* 					pr(array('checkpoint' => 'Begin Model Work', compact('alias', 'options'))); */
 
 					$modelClass     = $options['class'];
 					$modelAlias     = $options['alias'];
@@ -190,13 +154,11 @@ class LinkableBehavior extends ModelBehavior {
 
 					$_Model =& ClassRegistry::init($modelClass);         // the incoming model to be linked in query
 					$Reference =& ClassRegistry::init($referenceClass);  // the already in query model that links to $_Model
-					$this->pr(12,
-						  array('checkpoint' => 'Aliases Established',
-							'Model' => ($modelAlias .' : '. $modelClass .
-								    ' ('. $_Model->alias .' : '. $_Model->name .')'),
-							'Reference' => ($referenceAlias .' : '. $referenceClass .
-									' ('. $Reference->alias .' : '. $Reference->name .')'),
-							));
+/* 					pr(array('checkpoint' => 'Aliases Established', */
+/* 						 'Model' => ($modelAlias .' : '. $modelClass . */
+/* 							     ' ('. $_Model->alias .' : '. $_Model->name .')'), */
+/* 						 'Reference' => ($referenceAlias .' : '. $referenceClass . */
+/* 								 ' ('. $Reference->alias .' : '. $Reference->name .')'))); */
 
 					$db =& $_Model->getDataSource();
 
@@ -207,20 +169,20 @@ class LinkableBehavior extends ModelBehavior {
 					// a relationship if one doesn't otherwise already exists.
 				        if (($associations = $Reference->getAssociated()) &&
 					    isset($associations[$_Model->alias])) {
-						$this->pr(12, array('checkpoint' => "Reference defines association to _Model"));
+/* 						pr("Reference defines association to _Model"); */
 						$associatedThroughReference = 1;
 						$type = $associations[$_Model->alias];
 						$association = $Reference->{$type}[$_Model->alias];
 					}
 					elseif (($associations = $_Model->getAssociated()) &&
 					    isset($associations[$Reference->alias])) {
-						$this->pr(12, array('checkpoint' => "_Model defines association to Reference"));
+/* 						pr("_Model defines association to Reference"); */
 						$type = $associations[$Reference->alias];
 						$association = $_Model->{$type}[$Reference->alias];
 					}
 					else {
 						// No relationship... make our best effort to create one.
-						$this->pr(12, array('checkpoint' => "No assocation between _Model and Reference"));
+/* 						pr("No assocation between _Model and Reference"); */
 						$type = 'belongsTo';
 						$_Model->bind($Reference->alias);
 						// Grab the association now, since we'll unbind in a moment.
@@ -250,119 +212,93 @@ class LinkableBehavior extends ModelBehavior {
 								       $associationAlias,
 								       $association['conditions']);
 
-					$this->pr(15,
-						  array('checkpoint' => 'Models Established - Check Associations',
-							'primaryModel' => $primaryAlias .' : '. $primaryModel->name,
-							'foreignModel' => $foreignAlias .' : '. $foreignModel->name,
-							compact('type', 'association'),
-							));
+/* 					pr(array('checkpoint' => 'Models Established - Check Associations', */
+/* 						 'primaryModel' => $primaryAlias .' : '. $primaryModel->name, */
+/* 						 'foreignModel' => $foreignAlias .' : '. $foreignModel->name, */
+/* 						 compact('type', 'association'))); */
 
-					if ($type === 'hasAndBelongsToMany') {
-						if (isset($association['with']))
-							$linkClass = $association['with'];
-						else
-							$linkClass = Inflector::classify($association['joinTable']);
+					if (empty($options['conditions'])) {
+						if ($type === 'hasAndBelongsToMany') {
+							if (isset($association['with']))
+								$linkClass = $association['with'];
+							else
+								$linkClass = Inflector::classify($association['joinTable']);
 
-						$Link =& $_Model->{$linkClass};
+							$Link =& $_Model->{$linkClass};
 
-						if (isset($options['linkalias']))
-							$linkAlias = $options['linkalias'];
-						else
-							$linkAlias = $Link->alias;
+							if (isset($options['linkalias']))
+								$linkAlias = $options['linkalias'];
+							else
+								$linkAlias = $Link->alias;
 
-						$this->pr(17,
-							  array('checkpoint' => 'Linking HABTM',
-								compact('linkClass', 'linkAlias'),
-								));
+							// Get the foreign key fields (for the link table) directly from
+							// the defined model associations, if they exists.  This is the
+							// users direct specification, and therefore definitive if present.
+							$modelLink = $Link->escapeField($association['foreignKey'], $linkAlias);
+							$referenceLink = $Link->escapeField($association['associationForeignKey'], $linkAlias);
 
-						// Get the foreign key fields (for the link table) directly from
-						// the defined model associations, if they exists.  This is the
-						// users direct specification, and therefore definitive if present.
-						$modelLink = $Link->escapeField($association['foreignKey'], $linkAlias);
-						$referenceLink = $Link->escapeField($association['associationForeignKey'], $linkAlias);
+							// If we haven't figured out the foreign keys, see if there is a
+							// model for the link table, and if it has the appropriate
+							// associations with the two tables we're trying to join.
+							if (empty($modelLink) && isset($Link->belongsTo[$_Model->alias]))
+								$modelLink = $Link->escapeField($Link->belongsTo[$_Model->alias]['foreignKey'], $linkAlias);
+							if (empty($referenceLink) && isset($Link->belongsTo[$Reference->alias]))
+								$referenceLink = $Link->escapeField($Link->belongsTo[$Reference->alias]['foreignKey'], $linkAlias);
 
-						// If we haven't figured out the foreign keys, see if there is a
-						// model for the link table, and if it has the appropriate
-						// associations with the two tables we're trying to join.
-						if (empty($modelLink) && isset($Link->belongsTo[$_Model->alias]))
-							$modelLink = $Link->escapeField($Link->belongsTo[$_Model->alias]['foreignKey'], $linkAlias);
-						if (empty($referenceLink) && isset($Link->belongsTo[$Reference->alias]))
-							$referenceLink = $Link->escapeField($Link->belongsTo[$Reference->alias]['foreignKey'], $linkAlias);
+							// We're running quite thin here.  None of the models spell
+							// out the appropriate linkages.  We'll have to SWAG it.
+							if (empty($modelLink))
+								$modelLink = $Link->escapeField(Inflector::underscore($_Model->alias) . '_id', $linkAlias);
+							if (empty($referenceLink))
+								$referenceLink = $Link->escapeField(Inflector::underscore($Reference->alias) . '_id', $linkAlias);
 
-						// We're running quite thin here.  None of the models spell
-						// out the appropriate linkages.  We'll have to SWAG it.
-						if (empty($modelLink))
-							$modelLink = $Link->escapeField(Inflector::underscore($_Model->alias) . '_id', $linkAlias);
-						if (empty($referenceLink))
-							$referenceLink = $Link->escapeField(Inflector::underscore($Reference->alias) . '_id', $linkAlias);
-
-						// Get the primary key from the tables we're joining.
-						$referenceKey = $Reference->escapeField(null, $referenceAlias);
-						$modelKey = $_Model->escapeField(null, $modelAlias);
+							// Get the primary key from the tables we're joining.
+							$referenceKey = $Reference->escapeField(null, $referenceAlias);
+							$modelKey = $_Model->escapeField(null, $modelAlias);
 								
-						// Join the linkage table to our model.  We'll use an inner join,
-						// as the whole purpose of the linkage table is to make this
-						// connection.  As we are embedding this join, the INNER will not
-						// cause any problem with the overall query, should the user not
-						// be concerned with whether or not the join has any results.
-						// They control that with the 'type' parameter which will be at
-						// the top level join.
-						$options['joins'][] = array('type'  => 'INNER',
-									    'alias' => $modelAlias,
-									    'conditions' => "{$modelKey} = {$modelLink}",
-									    'table' => $db->fullTableName($_Model, true));
+							// Join the linkage table to our model.  We'll use an inner join,
+							// as the whole purpose of the linkage table is to make this
+							// connection.  As we are embedding this join, the INNER will not
+							// cause any problem with the overall query, should the user not
+							// be concerned with whether or not the join has any results.
+							// They control that with the 'type' parameter which will be at
+							// the top level join.
+							$options['joins'][] = array('type'  => 'INNER',
+							                            'alias' => $modelAlias,
+							                            'conditions' => "{$modelKey} = {$modelLink}",
+							                            'table' => $db->fullTableName($_Model, true));
+
+							// Now for the top level join.  This will be added into the list
+							// of joins down below, outside of the HABTM specific code.
+							$options['class'] = $linkClass;
+							$options['alias'] = $linkAlias;
+							$options['table'] = $Link->getDataSource()->fullTableName($Link);
+							$options['conditions'][] = "{$referenceLink} = {$referenceKey}";
+						}
+						elseif (isset($association['foreignKey']) && $association['foreignKey']) {
+							$foreignKey = $primaryModel->escapeField($association['foreignKey'], $primaryAlias);
+							$primaryKey = $foreignModel->escapeField($foreignModel->primaryKey, $foreignAlias);
+
+							// Only differentiating to help show the logical flow.
+							// Either way works and this test can be tossed out
+							if (($type === 'hasMany' || $type === 'hasOne') ^ $associatedThroughReference)
+								$options['conditions'][] = "{$primaryKey} = {$foreignKey}";
+							else
+								$options['conditions'][] = "{$foreignKey} = {$primaryKey}";
+						}
+						else {
+							// No Foreign Key... nothing we can do.
+							$options['conditions'] = array();
+						}
+
+						// The user may have specified conditions directly in the model
+						// for this join.  Make sure to adhere to those conditions.
+						if (isset($association['conditions']) && is_array($association['conditions']))
+							$options['conditions'] = array_merge($options['conditions'], $association['conditions']);
+						elseif (!empty($association['conditions']))
+							$options['conditions'][] = $association['conditions'];
 
-						// Now for the top level join.  This will be added into the list
-						// of joins down below, outside of the HABTM specific code.
-						$options['class'] = $linkClass;
-						$options['alias'] = $linkAlias;
-						$options['table'] = $Link->getDataSource()->fullTableName($Link);
-						$options['conditions'][] = "{$referenceLink} = {$referenceKey}";
 					}
-					elseif (isset($association['foreignKey']) && $association['foreignKey']) {
-						$foreignKey = $primaryModel->escapeField($association['foreignKey'], $primaryAlias);
-						$primaryKey = $foreignModel->escapeField($foreignModel->primaryKey, $foreignAlias);
-
-						$this->pr(17,
-							  array('checkpoint' => 'Linking due to foreignKey',
-								compact('foreignKey', 'primaryKey'),
-								));
-
-						// Only differentiating to help show the logical flow.
-						// Either way works and this test can be tossed out
-						if (($type === 'hasMany' || $type === 'hasOne') ^ $associatedThroughReference)
-							$options['conditions'][] = "{$primaryKey} = {$foreignKey}";
-						else
-							$options['conditions'][] = "{$foreignKey} = {$primaryKey}";
-					}
-					else {
-						$this->pr(17,
-							  array('checkpoint' => 'Linking with no logic (expecting user defined)',
-								));
-
-						// No Foreign Key... nothing we can do.
-					}
-
-					$this->pr(19,
-						  array('checkpoint' => 'Conditions',
-							array('options[conditions]' => $options['conditions'],
-							      'association[conditions]' => $association['conditions'],
-							      ),
-							));
-
-					// The user may have specified conditions directly in the model
-					// for this join.  Make sure to adhere to those conditions.
-					if (isset($association['conditions']) && is_array($association['conditions']))
-						$options['conditions'] = array_merge($options['conditions'], $association['conditions']);
-					elseif (!empty($association['conditions']))
-						$options['conditions'][] = $association['conditions'];
-
-					$this->pr(19,
-						  array('checkpoint' => 'Conditions2',
-							array('options[conditions]' => $options['conditions'],
-							      ),
-							));
-
 					if (empty($options['table'])) {
 						$options['table'] = $db->fullTableName($_Model, true);
 					}
@@ -376,31 +312,26 @@ class LinkableBehavior extends ModelBehavior {
 								       (empty($association['fields']) 
 									? array() : $db->fields($_Model, $modelAlias, $association['fields'])));
 									       
+/* 					pr(array('checkpoint' => 'Model Work Complete', compact('options', 'modelClass', 'modelAlias'))); */
+
 					$options[$this->_key] = am($options[$this->_key], array_diff_key($options, $optionsKeys));
 					$options = array_intersect_key($options, $optionsKeys);
 					if (!empty($options[$this->_key])) {
 						$iterators[] = $options[$this->_key] +
 							array('defaults' =>
-							      array_merge($defaults,
+										array_merge($defaults,
 							                  array('reference' =>
 							                        array('class' => $modelClass,
 							                              'alias' => $modelAlias))));
 					}
 					$query['joins'][] = array_intersect_key($options, array('type' => true, 'alias' => true, 'table' => true, 'joins' => true, 'conditions' => true));
-
-					$this->pr(19,
-						  array('checkpoint' => 'Model Join Complete',
-							compact('options', 'modelClass', 'modelAlias', 'query'),
-							));
 				}
 				++$cont;
 				$notDone = isset($iterators[$cont]);
 			} while ($notDone);
 		}
-		$this->pr(20,
-			  array('function' => 'Linkable::beforeFind',
-				'return' => compact('query'),
-				));
+/* 		pr(array('checkpoint' => 'Linkable::beforeFind() end', */
+/* 		         compact('query'))); */
 		return $query;
 	}
 }

models/customer.php

@@ -8,6 +8,7 @@ class Customer extends AppModel {
 	);
 
 	var $belongsTo = array(
+		'Account',
 		'PrimaryContact' => array(
 			'className' => 'Contact',
 		),
@@ -19,19 +20,11 @@ class Customer extends AppModel {
 			'conditions' => 'CurrentLease.close_date IS NULL',
 		),
 		'Lease',
-		'LedgerEntry',
-
-		// Cheat to get Account set as part of this class
-		'Account',
+		'Transaction',
 	);
 
 	var $hasAndBelongsToMany = array(
 		'Contact',
-		'Transaction' => array(
-			'joinTable' => 'ledger_entries',
-			'foreignKey' => 'customer_id',
-			'associationForeignKey' => 'transaction_id',
-		),
 	);
 
 
@@ -87,9 +80,16 @@ class Customer extends AppModel {
 /* 	     )); */
 
     $entries = $this->Account->findLedgerEntriesRelatedToAccount
-      ($this->Account->invoiceAccountID(),
+      ($this->accountId($id),
        $this->Account->securityDepositAccountID(),
-       true, array('LedgerEntry.customer_id' => $id), $link);
+       true, null, $link);
+
+    foreach ($this->leaseIds($id) AS $lease_id) {
+      $ledger_entries = $this->Lease->findSecurityDeposits($lease_id, $link);
+      $this->statsMerge($entries['summary'], $ledger_entries['summary']);
+      $entries['Entries'] = array_merge($entries['Entries'],
+					$ledger_entries['Entries']);
+    }
 
 /*     pr(array('function' => 'Customer::findSecurityDeposits', */
 /* 	     'args' => compact('id', 'link'), */
@@ -110,10 +110,22 @@ class Customer extends AppModel {
    */
 
   function findUnreconciledLedgerEntries($id = null, $fundamental_type = null) {
+
     $unreconciled = $this->Account->findUnreconciledLedgerEntries
-      ($this->Account->accountReceivableAccountID(),
-       $fundamental_type,
-       array('LedgerEntry.customer_id' => $id));
+      ($this->accountId($id), $fundamental_type);
+
+    foreach ($this->leaseIds($id) AS $lease_id) {
+      $unrec = $this->Lease->findUnreconciledLedgerEntries($lease_id,
+							   $fundamental_type);
+
+      foreach (array_keys($unreconciled) AS $type) {
+	$left  = &$unreconciled[$type];
+	$right = &$unrec[$type];
+
+	$left['entry']    = array_merge($left['entry'], $right['entry']);
+	$left['balance'] += $right['balance'];
+      }
+    }
 
     return $unreconciled;
   }
@@ -134,11 +146,25 @@ class Customer extends AppModel {
    */
 
   function reconcileNewLedgerEntry($id, $fundamental_type, $amount) {
+
     $reconciled = $this->Account->reconcileNewLedgerEntry
-      ($this->Account->accountReceivableAccountID(),
-       $fundamental_type,
-       $amount,
-       array('LedgerEntry.customer_id' => $id));
+      ($this->accountId($id), $fundamental_type, $amount);
+
+    foreach ($this->leaseIds($id) AS $lease_id) {
+      foreach (array_keys($reconciled) AS $type) {
+	$rec = $this->Lease->reconcileNewLedgerEntry($lease_id,
+						     $fundamental_type,
+						     $reconciled[$type]['unapplied']);
+
+	$left  = &$reconciled[$type];
+	$right = &$rec[$type];
+
+	$left['entry']      = array_merge($left['entry'], $right['entry']);
+	$left['balance']   += $right['balance'];
+	$left['applied']   += $right['applied'];
+	$left['unapplied']  = $right['unapplied'];
+      }
+    }
 
     return $reconciled;
   }
@@ -163,6 +189,7 @@ class Customer extends AppModel {
 	       'ContactEmail',
 	       'ContactAddress',
 	       ),
+	 'Account',
 	 'Lease' =>
 	 array('Unit' =>
 	       array('order'  => array('sort_order'),
@@ -174,12 +201,22 @@ class Customer extends AppModel {
 	'conditions' => array('Customer.id' => $id),
 	));
 
-    // Figure out the outstanding balance for this customer
+    // Add the lease balance to each lease.
+    foreach ($customer['Lease'] AS &$lease) {
+      $stats = $this->Lease->stats($lease['id']);
+      $lease['balance'] = $stats['Account']['Ledger']['balance'];
+    }
+
+    // Figure out the outstanding balance of the current lease.
     $customer['stats'] = $this->stats($id);
 
     // Figure out the total security deposit for the current lease.
     $customer['deposits'] = $this->findSecurityDeposits($id);
 
+    // Add statistics into the customer account.
+    $customer['Account'] = array_merge($customer['Account'],
+				       $customer['stats']['Account']['Ledger']);
+
     return $customer;
   }
 
@@ -195,11 +232,29 @@ class Customer extends AppModel {
     if (!$id)
       return null;
 
-    $stats = $this->Account->stats($this->Account->accountReceivableAccountID(), true,
-				   array('LedgerEntry.customer_id' => $id));
+    // Get the basic information necessary
+    $customer = $this->find('first',
+			    array('contain' =>
+				  array('Account' => array
+					('fields' => array('Account.id')),
+
+					'Lease'   => array
+					('fields' => array('Lease.id'))
+					),
+				  'conditions' => array
+				  (array('Customer.id' => $id))));
+
+    // Get stats from the customer account, and each lease
+    $stats['Account'] = $this->Account->stats($customer['Account']['id']);
+    foreach ($customer['Lease'] AS $lease) {
+      $this->statsMerge($stats['Lease'], $this->Lease->stats($lease['id']));
+    }
+
+    // Merge the stats from both the customer specific account, as
+    // well as the leases.  This will provide current customer standing.
+    $this->statsMerge($stats, $stats['Account']['Ledger']);
+    $this->statsMerge($stats, $stats['Lease']['Account']['Ledger']);
 
-    // Pull to the top level and return
-    $stats = $stats['Ledger'];
     return $stats;
   }
 

models/lease.php

@@ -25,18 +25,11 @@ class Lease extends AppModel {
 	var $belongsTo = array(
 		'LeaseType',
 		'Unit',
+		'Account',
 		'Customer',
 		'LateSchedule',
 	);
 
-	var $hasMany = array(
-		'LedgerEntry',
-
-		// Cheat to get Account set as part of this class
-		'Account',
-	);
-
-
   /**************************************************************************
    **************************************************************************
    **************************************************************************
@@ -44,7 +37,15 @@ class Lease extends AppModel {
    *  - Returns the accountId of the given lease
    */
   function accountId($id) {
-    return $this->Account->invoiceAccountID();
+    $this->cacheQueries = true;
+    $lease = $this->find('first', array
+			  ('recursive' => -1,
+			   'fields' => array('account_id'),
+			   'conditions' => array(array('id' => $id)),
+			   ));
+    $this->cacheQueries = false;
+
+    return $lease['Lease']['account_id'];
   }
 
 
@@ -60,10 +61,6 @@ class Lease extends AppModel {
 /* 	     'args' => compact('id', 'all', 'cond', 'link'), */
 /* 	     )); */
 
-    if (!isset($cond))
-      $cond = array();
-    $cond[] = array('LedgerEntry.lease_id' => $id);
-
     $entries = $this->Account->findLedgerEntries($this->accountId($id),
 						 $all, $cond, $link);
 
@@ -90,7 +87,7 @@ class Lease extends AppModel {
     $entries = $this->Account->findLedgerEntriesRelatedToAccount
       ($this->accountId($id),
        $this->Account->securityDepositAccountID(),
-       true, array('LedgerEntry.lease_id' => $id), $link);
+       true, null, $link);
 
 /*     pr(array('function' => 'Lease::findSecurityDeposits', */
 /* 	     'args' => compact('id', 'link'), */
@@ -111,7 +108,7 @@ class Lease extends AppModel {
 
   function findUnreconciledLedgerEntries($id = null, $fundamental_type = null) {
     return $this->Account->findUnreconciledLedgerEntries
-      ($this->accountId($id), $fundamental_type, array('LedgerEntry.lease_id' => $id));
+      ($this->accountId($id), $fundamental_type);
   }
 
 
@@ -131,7 +128,7 @@ class Lease extends AppModel {
 
   function reconcileNewLedgerEntry($id, $fundamental_type, $amount) {
     return $this->Account->reconcileNewLedgerEntry
-      ($this->accountId($id), $fundamental_type, $amount, array('LedgerEntry.lease_id' => $id));
+      ($this->accountId($id), $fundamental_type, $amount);
   }
 
 
@@ -146,15 +143,18 @@ class Lease extends AppModel {
     if (!$id)
       return null;
 
-    $stats = $this->Account->stats($this->Account->accountReceivableAccountID(), true,
-				   array('OR' => array
-					 (array('LedgerEntry.lease_id' => $id),
-					  array('ReconciledLedgerEntry.lease_id' => $id),
-					  ),
-					 ));
+    // Find the associated account.
+    $lease = $this->find('first',
+			 array('recursive' => -1,
+			       'conditions' => array(array('Lease.id' => $id))));
+
+    // Pull the stats from the account.
+    $stats['Account'] = $this->Account->stats($lease['Lease']['account_id']);
+
+    // Place a summary of the stats (one lease account in this case)
+    // at the top level for easy summarized access.
+    $this->statsMerge($stats, $stats['Account']['Ledger']);
 
-    // Pull to the top level and return
-    $stats = $stats['Ledger'];
     return $stats;
   }
 

models/ledger.php

@@ -115,9 +115,6 @@ class Ledger extends AppModel {
    *  - Returns summary data from the requested ledger.
    */
   function stats($id, $cond = null) {
-    if (!isset($cond))
-      $cond = array();
-    $cond[] = array('Ledger.id' => $id);
 
     $stats = $this->find
       ('first', array
@@ -141,7 +138,8 @@ class Ledger extends AppModel {
                       ) * IF(LedgerEntry.amount, LedgerEntry.amount, 0)
                    ) AS balance",
 	      "COUNT(LedgerEntry.id) AS entries"),
-	'conditions' => $cond,
+	'conditions' => array(isset($cond) ? $cond : array(),
+			      array('Ledger.id' => $id)),
 	'group' => 'Ledger.id',
 	));
 

models/ledger_entry.php

@@ -11,8 +11,6 @@ class LedgerEntry extends AppModel {
 	var $belongsTo = array(
 		'MonetarySource',
 		'Transaction',
-		'Customer',
-		'Lease',
 
 		'DebitLedger' => array(
 			'className' => 'Ledger',
@@ -25,11 +23,17 @@ class LedgerEntry extends AppModel {
 	);
 
 	var $hasAndBelongsToMany = array(
-		'ReconcilingTransaction' => array(
-			'className' => 'Transaction',
+		'DebitReconciliationLedgerEntry' => array(
+			'className' => 'LedgerEntry',
 			'joinTable' => 'reconciliations',
-			'foreignKey' => 'transaction_id',
-			'associationForeignKey' => 'ledger_entry_id',
+			'foreignKey' => 'credit_ledger_entry_id',
+			'associationForeignKey' => 'debit_ledger_entry_id',
+		),
+		'CreditReconciliationLedgerEntry' => array(
+			'className' => 'LedgerEntry',
+			'joinTable' => 'reconciliations',
+			'foreignKey' => 'debit_ledger_entry_id',
+			'associationForeignKey' => 'credit_ledger_entry_id',
 		),
 	);
 
@@ -80,7 +84,7 @@ class LedgerEntry extends AppModel {
     return $fields;
   }
 
-  function ledgerContextFields2($ledger_id = null, $account_id = null, $account_type = null) {
+  function ledgerContextFields2($ledger_id = null, $account_type = null) {
     $fields = array('id', 'name', 'comment', 'amount');
 
     if (isset($ledger_id)) {
@@ -89,29 +93,20 @@ class LedgerEntry extends AppModel {
       $fields[] = ("IF(LedgerEntry.credit_ledger_id = $ledger_id," .
 		   "   SUM(LedgerEntry.amount), NULL) AS credit");
 
-      if (isset($account_id) || isset($account_type)) {
-	$Account = new Account();
-	$account_ftype = $Account->fundamentalType($account_id ? $account_id : $account_type);
-	$fields[] = ("(IF(LedgerEntry.{$account_ftype}_ledger_id = $ledger_id," .
+      if (isset($account_type)) {
+	if (in_array($account_type, array('ASSET', 'EXPENSE')))
+	  $ledger_type = 'debit';
+	else
+	  $ledger_type = 'credit';
+
+	$fields[] = ("(IF(LedgerEntry.{$ledger_type}_ledger_id = $ledger_id," .
 		     "    1, -1) * SUM(LedgerEntry.amount)) AS balance");
       }
     }
-    elseif (isset($account_id)) {
-      $fields[] = ("IF(DebitLedger.account_id = $account_id," .
-		   "   SUM(LedgerEntry.amount), NULL) AS debit");
-      $fields[] = ("IF(CreditLedger.account_id = $account_id," .
-		   "   SUM(LedgerEntry.amount), NULL) AS credit");
-
-      $Account = new Account();
-      $account_ftype = ucfirst($Account->fundamentalType($account_id));
-      $fields[] = ("(IF({$account_ftype}Ledger.account_id = $account_id," .
-		   "    1, -1) * SUM(LedgerEntry.amount)) AS balance");
-    }
 
     return $fields;
   }
 
-
   function ledgerContextConditions($ledger_id, $account_type) {
     if (isset($ledger_id)) {
       return array
@@ -161,36 +156,36 @@ class LedgerEntry extends AppModel {
   /**************************************************************************
    **************************************************************************
    **************************************************************************
-   * function: findReconcilingTransactions
-   *  - Returns transactions that reconcile the given entry
-   *    (such as receipts that reconcile a charge).
+   * function: findReconciledLedgerEntries
+   *  - Returns ledger entries that are reconciled to the given entry.
+   *    (such as payments towards a charge).
    */
 
-  function findReconcilingTransactions($id = null, $fundamental_type = null) {
+  function findReconciledLedgerEntries($id = null, $fundamental_type = null) {
     foreach (($fundamental_type
 	      ? array($fundamental_type)
 	      : array('debit', 'credit')) AS $fund) {
-      $reconciled[$fund]['LedgerEntry'] = $this->find
+      $ucfund = ucfirst($fund);
+      $reconciled[$fund]['entry'] = $this->find
 	('all', array
 	 ('link' => array
-	  ('ReconcilingTransaction' => array
-	   ('fields' => array
+	  ("ReconciliationLedgerEntry" => array
+	   ('class' => "{$ucfund}ReconciliationLedgerEntry",
+	    'fields' => array
 	    ('id',
 	     "COALESCE(SUM(Reconciliation.amount),0) AS 'reconciled'",
-	     //"ReconcilingTransaction.amount - COALESCE(SUM(Reconciliation.amount),0) AS 'balance'",
-	     "COALESCE(7) AS 'balance'",
+	     "LedgerEntry.amount - COALESCE(SUM(Reconciliation.amount),0) AS 'balance'",
 	     ),
-	    'conditions' => array('Reconciliation.type' => $fund),
 	    ),
 	   ),
-	  'group' => ('ReconcilingTransaction.id'),
+	  'group' => ("ReconciliationLedgerEntry.id"),
 	  'conditions' => array('LedgerEntry.id' => $id),
 	  'fields' => array(),
 	  ));
       //pr($reconciled);
       $balance = 0;
-      foreach ($reconciled[$fund]['LedgerEntry'] AS &$entry) {
-	$entry = array_merge($entry["ReconcilingTransaction"], $entry[0]);
+      foreach ($reconciled[$fund]['entry'] AS &$entry) {
+	$entry = array_merge($entry["ReconciliationLedgerEntry"], $entry[0]);
 	$balance += $entry['balance'];
       }
       $reconciled[$fund]['balance'] = $balance;
@@ -206,31 +201,27 @@ class LedgerEntry extends AppModel {
    * function: stats
    *  - Returns summary data from the requested ledger entry
    */
-  function stats($id, $cond = null) {
-
-    if (!isset($cond))
-      $cond = array();
-
-    $cond[] = array('LedgerEntry.id' => $id);
+  function stats($id) {
 
     $query = array
       (
-       'link' => array('ReconcilingTransaction'),
        'fields' => array("SUM(Reconciliation.amount) AS 'reconciled'"),
+
+       'conditions' => array(isset($cond) ? $cond : array(),
+			     array('LedgerEntry.id' => $id)),
+
        'group' => 'LedgerEntry.id',
        );
 
     // Get the applied amounts on the debit side
-    $qcond = $cond;
-    $qcond[] = array('Reconciliation.type' => 'DEBIT');
-    $query['conditions'] = $qcond;
+    $query['link'] =
+      array('DebitReconciliationLedgerEntry' => array('alias' => 'DRLE', 'DRLETransaction' => array('class' => 'Transaction')));
     $tmpstats = $this->find('first', $query);
     $stats['debit_amount_reconciled'] = $tmpstats[0]['reconciled'];
 
     // Get the applied amounts on the credit side
-    $qcond = $cond;
-    $qcond[] = array('Reconciliation.type' => 'CREDIT');
-    $query['conditions'] = $qcond;
+    $query['link'] =
+      array('CreditReconciliationLedgerEntry' => array('alias' => 'CRLE', 'CRLETransaction' => array('class' => 'Transaction')));
     $tmpstats = $this->find('first', $query);
     $stats['credit_amount_reconciled'] = $tmpstats[0]['reconciled'];
 

models/transaction.php

@@ -0,0 +1,28 @@
+<?php
+class Transaction extends AppModel {
+
+	var $name = 'Transaction';
+/* 	var $validate = array( */
+/* 		'stamp' => array('date') */
+/* 	); */
+
+	var $belongsTo = array(
+		'Customer',
+	);
+
+	var $hasMany = array(
+		'LedgerEntry',
+	);
+
+
+	function beforeSave() {
+    
+		if(!empty($this->data['Transaction']['stamp'])) {
+			$this->data['Transaction']['stamp'] =
+				$this->dateFormatBeforeSave($this->data['Transaction']['stamp']);
+		}
+		return true;
+	}
+
+}
+?>

site/models/transaction.php

@@ -1,71 +0,0 @@
-<?php
-class Transaction extends AppModel {
-
-	var $name = 'Transaction';
-/* 	var $validate = array( */
-/* 		'stamp' => array('date') */
-/* 	); */
-
-	var $belongsTo = array(
-		'Customer',
-	);
-
-	var $hasMany = array(
-		'LedgerEntry',
-	);
-
-	var $hasAndBelongsToMany = array(
-		'ReconciledLedgerEntry' => array(
-			'className' => 'LedgerEntry',
-			'joinTable' => 'reconciliations',
-			'foreignKey' => 'ledger_entry_id',
-			'associationForeignKey' => 'transaction_id',
-		),
-	);
-
-	function beforeSave() {
-    
-		if(!empty($this->data['Transaction']['stamp'])) {
-			$this->data['Transaction']['stamp'] =
-				$this->dateFormatBeforeSave($this->data['Transaction']['stamp']);
-		}
-		return true;
-	}
-
-  /**************************************************************************
-   **************************************************************************
-   **************************************************************************
-   * function: findReconciledLedgerEntries
-   *  - Returns ledger entries that are reconciled by the given
-   *    transaction (such as charges reconciled by a receipt)
-   */
-
-  function findReconciledLedgerEntries($id = null) {
-    $reconciled['LedgerEntry'] = $this->find
-      ('all', array
-       ('link' => array
-	('ReconciledLedgerEntry' => array
-	 ('fields' => array
-	  ('id',
-	   "COALESCE(SUM(Reconciliation.amount),0) AS 'reconciled'",
-	   "ReconciledLedgerEntry.amount - COALESCE(SUM(Reconciliation.amount),0) AS 'balance'",
-	   ),
-	  ),
-	 ),
-	'group' => ('ReconciledLedgerEntry.id'),
-	'conditions' => array('Transaction.id' => $id),
-	'fields' => array(),
-	));
-    //pr($reconciled);
-    $balance = 0;
-    foreach ($reconciled['LedgerEntry'] AS &$entry) {
-      $entry = array_merge($entry["ReconciledLedgerEntry"], $entry[0]);
-      $balance += $entry['balance'];
-    }
-    $reconciled['balance'] = $balance;
-
-  return $reconciled;
-  }
-
-}
-?>

views/customers/view.ctp

@@ -63,14 +63,12 @@ echo $this->element('leases',
 
 
 /**********************************************************************
- * Customer Account History
+ * Account
  */
 
-echo $this->element('ledger_entries',
+echo $this->element('accounts',
 		    array('caption' => 'Account',
-			  'customer_id' => $customer['Customer']['id'],
-			  'ar_account' => true,
-			  ));
+			  'accounts' => array($customer['Account'])));
 
 
 /* End "detail supporting" div */