GeoAlchemy2 0.15.1__tar.gz → 0.15.2__tar.gz

{geoalchemy2-0.15.1 → geoalchemy2-0.15.2}/.pre-commit-config.yaml

@@ -2,7 +2,7 @@ default_language_version:
   python: python3.8
 repos:
   - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v4.5.0
+    rev: v4.6.0
     hooks:
       - id: check-added-large-files
       - id: check-case-conflict
@@ -17,11 +17,11 @@ repos:
     hooks:
       - id: isort
   - repo: https://github.com/psf/black
-    rev: 24.3.0
+    rev: 24.4.2
     hooks:
       - id: black
   - repo: https://github.com/codespell-project/codespell
-    rev: v2.2.6
+    rev: v2.3.0
     hooks:
       - id: codespell
   - repo: https://github.com/PyCQA/pydocstyle
@@ -31,7 +31,7 @@ repos:
         additional_dependencies: ["tomli"]
         exclude: "tests"
   - repo: https://github.com/PyCQA/flake8
-    rev: 7.0.0
+    rev: 7.1.0
     hooks:
       - id: flake8
 ci:

{geoalchemy2-0.15.1 → geoalchemy2-0.15.2}/CHANGES.txt

@@ -1,6 +1,11 @@
 GeoAlchemy 2 Changelog
 ======================
 
+0.15.2
+------
+
+* Fix: Can handle negative coordinates with Spatialite @adrien-berchet (#517)
+
 0.15.1
 ------
 

{geoalchemy2-0.15.1 → geoalchemy2-0.15.2/GeoAlchemy2.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: GeoAlchemy2
-Version: 0.15.1
+Version: 0.15.2
 Summary: Using SQLAlchemy with Spatial Databases
 Home-page: https://geoalchemy-2.readthedocs.io/en/stable/
 Author: Eric Lemoine

{geoalchemy2-0.15.1/GeoAlchemy2.egg-info → geoalchemy2-0.15.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: GeoAlchemy2
-Version: 0.15.1
+Version: 0.15.2
 Summary: Using SQLAlchemy with Spatial Databases
 Home-page: https://geoalchemy-2.readthedocs.io/en/stable/
 Author: Eric Lemoine

{geoalchemy2-0.15.1 → geoalchemy2-0.15.2}/geoalchemy2/elements.py

@@ -117,7 +117,7 @@ class WKTElement(_SpatialElement):
     """
 
     _REMOVE_SRID = re.compile("(SRID=([0-9]+); ?)?(.*)")
-    SPLIT_WKT_PATTERN = re.compile(r"((SRID=\d+) *; *)?([\w ]+) *(\([\d\. ,\(\)]+\))")
+    SPLIT_WKT_PATTERN = re.compile(r"((SRID=\d+) *; *)?([\w ]+) *(\([-\d\. ,\(\)]+\))")
 
     geom_from: str = "ST_GeomFromText"
     geom_from_extended_version: str = "ST_GeomFromEWKT"

{geoalchemy2-0.15.1 → geoalchemy2-0.15.2}/geoalchemy2/types/dialects/sqlite.py

@@ -1,6 +1,7 @@
 """This module defines specific functions for SQLite dialect."""
 
 import re
+import warnings
 
 from geoalchemy2.elements import RasterElement
 from geoalchemy2.elements import WKBElement
@@ -12,6 +13,10 @@ def format_geom_type(wkt, default_srid=None):
     """Format the Geometry type for SQLite."""
     match = re.match(WKTElement.SPLIT_WKT_PATTERN, wkt)
     if match is None:
+        warnings.warn(
+            "The given WKT could not be parsed by GeoAlchemy2, this could lead to undefined "
+            f"behavior with Z, M or ZM geometries or with incorrect SRID. The WKT string is: {wkt}"
+        )
         return wkt
     _, srid, geom_type, coords = match.groups()
     geom_type = geom_type.replace(" ", "")

{geoalchemy2-0.15.1 → geoalchemy2-0.15.2}/tests/__init__.py

@@ -5,8 +5,7 @@ import shutil
 import sys
 
 import pytest
-from packaging import version
-from pkg_resources import parse_version
+from packaging.version import parse as parse_version
 from sqlalchemy import __version__ as SA_VERSION
 from sqlalchemy import create_engine
 from sqlalchemy import select as raw_select
@@ -30,9 +29,9 @@ class test_only_with_dialects:
 
 def get_postgis_major_version(bind):
     try:
-        return version.parse(bind.execute(func.postgis_lib_version()).scalar()).major
+        return parse_version(bind.execute(func.postgis_lib_version()).scalar()).major
     except OperationalError:
-        return version.parse("0").major
+        return parse_version("0").major
 
 
 def get_postgres_major_version(bind):
@@ -81,7 +80,7 @@ def skip_pypy(msg=None):
 
 
 def select(args):
-    if version.parse(SA_VERSION) < version.parse("1.4"):
+    if parse_version(SA_VERSION) < parse_version("1.4"):
         return raw_select(args)
     else:
         return raw_select(*args)

{geoalchemy2-0.15.1 → geoalchemy2-0.15.2}/tests/test_functional.py

@@ -445,6 +445,32 @@ class TestInsertionCore:
             assert srid == 4326
             assert row[1] == from_shape(Point(1, 1), srid=4326, extended=True)
 
+    def test_insert_negative_coords(self, conn, Poi, setup_tables, dialect_name):
+        conn.execute(
+            Poi.__table__.insert(),
+            [
+                {"geom": "SRID=4326;POINT(-1 1)"},
+                {"geom": WKTElement("POINT(-1 1)", srid=4326)},
+                {"geom": WKTElement("SRID=4326;POINT(-1 1)", extended=True)},
+                {"geom": from_shape(Point(-1, 1), srid=4326)},
+                {"geom": from_shape(Point(-1, 1), srid=4326, extended=True)},
+            ],
+        )
+
+        results = conn.execute(Poi.__table__.select())
+        rows = results.fetchall()
+
+        for row in rows:
+            assert isinstance(row[1], WKBElement)
+            wkt = conn.execute(row[1].ST_AsText()).scalar()
+            assert format_wkt(wkt) == "POINT(-1 1)"
+            srid = conn.execute(row[1].ST_SRID()).scalar()
+            assert srid == 4326
+            if dialect_name == "mysql":
+                assert row[1] == from_shape(Point(-1, 1), srid=4326)
+            else:
+                assert row[1] == from_shape(Point(-1, 1), srid=4326, extended=True)
+
 
 class TestSelectBindParam:
     @pytest.fixture

{geoalchemy2-0.15.1 → geoalchemy2-0.15.2}/tests/test_functional_sqlite.py

@@ -1,7 +1,9 @@
 import re
 
 import pytest
+from shapely.geometry import GeometryCollection
 from shapely.geometry import LineString
+from shapely.geometry import Point
 from sqlalchemy import CheckConstraint
 from sqlalchemy import Column
 from sqlalchemy import Integer
@@ -22,6 +24,7 @@ from geoalchemy2.elements import WKTElement
 from geoalchemy2.shape import from_shape
 from geoalchemy2.shape import to_shape
 
+from . import format_wkt
 from . import select
 from . import skip_case_insensitivity
 from . import skip_pypy
@@ -310,6 +313,51 @@ class TestMiscellaneous:
             load_spatialite(conn.connection.dbapi_connection)
 
 
+class TestInsertionCore:
+    @pytest.fixture
+    def GeomObject(self, base):
+        class GeomObject(base):
+            __tablename__ = "any_geom_object"
+            id = Column(Integer, primary_key=True)
+            geom = Column(Geometry(srid=4326))
+
+        return GeomObject
+
+    def test_insert_unparsable_WKT(self, conn, GeomObject, setup_tables, dialect_name):
+        with pytest.warns(
+            UserWarning,
+            match=(
+                "The given WKT could not be parsed by GeoAlchemy2, this could lead to undefined "
+                "behavior"
+            ),
+        ):
+            conn.execute(
+                GeomObject.__table__.insert(),
+                [
+                    {"geom": "SRID=4326;GeometryCollection(POINT (-1 1),LINESTRING (2 2, 3 3))"},
+                ],
+            )
+
+        results = conn.execute(GeomObject.__table__.select())
+        rows = results.fetchall()
+
+        for row in rows:
+            assert isinstance(row[1], WKBElement)
+            wkt = conn.execute(row[1].ST_AsText()).scalar()
+            assert format_wkt(wkt) == "GEOMETRYCOLLECTION(POINT(-1 1),LINESTRING(2 2,3 3))"
+            srid = conn.execute(row[1].ST_SRID()).scalar()
+            assert srid == 4326
+            if dialect_name == "mysql":
+                extended = None
+            else:
+                extended = True
+            assert row[1] == from_shape(
+                GeometryCollection([Point(-1, 1), LineString([[2, 2], [3, 3]])]),
+                srid=4326,
+                extended=extended,
+            )
+
+
 class TestInsertionORM:
     @pytest.fixture
     def LocalPoint(self, base):