Page MenuHomePhabricator

D18012.diff
No OneTemporary

D18012.diff

diff --git a/src/infrastructure/query/policy/PhabricatorCursorPagedPolicyAwareQuery.php b/src/infrastructure/query/policy/PhabricatorCursorPagedPolicyAwareQuery.php
--- a/src/infrastructure/query/policy/PhabricatorCursorPagedPolicyAwareQuery.php
+++ b/src/infrastructure/query/policy/PhabricatorCursorPagedPolicyAwareQuery.php
@@ -1811,11 +1811,16 @@
case PhabricatorQueryConstraint::OPERATOR_NOT:
case PhabricatorQueryConstraint::OPERATOR_AND:
case PhabricatorQueryConstraint::OPERATOR_OR:
- case PhabricatorQueryConstraint::OPERATOR_ANCESTOR:
if (count($list) > 1) {
return true;
}
break;
+ case PhabricatorQueryConstraint::OPERATOR_ANCESTOR:
+ // NOTE: We must always group query results rows when using an
+ // "ANCESTOR" operator because a single task may be related to
+ // two different descendants of a particular ancestor. For
+ // discussion, see T12753.
+ return true;
case PhabricatorQueryConstraint::OPERATOR_NULL:
return true;
}

File Metadata

Mime Type
text/plain
Expires
Tue, Mar 11, 6:52 PM (1 w, 12 h ago)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
7545518
Default Alt Text
D18012.diff (1 KB)

Event Timeline