Use Eloquent Attributes instead of magic methods

cybercog · Mar 9, 2024 · 8e0e4bd · 8e0e4bd
1 parent d9fae25
commit 8e0e4bd
Show file tree

Hide file tree

Showing 8 changed files with 93 additions and 75 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -12,6 +12,7 @@ All notable changes to `laravel-love` will be documented in this file.
 
 ### Changed
 
+- ([#251]) Changed Models attributes casting style
 - ([#252]) All Models are unguarded now
 
 ### Removed
@@ -596,6 +597,7 @@ Follow [upgrade instructions](UPGRADING.md#from-v5-to-v6) to migrate database to
 
 [#258]: https://github.com/cybercog/laravel-love/pull/258
 [#252]: https://github.com/cybercog/laravel-love/pull/252
+[#251]: https://github.com/cybercog/laravel-love/pull/251
 [#248]: https://github.com/cybercog/laravel-love/pull/248
 [#247]: https://github.com/cybercog/laravel-love/pull/247
 [#240]: https://github.com/cybercog/laravel-love/pull/240

diff --git a/src/Reactant/Models/Reactant.php b/src/Reactant/Models/Reactant.php
@@ -30,6 +30,7 @@
 use Cog\Laravel\Love\Reactant\ReactionTotal\Models\ReactionTotal;
 use Cog\Laravel\Love\Reaction\Models\Reaction;
 use Cog\Laravel\Love\Support\Database\Eloquent\Model;
+use Illuminate\Database\Eloquent\Casts\Attribute;
 use Illuminate\Database\Eloquent\Factories\HasFactory;
 use Illuminate\Database\Eloquent\Relations\HasMany;
 use Illuminate\Database\Eloquent\Relations\HasOne;
@@ -45,12 +46,13 @@ final class Reactant extends Model implements
 
     protected static $unguarded = true;
 
-    /**
-     * @var string[]
-     */
-    protected $casts = [
-        'id' => 'string',
-    ];
+    public function id(): Attribute
+    {
+        return new Attribute(
+            get: fn (string | null $value) => $value,
+            set: fn (string | null $value) => $value,
+        );
+    }
 
     public function reactable(): MorphTo
     {

diff --git a/src/Reactant/ReactionCounter/Models/ReactionCounter.php b/src/Reactant/ReactionCounter/Models/ReactionCounter.php
@@ -19,6 +19,7 @@
 use Cog\Laravel\Love\Reactant\Models\Reactant;
 use Cog\Laravel\Love\ReactionType\Models\ReactionType;
 use Cog\Laravel\Love\Support\Database\Eloquent\Model;
+use Illuminate\Database\Eloquent\Casts\Attribute;
 use Illuminate\Database\Eloquent\Factories\HasFactory;
 use Illuminate\Database\Eloquent\Relations\BelongsTo;
 
@@ -43,13 +44,21 @@ final class ReactionCounter extends Model implements
         'weight' => self::WEIGHT_DEFAULT,
     ];
 
-    /**
-     * @var string[]
-     */
-    protected $casts = [
-        'count' => 'integer',
-        'weight' => 'float',
-    ];
+    public function count(): Attribute
+    {
+        return new Attribute(
+            get: fn (int | null $value) => $value ?? self::COUNT_DEFAULT,
+            set: fn (int | null $value) => $value ?? self::COUNT_DEFAULT,
+        );
+    }
+
+    public function weight(): Attribute
+    {
+        return new Attribute(
+            get: fn (float | null $value) => $value ?? self::WEIGHT_DEFAULT,
+            set: fn (float | null $value) => $value ?? self::WEIGHT_DEFAULT,
+        );
+    }
 
     public function reactant(): BelongsTo
     {
@@ -116,16 +125,4 @@ public function decrementWeight(
     ): void {
         $this->decrement('weight', $amount);
     }
-
-    public function setCountAttribute(
-        int | null $count,
-    ): void {
-        $this->attributes['count'] = $count ?? self::COUNT_DEFAULT;
-    }
-
-    public function setWeightAttribute(
-        float | null $weight,
-    ): void {
-        $this->attributes['weight'] = $weight ?? self::WEIGHT_DEFAULT;
-    }
 }
diff --git a/src/Reactant/ReactionTotal/Models/ReactionTotal.php b/src/Reactant/ReactionTotal/Models/ReactionTotal.php
@@ -17,6 +17,7 @@
 use Cog\Contracts\Love\Reactant\ReactionTotal\Models\ReactionTotal as ReactionTotalInterface;
 use Cog\Laravel\Love\Reactant\Models\Reactant;
 use Cog\Laravel\Love\Support\Database\Eloquent\Model;
+use Illuminate\Database\Eloquent\Casts\Attribute;
 use Illuminate\Database\Eloquent\Factories\HasFactory;
 use Illuminate\Database\Eloquent\Relations\BelongsTo;
 
@@ -41,13 +42,21 @@ final class ReactionTotal extends Model implements
         'weight' => self::WEIGHT_DEFAULT,
     ];
 
-    /**
-     * @var string[]
-     */
-    protected $casts = [
-        'count' => 'integer',
-        'weight' => 'float',
-    ];
+    public function count(): Attribute
+    {
+        return new Attribute(
+            get: fn (int | null $value) => $value ?? self::COUNT_DEFAULT,
+            set: fn (int | null $value) => $value ?? self::COUNT_DEFAULT,
+        );
+    }
+
+    public function weight(): Attribute
+    {
+        return new Attribute(
+            get: fn (float | null $value) => $value ?? self::WEIGHT_DEFAULT,
+            set: fn (float | null $value) => $value ?? self::WEIGHT_DEFAULT,
+        );
+    }
 
     public function reactant(): BelongsTo
     {
@@ -92,16 +101,4 @@ public function decrementWeight(
     ): void {
         $this->decrement('weight', $amount);
     }
-
-    public function setCountAttribute(
-        int | null $count,
-    ): void {
-        $this->attributes['count'] = $count ?? self::COUNT_DEFAULT;
-    }
-
-    public function setWeightAttribute(
-        float | null $weight,
-    ): void {
-        $this->attributes['weight'] = $weight ?? self::WEIGHT_DEFAULT;
-    }
 }
diff --git a/src/Reacter/Models/Reacter.php b/src/Reacter/Models/Reacter.php
@@ -24,6 +24,7 @@
 use Cog\Contracts\Love\ReactionType\Models\ReactionType as ReactionTypeInterface;
 use Cog\Laravel\Love\Reaction\Models\Reaction;
 use Cog\Laravel\Love\Support\Database\Eloquent\Model;
+use Illuminate\Database\Eloquent\Casts\Attribute;
 use Illuminate\Database\Eloquent\Factories\HasFactory;
 use Illuminate\Database\Eloquent\Relations\HasMany;
 use Illuminate\Database\Eloquent\Relations\MorphTo;
@@ -37,12 +38,13 @@ final class Reacter extends Model implements
 
     protected static $unguarded = true;
 
-    /**
-     * @var string[]
-     */
-    protected $casts = [
-        'id' => 'string',
-    ];
+    public function id(): Attribute
+    {
+        return new Attribute(
+            get: fn (string | null $value) => $value,
+            set: fn (string | null $value) => $value,
+        );
+    }
 
     public function reacterable(): MorphTo
     {

diff --git a/src/Reaction/Models/Reaction.php b/src/Reaction/Models/Reaction.php
@@ -23,6 +23,7 @@
 use Cog\Laravel\Love\Reacter\Models\Reacter;
 use Cog\Laravel\Love\ReactionType\Models\ReactionType;
 use Cog\Laravel\Love\Support\Database\Eloquent\Model;
+use Illuminate\Database\Eloquent\Casts\Attribute;
 use Illuminate\Database\Eloquent\Factories\HasFactory;
 use Illuminate\Database\Eloquent\Relations\BelongsTo;
 
@@ -46,13 +47,31 @@ final class Reaction extends Model implements
         'rate' => self::RATE_DEFAULT,
     ];
 
-    /**
-     * @var string[]
-     */
-    protected $casts = [
-        'id' => 'string',
-        'rate' => 'float',
-    ];
+    public function id(): Attribute
+    {
+        return new Attribute(
+            get: fn (string | null $value) => $value,
+            set: fn (string | null $value) => $value,
+        );
+    }
+
+    public function rate(): Attribute
+    {
+        return new Attribute(
+            get: fn (float | null $value) => $value ?? self::RATE_DEFAULT,
+            set: function (float | null $value) {
+                if ($value !== null && ($value < self::RATE_MIN || $value > self::RATE_MAX)) {
+                    throw RateOutOfRange::withValueBetween(
+                        $value,
+                        self::RATE_MIN,
+                        self::RATE_MAX,
+                    );
+                }
+
+                return $value ?? self::RATE_DEFAULT;
+            },
+        );
+    }
 
     public function reactant(): BelongsTo
     {
@@ -99,16 +118,6 @@ public function getWeight(): float
         return $this->getType()->getMass() * $this->getRate();
     }
 
-    public function setRateAttribute(
-        float | null $rate,
-    ): void {
-        if ($rate !== null && ($rate < self::RATE_MIN || $rate > self::RATE_MAX)) {
-            throw RateOutOfRange::withValueBetween($rate, self::RATE_MIN, self::RATE_MAX);
-        }
-
-        $this->attributes['rate'] = $rate ?? self::RATE_DEFAULT;
-    }
-
     public function isOfType(
         ReactionTypeInterface $reactionType,
     ): bool {

diff --git a/src/ReactionType/Models/ReactionType.php b/src/ReactionType/Models/ReactionType.php
@@ -17,6 +17,7 @@
 use Cog\Contracts\Love\ReactionType\Models\ReactionType as ReactionTypeInterface;
 use Cog\Laravel\Love\Reaction\Models\Reaction;
 use Cog\Laravel\Love\Support\Database\Eloquent\Model;
+use Illuminate\Database\Eloquent\Casts\Attribute;
 use Illuminate\Database\Eloquent\Factories\HasFactory;
 use Illuminate\Database\Eloquent\Relations\HasMany;
 
@@ -38,13 +39,21 @@ final class ReactionType extends Model implements
         'mass' => self::MASS_DEFAULT,
     ];
 
-    /**
-     * @var string[]
-     */
-    protected $casts = [
-        'id' => 'string',
-        'mass' => 'integer',
-    ];
+    public function id(): Attribute
+    {
+        return new Attribute(
+            get: fn (string | null $value) => $value,
+            set: fn (string | null $value) => $value,
+        );
+    }
+
+    public function mass(): Attribute
+    {
+        return new Attribute(
+            get: fn (int | null $value) => $value ?? self::MASS_DEFAULT,
+            set: fn (int | null $value) => $value ?? self::MASS_DEFAULT,
+        );
+    }
 
     /**
      * @var array<self>

diff --git a/tests/Unit/Reaction/Models/ReactionTest.php b/tests/Unit/Reaction/Models/ReactionTest.php
@@ -82,7 +82,7 @@ public function it_throws_rate_out_of_range_on_fill_rate_with_overflow_value():
     /** @test */
     public function it_casts_id_to_string(): void
     {
-        $reaction = Reaction::factory()->make([
+        $reaction = new Reaction([
             'id' => 4,
         ]);