diff --git a/IRaCIS.Core.API/Controllers/UploadDownLoadController.cs b/IRaCIS.Core.API/Controllers/UploadDownLoadController.cs
index cc755a617..77987a8ca 100644
--- a/IRaCIS.Core.API/Controllers/UploadDownLoadController.cs
+++ b/IRaCIS.Core.API/Controllers/UploadDownLoadController.cs
@@ -894,11 +894,12 @@ namespace IRaCIS.Core.API.Controllers
         #endregion
 
 
-        [HttpPost("download/PatientStudyBatchDownload")]
+        [HttpGet("download/PatientStudyBatchDownload")]
         public async Task<IActionResult> DownloadPatientStudyBatch([FromServices] IPatientService _patientService, [FromServices] IOSSService _oSSService,
              [FromServices] IHubContext<DownloadHub, IDownloadClient> _downLoadHub,
-            PatientImageDownloadCommand inCommand)
+             [FromQuery] PatientImageDownloadCommand inCommand)
         {
+            inCommand.SCPStudyIdList = inCommand.SCPStudyIdList.Where(t => t != Guid.Empty).ToList();
             var rusult = await _patientService.GetDownloadPatientStudyInfo(inCommand);
 
             var patientList = rusult.Data;
diff --git a/IRaCIS.Core.Application/Service/Visit/DTO/PatientViewModel.cs b/IRaCIS.Core.Application/Service/Visit/DTO/PatientViewModel.cs
index ed0661f3f..e9c0eb951 100644
--- a/IRaCIS.Core.Application/Service/Visit/DTO/PatientViewModel.cs
+++ b/IRaCIS.Core.Application/Service/Visit/DTO/PatientViewModel.cs
@@ -984,9 +984,9 @@ namespace IRaCIS.Application.Contracts
 
     public class PatientImageDownloadCommand
     {
-        public List<Guid> PatientIdList { get; set; }
+        public List<Guid> PatientIdList { get; set; } = new List<Guid>();
 
-        public List<Guid> SCPStudyIdList { get; set; }
+        public List<Guid> SCPStudyIdList { get; set; } = new List<Guid>();
 
         [NotDefault]
         public string CurrentNoticeId { get; set; }