changeset 677:a5241defb36f

cleanup + fix
author Alain Mazy <am@orthanc.team>
date Wed, 09 Oct 2024 16:04:41 +0200 (7 months ago)
parents 6ae0e07512b8
children 9e43494cb054 280980f6dda3
files NEWS Plugin/WadoRsRetrieveFrames.cpp
diffstat 2 files changed, 2 insertions(+), 4 deletions(-) [+]
line wrap: on
line diff
--- a/NEWS	Wed Oct 09 15:52:24 2024 +0200
+++ b/NEWS	Wed Oct 09 16:04:41 2024 +0200
@@ -6,7 +6,7 @@
   the retrieval of NumberOfStudyRelatedInstances, NumberOfStudyRelatedSeries, ...
 * Fixed non latin PatientName values that were empty in some QIDO-RS responses.
 * Optimized the retrieval of single frame in WADO-RS when no transcoding is required.
-  This greatly improve download time of multi-frame images in OHIF.
+  This greatly improves the download time of multi-frame images in OHIF.
 
 
 Version 1.17 (2024-06-05)
--- a/Plugin/WadoRsRetrieveFrames.cpp	Wed Oct 09 15:52:24 2024 +0200
+++ b/Plugin/WadoRsRetrieveFrames.cpp	Wed Oct 09 16:04:41 2024 +0200
@@ -456,7 +456,7 @@
   const std::string base = OrthancPlugins::Configuration::GetBasePublicUrl(request);
   std::string location = (
     OrthancPlugins::Configuration::GetWadoUrl(base, studyInstanceUid, seriesInstanceUid, sopInstanceUid) +
-    "frames/" + boost::lexical_cast<std::string>(frame));
+    "frames/" + boost::lexical_cast<std::string>(frame + 1));
   const char *keys[] = { "Content-Location" };
   const char *values[] = { location.c_str() };
   error = OrthancPluginSendMultipartItem2(OrthancPlugins::GetGlobalContext(), output, instanceContent.GetData(), instanceContent.GetSize(), 1, keys, values);
@@ -548,10 +548,8 @@
         throw Orthanc::OrthancException(Orthanc::ErrorCode_InternalError, "DICOMWeb: Unable to get file for instance " + orthancId);
       }
 
-      LOG(INFO) << "DICOMweb RetrieveFrames, before AnswerFrame";
       AnswerFrame(output, request, content, studyInstanceUid, seriesInstanceUid,
                   sopInstanceUid, frames.front(), targetSyntax);
-      LOG(INFO) << "DICOMweb RetrieveFrame, leaving";
       return;
     }
     else