fix: Enhance code quality and shell provider factoryr

factorial-io · Mar 1, 2025 · f867dcf · f867dcf
1 parent f8a8228
commit f867dcf
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 7 deletions.
diff --git a/src/Configuration/ConfigurationService.php b/src/Configuration/ConfigurationService.php
@@ -834,12 +834,8 @@ public function getDockerConfig(string $config_name): ?DockerConfig
         $shell_provider = ShellProviderFactory::create($data['shellProvider'], $this->logger);
 
         $errors = new ValidationErrorBag();
-        if (!$shell_provider) {
-            $errors->addError('shellProvider', 'Unhandled shell-provider: `'.$data['shellProvider'].'`');
-        } else {
-            $data = Node::mergeData($shell_provider->getDefaultConfig($this, $data), $data);
-            $shell_provider->validateConfig($data, $errors);
-        }
+        $data = Node::mergeData($shell_provider->getDefaultConfig($this, $data), $data);
+        $shell_provider->validateConfig($data, $errors);
         if ($errors->hasErrors()) {
             throw new ValidationFailedException($errors);
         }

diff --git a/src/ShellProvider/ShellProviderFactory.php b/src/ShellProvider/ShellProviderFactory.php
@@ -2,9 +2,11 @@
 
 namespace Phabalicious\ShellProvider;
 
+use Psr\Log\LoggerInterface;
+
 class ShellProviderFactory
 {
-    public static function create($shell_provider_name, $logger): ?ShellProviderInterface
+    public static function create(string $shell_provider_name, LoggerInterface $logger): ShellProviderInterface
     {
         switch ($shell_provider_name) {
             case LocalShellProvider::PROVIDER_NAME: