- return 204 for tiles that are within the tile matrix set limits

Author: doublebyte1

pygeoapi/api/tiles.py

@@ -247,7 +247,7 @@ def get_collection_tiles_data(
         if content is None:
             msg = 'identifier not found'
             return api.get_exception(
-                HTTPStatus.NOT_FOUND, headers, format_, 'NotFound', msg)
+                HTTPStatus.NO_CONTENT, headers, format_, 'NocContent', msg)
         else:
             return headers, HTTPStatus.OK, content
 
@@ -444,7 +444,6 @@ def tilematrixset(api: API,
 
     return headers, HTTPStatus.OK, to_json(tms, api.pretty_print)
 
-
 def get_oas_30(cfg: dict, locale: str) -> tuple[list[dict[str, str]], dict[str, dict]]:  # noqa
     """
     Get OpenAPI fragments

pygeoapi/provider/base_mvt.py

@@ -248,3 +248,45 @@ def get_tms_links(self):
             ]
         }
         return links
+
+    def get_tilematrixset(self, tileMatrixSetId):
+        """
+        Get tilematrixset
+
+        :param tileMatrixSetId: tilematrixsetid str
+
+        :returns: tilematrixset enum object
+        """
+
+        enums = [e.value for e in TileMatrixSetEnum]
+        enum = None
+
+        try:
+            for e in enums:
+                if tileMatrixSetId == e.tileMatrixSet:
+                    enum = e
+            if not enum:
+                raise ValueError('could not find this tilematrixset')
+            return enum
+
+        except ValueError as err:
+            LOGGER.error(err)
+
+    def is_in_limits(self, tilematrixset, z, x, y):
+        """
+        Is within the limits of the tilematrixset
+
+        :param z: tilematrix
+        :param x: x
+        :param y: y
+
+        :returns: wether this tile is within the tile matrix
+        set limits (Boolean)
+        """
+
+        try:
+            if int(x) <= tilematrixset.tileMatrices[int(z)]['matrixWidth'] and int(y) <= tilematrixset.tileMatrices[int(z)]['matrixHeight']: # noqa
+                return True
+            return False
+        except ValueError as err:
+            LOGGER.error(err)

pygeoapi/provider/mvt_elastic.py

@@ -179,6 +179,8 @@ def get_tiles(self, layer=None, tileset=None,
                     resp = session.get(f'{base_url}/{layer}/{z}/{y}/{x}{url_query}', json=data)  # noqa
 
                     if resp.status_code == 404:
+                        if (self.is_in_limits(self.get_tilematrixset(tileset), z, x, y)): # noqa
+                            return None
                         raise ProviderTileNotFoundError
 
                     resp.raise_for_status()

pygeoapi/provider/mvt_proxy.py

@@ -175,6 +175,8 @@ def get_tiles(self, layer=None, tileset=None,
                         resp = session.get(f'{base_url}/{layer}/{z}/{y}/{x}{url_query}')  # noqa
 
                     if resp.status_code == 404:
+                        if (self.is_in_limits(self.get_tilematrixset(tileset), z, x, y)): # noqa
+                            return None
                         raise ProviderTileNotFoundError
 
                     resp.raise_for_status()

pygeoapi/provider/mvt_tippecanoe.py

@@ -188,6 +188,8 @@ def get_tiles_from_url(self, layer=None, tileset=None,
                 resp = session.get(f'{base_url}/{layer}/{z}/{y}/{x}{extension}')  # noqa
 
                 if resp.status_code == 404:
+                    if (self.is_in_limits(self.get_tilematrixset(tileset), z, x, y)): # noqa
+                        return None
                     raise ProviderTileNotFoundError
 
                 resp.raise_for_status()
@@ -218,6 +220,8 @@ def get_tiles_from_disk(self, layer=None, tileset=None,
             with open(service_url_path, mode='rb') as tile:
                 return tile.read()
         except FileNotFoundError as err:
+            if (self.is_in_limits(self.get_tilematrixset(tileset), z, x, y)): # noqa
+                return None
             raise ProviderTileNotFoundError(err)
 
     def get_tiles(self, layer=None, tileset=None,