Adjust GeometryCast regex to support geometry expression without SRID

Author: nickknissen

src/GeometryCast.php

@@ -36,10 +36,9 @@ 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());
+            $expressionValues = $this->extractFromExpression($value, $model->getConnection());
 
-            return $this->className::fromWkt($wkt, $srid);
+            return $this->className::fromWkt($expressionValues['wkt'], $expressionValues['srid']);
         }
 
         return $this->className::fromWkb($value);
@@ -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{wkt: string, srid: int}
+     */
+    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 ['wkt' => $matches[1], 'srid' => (int) ($matches[2] ?? 0)];
     }
 }