Don't automatically transform relation names

spatie · Oct 20, 2024 · 380942b · 380942b
1 parent e7b5ce0
commit 380942b
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 19 deletions.
diff --git a/src/Normalizers/Normalized/NormalizedModel.php b/src/Normalizers/Normalized/NormalizedModel.php
@@ -24,11 +24,9 @@ public function __construct(
 
     public function getProperty(string $name, DataProperty $dataProperty): mixed
     {
-        $propertyName = $this->model::$snakeAttributes ? Str::snake($name) : $name;
-
-        $value = array_key_exists($propertyName, $this->properties)
-            ? $this->properties[$propertyName]
-            : $this->fetchNewProperty($propertyName, $dataProperty);
+        $value = array_key_exists($name, $this->properties)
+            ? $this->properties[$name]
+            : $this->fetchNewProperty($name, $dataProperty);
 
         if ($value === null && ! $dataProperty->type->isNullable) {
             return UnknownProperty::create();
@@ -39,26 +37,20 @@ public function getProperty(string $name, DataProperty $dataProperty): mixed
 
     protected function fetchNewProperty(string $name, DataProperty $dataProperty): mixed
     {
-        $camelName = Str::camel($name);
-
         if ($dataProperty->attributes->contains(fn (object $attribute) => $attribute::class === LoadRelation::class)) {
             if (method_exists($this->model, $name)) {
                 $this->model->loadMissing($name);
-            } elseif (method_exists($this->model, $camelName)) {
-                $this->model->loadMissing($camelName);
             }
         }
 
         if ($this->model->relationLoaded($name)) {
             return $this->properties[$name] = $this->model->getRelation($name);
         }
-        if ($this->model->relationLoaded($camelName)) {
-            return $this->properties[$name] = $this->model->getRelation($camelName);
-        }
 
-        if (!$this->model->isRelation($name) && !$this->model->isRelation($camelName)) {
+        if (!$this->model->isRelation($name)) {
             try {
-                return $this->properties[$name] = $this->model->getAttribute($name);
+                $propertyName = $this->model::$snakeAttributes ? Str::snake($name) : $name;
+                return $this->properties[$name] = $this->model->getAttribute($propertyName);
             } catch (MissingAttributeException) {
                 // Fallback if missing Attribute
             }

diff --git a/tests/Fakes/FakeModelData.php b/tests/Fakes/FakeModelData.php
@@ -4,8 +4,10 @@
 
 use Carbon\CarbonImmutable;
 use Spatie\LaravelData\Attributes\DataCollectionOf;
+use Spatie\LaravelData\Attributes\MapInputName;
 use Spatie\LaravelData\Data;
 use Spatie\LaravelData\DataCollection;
+use Spatie\LaravelData\Mappers\CamelCaseMapper;
 use Spatie\LaravelData\Optional;
 
 class FakeModelData extends Data
@@ -14,7 +16,7 @@ public function __construct(
         public string $string,
         public ?string $nullable,
         public CarbonImmutable $date,
-        #[DataCollectionOf(FakeNestedModelData::class)]
+        #[DataCollectionOf(FakeNestedModelData::class), MapInputName(CamelCaseMapper::class)]
         public Optional|null|DataCollection $fake_nested_models,
         public string $accessor,
         public string $old_accessor,

diff --git a/tests/Normalizers/ModelNormalizerTest.php b/tests/Normalizers/ModelNormalizerTest.php
@@ -118,7 +118,7 @@
 
     $dataClass = new class () extends Data {
         #[LoadRelation, DataCollectionOf(FakeNestedModelData::class)]
-        public array $fake_nested_models;
+        public array $fakeNestedModels;
 
         #[LoadRelation, DataCollectionOf(FakeNestedModelData::class)]
         public array $fake_nested_models_snake_cased;
@@ -131,7 +131,7 @@
 
     $queryLog = DB::getQueryLog();
 
-    expect($data->fake_nested_models)
+    expect($data->fakeNestedModels)
         ->toHaveCount(2)
         ->each->toBeInstanceOf(FakeNestedModelData::class);
 
@@ -183,15 +183,18 @@
 
     $dataClass = new class () extends Data {
         #[DataCollectionOf(FakeNestedModelData::class), MapInputName(SnakeCaseMapper::class)]
-        public array|Optional $fakeNestedModels;
+        public array $fakeNestedModels;
+        #[DataCollectionOf(FakeNestedModelData::class), MapInputName(SnakeCaseMapper::class), LoadRelation]
+        public array|Optional $fakeNestedModelsSnakeCased;
 
         #[MapInputName(SnakeCaseMapper::class)]
         public string $oldAccessor;
     };
 
     $data = $dataClass::from($model->load('fakeNestedModels'));
 
-    expect($data->fakeNestedModels)
+    expect(isset($data->fakeNestedModels))->toBeFalse();
+    expect($data->fakeNestedModelsSnakeCased)
         ->toHaveCount(2)
         ->each()
         ->toBeInstanceOf(FakeNestedModelData::class);