Fix Issue #104 added ShouldProcess Arguments (#107)

Co-authored-by: Andrew Hoggins <[email protected]>
JosephMcEvoy · Mar 17, 2024 · da33b3d · da33b3d
1 parent 6ed9303
commit da33b3d
Show file tree

Hide file tree

Showing 6 changed files with 7 additions and 7 deletions.
diff --git a/PSZoom/Public/CloudRecording/Update-ZoomMeetingRecordingsSettings.ps1 b/PSZoom/Public/CloudRecording/Update-ZoomMeetingRecordingsSettings.ps1
@@ -179,7 +179,7 @@ function Update-ZoomMeetingRecordingsSettings {
 
         $requestBody = ConvertTo-Json $requestBody
 
-        if ($pscmdlet.ShouldProcess) {
+        if ($pscmdlet.ShouldProcess($MeetingId, 'Update')) {
             $response = Invoke-ZoomRestMethod -Uri $request.Uri -Body $requestBody -Method PATCH
 
             Write-Output $response

diff --git a/PSZoom/Public/Groups/Remove-ZoomGroupMembers.ps1 b/PSZoom/Public/Groups/Remove-ZoomGroupMembers.ps1
@@ -49,7 +49,7 @@ function Remove-ZoomGroupMembers {
             foreach ($MemberId in $MemberIds) {
                 $Request = [System.UriBuilder]"https://api.$ZoomURI/v2/groups/$GroupId/members/$MemberId"
 
-                if ($PScmdlet.ShouldProcess) {
+                if ($PScmdlet.ShouldProcess("Removing $MemberId from $GroupId", $MemberId, 'Remove Group Member')) {
                     Write-Verbose "Removing $MemberId from $GroupId."
                     $response = Invoke-ZoomRestMethod -Uri $request.Uri -Method DELETE
 

diff --git a/PSZoom/Public/Users/New-ZoomUser.ps1 b/PSZoom/Public/Users/New-ZoomUser.ps1
@@ -150,7 +150,7 @@ function New-ZoomUser {
 
         $requestBody = $requestBody | ConvertTo-Json
 
-        if ($PScmdlet.ShouldProcess) {
+        if ($PScmdlet.ShouldProcess($Email, "Create Zoom Account")) {
             $response = Invoke-ZoomRestMethod -Uri $request.Uri -Body $requestBody -Method POST
 
             if ($passthru) {

diff --git a/PSZoom/Public/Users/Update-ZoomUser.ps1 b/PSZoom/Public/Users/Update-ZoomUser.ps1
@@ -233,7 +233,7 @@ function Update-ZoomUser {
 
             $RequestBody = $RequestBody | ConvertTo-Json
 
-            if ($pscmdlet.ShouldProcess) {
+            if ($pscmdlet.ShouldProcess($user, 'Update')) {
                 $response = Invoke-ZoomRestMethod -Uri $request.Uri -Body $requestBody -Method PATCH
 
                 if (-not $PassThru) {

diff --git a/PSZoom/Public/Users/Update-ZoomUserPassword.ps1 b/PSZoom/Public/Users/Update-ZoomUserPassword.ps1
@@ -58,7 +58,7 @@ function Update-ZoomUserpassword {
 
             $requestBody = $requestBody | ConvertTo-Json
 
-            if ($PSCmdlet.ShouldProcess) {
+            if ($PSCmdlet.ShouldProcess($UserId, 'Update Password')) {
                 $response = Invoke-ZoomRestMethod -Uri $request.Uri -Body $requestBody -Method PUT
 
                 if (-not $PassThru) {

diff --git a/PSZoom/Public/Users/Update-ZoomUserSettings.ps1 b/PSZoom/Public/Users/Update-ZoomUserSettings.ps1
@@ -259,7 +259,7 @@ Update-ZoomUserSettings -UserId '[email protected]' -JoinBeforeHost $True
 #>
 
 function Update-ZoomUserSettings {    
-    [CmdletBinding()]
+    [CmdletBinding(SupportsShouldProcess = $True)]
     Param(
         [Parameter(
             Mandatory = $True,
@@ -944,7 +944,7 @@ function Update-ZoomUserSettings {
 
             $requestBody = $requestBody | ConvertTo-Json
 
-            if ($pscmdlet.ShouldProcess) {
+            if ($pscmdlet.ShouldProcess($user,'Update')) {
                 $response = Invoke-ZoomRestMethod -Uri $request.Uri -Body $requestBody -Method PATCH
 
                 if (-not $PassThru) {