6.5.4

JulianHayward · Sep 19, 2024 · 75abaf2 · 75abaf2
1 parent 42618ac
commit 75abaf2
Show file tree

Hide file tree

Showing 6 changed files with 15 additions and 6 deletions.
diff --git a/README.md b/README.md
@@ -84,9 +84,13 @@ The [Azure Governance Visualizer accelerator](https://github.com/Azure/Azure-Gov
 
 ## Release history
 
+**Changes** (2024-September-19 / 6.5.4 Patch)
+
+- minor PSScriptAnalyzer finding resolved
+
 **Changes** (2024-September-17 / 6.5.3 Patch)
 
-- fix stop error for subscriptions with null valued quotaId. the function detailSubscription uses `.startsWith()` method to check for `AAD_` but cannot validate when a null-valued `.quotaId` occurs. 
+- fix stop error for subscriptions with null valued quotaId. the function detailSubscription uses `.startsWith()` method to check for `AAD_` but cannot validate when a null-valued `.quotaId` occurs.
 
 [Full release history](history.md)
 

diff --git a/history.md b/history.md
@@ -4,6 +4,11 @@
 
 ### Azure Governance Visualizer version 6
 
+
+**Changes** (2024-September-19 / 6.5.4 Patch)
+
+- minor PSScriptAnalyzer finding resolved
+
 **Changes** (2024-September-17 / 6.5.3 Patch)
 
 - fix stop error for subscriptions with null valued quotaId. the function detailSubscription uses `.startsWith()` method to check for `AAD_` but cannot validate when a null-valued `.quotaId` occurs. 

diff --git a/pwsh/AzGovVizParallel.ps1 b/pwsh/AzGovVizParallel.ps1
@@ -365,7 +365,7 @@ Param
     $Product = 'AzGovViz',
 
     [string]
-    $ProductVersion = '6.5.3',
+    $ProductVersion = '6.5.4',
 
     [string]
     $GithubRepository = 'aka.ms/AzGovViz',
@@ -2435,7 +2435,7 @@ function detailSubscriptions {
     foreach ($childrenSubscription in $childrenSubscriptions) {
 
         $sub = $htAllSubscriptionsFromAPI.($childrenSubscription.name)
-        if ($sub.subDetails.subscriptionPolicies.quotaId -eq $null) {
+        if ($null -eq $sub.subDetails.subscriptionPolicies.quotaId) {
             $null = $script:outOfScopeSubscriptions.Add([PSCustomObject]@{
                     subscriptionId      = $childrenSubscription.name
                     subscriptionName    = $childrenSubscription.properties.displayName

diff --git a/pwsh/dev/devAzGovVizParallel.ps1 b/pwsh/dev/devAzGovVizParallel.ps1
@@ -365,7 +365,7 @@ Param
     $Product = 'AzGovViz',
 
     [string]
-    $ProductVersion = '6.5.3',
+    $ProductVersion = '6.5.4',
 
     [string]
     $GithubRepository = 'aka.ms/AzGovViz',

diff --git a/pwsh/dev/functions/detailSubscriptions.ps1 b/pwsh/dev/functions/detailSubscriptions.ps1
@@ -37,7 +37,7 @@
     foreach ($childrenSubscription in $childrenSubscriptions) {
 
         $sub = $htAllSubscriptionsFromAPI.($childrenSubscription.name)
-        if ($sub.subDetails.subscriptionPolicies.quotaId -eq $null) {
+        if ($null -eq $sub.subDetails.subscriptionPolicies.quotaId) {
             $null = $script:outOfScopeSubscriptions.Add([PSCustomObject]@{
                     subscriptionId      = $childrenSubscription.name
                     subscriptionName    = $childrenSubscription.properties.displayName

diff --git a/version.json b/version.json
@@ -1,3 +1,3 @@
 {
-  "ProductVersion": "6.5.3"
+  "ProductVersion": "6.5.4"
 }