mirror of
https://we.phorge.it/source/phorge.git
synced 2025-01-25 22:18:19 +01:00
Fix an issue with Diffusion where paging by commit date would fail
Summary: When loading the cursor repository, we need to load the most recent commit too if we're paging by commit date. This fixes a fatal for installs with more than 100 repositories. Auditors: btrahan
This commit is contained in:
parent
806ba1e7a1
commit
b928e7f8f5
1 changed files with 8 additions and 3 deletions
|
@ -163,10 +163,15 @@ final class PhabricatorRepositoryQuery
|
||||||
}
|
}
|
||||||
|
|
||||||
private function loadCursorObject($id) {
|
private function loadCursorObject($id) {
|
||||||
$results = id(new PhabricatorRepositoryQuery())
|
$query = id(new PhabricatorRepositoryQuery())
|
||||||
->setViewer($this->getPagingViewer())
|
->setViewer($this->getPagingViewer())
|
||||||
->withIDs(array((int)$id))
|
->withIDs(array((int)$id));
|
||||||
->execute();
|
|
||||||
|
if ($this->order == self::ORDER_COMMITTED) {
|
||||||
|
$query->needMostRecentCommits(true);
|
||||||
|
}
|
||||||
|
|
||||||
|
$results = $query->execute();
|
||||||
return head($results);
|
return head($results);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Add table
Reference in a new issue