fix has servers check

Signed-off-by: Dieter Coopman <[email protected]>
dietercoopman · Mar 21, 2023 · c557efb · c557efb
1 parent 8fffa6b
commit c557efb
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 6 deletions.
diff --git a/src/ServerList.php b/src/ServerList.php
@@ -37,7 +37,6 @@ public function execute( InputInterface $input, OutputInterface $output): int
     {
         $this->title();
 
-        render('<div class="ml-1 mb-1">Here\'s a list of all saved servers.</div>');
         (new Configurator())->list();
 
         return 0;

diff --git a/src/Services/Configurator.php b/src/Services/Configurator.php
@@ -40,6 +40,8 @@ public function list()
     {
         $counter = 1;
         if (isset($this->getConfig()['servers'])) {
+            render('<div class="ml-1 mb-1">Here\'s a list of all saved servers.</div>');
+
             collect($this->getConfig()['servers'])->each(function ($server) use (&$counter) {
                 render("<span class='ml-1'>{$counter}. {$server['name']} ({$server['host']})</span>");
                 $counter++;

diff --git a/src/Traits/HasServer.php b/src/Traits/HasServer.php
@@ -5,12 +5,15 @@
 
 trait HasServer
 {
-    public function getServers($configurator){
+    public function getServers($configurator)
+    {
         $ret = [];
-        $counter = 1;
-        foreach($configurator->getConfig()['servers'] as $key => $server){
-            $ret[$counter] = $server['name'];
-            $counter++;
+        if (isset($configurator->getConfig()['servers'])) {
+            $counter = 1;
+            foreach ($configurator->getConfig()['servers'] as $key => $server) {
+                $ret[$counter] = $server['name'];
+                $counter++;
+            }
         }
         return $ret;
     }