Fixes microsoft#3217

NikCharlebois · Jun 16, 2023 · 369f8d1 · 369f8d1
1 parent 4a27ecc
commit 369f8d1
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -2,6 +2,9 @@
 
 # UNRELEASED
 
+* EXORoleGroup
+  * Fixes an issue where the role group wasn't getting created when members were null.
+    FIXES [#3217](https://github.com/microsoft/Microsoft365DSC/issues/3217)
 * SCProtectionAlert
   * Prevents extracting system rules.
     FIXES [#3224](https://github.com/microsoft/Microsoft365DSC/issues/3224)

diff --git a/Modules/Microsoft365DSC/DSCResources/MSFT_EXORoleGroup/MSFT_EXORoleGroup.psm1 b/Modules/Microsoft365DSC/DSCResources/MSFT_EXORoleGroup/MSFT_EXORoleGroup.psm1
@@ -208,7 +208,6 @@ function Set-TargetResource
     $NewRoleGroupParams = @{
         Name        = $Name
         Description = $Description
-        Members     = $Members
         Roles       = $Roles
         Confirm     = $false
     }
@@ -222,6 +221,10 @@ function Set-TargetResource
     {
         Write-Verbose -Message "Role Group '$($Name)' does not exist but it should. Create and configure it."
         # Create Role Group
+        if ($Members.Length -gt 0)
+        {
+            $NewRoleGroupParams.Add("Members", $Members)
+        }
         New-RoleGroup @NewRoleGroupParams
     }
     # CASE: Role Group exists but it shouldn't;