diff --git a/src/GeometryCast.php b/src/GeometryCast.php index a994059..213f4f6 100644 --- a/src/GeometryCast.php +++ b/src/GeometryCast.php @@ -69,7 +69,7 @@ public function set($model, string $key, $value, array $attributes): ?Expression if (! ($value instanceof $this->className)) { $geometryType = is_object($value) ? $value::class : gettype($value); throw new InvalidArgumentException( - sprintf('Expected %s, %s given.', static::class, $geometryType) + sprintf('Expected %s, %s given.', $this->className, $geometryType) ); } diff --git a/src/Traits/HasSpatial.php b/src/Traits/HasSpatial.php index ad6ee75..14816f0 100644 --- a/src/Traits/HasSpatial.php +++ b/src/Traits/HasSpatial.php @@ -11,6 +11,26 @@ trait HasSpatial { + public function originalIsEquivalent($key) + { + if (! array_key_exists($key, $this->original)) { + return false; + } + + $casts = $this->getCasts(); + + if (array_key_exists($key, $casts)) { + $original = $this->getOriginal($key); + $attribute = $this->getAttributeValue($key); + + if ($original instanceof Geometry && $attribute instanceof Geometry) { + return $original->getWktData() === $attribute->getWktData(); + } + } + + return parent::originalIsEquivalent($key); + } + public function scopeWithDistance( Builder $query, ExpressionContract|Geometry|string $column, diff --git a/tests/GeometryCastTest.php b/tests/GeometryCastTest.php index 4073d14..5f98015 100644 --- a/tests/GeometryCastTest.php +++ b/tests/GeometryCastTest.php @@ -126,3 +126,66 @@ expect($testPlace->point)->toEqual($point); }); + +it('checks a model record is not dirty after creation', function (): void { + $point = new Point(0, 180); + + /** @var TestPlace $testPlace */ + $testPlace = TestPlace::factory()->create(['point' => $point]); + + expect($testPlace->isDirty())->toBeFalse(); +}); + +it('checks a model record is not dirty after fetch', function (): void { + $point = new Point(0, 180); + TestPlace::factory()->create(['point' => $point]); + + /** @var TestPlace $testPlace */ + $testPlace = TestPlace::firstOrFail(); + + expect($testPlace->isDirty())->toBeFalse(); +}); + +it('checks a model record is dirty after update from null before save', function (): void { + $point = new Point(0, 180); + /** @var TestPlace $testPlace */ + $testPlace = TestPlace::factory()->create([]); + + $testPlace->point = $point; + + expect($testPlace->isDirty())->toBeTrue(); +}); + +it('checks a model record is dirty after update before save', function (): void { + $point = new Point(0, 180); + $point2 = new Point(0, 0); + /** @var TestPlace $testPlace */ + $testPlace = TestPlace::factory()->create(['point' => $point]); + + $testPlace->point = $point2; + + expect($testPlace->isDirty())->toBeTrue(); +}); + +it('checks a model record is not dirty after update and save', function (): void { + $point = new Point(0, 180); + $point2 = new Point(0, 0); + /** @var TestPlace $testPlace */ + $testPlace = TestPlace::factory()->create(['point' => $point]); + + $testPlace->point = $point2; + $testPlace->save(); + + expect($testPlace->isDirty())->toBeFalse(); +}); + +it('checks a model record is not dirty after update to same value before save', function (): void { + $point = new Point(0, 180); + $point2 = new Point(0, 180); + /** @var TestPlace $testPlace */ + $testPlace = TestPlace::factory()->create(['point' => $point]); + + $testPlace->point = $point2; + + expect($testPlace->isDirty())->toBeFalse(); +});