Adjust GeometryCast regex to support geometry expression without SRID

Author: nickknissen

Diff for: src/GeometryCast.php

@@ -36,8 +36,7 @@ public function get($model, string $key, $value, array $attributes): ?Geometry
         }
 
         if ($value instanceof ExpressionContract) {
-            $wkt = $this->extractWktFromExpression($value, $model->getConnection());
-            $srid = $this->extractSridFromExpression($value, $model->getConnection());
+            [$wkt, $srid, $options] = $this->extractFromExpression($value, $model->getConnection());
 
             return $this->className::fromWkt($wkt, $srid);
         }
@@ -76,23 +75,16 @@ public function set($model, string $key, $value, array $attributes): ?Expression
         return $value->toSqlExpression($model->getConnection());
     }
 
-    private function extractWktFromExpression(ExpressionContract $expression, Connection $connection): string
-    {
-        $grammar = $connection->getQueryGrammar();
-        $expressionValue = $expression->getValue($grammar);
-
-        preg_match('/ST_GeomFromText\(\'(.+)\', .+(, .+)?\)/', (string) $expressionValue, $match);
-
-        return $match[1];
-    }
-
-    private function extractSridFromExpression(ExpressionContract $expression, Connection $connection): int
+    /**
+     * @return array{string, int, string}
+     */
+    private function extractFromExpression(ExpressionContract $expression, Connection $connection): array
     {
         $grammar = $connection->getQueryGrammar();
         $expressionValue = $expression->getValue($grammar);
 
-        preg_match('/ST_GeomFromText\(\'.+\', (.+)(, .+)?\)/', (string) $expressionValue, $match);
+        preg_match("/ST_GeomFromText\(\s*'([^']+)'\s*(?:,\s*(\d+))?\s*(?:,\s*'([^']+)')?\s*\)/", (string) $expressionValue, $matches);
 
-        return (int) $match[1];
+        return [$matches[1], (int) ($matches[2] ?? 0), $matches[3] ?? ''];
     }
 }

Diff for: tests/GeometryCastTest.php

@@ -1,7 +1,6 @@
 <?php
 
 use Illuminate\Support\Facades\DB;
-use MatanYadaev\EloquentSpatial\AxisOrder;
 use MatanYadaev\EloquentSpatial\Enums\Srid;
 use MatanYadaev\EloquentSpatial\GeometryExpression;
 use MatanYadaev\EloquentSpatial\Objects\LineString;
@@ -197,6 +196,6 @@
     $testPlace->point = null;
     expect($testPlace->isDirty())->toBeTrue();
 })->with([
-    "without SRID" => "ST_GeomFromText('POINT(12.38057 55.73406)')",
-    "with SRID" => "ST_GeomFromText('POINT(12.38057 55.73406)', 4326)",
+    'without SRID' => "ST_GeomFromText('POINT(12.38057 55.73406)')",
+    'with SRID' => "ST_GeomFromText('POINT(12.38057 55.73406)', 4326)",
 ]);