Merge pull request KelvinTegelaar#1608 from Zacgoose/quota-alert

JohnDuprey · web-flow · commit 815cb3318e92 · 2025-09-05T10:58:24.000-04:00
Add null checks to mailbox quota alert
diff --git a/Modules/CIPPCore/Public/Alerts/Get-CIPPAlertQuotaUsed.ps1 b/Modules/CIPPCore/Public/Alerts/Get-CIPPAlertQuotaUsed.ps1
@@ -17,7 +17,7 @@ function Get-CIPPAlertQuotaUsed {
         return
     }
     $OverQuota = $AlertData | ForEach-Object {
-        if ($_.StorageUsedInBytes -eq 0 -or $_.prohibitSendReceiveQuotaInBytes -eq 0) { return }
+        if ([string]::IsNullOrEmpty($_.StorageUsedInBytes) -or [string]::IsNullOrEmpty($_.prohibitSendReceiveQuotaInBytes) -or $_.StorageUsedInBytes -eq 0 -or $_.prohibitSendReceiveQuotaInBytes -eq 0) { return }
         try {
             $PercentLeft = [math]::round(($_.storageUsedInBytes / $_.prohibitSendReceiveQuotaInBytes) * 100)
         } catch { $PercentLeft = 100 }

Original file line number	Diff line number	Diff line change
`@@ -17,7 +17,7 @@ function Get-CIPPAlertQuotaUsed {`
`17`	`17`	`return`
`18`	`18`	`}`
`19`	`19`	`$OverQuota = $AlertData \| ForEach-Object {`
`20`		`- if ($_.StorageUsedInBytes -eq 0 -or $_.prohibitSendReceiveQuotaInBytes -eq 0) { return }`
	`20`	`+ if ([string]::IsNullOrEmpty($_.StorageUsedInBytes) -or [string]::IsNullOrEmpty($_.prohibitSendReceiveQuotaInBytes) -or $_.StorageUsedInBytes -eq 0 -or $_.prohibitSendReceiveQuotaInBytes -eq 0) { return }`
`21`	`21`	`try {`
`22`	`22`	`$PercentLeft = [math]::round(($_.storageUsedInBytes / $_.prohibitSendReceiveQuotaInBytes) * 100)`
`23`	`23`	`} catch { $PercentLeft = 100 }`