diff --git a/src/applications/drydock/management/DrydockManagementLeaseWorkflow.php b/src/applications/drydock/management/DrydockManagementLeaseWorkflow.php --- a/src/applications/drydock/management/DrydockManagementLeaseWorkflow.php +++ b/src/applications/drydock/management/DrydockManagementLeaseWorkflow.php @@ -62,6 +62,10 @@ $drydock_phid = id(new PhabricatorDrydockApplication())->getPHID(); $lease->setAuthorizingPHID($drydock_phid); + if ($attributes) { + $lease->setAttributes($attributes); + } + // TODO: This is not hugely scalable, although this is a debugging workflow // so maybe it's fine. Do we even need `bin/drydock lease` in the long run? $all_blueprints = id(new DrydockBlueprintQuery()) @@ -76,10 +80,6 @@ } $lease->setAllowedBlueprintPHIDs($allowed_phids); - if ($attributes) { - $lease->setAttributes($attributes); - } - if ($until) { $lease->setUntil($until); }