refactor: Refactor NBIAClient's _downloadSingleSeries method

This commit refactors the _downloadSingleSeries method in the NBIAClient class. The changes include:
- Reformatting the method signature for better readability
- Adding exception handling for MD5 hash validation
- Commenting out a line of code that is not currently being used
- Adding error handling and logging for sorting DICOM files

src/nbiatoolkit/nbia.py

@@ -202,7 +202,8 @@ def downloadSeries(
     # downloads in the future
     def _downloadSingleSeries(
         self, SeriesInstanceUID: str, downloadDir: str,
-        filePattern: str, overwrite: bool) -> bool:
+        filePattern: str, overwrite: bool
+    ) -> bool:
 
         # create temporary directory
         from tempfile import TemporaryDirectory
@@ -223,8 +224,11 @@ def _downloadSingleSeries(
 
         with TemporaryDirectory() as tempDir:
             file.extractall(path=tempDir)
-            if not validateMD5(seriesDir=tempDir) and not overwrite:
-                self.log.error("MD5 validation failed. Exiting...")
+
+            try:
+                validateMD5(seriesDir=tempDir)
+            except Exception as e:
+                self.log.error("Error validating MD5 hash: %s", e)
                 return False
 
             # Create an instance of DICOMSorter with the desired target pattern
@@ -235,8 +239,12 @@ def _downloadSingleSeries(
                 truncateUID=True,
                 sanitizeFilename=True
                 )
-
-            sorter.sortDICOMFiles(option="move", overwrite=overwrite)
+            # sorter.sortDICOMFiles(option="move", overwrite=overwrite)
+            if not sorter.sortDICOMFiles(option="move", overwrite=overwrite):
+                self.log.error(
+                    "Error sorting DICOM files for series %s\n \
+                        failed files located at %s", SeriesInstanceUID, tempDir)
+                return False
 
         return True