1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-12-30 01:10:58 +01:00

Namespace dateCreated in Maniphest query construction

Summary: Fixes T5661. We may now pick up a conflicting `dateCreated` field from an edge table join.

Test Plan: Ran a project + dateCreated filtering query, no longer got an exception.

Reviewers: btrahan, joshuaspence

Reviewed By: joshuaspence

Subscribers: epriestley

Maniphest Tasks: T5661

Differential Revision: https://secure.phabricator.com/D9997
This commit is contained in:
epriestley 2014-07-21 06:44:35 -07:00
parent af214ecb65
commit 9d64beeaa3

View file

@ -206,28 +206,28 @@ final class ManiphestTaskQuery extends PhabricatorCursorPagedPolicyAwareQuery {
if ($this->dateCreatedAfter) { if ($this->dateCreatedAfter) {
$where[] = qsprintf( $where[] = qsprintf(
$conn, $conn,
'dateCreated >= %d', 'task.dateCreated >= %d',
$this->dateCreatedAfter); $this->dateCreatedAfter);
} }
if ($this->dateCreatedBefore) { if ($this->dateCreatedBefore) {
$where[] = qsprintf( $where[] = qsprintf(
$conn, $conn,
'dateCreated <= %d', 'task.dateCreated <= %d',
$this->dateCreatedBefore); $this->dateCreatedBefore);
} }
if ($this->dateModifiedAfter) { if ($this->dateModifiedAfter) {
$where[] = qsprintf( $where[] = qsprintf(
$conn, $conn,
'dateModified >= %d', 'task.dateModified >= %d',
$this->dateModifiedAfter); $this->dateModifiedAfter);
} }
if ($this->dateModifiedBefore) { if ($this->dateModifiedBefore) {
$where[] = qsprintf( $where[] = qsprintf(
$conn, $conn,
'dateModified <= %d', 'task.dateModified <= %d',
$this->dateModifiedBefore); $this->dateModifiedBefore);
} }