fix bulk requests

Author: JohnDuprey

Modules/CIPPCore/Public/Entrypoints/HTTP Functions/Identity/Administration/Users/Invoke-ListUserMailboxDetails.ps1

@@ -1,6 +1,6 @@
 using namespace System.Net
 
-Function Invoke-ListUserMailboxDetails {
+function Invoke-ListUserMailboxDetails {
     <#
     .FUNCTIONALITY
         Entrypoint
@@ -67,6 +67,7 @@ Function Invoke-ListUserMailboxDetails {
         $usernames = New-GraphGetRequest -tenantid $TenantFilter -uri 'https://graph.microsoft.com/beta/users?$select=id,userPrincipalName&$top=999'
         $Results = New-ExoBulkRequest -TenantId $TenantFilter -CmdletArray $Requests -returnWithCommand $true -Anchor $username
         Write-Host "First line of usernames is $($usernames[0] | ConvertTo-Json)"
+
         # Assign variables from $Results
         $MailboxDetailedRequest = $Results.'Get-Mailbox'
         $PermsRequest = $Results.'Get-MailboxPermission'
@@ -75,7 +76,8 @@ Function Invoke-ListUserMailboxDetails {
         $ArchiveSizeRequest = $Results.'Get-MailboxStatistics'
         $BlockedSender = $Results.'Get-BlockedSenderAddress'
         $PermsRequest2 = $Results.'Get-RecipientPermission'
-        $StatsRequest = New-GraphGetRequest -uri "https://outlook.office365.com/adminapi/beta/$($TenantFilter)/Mailbox('$($MailboxDetailedRequest.UserPrincipalName)')/Exchange.GetMailboxStatistics()" -Tenantid $TenantFilter -scope ExchangeOnline -noPagination $true
+
+        $StatsRequest = New-GraphGetRequest -uri "https://outlook.office365.com/adminapi/beta/$($TenantFilter)/Mailbox('$($UserID)')/Exchange.GetMailboxStatistics()" -Tenantid $TenantFilter -scope ExchangeOnline -noPagination $true
 
 
         # Handle ArchiveEnabled and AutoExpandingArchiveEnabled

Modules/CIPPCore/Public/GraphHelper/New-ExoBulkRequest.ps1

@@ -85,7 +85,7 @@ function New-ExoBulkRequest {
                 }
                 $BatchBodyJson = ConvertTo-Json -InputObject $BatchBodyObj -Depth 10
                 $Results = Invoke-RestMethod $BatchURL -ResponseHeadersVariable responseHeaders -Method POST -Body $BatchBodyJson -Headers $Headers -ContentType 'application/json; charset=utf-8'
-                $ReturnedData.Add($Results.responses)
+                $ReturnedData.AddRange(@($Results.responses))
 
                 Write-Host "Batch #$($batches.IndexOf($batch) + 1) of $($batches.Count) processed"
             }
@@ -101,7 +101,7 @@ function New-ExoBulkRequest {
             foreach ($item in $ReturnedData) {
                 $itemId = $item.id
                 $CmdletName = $IdToCmdletName[$itemId]
-                $body = $item.body
+                $body = $item.body.PSObject.Copy()
 
                 if ($body.'@adminapi.warnings') {
                     Write-Warning ($body.'@adminapi.warnings' | Out-String)
@@ -116,16 +116,16 @@ function New-ExoBulkRequest {
                 }
                 $resultValue = $body.value
 
-                # Assign results without using += or ArrayList
                 if (-not $FinalData.ContainsKey($CmdletName)) {
-                    $FinalData[$CmdletName] = @($resultValue)
+                    $FinalData[$CmdletName] = [System.Collections.Generic.List[object]]::new()
+                    $FinalData.$CmdletName.Add($resultValue)
                 } else {
-                    $FinalData[$CmdletName] = $FinalData[$CmdletName] + $resultValue
+                    $FinalData.$CmdletName.Add($resultValue)
                 }
             }
         } else {
             $FinalData = foreach ($item in $ReturnedData) {
-                $body = $item.body
+                $body = $item.body.PSObject.Copy()
 
                 if ($body.'@adminapi.warnings') {
                     Write-Warning ($body.'@adminapi.warnings' | Out-String)
@@ -141,7 +141,6 @@ function New-ExoBulkRequest {
                 $body.value
             }
         }
-
         return $FinalData
 
     } else {