Page MenuHomePhabricator

D18727.diff
No OneTemporary

D18727.diff

diff --git a/src/applications/differential/query/DifferentialRevisionQuery.php b/src/applications/differential/query/DifferentialRevisionQuery.php
--- a/src/applications/differential/query/DifferentialRevisionQuery.php
+++ b/src/applications/differential/query/DifferentialRevisionQuery.php
@@ -38,8 +38,6 @@
private $needDrafts;
private $needFlags;
- private $buildingGlobalOrder;
-
/* -( Query Configuration )------------------------------------------------ */
@@ -484,12 +482,11 @@
}
if (count($selects) > 1) {
- $this->buildingGlobalOrder = true;
$query = qsprintf(
$conn_r,
'%Q %Q %Q',
implode(' UNION DISTINCT ', $selects),
- $this->buildOrderClause($conn_r),
+ $this->buildOrderClause($conn_r, true),
$this->buildLimitClause($conn_r));
} else {
$query = head($selects);
@@ -513,7 +510,6 @@
$group_by = $this->buildGroupClause($conn_r);
$having = $this->buildHavingClause($conn_r);
- $this->buildingGlobalOrder = false;
$order_by = $this->buildOrderClause($conn_r);
$limit = $this->buildLimitClause($conn_r);
@@ -758,17 +754,9 @@
}
public function getOrderableColumns() {
- $primary = ($this->buildingGlobalOrder ? null : 'r');
-
return array(
- 'id' => array(
- 'table' => $primary,
- 'column' => 'id',
- 'type' => 'int',
- 'unique' => true,
- ),
'updated' => array(
- 'table' => $primary,
+ 'table' => $this->getPrimaryTableAlias(),
'column' => 'dateModified',
'type' => 'int',
),

File Metadata

Mime Type
text/plain
Expires
Sat, Nov 30, 8:51 PM (9 h, 10 m)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
6809746
Default Alt Text
D18727.diff (1 KB)

Event Timeline