From bad08fa6db72de5744474d7ac5e65181ba1698fc Mon Sep 17 00:00:00 2001
From: Neil Dorin <ndorin@pepperdash.com>
Date: Thu, 7 May 2026 12:08:26 -0600
Subject: [PATCH] refactor: Extract portal URL display logic into a separate
 method for improved readability

---
 src/PepperDash.Essentials/ControlSystem.cs | 34 +++++++++++++++++-----
 1 file changed, 26 insertions(+), 8 deletions(-)

diff --git a/src/PepperDash.Essentials/ControlSystem.cs b/src/PepperDash.Essentials/ControlSystem.cs
index 700b30d8..1539fcf0 100644
--- a/src/PepperDash.Essentials/ControlSystem.cs
+++ b/src/PepperDash.Essentials/ControlSystem.cs
@@ -177,14 +177,8 @@ public class ControlSystem : CrestronControlSystem, ILoadConfig, IInitialization
                     (ConfigReader.ConfigObject, Newtonsoft.Json.Formatting.Indented));
             }, "showconfig", "Shows the current running merged config", ConsoleAccessLevelEnum.AccessOperator);
 
-            CrestronConsole.AddNewConsoleCommand(s =>
-                CrestronConsole.ConsoleCommandResponse(
-                "This system can be found at the following URLs:{2}" +
-                "System URL:   {0}{2}" +
-                "Template URL: {1}{2}",
-                ConfigReader.ConfigObject.SystemUrl,
-                ConfigReader.ConfigObject.TemplateUrl,
-                CrestronEnvironment.NewLine),
+            CrestronConsole.AddNewConsoleCommand(
+                PrintPortalInfo,
                 "portalinfo",
                 "Shows portal URLS from configuration",
                 ConsoleAccessLevelEnum.AccessOperator);
@@ -216,6 +210,30 @@ public class ControlSystem : CrestronControlSystem, ILoadConfig, IInitialization
         }
     }
 
+
+    private void PrintPortalInfo(string args)
+    {
+        if (ConfigReader.ConfigObject == null)
+        {
+            CrestronConsole.ConsoleCommandResponse("No configuration loaded. Cannot show portal URLs.");
+            return;
+        }
+
+        if (string.IsNullOrEmpty(ConfigReader.ConfigObject.SystemUrl) && string.IsNullOrEmpty(ConfigReader.ConfigObject.TemplateUrl))
+        {
+            CrestronConsole.ConsoleCommandResponse("No portal URLs defined in config.");
+            return;
+        }
+
+        CrestronConsole.ConsoleCommandResponse(
+            "This system can be found at the following URLs:{2}" +
+            "System URL:   {0}{2}" +
+            "Template URL: {1}{2}",
+            ConfigReader.ConfigObject?.SystemUrl,
+            ConfigReader.ConfigObject?.TemplateUrl,
+            CrestronEnvironment.NewLine);
+    }
+
     /// <summary>
     /// Determines if the program is running on a processor (appliance) or server (VC-4).
     ///