Merge pull request #62 from linc/fix/config-clues

Config validation: Add more clues
linc · Dec 27, 2024 · d05b04e · d05b04e
2 parents bfe318a + 9c426df
commit d05b04e
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/src/Command/RunCommand.php b/src/Command/RunCommand.php
@@ -15,8 +15,8 @@ public function __construct()
         $this
             ->option('-s --source', 'Name of source package')
             ->option('-t --target', 'Name of target package')
-            ->option('-i --input', 'Source connection (defined in config)')
-            ->option('-o --output', 'Target connection (defined in config)')
+            ->option('-i --input', 'Source connection alias (defined in config)')
+            ->option('-o --output', 'Target connection alias (defined in config)')
             ->option('-x --sourceprefix', 'Source prefix')
             ->option('--cdn', 'CDN prefix')
             ->option('--dumpsql', 'Output SQL instead of migrating')

diff --git a/src/Config.php b/src/Config.php
@@ -86,6 +86,11 @@ public function getConnectionAlias(string $alias): array
      */
     protected function validateConnectionInfo(string $alias, array $info): void
     {
+        // Alias not in config.
+        if (empty($info)) {
+            trigger_error('Config error: Alias "' . $alias . '" not found', E_USER_ERROR);
+        }
+
         // Type is required.
         if (empty($info['type'])) {
             trigger_error('Config error: No connection `type` for alias "' . $alias . '"', E_USER_ERROR);