improved reports query, could be further optimized

app/Http/Controllers/ReportsController.php

@@ -11,6 +11,7 @@ use App\Models\AssetModel;
 use App\Models\Category;
 use App\Models\AssetMaintenance;
 use App\Models\CheckoutAcceptance;
+use App\Models\Company;
 use App\Models\CustomField;
 use App\Models\Depreciation;
 use App\Models\License;
@@ -18,6 +19,7 @@ use App\Models\ReportTemplate;
 use App\Models\Setting;
 use App\Notifications\CheckoutAssetNotification;
 use Carbon\Carbon;
+use Illuminate\Database\Eloquent\Relations\MorphTo;
 use Illuminate\Http\Request;
 use Illuminate\Http\Response;
 use Illuminate\Support\Facades\Mail;
@@ -1109,43 +1111,44 @@ class ReportsController extends Controller
         $this->authorize('reports.view');
         $showDeleted = $deleted == 'deleted';
 
+//        $acceptances = CheckoutAcceptance::pending()
+//            ->where('checkoutable_type', 'App\Models\Asset')
+//            ->withTrashed()
+//            ->with([
+//                'assignedTo',
+//                'checkoutable.assignedTo',
+//                'checkoutable.model'
+//            ])->get();
         $assetsForReport = collect();
 
         $query = CheckoutAcceptance::pending()
             ->where('checkoutable_type', 'App\Models\Asset')
-            ->with(['checkoutable.assignedTo', 'checkoutable.model']); // Eager load common relationships
+            ->with([
+                'checkoutable' => function (MorphTo $query) {
+                    $query->morphWith([
+                        AssetModel::class => ['model'],
+                        Company::class => ['company'],
+                        Asset::class => ['assignedTo'],
+                    ])->with('model.category');
+                },
+                'assignedTo' => function($query){
+                         $query->withTrashed();
+                    }
+            ]);
 
         if ($showDeleted) {
-            $query->withTrashed()->with(['assignedTo']);
-        }
-        else {
-            $query->with(['assignedTo' => function ($query) {
-                $query->withTrashed();
-            }]);
+            $query->withTrashed();
         }
 
-// Process records in chunks
-        $query->chunk(100, function ($chunk) use (&$assetsForReport) {
-
-            $filtered = $chunk->filter(function ($acceptance) {
-                $acceptance_checkoutable_flag = false;
-
-                if ($acceptance->checkoutable){
-                    $acceptance_checkoutable_flag = $acceptance->checkoutable->assignedTo();
-                }
-                // Return true if user
-                return $acceptance->checkoutable_type === 'App\Models\Asset' && $acceptance_checkoutable_flag;
-            })->map(function ($acceptance) {
-
-                return [
-                    'assetItem' => $acceptance->checkoutable,
-                    'acceptance' => $acceptance,
-                ];
+        $assetsForReport = $query->get()
+                ->map(function ($acceptance) {
+                    return [
+                        'assetItem' => $acceptance->checkoutable,
+                        'acceptance' => $acceptance,
+                    ];
             });
-
             // Merge results into the main collection
-            $assetsForReport = $assetsForReport->merge($filtered);
-        });
+
 
         return view('reports/unaccepted_assets', compact('assetsForReport','showDeleted' ));
     }

app/Http/Traits/CheckoutableTrait.php

@@ -1,24 +0,0 @@
-<?php
-
-namespace App\Http\Traits;
-
-trait CheckoutableTrait
-{
-    public const LOCATION = 'location';
-    public const ASSET = 'asset';
-    public const USER = 'user';
-    public function checkedOutToUser(): bool
-    {
-        return $this->assignedType() === self::USER;
-    }
-
-    public function checkedOutToLocation(): bool
-    {
-        return $this->assignedType() === self::LOCATION;
-    }
-
-    public function checkedOutToAsset(): bool
-    {
-        return $this->assignedType() === self::ASSET;
-    }
-}