extern/egroupware_official
1
0
Fork 0
mirror of https://github.com/EGroupware/egroupware.git synced 2025-02-22 21:30:54 +01:00
Code Issues Packages Projects Releases Wiki Activity

* SyncML/Calendar/PostgreSQL: fixed SQL error when searching participants (SELECT DISTINCT requires ORDER-BY-expressions to be in SELECT-list)

Browse Source
This commit is contained in:
Ralf Becker 2011-04-11 12:10:23 +00:00
parent 3ee51f7a74
commit 79e6d38c41
1 changed files with 15 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
addressbook/inc/class.addressbook_sql.inc.php
View File

@ -432,17 +432,25 @@ class addressbook_sql extends so_sql
break;
}
// postgres requires that expressions in order by appear in the columns of a distinct select
if ($this->db->Type != 'mysql' && preg_match("/([a-zA-Z_.]+)<>''/",$order_by,$matches))
if ($this->db->Type != 'mysql' && preg_match_all("/([a-zA-Z_.]+) *(<> *''|IS NULL|IS NOT NULL)/u",$order_by,$all_matches,PREG_SET_ORDER))
{
if (!is_array($extra_cols)) $extra_cols = $extra_cols ? explode(',',$extra_cols) : array();
$extra_cols[] = $matches[1];
$extra_cols[] = $matches[1]."<>''";
//_debug_array($matches[1]);
if (!empty($order_by) && $matches[1] != 'extra_order.contact_value' && stripos($matches[1],'.')===false) // postgres requires explizit order by
foreach($all_matches as $matches)
{
$order_by = str_replace($matches[1],'egw_addressbook.'.$matches[1],$order_by);
$table = '';
if (strpos($matches[1],'.') === false)
{
$table = ($matches[1] == $this->extra_value ? $this->extra_table : $this->table_name).'.';
}
$extra_cols[] = $table.$matches[1];
$extra_cols[] = $table.$matches[0];
//_debug_array($matches);
if (!empty($order_by) && $table) // postgres requires explizit order by
{
$order_by = str_replace($matches[0],$table.$matches[0],$order_by);
}
}
//_debug_array($order_by);
//_debug_array($order_by); _debug_array($extra_cols);
}
}
// add join to show only active accounts (only if accounts are shown and in sql and we not already join the accounts table, eg. used by admin)