CLI-1175: [auth:acsf-logout] undefined array key

acquia · Nov 13, 2023 · 3be9805 · 3be9805
1 parent 7f044a2
commit 3be9805
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/config/prod/services.yml b/config/prod/services.yml
@@ -47,7 +47,6 @@ services:
     parent: Acquia\Cli\Command\CommandBase
     exclude:
       - ../../src/Command/CommandBase.php
-      - ../../src/Command/Acsf/AcsfCommandBase.php
       - ../../src/Command/Acsf/AcsfListCommand.php
       - ../../src/Command/Acsf/AcsfListCommandBase.php
       - ../../src/Command/Acsf/AcsfApiBaseCommand.php

diff --git a/src/Command/Auth/AuthAcsfLogoutCommand.php b/src/Command/Auth/AuthAcsfLogoutCommand.php
@@ -22,11 +22,11 @@ protected function commandRequiresAuthentication(): bool {
   }
 
   protected function execute(InputInterface $input, OutputInterface $output): int {
-    if (!$this->cloudApiClientService->isMachineAuthenticated()) {
+    $factories = $this->datastoreCloud->get('acsf_factories');
+    if (empty($factories)) {
       $this->io->error(['You are not logged into any factories.']);
-      return 1;
+      return Command::FAILURE;
     }
-    $factories = $this->datastoreCloud->get('acsf_factories');
     foreach ($factories as $url => $factory) {
       $factories[$url]['url'] = $url;
     }