Update Invoke-AddGroup.ps1

JohnDuprey · JohnDuprey · commit 82c6a0fdcb06 · 2025-03-12T12:16:41.000-04:00
diff --git a/Modules/CIPPCore/Public/Entrypoints/HTTP Functions/Identity/Administration/Groups/Invoke-AddGroup.ps1 b/Modules/CIPPCore/Public/Entrypoints/HTTP Functions/Identity/Administration/Groups/Invoke-AddGroup.ps1
@@ -39,11 +39,11 @@ Function Invoke-AddGroup {
                     $BodyParams | Add-Member -NotePropertyName 'groupTypes' -NotePropertyValue @('Unified')
                 }
                 if ($GroupObject.owners -AND $GroupObject.groupType -in 'generic', 'azurerole', 'security') {
-                    $BodyParams | Add-Member -NotePropertyName 'owners@odata.bind' -NotePropertyValue (($GroupObject.AddOwner) | ForEach-Object { "https://graph.microsoft.com/v1.0/users/$($_.value)" })
+                    $BodyParams | Add-Member -NotePropertyName 'owners@odata.bind' -NotePropertyValue (($GroupObject.owners) | ForEach-Object { "https://graph.microsoft.com/v1.0/users/$($_.value)" })
                     $BodyParams.'owners@odata.bind' = @($BodyParams.'owners@odata.bind')
                 }
                 if ($GroupObject.members -AND $GroupObject.groupType -in 'generic', 'azurerole', 'security') {
-                    $BodyParams | Add-Member -NotePropertyName 'members@odata.bind' -NotePropertyValue (($GroupObject.AddMember) | ForEach-Object { "https://graph.microsoft.com/v1.0/users/$($_.value)" })
+                    $BodyParams | Add-Member -NotePropertyName 'members@odata.bind' -NotePropertyValue (($GroupObject.members) | ForEach-Object { "https://graph.microsoft.com/v1.0/users/$($_.value)" })
                     $BodyParams.'members@odata.bind' = @($BodyParams.'members@odata.bind')
                 }
                 $GraphRequest = New-GraphPostRequest -uri 'https://graph.microsoft.com/beta/groups' -tenantid $tenant -type POST -body (ConvertTo-Json -InputObject $BodyParams -Depth 10) -Verbose

Original file line number	Diff line number	Diff line change
`@@ -39,11 +39,11 @@ Function Invoke-AddGroup {`
`39`	`39`	`$BodyParams \| Add-Member -NotePropertyName 'groupTypes' -NotePropertyValue @('Unified')`
`40`	`40`	`}`
`41`	`41`	`if ($GroupObject.owners -AND $GroupObject.groupType -in 'generic', 'azurerole', 'security') {`
`42`		`- $BodyParams \| Add-Member -NotePropertyName '[email protected]' -NotePropertyValue (($GroupObject.AddOwner) \| ForEach-Object { "https://graph.microsoft.com/v1.0/users/$($_.value)" })`
	`42`	`+ $BodyParams \| Add-Member -NotePropertyName '[email protected]' -NotePropertyValue (($GroupObject.owners) \| ForEach-Object { "https://graph.microsoft.com/v1.0/users/$($_.value)" })`
`43`	`43`	`$BodyParams.'[email protected]' = @($BodyParams.'[email protected]')`
`44`	`44`	`}`
`45`	`45`	`if ($GroupObject.members -AND $GroupObject.groupType -in 'generic', 'azurerole', 'security') {`
`46`		`- $BodyParams \| Add-Member -NotePropertyName '[email protected]' -NotePropertyValue (($GroupObject.AddMember) \| ForEach-Object { "https://graph.microsoft.com/v1.0/users/$($_.value)" })`
	`46`	`+ $BodyParams \| Add-Member -NotePropertyName '[email protected]' -NotePropertyValue (($GroupObject.members) \| ForEach-Object { "https://graph.microsoft.com/v1.0/users/$($_.value)" })`
`47`	`47`	`$BodyParams.'[email protected]' = @($BodyParams.'[email protected]')`
`48`	`48`	`}`
`49`	`49`	`$GraphRequest = New-GraphPostRequest -uri 'https://graph.microsoft.com/beta/groups' -tenantid $tenant -type POST -body (ConvertTo-Json -InputObject $BodyParams -Depth 10) -Verbose`