Customers is working somewhat. I want to see if the grid query logic can be simplified, so I'm checking in first

git-svn-id: file:///svn-source/pmgr/branches/yafr_20090716@359 97e9348a-65ac-dc4b-aefc-98561f571b83
This commit is contained in:
abijah
2009-07-21 09:40:51 +00:00
parent 59e6379977
commit fd856323a5
3 changed files with 41 additions and 73 deletions

View File

@@ -47,8 +47,6 @@ class CustomersController extends AppController {
function jqGridDataSetup(&$params) {
parent::jqGridDataSetup($params);
if (!isset($params['action']))
$params['action'] = 'all';
}
function jqGridDataCountTables(&$params, &$model) {
@@ -63,20 +61,7 @@ class CustomersController extends AppController {
function jqGridDataTables(&$params, &$model) {
$link = $this->jqGridDataCountTables($params, $model);
$link['link']['DoubleEntry'] = array('fields' => array());
$link['link']['DoubleEntry']['Ledger'] = array('fields' => array());
$link['link']['DoubleEntry']['Ledger']['Account'] = array('fields' => array());
// INNER JOIN would be great, as it would ensure we're only looking
// at the ledger entries that we truly want. However, this also
// removes from the query any units that do not yet have a ledger
// entry in A/R. A solution would be to INNER JOIN these tables,
// and LEFT JOIN it to the rest. Grouping of JOINs, however, is
// implemented with the 'joins' tag, and is not available through
// the Linkable behavior interface.
//$link['link']['DoubleEntry']['Ledger']['Account']['type'] = 'INNER';
$link['link']['DoubleEntry']['Ledger']['Account']['conditions']
= array('Account.id' =>
$this->Customer->DoubleEntry->Ledger->Account->accountReceivableAccountID());
$link['link']['StatementEntry'] = array('fields' => array());
return $link;
}
@@ -84,11 +69,8 @@ class CustomersController extends AppController {
$db = &$model->getDataSource();
$fields = $db->fields($model, $model->alias);
$fields[] = ('COUNT(DISTINCT CurrentLease.id) AS lease_count');
$fields[] = ("SUM(IF(Account.id IS NULL, 0," .
" IF(DoubleEntry.debit_ledger_id = Account.id," .
" 1, -1))" .
" * IF(DoubleEntry.amount IS NULL, 0, DoubleEntry.amount))" .
" AS 'balance'");
$fields = array_merge($fields,
$this->Customer->StatementEntry->chargePaymentFields(true));
return $fields;
}
@@ -137,7 +119,7 @@ class CustomersController extends AppController {
array('CurrentLease'=>1))),
array_diff_key($query, array('link'=>1))));
if ($params['action'] === 'all')
if ($params['action'] == 'all')
return $count;
$query['conditions'][] = 'CurrentLease.id IS NULL';
@@ -155,18 +137,6 @@ class CustomersController extends AppController {
return $count;
}
function jqGridDataRecords(&$params, &$model, $query) {
$customers = parent::jqGridDataRecords($params, $model, $query);
// Get the balance on each customer.
foreach ($customers AS &$customer) {
$stats = $this->Customer->stats($customer['Customer']['id']);
$customer['Customer']['balance'] = $stats['balance'];
}
return $customers;
}
function jqGridRecordLinks(&$params, &$model, &$records, $links) {
$links['Customer'] = array('name');
return parent::jqGridRecordLinks($params, $model, $records, $links);
@@ -437,17 +407,17 @@ class CustomersController extends AppController {
*/
function refund() {
$entries = $this->Customer->DoubleEntry->find
$entries = $this->Customer->StatementEntry->find
('all', array
('contain' => false,
'conditions' => array('DoubleEntry.id' =>
'conditions' => array('StatementEntry.id' =>
//array(199,200,201)
61
),
));
pr(compact('entries'));
$this->Customer->DoubleEntry->reverse($entries);
$this->Customer->StatementEntry->reverse($entries);
}
@@ -482,7 +452,7 @@ class CustomersController extends AppController {
unset($unreconciled['entries']['entry']);
//pr($unreconciled);
//$reconciled = $cust->reconcileNewDoubleEntry($cust_id, 'credit', $amount);
//$reconciled = $cust->reconcileNewStatementEntry($cust_id, 'credit', $amount);
$opts = array();
//$opts['format'] = 'tags';