fix: use the project id when given as an argument

carlalexander · carlalexander · commit ec830aaadb80 · 2023-01-12T10:39:51.000-05:00
diff --git a/src/Command/AbstractCommand.php b/src/Command/AbstractCommand.php
@@ -170,9 +170,13 @@ protected function determineProject(string $question, InputInterface $input, Out
 
         $projectIdOrName = $this->getStringArgument($input, 'project');
 
-        if (empty($projectIdOrName)) {
+        if (empty($projectIdOrName) && $this->projectConfiguration->exists()) {
+            $projectIdOrName = $this->projectConfiguration->getProjectId();
+        } elseif (empty($projectIdOrName)) {
             $projectIdOrName = $output->choiceWithId($question, $projects);
-        } elseif (1 < $projects->where('name', $projectIdOrName)->count()) {
+        }
+
+        if (1 < $projects->where('name', $projectIdOrName)->count()) {
             throw new RuntimeException(sprintf('Unable to select a project because more than one project has the name "%s"', $projectIdOrName));
         }
 
diff --git a/src/Command/Project/DeleteProjectCommand.php b/src/Command/Project/DeleteProjectCommand.php
@@ -51,12 +51,7 @@ protected function configure()
      */
     protected function perform(InputInterface $input, OutputInterface $output)
     {
-        $projectId = $this->projectConfiguration->exists() ? $this->projectConfiguration->getProjectId() : null;
-
-        if (null === $projectId) {
-            $projectId = $this->determineProject('Which project would you like to delete', $input, $output);
-        }
-
+        $projectId = $this->determineProject('Which project would you like to delete', $input, $output);
         $project = $this->apiClient->getProject($projectId);
 
         if (!$output->confirm(sprintf('Are you sure you want to delete the <comment>%s</comment> project?', $project['name']), false)) {
