fix: get_study_metadata_path method update

EBI-Metabolights · Dec 1, 2024 · 75d221c · 75d221c
1 parent 98b4303
commit 75d221c
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/metabolights_utils/provider/async_provider/study_provider.py b/metabolights_utils/provider/async_provider/study_provider.py
@@ -234,14 +234,14 @@ async def get_file_path(
 
     async def get_study_metadata_path(
         self,
-        folder: Union[None, str],
         study_id: Union[None, str],
+        folder: Union[None, str],
     ) -> Tuple[str, bool]:
         if not folder:
             study_path = await self.metadata_file_provider.get_study_metadata_path(
                 study_id
             )
-            exist = await self.metadata_file_provider.exists(study_path)
+            exist = await self.metadata_file_provider.exists(study_id)
         else:
             study_path = folder
             real_path = os.path.realpath(folder)
@@ -831,7 +831,7 @@ async def load_study(
         if not study_id:
             raise ValueError("invalid study_id")
         exist = False
-        study_path, exist = await self.get_study_metadata_path(study_path, study_id)
+        study_path, exist = await self.get_study_metadata_path(study_id, study_path)
 
         if not study_path:
             raise ValueError("invalid study_path")

diff --git a/metabolights_utils/provider/study_provider.py b/metabolights_utils/provider/study_provider.py
@@ -232,12 +232,12 @@ def get_file_path(
 
     def get_study_metadata_path(
         self,
-        folder: Union[None, str],
         study_id: Union[None, str],
+        folder: Union[None, str],
     ) -> Tuple[str, bool]:
         if not folder:
             study_path = self.metadata_file_provider.get_study_metadata_path(study_id)
-            exist = self.metadata_file_provider.exists(study_path)
+            exist = self.metadata_file_provider.exists(study_id)
         else:
             study_path = folder
             real_path = os.path.realpath(folder)
@@ -821,7 +821,7 @@ def load_study(
         if not study_id:
             raise ValueError("invalid study_id")
         exist = False
-        study_path, exist = self.get_study_metadata_path(study_path, study_id)
+        study_path, exist = self.get_study_metadata_path(study_id, study_path)
 
         if not study_path:
             raise ValueError("invalid study_path")