is_directory is required

giancarloromeo · Dec 13, 2024 · 0c664fa · 0c664fa
1 parent a0d4011
commit 0c664fa
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/services/storage/src/simcore_service_storage/db_file_meta_data.py b/services/storage/src/simcore_service_storage/db_file_meta_data.py
@@ -72,7 +72,7 @@ def _list_filter_with_partial_file_id_stmt(
     file_id_prefix: str | None,
     partial_file_id: str | None,
     sha256_checksum: SHA256Str | None,
-    is_directory: bool | None = None,
+    is_directory: bool | None,
     limit: int | None = None,
     offset: int | None = None,
 ):
@@ -119,7 +119,7 @@ async def list_filter_with_partial_file_id(
     file_id_prefix: str | None,
     partial_file_id: str | None,
     sha256_checksum: SHA256Str | None,
-    is_directory: bool | None = None,
+    is_directory: bool | None,
     limit: int | None = None,
     offset: int | None = None,
 ) -> list[FileMetaDataAtDB]:

diff --git a/services/storage/src/simcore_service_storage/simcore_s3_dsm.py b/services/storage/src/simcore_service_storage/simcore_s3_dsm.py
@@ -185,6 +185,7 @@ async def list_files(  # noqa C901
                     user_id=uid, project_ids=accessible_projects_ids
                 ),
                 file_id_prefix=None,
+                is_directory=None,
                 partial_file_id=uuid_filter,
                 sha256_checksum=None,
             )

diff --git a/services/storage/src/simcore_service_storage/simcore_s3_dsm_utils.py b/services/storage/src/simcore_service_storage/simcore_s3_dsm_utils.py
@@ -123,5 +123,4 @@ async def _get_fmd(
 
 def compute_file_id_prefix(file_id: str, levels: int):
     components = file_id.strip("/").split("/")
-    subpath = "/".join(components[:levels])
-    return "/" + subpath if subpath else "/"
+    return "/".join(components[:levels])