Only rely on the file ID, not PID

fatiando · Feb 19, 2024 · 15f7536 · 15f7536
1 parent 93d9835
commit 15f7536
Showing 1 changed file with 5 additions and 13 deletions.
diff --git a/pooch/downloaders.py b/pooch/downloaders.py
@@ -1134,19 +1134,11 @@ def download_url(self, file_name):
                 f"File '{file_name}' not found in data archive "
                 f"{self.archive_url} (doi:{self.doi})."
             )
-        # Generate download_url using persistentId or file id
-        persistent_id = files[file_name].get("persistentId")
-        if persistent_id:
-            download_url = (
-                f"{parsed['protocol']}://{parsed['netloc']}/api/access/datafile/"
-                f":persistentId?persistentId={persistent_id}"
-            )
-        else:
-            file_id = files[file_name]["id"]
-            download_url = (
-                f"{parsed['protocol']}://{parsed['netloc']}/api/access/datafile/"
-                f"{file_id}"
-            )
+        # Generate download_url using the file id
+        download_url = (
+            f"{parsed['protocol']}://{parsed['netloc']}/api/access/datafile/"
+            f"{files[file_name]['id']}"
+        )
         return download_url
 
     def populate_registry(self, pooch):