abijah/pmgr
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Half-kludgy fix to a filtering bug when a table is already specified using the simple notation, e.g. array('Table'), instead of the more complete form array('Table' => array(...))

Browse Source 
git-svn-id: file:///svn-source/pmgr/branches/yafr_20090716/site@397 97e9348a-65ac-dc4b-aefc-98561f571b83
This commit is contained in:
abijah
2009-07-28 02:39:44 +00:00
parent cb5b017c7e
commit 1372bb68e4
1 changed files with 13 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
app_controller.php
View File

@@ -328,9 +328,20 @@ class AppController extends Controller {
array_merge_recursive($query[$link][$table], $config); array_merge_recursive($query[$link][$table], $config);
} }
elseif (in_array($table, $query[$link])) { elseif (in_array($table, $query[$link])) {
// REVISIT <AP>: 20090727
// This presents a dilema. $config may specify fields
// as array(), whereas the user has already indicated
// they desire ALL fields (by the fact that table is
// a member of the query['link'] array without anything
// specified). However, $config _could_ specify a
// non-standard field in the array, such as using SUM()
// or other equation. In that case, we'd have to add
// in all the fields of table to the array. That
// wouldn't be very hard, but I'll ignore it at the
// moment and wait until it's needed.
$key = array_search($table, $query[$link]); $key = array_search($table, $query[$link]);
$query[$link][$key] = $query[$link][$table] = array('fields' => null) + $config;
array_merge_recursive($query[$link][$key], $config); unset($query[$link][$key]);
} }
else { else {
// Table is NOT already part of the query... set it now // Table is NOT already part of the query... set it now