test(laravel): standard put model should update when it exists

api-platform · Sep 20, 2024 · a76aca1 · a76aca1
1 parent 72a0b66
commit a76aca1
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/src/Laravel/Eloquent/State/PersistProcessor.php b/src/Laravel/Eloquent/State/PersistProcessor.php
@@ -14,6 +14,7 @@
 namespace ApiPlatform\Laravel\Eloquent\State;
 
 use ApiPlatform\Laravel\Eloquent\Metadata\ModelMetadata;
+use ApiPlatform\Metadata\HttpOperation;
 use ApiPlatform\Metadata\Operation;
 use ApiPlatform\State\ProcessorInterface;
 use Illuminate\Database\Eloquent\Relations\BelongsTo;
@@ -41,6 +42,15 @@ public function process(mixed $data, Operation $operation, array $uriVariables =
             }
         }
 
+        if (($previousData = $context['previous_data'] ?? null) && $operation instanceof HttpOperation && 'PUT' === $operation->getMethod() && ($operation->getExtraProperties()['standard_put'] ?? true)) {
+            foreach ($this->modelMetadata->getAttributes($data) as $attribute) {
+                if ($attribute['primary'] ?? false) {
+                    $data->{$attribute['name']} = $previousData->{$attribute['name']};
+                }
+            }
+            $data->exists = true;
+        }
+
         $data->saveOrFail();
         $data->refresh();
 

diff --git a/src/Laravel/Tests/JsonLdTest.php b/src/Laravel/Tests/JsonLdTest.php
@@ -97,7 +97,6 @@ public function testUpdateBook(): void
         );
         $response->assertStatus(200);
         $response->assertJsonFragment([
-            '@id' => $iri,
             'name' => 'updated title',
         ]);
     }