# HG changeset patch
# User Sebastien Jodogne <s.jodogne@gmail.com>
# Date 1583767402 -3600
# Node ID 7324fee63e91fce1f4392c8eae80f879987d8fa8
# Parent  be4ffd525f1d9c826b65a7a17de07d73a9c41d1c
fix

diff -r be4ffd525f1d -r 7324fee63e91 Plugin/WadoRsRetrieveFrames.cpp
--- a/Plugin/WadoRsRetrieveFrames.cpp	Mon Mar 09 15:47:07 2020 +0100
+++ b/Plugin/WadoRsRetrieveFrames.cpp	Mon Mar 09 16:23:22 2020 +0100
@@ -478,7 +478,11 @@
     std::string photometric;
     if (samplesPerPixel == 3 &&
         dicom.GetStringTag(photometric, DICOM_TAG_PHOTOMETRIC_INTERPRETATION, true) &&
-        photometric == "YBR_FULL")
+        photometric == "YBR_FULL" &&
+        // Only applicable to uncompressed transfer syntaxes
+        (syntax == Orthanc::DicomTransferSyntax_LittleEndianImplicit ||
+         syntax == Orthanc::DicomTransferSyntax_LittleEndianExplicit ||
+         syntax == Orthanc::DicomTransferSyntax_BigEndianExplicit))
     {
       convertYbr = true;
     }
@@ -658,13 +662,10 @@
     }
     else
     {
-      // Need to convert the transfer syntax
-
-      {
-        OrthancPlugins::LogInfo("DICOMweb RetrieveFrames: Transcoding instance " + orthancId + 
-                                " from transfer syntax " + Orthanc::GetTransferSyntaxUid(sourceSyntax) +
-                                " to " + Orthanc::GetTransferSyntaxUid(targetSyntax));
-      }
+      // Need to convert the transfer syntax (transcoding)
+      OrthancPlugins::LogInfo("DICOMweb RetrieveFrames: Transcoding instance " + orthancId + 
+                              " from transfer syntax " + Orthanc::GetTransferSyntaxUid(sourceSyntax) +
+                              " to " + Orthanc::GetTransferSyntaxUid(targetSyntax));
 
       gdcm::ImageChangeTransferSyntax change;
       change.SetTransferSyntax(OrthancPlugins::GdcmParsedDicomFile::GetGdcmTransferSyntax(targetSyntax));