1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-12-26 23:40:57 +01:00

Fix missing 'user' in PhabricatorOwnerPathQuery

Summary:
See:

  - https://github.com/facebook/phabricator/issues/317
  - https://github.com/facebook/phabricator/issues/319

This callsite was just overlooked in D5928, I think.

Test Plan: Ran `scripts/repository/reparse.php --owners rXnnnn` without hitting an exception.

Reviewers: btrahan

Reviewed By: btrahan

CC: aran

Differential Revision: https://secure.phabricator.com/D5930
This commit is contained in:
epriestley 2013-05-15 08:12:39 -07:00
parent cab28cc91d
commit 571d191a90
3 changed files with 9 additions and 3 deletions

View file

@ -53,7 +53,9 @@ final class HeraldCommitAdapter extends HeraldObjectAdapter {
public function loadAffectedPaths() {
if ($this->affectedPaths === null) {
$result = PhabricatorOwnerPathQuery::loadAffectedPaths(
$this->repository, $this->commit);
$this->repository,
$this->commit,
PhabricatorUser::getOmnipotentUser());
$this->affectedPaths = $result;
}
return $this->affectedPaths;

View file

@ -4,10 +4,12 @@ final class PhabricatorOwnerPathQuery {
public static function loadAffectedPaths(
PhabricatorRepository $repository,
PhabricatorRepositoryCommit $commit) {
PhabricatorRepositoryCommit $commit,
PhabricatorUser $user) {
$drequest = DiffusionRequest::newFromDictionary(
array(
'user' => $user,
'repository' => $repository,
'commit' => $commit->getCommitIdentifier(),
));

View file

@ -8,7 +8,9 @@ final class PhabricatorRepositoryCommitOwnersWorker
PhabricatorRepositoryCommit $commit) {
$affected_paths = PhabricatorOwnerPathQuery::loadAffectedPaths(
$repository, $commit);
$repository,
$commit,
PhabricatorUser::getOmnipotentUser());
$affected_packages = PhabricatorOwnersPackage::loadAffectedPackages(
$repository,
$affected_paths);