revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-11-29 18:22:41 +01:00
Code Issues Releases Wiki Activity

Don't fetch all commits without blame in Diffusion

Browse source 
Summary:
Otherwise useless query is executed:

  lang=sql
  SELECT c.*
  FROM `repository_commit` c
  ORDER BY c.epoch DESC

Test Plan: /diffusion/X/browse/x

Reviewers: epriestley

Reviewed By: epriestley

CC: aran

Differential Revision: https://secure.phabricator.com/D2186
This commit is contained in:
vrana 2012-04-10 01:32:47 -07:00
parent 347bc357fd
commit 76b534b560
1 changed files with 8 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
src/applications/diffusion/controller/file/DiffusionBrowseFileController.php
View file

@ -342,13 +342,14 @@ final class DiffusionBrowseFileController extends DiffusionController {
++$line_number; ++$line_number;
} }
$commits = array_filter(ipull($display, 'commit'));
if ($commits) {
$commits = id(new PhabricatorAuditCommitQuery()) $commits = id(new PhabricatorAuditCommitQuery())
->withIdentifiers( ->withIdentifiers($drequest->getRepository()->getID(), $commits)
$drequest->getRepository()->getID(),
array_filter(ipull($display, 'commit')))
->needCommitData(true) ->needCommitData(true)
->execute(); ->execute();
$commits = mpull($commits, null, 'getCommitIdentifier'); $commits = mpull($commits, null, 'getCommitIdentifier');
}
$request = $this->getRequest(); $request = $this->getRequest();
$user = $request->getUser(); $user = $request->getUser();