From b21270d498d3278b8d99c0ccca10759d72b3250f Mon Sep 17 00:00:00 2001 From: epriestley Date: Tue, 18 Aug 2015 10:46:41 -0700 Subject: [PATCH] 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 --- .../step/HarbormasterCommandBuildStepImplementation.php | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/applications/harbormaster/step/HarbormasterCommandBuildStepImplementation.php b/src/applications/harbormaster/step/HarbormasterCommandBuildStepImplementation.php index 2a6b633a72..a99c56d0a3 100644 --- a/src/applications/harbormaster/step/HarbormasterCommandBuildStepImplementation.php +++ b/src/applications/harbormaster/step/HarbormasterCommandBuildStepImplementation.php @@ -39,13 +39,14 @@ final class HarbormasterCommandBuildStepImplementation public function execute( HarbormasterBuild $build, HarbormasterBuildTarget $build_target) { + $viewer = PhabricatorUser::getOmnipotentUser(); $settings = $this->getSettings(); $variables = $build_target->getVariables(); $artifact = $build_target->loadArtifact($settings['hostartifact']); $impl = $artifact->getArtifactImplementation(); - $lease = $impl->loadArtifactLease(); + $lease = $impl->loadArtifactLease($viewer); $this->platform = $lease->getAttribute('platform');