Fix url_builder

galaxyproject · Sep 26, 2024 · f3e3f6d · f3e3f6d
1 parent 73c6ddc
commit f3e3f6d
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/lib/galaxy/webapps/galaxy/api/library_contents.py b/lib/galaxy/webapps/galaxy/api/library_contents.py
@@ -65,6 +65,7 @@ def index(
 
     @router.get(
         "/api/libraries/{library_id}/contents/{id}",
+        name="library_content",
         summary="Return a library file or folder.",
         deprecated=True,
     )

diff --git a/lib/galaxy/webapps/galaxy/services/library_contents.py b/lib/galaxy/webapps/galaxy/services/library_contents.py
@@ -257,11 +257,16 @@ def _decode_library_content_id(
                 f"Malformed library content id ( {str(content_id)} ) specified, unable to decode."
             )
 
-    def _url_for(self, trans: ProvidesUserContext, library_id: DecodedDatabaseIdField, id, type):
+    def _url_for(self, trans: ProvidesUserContext, library_id, id, type):
+        encoded_library_id = trans.security.encode_id(library_id)
         encoded_id = trans.security.encode_id(id)
         if type == "folder":
             encoded_id = f"F{encoded_id}"
-        return trans.url_builder("library_content", library_id=library_id, id=encoded_id) if trans.url_builder else None
+        return (
+            trans.url_builder("library_content", library_id=encoded_library_id, id=encoded_id)
+            if trans.url_builder
+            else None
+        )
 
     def _traverse(self, trans: ProvidesUserContext, folder, current_user_roles):
         admin = trans.user_is_admin