Check to see if role already exists, not that roles exist. (#131)

microsoft · Oct 24, 2018 · 87030e2 · 87030e2
1 parent 470fd0d
commit 87030e2
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/release/scripts/PowerShell/FhirServerRelease/Private/Set-FhirServerApiApplicationRoles.ps1 b/release/scripts/PowerShell/FhirServerRelease/Private/Set-FhirServerApiApplicationRoles.ps1
@@ -37,8 +37,10 @@ function Set-FhirServerApiApplicationRoles {
     $desiredAppRoles = @()
 
     foreach ($role in $RoleConfiguration) {
-        if($azureAdApplication.AppRoles) {
-            $id = ($azureAdApplication.AppRoles | Where-Object Value -eq $role.name).Id
+        $existingAppRole = $azureAdApplication.AppRoles | Where-Object Value -eq $role.name
+
+        if($existingAppRole) {
+            $id = $existingAppRole.Id
         }
         else {
             $id = New-Guid