Fix improper handling of null data in cache

Author: itm4n

info/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## 2025-09-01
+
+### Fixed
+
+- Improper handling of null data in 'Set-CachedData'.
+
 ## 2025-08-30
 
 ### Added

src/helper/Cache.ps1

@@ -53,6 +53,7 @@ function Set-CachedData {
         [String] $Name,
 
         [Parameter(Mandatory = $true)]
+        [AllowEmptyCollection()]
         [Object[]] $Data
     )
 

src/helper/Environment.ps1

@@ -20,7 +20,10 @@ function Get-CurrentUserDenySid {
     param()
 
     if (-not (Test-CachedData -Name "CurrentUserDenySids")) {
-        $CurrentUserDenySids = [string[]](Get-TokenInformationGroup -InformationClass Groups | Where-Object { $_.Attributes.Equals("UseForDenyOnly") } | Select-Object -ExpandProperty SID)
+        $CurrentUserDenySids = @()
+        Get-TokenInformationGroup -InformationClass Groups | Where-Object { $_.Attributes.Equals("UseForDenyOnly") } | Select-Object -ExpandProperty SID | ForEach-Object {
+            $CurrentUserDenySids += $_
+        }
         Set-CachedData -Name "CurrentUserDenySids" -Data $CurrentUserDenySids
     }
 
@@ -33,7 +36,8 @@ function Get-InstalledApplication {
     param ()
 
     if (-not (Test-CachedData -Name "InstalledApplicationList")) {
-        $InstalledApplicationList = Get-InstalledApplicationHelper
+        $InstalledApplicationList = @()
+        Get-InstalledApplicationHelper | ForEach-Object { $InstalledApplicationList += $_ }
         Set-CachedData -Name "InstalledApplicationList" -Data $InstalledApplicationList
     }