@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
Probably fix bad loadArtifactLease() call
Summary: Ref T9205. This is a likely fix.
Test Plan: This isn't straightforward to test in the upstream unless you have custom code on top of it.
Reviewers: chad
Reviewed By: chad
Maniphest Tasks: T9205
Differential Revision: https://secure.phabricator.com/D13928
+2
-1
+2
-1
src/applications/harbormaster/step/HarbormasterCommandBuildStepImplementation.php
+2
-1
src/applications/harbormaster/step/HarbormasterCommandBuildStepImplementation.php
···
39
39
public function execute(
40
40
HarbormasterBuild $build,
41
41
HarbormasterBuildTarget $build_target) {
42
+
$viewer = PhabricatorUser::getOmnipotentUser();
42
43
43
44
$settings = $this->getSettings();
44
45
$variables = $build_target->getVariables();
45
46
46
47
$artifact = $build_target->loadArtifact($settings['hostartifact']);
47
48
$impl = $artifact->getArtifactImplementation();
48
-
$lease = $impl->loadArtifactLease();
49
+
$lease = $impl->loadArtifactLease($viewer);
49
50
50
51
$this->platform = $lease->getAttribute('platform');
51
52