Fixed optional parameter (#16)

* Added logging and env variable * One more error checking * Updated reademe * Changed to an optional parameter
UNT-CAS · Sep 28, 2018 · 7826fae · 7826fae
1 parent 9b4cb4f
commit 7826fae
Showing 1 changed file with 12 additions and 6 deletions.
diff --git a/ePOwerShell/Public/Set-ePOwerShellServer.ps1 b/ePOwerShell/Public/Set-ePOwerShellServer.ps1
@@ -45,13 +45,13 @@ function Set-ePOwerShellServer {
         [String]
         $Server,
 
-        [Parameter(Mandatory = $True, ParameterSetName = 'ManualEntry')]
-        [Int]
-        $Port,
-
         [Parameter(Mandatory = $True, ParameterSetName = 'ManualEntry')]
         [System.Management.Automation.PSCredential]
         $Credentials,
+
+        [Parameter(Mandatory = $False, ParameterSetName = 'ManualEntry')]
+        [Int]
+        $Port,
 
         [Parameter(ParameterSetName = 'Env')]
         [String]
@@ -101,16 +101,22 @@ function Set-ePOwerShellServer {
 
             $ePOwerShellVariables = @{
                 Server      = $Settings.Server
-                Port        = $Settings.Port
                 Credentials = $Credentials
             }
+
+            if ($settings.Port) {
+                [void]$ePOwerShellVariables.Add("Port", $Settings.Port)
+            }
         }
         'ManualEntry' {
             $ePOwerShellVariables = @{
                 Server      = $Server
-                Port        = $Port
                 Credentials = $Credentials
             }
+
+            if ($Port) {
+                [void]$ePOwerShellVariables.Add("Port", $Port)
+            }
         }
     }