Fix error for CSV/GeoJSON provider when the bbox parameter has 6 coordinates (#2187)

* Fix error for CSV provider when the bbox parameter has 6 coordinates

* Fix error for GeoJSON provider when the bbox parameter has 6 coordinates

Fix error for GeoJSON provider when the bbox parameter has 6 coordinates

* Add one more test for the CSW provider that uses the same function
giving error

Author: francbartoli

pygeoapi/provider/csv_.py

@@ -1,8 +1,10 @@
 # =================================================================
 #
 # Authors: Tom Kralidis <[email protected]>
+#          Francesco Bartoli <[email protected]>
 #
 # Copyright (c) 2025 Tom Kralidis
+# Copyright (c) 2025 Francesco Bartoli
 #
 # Permission is hereby granted, free of charge, to any person
 # obtaining a copy of this software and associated documentation
@@ -138,6 +140,9 @@ def _load(self, offset=0, limit=10, resulttype='results',
 
             if bbox:
                 LOGGER.debug('processing bbox parameter')
+                if len(bbox) > 4:
+                    LOGGER.debug("bbox reduced to 4 elements")
+                    bbox = bbox[:4]
                 data_ = filter(
                     lambda f: all(
                         [self._intersects(f, bbox)]), data_)

pygeoapi/provider/geojson.py

@@ -127,6 +127,9 @@ def _load(self, bbox=[], skip_geometry=None, properties=[],
         # filter by bbox if set
         if bbox:
             LOGGER.debug('processing bbox parameter')
+            if len(bbox) > 4:
+                LOGGER.debug("bbox reduced to 4 elements")
+                bbox = bbox[:4]
             data['features'] = [f for f in data['features'] if \
                 self._intersects(f['geometry'], bbox)]  # noqa
 

tests/provider/test_csv__provider.py

@@ -140,6 +140,9 @@ def test_query(config):
     results = p.query(bbox=[-75, 45, -64, 55])
     assert len(results['features'][0]['properties']) == 2
 
+    results = p.query(bbox=[-75, 45, -64, 55, 0, 0])
+    assert len(results['features'][0]['properties']) == 2
+
 
 def test_get_invalid_property(config):
     """Testing query for an invalid property name"""

tests/provider/test_csw_provider.py

@@ -1,8 +1,10 @@
 # =================================================================
 #
 # Authors: Tom Kralidis <[email protected]>
+#          Francesco Bartoli <[email protected]>
 #
 # Copyright (c) 2025 Tom Kralidis
+# Copyright (c) 2025 Francesco Bartoli
 #
 # Permission is hereby granted, free of charge, to any person
 # obtaining a copy of this software and associated documentation
@@ -109,6 +111,9 @@ def test_query(config):
     results = p.query(bbox=[-10, 40, 0, 60])
     assert len(results['features']) == 2
 
+    results = p.query(bbox=[-10, 40, 0, 60, 0, 0])
+    assert len(results['features']) == 2
+
     results = p.query(properties=[('title', 'Maecenas enim')])
     assert len(results['features']) == 1
 

tests/provider/test_geojson_provider.py

@@ -104,6 +104,11 @@ def test_query(fixture, config):
     assert results['numberMatched'] == 1
     assert results['numberReturned'] == 1
 
+    results = p.query(bbox=[120, 10, 126, 11, 0, 0])
+    assert len(results['features']) == 1
+    assert results['numberMatched'] == 1
+    assert results['numberReturned'] == 1
+
 
 def test_get(fixture, config):
     p = GeoJSONProvider(config)