Merge pull request #3 from clickbar/fix-double-get-query

fix: Double get query for resource classes

src/AgGridExport.php

@@ -15,7 +15,7 @@
 use Maatwebsite\Excel\Concerns\WithMapping;
 use PhpOffice\PhpSpreadsheet\Cell\Coordinate;
 
-class AgGridExport implements FromQuery, WithHeadings, WithColumnFormatting, ShouldAutoSize, WithMapping
+class AgGridExport implements FromQuery, ShouldAutoSize, WithColumnFormatting, WithHeadings, WithMapping
 {
     /**
      * @var Collection<string, AgGridColumnDefinition>

src/AgGridQueryBuilder.php

@@ -151,10 +151,10 @@ public function toResponse($request): mixed
             $resourceClass = $this->resourceClass;
             if (is_a($resourceClass, ResourceCollection::class, true)) {
                 // the resource is already a collection
-                $data = new $resourceClass($this->get());
+                $data = new $resourceClass($data);
             } else {
                 // wrap in an anonymous collection
-                $data = $resourceClass::collection($this->get());
+                $data = $resourceClass::collection($data);
             }
         }
 

tests/TestClasses/Models/Flamingo.php

@@ -15,7 +15,7 @@
 use Illuminate\Database\Eloquent\Relations\BelongsTo;
 use Illuminate\Database\Eloquent\SoftDeletes;
 
-class Flamingo extends Model implements AgGridExportable, AgGridCustomFilterable
+class Flamingo extends Model implements AgGridCustomFilterable, AgGridExportable
 {
     use HasFactory;
     use SoftDeletes;