tonyair/silverstripe-framework
1
0
Fork 0
mirror of https://github.com/silverstripe/silverstripe-framework synced 2024-10-22 14:05:37 +02:00
Code Issues Projects Releases Wiki Activity

Merge remote branch 'origin/master' into translation-staging

Browse Source
This commit is contained in:
TeamCity 2012-09-24 13:19:30 +12:00
commit 21b6e3d0ed
3 changed files with 179 additions and 47 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
docs/en/topics/datamodel.md
View File

@ -248,10 +248,13 @@ use case could be when you want to find all the members that does not exist in a
### Raw SQL options for advanced users ### Raw SQL options for advanced users
Occasionally, the system described above won't let you do exactly what you need to do. In these situtations, we have Occasionally, the system described above won't let you do exactly what you need to do. In these situations, we have
methods that manipulate the SQL query at a lower level. When using these, please ensure that all table & field names methods that manipulate the SQL query at a lower level. When using these, please ensure that all table & field names
are escaped with double quotes, otherwise some DB back-ends (e.g. PostgreSQL) won't work. are escaped with double quotes, otherwise some DB back-ends (e.g. PostgreSQL) won't work.
Under the hood, query generation is handled by the `[api:DataQuery]` class. This class does provide more direct
access to certain SQL features that `DataList` abstracts away from you.
In general, we advise against using these methods unless it's absolutely necessary. If the ORM doesn't do quite what In general, we advise against using these methods unless it's absolutely necessary. If the ORM doesn't do quite what
you need it to, you may also consider extending the ORM with new data types or filter modifiers (that documentation you need it to, you may also consider extending the ORM with new data types or filter modifiers (that documentation
still needs to be written) still needs to be written)

157
model/DataQuery.php
View File

@ -5,6 +5,9 @@
* Acts as a wrapper over {@link SQLQuery} and performs all of the query generation. * Acts as a wrapper over {@link SQLQuery} and performs all of the query generation.
* Used extensively by {@link DataList}. * Used extensively by {@link DataList}.
* *
* Unlike DataList, modifiers on DataQuery modify the object rather than returning a clone.
* DataList is immutable, DataQuery is mutable.
*
* @subpackage model * @subpackage model
* @package framework * @package framework
*/ */
@ -417,12 +420,31 @@ public function max($field) {
*/ */
public function having($having) { public function having($having) {
if($having) { if($having) {
$clone = $this; $this->query->addHaving($having);
$clone->query->addHaving($having); }
return $clone;
} else {
return $this; return $this;
} }
/**
* Create a disjunctive subgroup.
*
* That is a subgroup joined by OR
*
* @return DataQuery_SubGroup
*/
public function disjunctiveGroup() {
return new DataQuery_SubGroup($this, 'OR');
}
/**
* Create a conjunctive subgroup
*
* That is a subgroup joined by AND
*
* @return DataQuery_SubGroup
*/
public function conjunctiveGroup() {
return new DataQuery_SubGroup($this, 'AND');
} }
/** /**
@ -441,12 +463,9 @@ public function max($field) {
*/ */
public function where($filter) { public function where($filter) {
if($filter) { if($filter) {
$clone = $this; $this->query->addWhere($filter);
$clone->query->addWhere($filter);
return $clone;
} else {
return $this;
} }
return $this;
} }
/** /**
@ -460,12 +479,9 @@ public function max($field) {
*/ */
public function whereAny($filter) { public function whereAny($filter) {
if($filter) { if($filter) {
$clone = $this; $this->query->addWhereAny($filter);
$clone->query->addWhereAny($filter);
return $clone;
} else {
return $this;
} }
return $this;
} }
/** /**
@ -479,14 +495,13 @@ public function max($field) {
* @return DataQuery * @return DataQuery
*/ */
public function sort($sort = null, $direction = null, $clear = true) { public function sort($sort = null, $direction = null, $clear = true) {
$clone = $this;
if($clear) { if($clear) {
$clone->query->setOrderBy($sort, $direction); $this->query->setOrderBy($sort, $direction);
} else { } else {
$clone->query->addOrderBy($sort, $direction); $this->query->addOrderBy($sort, $direction);
} }
return $clone; return $this;
} }
/** /**
@ -495,10 +510,8 @@ public function max($field) {
* @return DataQuery * @return DataQuery
*/ */
public function reverseSort() { public function reverseSort() {
$clone = $this; $this->query->reverseOrderBy();
return $this;
$clone->query->reverseOrderBy();
return $clone;
} }
/** /**
@ -508,9 +521,8 @@ public function max($field) {
* @param int $offset * @param int $offset
*/ */
public function limit($limit, $offset = 0) { public function limit($limit, $offset = 0) {
$clone = $this; $this->query->setLimit($limit, $offset);
$clone->query->setLimit($limit, $offset); return $this;
return $clone;
} }
/** /**
@ -520,19 +532,16 @@ public function max($field) {
public function join($join) { public function join($join) {
Deprecation::notice('3.0', 'Use innerJoin() or leftJoin() instead.'); Deprecation::notice('3.0', 'Use innerJoin() or leftJoin() instead.');
if($join) { if($join) {
$clone = $this; $this->query->addFrom($join);
$clone->query->addFrom($join);
// TODO: This needs to be resolved for all databases // TODO: This needs to be resolved for all databases
if(DB::getConn() instanceof MySQLDatabase) { if(DB::getConn() instanceof MySQLDatabase) {
$from = $clone->query->getFrom(); $from = $this->query->getFrom();
$clone->query->setGroupBy(reset($from) . ".\"ID\""); $this->query->setGroupBy(reset($from) . ".\"ID\"");
}
} }
return $clone;
} else {
return $this; return $this;
} }
}
/** /**
* Add an INNER JOIN clause to this query. * Add an INNER JOIN clause to this query.
@ -543,12 +552,9 @@ public function max($field) {
*/ */
public function innerJoin($table, $onClause, $alias = null) { public function innerJoin($table, $onClause, $alias = null) {
if($table) { if($table) {
$clone = $this; $this->query->addInnerJoin($table, $onClause, $alias);
$clone->query->addInnerJoin($table, $onClause, $alias);
return $clone;
} else {
return $this;
} }
return $this;
} }
/** /**
@ -560,12 +566,9 @@ public function max($field) {
*/ */
public function leftJoin($table, $onClause, $alias = null) { public function leftJoin($table, $onClause, $alias = null) {
if($table) { if($table) {
$clone = $this; $this->query->addLeftJoin($table, $onClause, $alias);
$clone->query->addLeftJoin($table, $onClause, $alias);
return $clone;
} else {
return $this;
} }
return $this;
} }
/** /**
@ -696,7 +699,7 @@ public function max($field) {
/** /**
* @param String $field Select statement identifier, either the unquoted column name, * @param String $field Select statement identifier, either the unquoted column name,
* the full composite SQL statement, or the alias set through {@link SQLQquery->selectField()}. * the full composite SQL statement, or the alias set through {@link SQLQuery->selectField()}.
* @param SQLQuery $query * @param SQLQuery $query
* @return String * @return String
*/ */
@ -744,5 +747,71 @@ public function max($field) {
if(isset($this->queryParams[$key])) return $this->queryParams[$key]; if(isset($this->queryParams[$key])) return $this->queryParams[$key];
else return null; else return null;
} }
}
/**
* Represents a subgroup inside a WHERE clause in a {@link DataQuery}
*
* Stores the clauses for the subgroup inside a specific {@link SQLQuery} object.
* All non-where methods call their DataQuery versions, which uses the base
* query object.
*/
class DataQuery_SubGroup extends DataQuery {
protected $whereQuery;
public function __construct(DataQuery $base, $connective) {
$this->dataClass = $base->dataClass;
$this->query = $base->query;
$this->whereQuery = new SQLQuery;
$this->whereQuery->setConnective($connective);
$base->where($this);
}
/**
* Set the WHERE clause of this query.
* There are two different ways of doing this:
*
* <code>
* // the entire predicate as a single string
* $query->where("Column = 'Value'");
*
* // multiple predicates as an array
* $query->where(array("Column = 'Value'", "Column != 'Value'"));
* </code>
*
* @param string|array $where Predicate(s) to set, as escaped SQL statements.
*/
public function where($filter) {
if($filter) {
$this->whereQuery->addWhere($filter);
}
return $this;
}
/**
* Set a WHERE with OR.
*
* @example $dataQuery->whereAny(array("Monkey = 'Chimp'", "Color = 'Brown'"));
* @see where()
*
* @param array $filter Escaped SQL statement.
* @return DataQuery
*/
public function whereAny($filter) {
if($filter) {
$this->whereQuery->addWhereAny($filter);
}
return $this;
}
public function __toString() {
if(!$this->whereQuery->getWhere()) {
// We always need to have something so we don't end up with something like '... AND () AND ...'
return '1=1';
}
$sql = DB::getConn()->sqlWhereToString($this->whereQuery->getWhere(), $this->whereQuery->getConnective());
$sql = preg_replace('[^\s*WHERE\s*]', '', $sql);
return $sql;
}
} }

60
tests/model/DataQueryTest.php
View File

@ -24,6 +24,66 @@ class DataQueryTest extends SapphireTest {
$this->assertEquals('DataQueryTest_B', $dq->applyRelation('TestBs'), 'DataQuery::applyRelation should return the name of the related object.'); $this->assertEquals('DataQueryTest_B', $dq->applyRelation('TestBs'), 'DataQuery::applyRelation should return the name of the related object.');
$this->assertEquals('DataQueryTest_B', $dq->applyRelation('ManyTestBs'), 'DataQuery::applyRelation should return the name of the related object.'); $this->assertEquals('DataQueryTest_B', $dq->applyRelation('ManyTestBs'), 'DataQuery::applyRelation should return the name of the related object.');
} }
public function testDisjunctiveGroup() {
$dq = new DataQuery('DataQueryTest_A');
$dq->where('DataQueryTest_A.ID = 2');
$subDq = $dq->disjunctiveGroup();
$subDq->where('DataQueryTest_A.Name = \'John\'');
$subDq->where('DataQueryTest_A.Name = \'Bob\'');
$this->assertContains("WHERE (DataQueryTest_A.ID = 2) AND ((DataQueryTest_A.Name = 'John') OR (DataQueryTest_A.Name = 'Bob'))", $dq->sql());
}
public function testConjunctiveGroup() {
$dq = new DataQuery('DataQueryTest_A');
$dq->where('DataQueryTest_A.ID = 2');
$subDq = $dq->conjunctiveGroup();
$subDq->where('DataQueryTest_A.Name = \'John\'');
$subDq->where('DataQueryTest_A.Name = \'Bob\'');
$this->assertContains("WHERE (DataQueryTest_A.ID = 2) AND ((DataQueryTest_A.Name = 'John') AND (DataQueryTest_A.Name = 'Bob'))", $dq->sql());
}
public function testNestedGroups() {
$dq = new DataQuery('DataQueryTest_A');
$dq->where('DataQueryTest_A.ID = 2');
$subDq = $dq->disjunctiveGroup();
$subDq->where('DataQueryTest_A.Name = \'John\'');
$subSubDq = $subDq->conjunctiveGroup();
$subSubDq->where('DataQueryTest_A.Age = 18');
$subSubDq->where('DataQueryTest_A.Age = 50');
$subDq->where('DataQueryTest_A.Name = \'Bob\'');
$this->assertContains("WHERE (DataQueryTest_A.ID = 2) AND ((DataQueryTest_A.Name = 'John') OR ((DataQueryTest_A.Age = 18) AND (DataQueryTest_A.Age = 50)) OR (DataQueryTest_A.Name = 'Bob'))", $dq->sql());
}
public function testEmptySubgroup() {
$dq = new DataQuery('DataQueryTest_A');
$dq->conjunctiveGroup();
$this->assertContains('WHERE (1=1)', $dq->sql());
}
public function testSubgroupHandoff() {
$dq = new DataQuery('DataQueryTest_A');
$subDq = $dq->disjunctiveGroup();
$orgDq = clone $dq;
$subDq->sort('"DataQueryTest_A"."Name"');
$orgDq->sort('"DataQueryTest_A"."Name"');
$this->assertEquals($dq->sql(), $orgDq->sql());
$subDq->limit(5, 7);
$orgDq->limit(5, 7);
$this->assertEquals($dq->sql(), $orgDq->sql());
}
} }