@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
Remove call to loadDrydockLease
Summary: Fixes T9219... I think.
Test Plan: swagging a guess for epriestley
Reviewers: epriestley
Reviewed By: epriestley
Subscribers: Korvin
Maniphest Tasks: T9219
Differential Revision: https://secure.phabricator.com/D13932
+4
-2
+4
-2
src/applications/harbormaster/step/HarbormasterUploadArtifactBuildStepImplementation.php
+4
-2
src/applications/harbormaster/step/HarbormasterUploadArtifactBuildStepImplementation.php
···
25
25
public function execute(
26
26
HarbormasterBuild $build,
27
27
HarbormasterBuildTarget $build_target) {
28
+
$viewer = PhabricatorUser::getOmnipotentUser();
28
29
29
30
$settings = $this->getSettings();
30
31
$variables = $build_target->getVariables();
···
35
36
$variables);
36
37
37
38
$artifact = $build_target->loadArtifact($settings['hostartifact']);
38
-
$lease = $artifact->loadDrydockLease();
39
+
$impl = $artifact->getArtifactImplementation();
40
+
$lease = $impl->loadArtifactLease($viewer);
39
41
40
42
$interface = $lease->getInterface('filesystem');
41
43
···
44
46
45
47
// Insert the artifact record.
46
48
$artifact = $build_target->createArtifact(
47
-
PhabricatorUser::getOmnipotentUser(),
49
+
$viewer,
48
50
$settings['name'],
49
51
HarbormasterFileArtifact::ARTIFACTCONST,
50
52
array(