Merge pull request #338 from ocefpaf/get_info_url

Allow for empty dataset_id info URL
ioos · Mar 8, 2024 · f2947fc · f2947fc
2 parents 5e45a0e + 3776868
commit f2947fc
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 5 deletions.
diff --git a/erddapy/core/url.py b/erddapy/core/url.py
@@ -343,17 +343,16 @@ def get_info_url(
     ----
         server: data server endpoint.
         dataset_id: a dataset unique id.
+        If empty the full dataset listing will be returned.
         response: default is HTML.
 
     Returns:
     -------
         url: the info URL for the `response` chosen.
 
     """
-    if not dataset_id:
-        msg = f"You must specify a valid dataset_id, got {dataset_id}"
-        raise ValueError(msg)
-
+    if dataset_id is None:
+        return f"{server}/info/index.{response}?itemsPerPage=1000000"
     return f"{server}/info/{dataset_id}/index.{response}"
 
 

diff --git a/erddapy/erddapy.py b/erddapy/erddapy.py
@@ -263,6 +263,7 @@ def get_info_url(
         Args:
         ----
             dataset_id: a dataset unique id.
+            If empty the full dataset listing will be returned.
             response: default is HTML.
 
         Returns:
@@ -273,7 +274,11 @@ def get_info_url(
         dataset_id = dataset_id if dataset_id else self.dataset_id
         response = response if response else self.response
 
-        return get_info_url(self.server, dataset_id, response)
+        return get_info_url(
+            self.server,
+            dataset_id=dataset_id,
+            response=response,
+        )
 
     def get_categorize_url(
         self: ERDDAP,