t.rast.list: allow method gran with where option (#7678)

* enable where with method gran

* enable where with method gran, set '' to None for gran

* add gran-where test, move to tools

---------

Co-authored-by: SAKET KUMAR MALL <72020337+saket0187@users.noreply.github.com>
Co-authored-by: Anna Petrasova <kratochanna@gmail.com>

Author: 3 people

python/grass/temporal/abstract_space_time_dataset.py

@@ -1104,7 +1104,9 @@ def sample_by_dataset_sql(
 
         return obj_list
 
-    def get_registered_maps_as_objects_by_granularity(self, gran=None, dbif=None):
+    def get_registered_maps_as_objects_by_granularity(
+        self, gran: str | None = None, where: str | None = None, dbif=None
+    ):
         """Return all registered maps as ordered (by start_time) object list
         with "gap" map objects (id==None) for spatio-temporal topological
         operations that require the temporal extent only.
@@ -1143,6 +1145,8 @@ def get_registered_maps_as_objects_by_granularity(self, gran=None, dbif=None):
                     weeks, month, months, year, years". The unit of the
                     relative time granule is always the space time dataset
                     unit and can not be changed.
+        :param where: The SQL where statement to select a subset of
+                      the registered maps without "WHERE"
         :param dbif: The database interface to be used
 
         :return: ordered list of map lists. Each list represents a single
@@ -1158,16 +1162,19 @@ def get_registered_maps_as_objects_by_granularity(self, gran=None, dbif=None):
         if not check:
             self.msgr.fatal(_('Wrong granularity: "%s"') % str(gran))
 
-        start, end = self.get_temporal_extent_as_tuple()
+        maps = self.get_registered_maps_as_objects(
+            dbif=dbif, order="start_time", where=where
+        )
 
-        if start is None or end is None:
+        if not maps:
             return None
 
-        maps = self.get_registered_maps_as_objects(dbif=dbif, order="start_time")
+        start = maps[0].get_temporal_extent_as_tuple()[0]
+        end_extend = maps[-1].get_temporal_extent_as_tuple()
+        end = end_extend[1] if end_extend[1] is not None else end_extend[0]
 
-        if not maps:
+        if start is None or end is None:
             return None
-
         # We need to adjust the end time in case the dataset has no
         # interval time, so we can catch time instances at the end
         if self.get_map_time() != "interval":

python/grass/temporal/list_stds.py

@@ -279,17 +279,12 @@ def _get_get_registered_maps_as_objects_with_method(dataset, where, method, gran
         return dataset.get_registered_maps_as_objects(
             where=where, order="start_time", dbif=dbif
         )
-    if method != "gran":
-        msg = f"Invalid method '{method}'"
-        raise ValueError(msg)
-    if where:
-        msg = f"The where parameter is not supported with method={method}"
-        raise ValueError(msg)
-    if gran is not None and gran != "":
+    if method == "gran":
         return dataset.get_registered_maps_as_objects_by_granularity(
-            gran=gran, dbif=dbif
+            gran=gran, where=where, dbif=dbif
         )
-    return dataset.get_registered_maps_as_objects_by_granularity(dbif=dbif)
+    msg = f"Invalid method '{method}'"
+    raise ValueError(msg)
 
 
 def _get_get_registered_maps_as_objects_delta_gran(
@@ -511,6 +506,8 @@ def list_maps_of_stds(
                  dataset is used
     :param outpath: The path to file where to save output
     """
+    if gran == "":
+        gran = None
     if not output_format:
         if method == "comma":
             output_format = "line"

temporal/t.rast.list/t.rast.list.py

@@ -272,15 +272,6 @@ def main():
                         ).format(name=column, method=method),
                     )
                 )
-    if method == "gran" and where:
-        gs.fatal(
-            message_option_value_excludes_option(
-                option_name="method",
-                option_value=method,
-                excluded_option_name="where",
-                reason=_("All maps are always listed"),
-            )
-        )
 
     # Make sure the temporal database exists
     tgis.init()