From d203e279f0c78ba18482a6559eaf32488bba2e05 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Lo=C3=AFc=20MICHEL?= <loic.michel@gmail.com>
Date: Thu, 12 Dec 2024 22:03:36 +0100
Subject: [PATCH] resolve #71 set-approval with no approvers

---
 EasyPIM/internal/functions/Set-Approval.ps1 | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/EasyPIM/internal/functions/Set-Approval.ps1 b/EasyPIM/internal/functions/Set-Approval.ps1
index e373873..4b16eb1 100644
--- a/EasyPIM/internal/functions/Set-Approval.ps1
+++ b/EasyPIM/internal/functions/Set-Approval.ps1
@@ -21,7 +21,7 @@
 #>
 function Set-Approval ($ApprovalRequired, $Approvers, [switch]$entraRole) {
     try {
-        Write-Verbose "Set-Approval"
+        Write-Verbose "Set-Approval started with ApprovalRequired=$ApprovalRequired and Approvers=$Approvers and entraRole=$entraRole"
         if ($null -eq $Approvers) { $Approvers = $script:config.Approvers }
         if ($ApprovalRequired -eq $false) { $req = "false" }else { $req = "true" }
         <#working sample
@@ -62,6 +62,7 @@ function Set-Approval ($ApprovalRequired, $Approvers, [switch]$entraRole) {
             "isEscalationEnabled":false,
             "primaryApprovers":[
                 '
+        if ($PSBoundParameters.Keys.Contains('Approvers') -and ($null -ne $Approvers)) {
         $cpt = 0
         $Approvers | ForEach-Object {
             #write-host $_
@@ -83,6 +84,7 @@ function Set-Approval ($ApprovalRequired, $Approvers, [switch]$entraRole) {
             $cpt++
         }
         $rule=$rule -replace ",$" #remove last comma
+    }
 
         <#{"id":"5dba24e0-00ef-4c21-9702-7c093a0775eb","userType":"Group","description":"0Ext_Partners","isBackup":false},
                 {"id":"00b34bb3-8a6b-45ce-a7bb-c7f7fb400507","userType":"User","description":"Bob MARLEY","isBackup":false},