changeset 1856:36ab170733d6

Made eFilm version explicit
author Sebastien Jodogne <s.jodogne@gmail.com>
date Tue, 01 Dec 2015 16:57:03 +0100
parents aa95aea0a352
children 4f2386d0f326 4e7c318a3f69
files NEWS OrthancServer/OrthancFindRequestHandler.cpp OrthancServer/ServerEnumerations.cpp OrthancServer/ServerEnumerations.h Resources/Configuration.json UnitTestsSources/UnitTestsMain.cpp
diffstat 6 files changed, 10 insertions(+), 10 deletions(-) [+]
line wrap: on
line diff
--- a/NEWS	Tue Dec 01 16:34:24 2015 +0100
+++ b/NEWS	Tue Dec 01 16:57:03 2015 +0100
@@ -72,7 +72,7 @@
 * "--logdir" flag creates a single log file instead of 3 separate files for errors/warnings/infos
 * "--errors" flag lists the error codes that could be returned by Orthanc
 * Under Windows, the exit status of Orthanc corresponds to the encountered error code
-* New "AgfaImpax", "Efilm" and "Vitrea" modality manufacturers
+* New "AgfaImpax", "EFilm2" and "Vitrea" modality manufacturers
 * Upgrade to Boost 1.59.0 for static builds
 
 
--- a/OrthancServer/OrthancFindRequestHandler.cpp	Tue Dec 01 16:34:24 2015 +0100
+++ b/OrthancServer/OrthancFindRequestHandler.cpp	Tue Dec 01 16:57:03 2015 +0100
@@ -96,7 +96,7 @@
   {
     switch (manufacturer)
     {
-      case ModalityManufacturer_Efilm:
+      case ModalityManufacturer_EFilm2:
         // Following Denis Nesterov's mail on 2015-11-30
         if (tag == DicomTag(0x0008, 0x0000) ||  // "GenericGroupLength"
             tag == DicomTag(0x0010, 0x0000) ||  // "GenericGroupLength"
--- a/OrthancServer/ServerEnumerations.cpp	Tue Dec 01 16:34:24 2015 +0100
+++ b/OrthancServer/ServerEnumerations.cpp	Tue Dec 01 16:57:03 2015 +0100
@@ -340,8 +340,8 @@
       case ModalityManufacturer_AgfaImpax:
         return "AgfaImpax";
       
-      case ModalityManufacturer_Efilm:
-        return "Efilm";
+      case ModalityManufacturer_EFilm2:
+        return "EFilm2";
       
       case ModalityManufacturer_Vitrea:
         return "Vitrea";
@@ -417,9 +417,9 @@
     {
       return ModalityManufacturer_Vitrea;
     }
-    else if (manufacturer == "Efilm")
+    else if (manufacturer == "EFilm2")
     {
-      return ModalityManufacturer_Efilm;
+      return ModalityManufacturer_EFilm2;
     }
     else
     {
--- a/OrthancServer/ServerEnumerations.h	Tue Dec 01 16:34:24 2015 +0100
+++ b/OrthancServer/ServerEnumerations.h	Tue Dec 01 16:57:03 2015 +0100
@@ -64,7 +64,7 @@
     ModalityManufacturer_Dcm4Chee,
     ModalityManufacturer_SyngoVia,
     ModalityManufacturer_AgfaImpax,
-    ModalityManufacturer_Efilm,
+    ModalityManufacturer_EFilm2,
     ModalityManufacturer_Vitrea
   };
 
--- a/Resources/Configuration.json	Tue Dec 01 16:34:24 2015 +0100
+++ b/Resources/Configuration.json	Tue Dec 01 16:57:03 2015 +0100
@@ -151,8 +151,8 @@
      * specific PACS manufacturer. The allowed values are currently
      * "Generic" (default value), "StoreScp" (storescp tool from
      * DCMTK), "ClearCanvas", "MedInria", "Dcm4Chee", "SyngoVia",
-     * "AgfaImpax", "Efilm" and "Vitrea". This parameter is
-     * case-sensitive.
+     * "AgfaImpax" (Agfa IMPAX), "EFilm2" (eFilm version 2), and
+     * "Vitrea". This parameter is case-sensitive.
      **/
     // "clearcanvas" : [ "CLEARCANVAS", "192.168.1.1", 104, "ClearCanvas" ]
   },
--- a/UnitTestsSources/UnitTestsMain.cpp	Tue Dec 01 16:34:24 2015 +0100
+++ b/UnitTestsSources/UnitTestsMain.cpp	Tue Dec 01 16:57:03 2015 +0100
@@ -530,7 +530,7 @@
   ASSERT_STREQ("Dcm4Chee", EnumerationToString(StringToModalityManufacturer("Dcm4Chee")));
   ASSERT_STREQ("SyngoVia", EnumerationToString(StringToModalityManufacturer("SyngoVia")));
   ASSERT_STREQ("AgfaImpax", EnumerationToString(StringToModalityManufacturer("AgfaImpax")));
-  ASSERT_STREQ("Efilm", EnumerationToString(StringToModalityManufacturer("Efilm")));
+  ASSERT_STREQ("EFilm2", EnumerationToString(StringToModalityManufacturer("EFilm2")));
   ASSERT_STREQ("Vitrea", EnumerationToString(StringToModalityManufacturer("Vitrea")));
 }