tonyair/silverstripe-framework
1
0
Fork 0
mirror of https://github.com/silverstripe/silverstripe-framework synced 2024-06-30 16:39:38 +02:00
Code Issues Projects Releases Wiki Activity

BUGFIX Ensure end bracket for HAVING clause is put in the right place

Browse Source
This commit is contained in:
Sean Harvey 2012-05-07 16:35:01 +12:00
parent 5f5197b4c1
commit e50f90bf8e
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
model/Database.php
View File

@ -783,7 +783,7 @@ abstract class SS_Database {
* @return string
*/
public function sqlHavingToString($having) {
return ' HAVING ( ' . implode(' ) AND ( ', $having);
return ' HAVING ( ' . implode(' ) AND ( ', $having) . ')';
}
/**
@ -829,7 +829,7 @@ abstract class SS_Database {
// these clauses only make sense in SELECT queries, not DELETE
if(!$query->getDelete()) {
if($query->getGroupBy()) $text .= $this->sqlGroupByToString($query->getGroupBy());
if($query->getHaving()) $text .= $this->sqlHavingToString($query->getHaving()) . ' )';
if($query->getHaving()) $text .= $this->sqlHavingToString($query->getHaving());
if($query->getOrderBy()) $text .= $this->sqlOrderByToString($query->getOrderBy());
if($query->getLimit()) $text .= $this->sqlLimitToString($query->getLimit());
}