Skip to content

Commit

Permalink
increse time limit for validate
Browse files Browse the repository at this point in the history
  • Loading branch information
salimkanoun committed Jun 6, 2023
1 parent 2acb97d commit 2e12cf5
Showing 1 changed file with 9 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,10 @@ public function execute(ValidateDicomUploadRequest $validateDicomUploadRequest,
{

try {

//Set Time Limit at 30min as operation could be really long
set_time_limit(1800);

//Retrieve Visit Context
$this->visitService->setVisitId($validateDicomUploadRequest->visitId);
$visitContext = $this->visitService->getVisitContext();
Expand All @@ -69,6 +73,9 @@ public function execute(ValidateDicomUploadRequest $validateDicomUploadRequest,
$currentUserId = $validateDicomUploadRequest->currentUserId;
$visitId = $validateDicomUploadRequest->visitId;

$expectedNumberOfInstances = $validateDicomUploadRequest->numberOfInstances;
$originalOrthancId = $validateDicomUploadRequest->originalOrthancId;

$this->checkAuthorization($currentUserId, $visitId, $studyName, $visitContext);

//Make Visit as being upload processing
Expand Down Expand Up @@ -98,8 +105,6 @@ public function execute(ValidateDicomUploadRequest $validateDicomUploadRequest,
}
$this->orthancService->setOrthancServer(false);

$expectedNumberOfInstances = $validateDicomUploadRequest->numberOfInstances;

$orthancStudyImport = $this->orthancService->importDicomFolder($unzipedPath);
if ($expectedNumberOfInstances !== $orthancStudyImport->getNumberOfInstances()) {
throw new GaelOValidateDicomException("Imported DICOM not matching announced number of Instances");
Expand Down Expand Up @@ -130,7 +135,7 @@ public function execute(ValidateDicomUploadRequest $validateDicomUploadRequest,
$this->orthancService->setOrthancServer(true);

$statistics = $this->orthancService->getOrthancRessourcesStatistics('studies', $anonymizedOrthancStudyID);
if ($statistics['CountInstances'] !== $validateDicomUploadRequest->numberOfInstances) {
if ($statistics['CountInstances'] !== $expectedNumberOfInstances) {
throw new GaelOValidateDicomException("Error during Peer transfers");
}

Expand All @@ -140,7 +145,7 @@ public function execute(ValidateDicomUploadRequest $validateDicomUploadRequest,
$studyName,
$currentUserId,
$anonymizedOrthancStudyID,
$validateDicomUploadRequest->originalOrthancId
$originalOrthancId
);

$studyInstanceUID = $this->registerDicomStudyService->execute();
Expand Down

0 comments on commit 2e12cf5

Please sign in to comment.