Merge pull request #7 from tarfin-labs/nullable-cast-bugfix

Nullable cast bugfix

Author: tkaratug

CHANGELOG.md

@@ -2,6 +2,16 @@
 
 All notable changes to `laravel-spatial` will be documented in this file
 
-## 1.0.0 - 201X-XX-XX
+# 1.1.2 - 2022-03-08
+- Bug fixed for casting location columns that is null.
 
+# 1.1.1 - 2022-01-20
+- `axis-order` added to setter of `LocationCast`.
+- `axis-order` added to query in `HasSpatial` trait.
+
+# 1.1.0 - 2022-01-20
+- `Point` type added to Doctrine mapped types in the service provider.
+- SRID support added to setter of `LocationCast`.
+
+## 1.0.0 - 2022-01-18
 - initial release

src/Casts/LocationCast.php

@@ -13,8 +13,12 @@
 
 class LocationCast implements CastsAttributes, SerializesCastableAttributes
 {
-    public function get($model, string $key, $value, array $attributes): Point
+    public function get($model, string $key, $value, array $attributes): ?Point
     {
+        if (is_null($value)) {
+            return null;
+        }
+
         $coordinates = explode(',', $value);
 
         if (count($coordinates) > 1) {

tests/LocationCastTest.php

@@ -73,6 +73,19 @@ public function it_can_get_a_casted_attribute(): void
         $this->assertEquals($point->getSrid(), $address->location->getSrid());
     }
 
+    /** @test */
+    public function it_returns_null_if_the_value_of_the_casted_column_is_null(): void
+    {
+        // 1. Arrange
+        $address = new Address();
+
+        // 2. Act
+        $address->save();
+
+        // 3. Assert
+        $this->assertNull($address->location);
+    }
+
     /** @test */
     public function it_can_serialize_a_casted_attribute(): void
     {

tests/database/migrations/0000_00_00_000000_create_addresses_table.php

@@ -10,7 +10,7 @@ public function up(): void
     {
         Schema::create('addresses', function (Blueprint $table) {
             $table->id();
-            $table->point('location');
+            $table->point('location')->nullable();
             $table->timestamps();
         });
     }