@recaptime-dev's working patches + fork for Phorge, a community fork of Phabricator. (Upstream dev and stable branches are at upstream/main and upstream/stable respectively.)
hq.recaptime.dev/wiki/Phorge
phorge
phabricator
Policy - convert Drydock query for repository to policy-based query
Summary: Ref T7094. Switch to OmnipotentUser policy-based query since this is usually done offline, etc.
Test Plan: pretty simple code change so I just have my fingers crossed while I am typing this
Reviewers: epriestley
Reviewed By: epriestley
Subscribers: Korvin, epriestley
Maniphest Tasks: T7094
Differential Revision: https://secure.phabricator.com/D11655
+4
-3
+4
-3
src/applications/drydock/blueprint/DrydockWorkingCopyBlueprintImplementation.php
+4
-3
src/applications/drydock/blueprint/DrydockWorkingCopyBlueprintImplementation.php
···
40
40
"Lease is missing required 'repositoryID' attribute.");
41
41
}
42
42
43
-
// TODO: (T603) Figure out the interaction between policies and
44
-
// Drydock.
45
-
$repository = id(new PhabricatorRepository())->load($repository_id);
43
+
$repository = id(new PhabricatorRepositoryQuery())
44
+
->setViewer(PhabricatorUser::getOmnipotentUser())
45
+
->withIDs(array($repository_id))
46
+
->executeOne();
46
47
47
48
if (!$repository) {
48
49
throw new Exception(