# HG changeset patch
# User Sebastien Jodogne <s.jodogne@gmail.com>
# Date 1542278135 -3600
# Node ID 00504dcc996f09a843d9f5782e9e4250c3326b22
# Parent  89f2c302fc377044fd3963d73ccb31b31985bd40
"SynchronousCMove" is now "true" by default

diff -r 89f2c302fc37 -r 00504dcc996f NEWS
--- a/NEWS	Tue Nov 13 17:50:03 2018 +0100
+++ b/NEWS	Thu Nov 15 11:35:35 2018 +0100
@@ -26,6 +26,7 @@
 Maintenance
 -----------
 
+* "SynchronousCMove" is now "true" by default
 * New modality manufacturer: "GE" for GE Healthcare EA and AW
 * Executing a query/retrieve from the REST API now creates a job
 * Fix: Closing DICOM associations after running query/retrieve from REST API
diff -r 89f2c302fc37 -r 00504dcc996f OrthancServer/OrthancMoveRequestHandler.cpp
--- a/OrthancServer/OrthancMoveRequestHandler.cpp	Tue Nov 13 17:50:03 2018 +0100
+++ b/OrthancServer/OrthancMoveRequestHandler.cpp	Thu Nov 15 11:35:35 2018 +0100
@@ -238,7 +238,7 @@
                                               const std::string& originatorAet,
                                               uint16_t originatorId)
   {
-    bool synchronous = Configuration::GetGlobalBoolParameter("SynchronousCMove", false);
+    bool synchronous = Configuration::GetGlobalBoolParameter("SynchronousCMove", true);
 
     if (synchronous)
     {
diff -r 89f2c302fc37 -r 00504dcc996f Resources/Configuration.json
--- a/Resources/Configuration.json	Tue Nov 13 17:50:03 2018 +0100
+++ b/Resources/Configuration.json	Thu Nov 15 11:35:35 2018 +0100
@@ -414,8 +414,11 @@
   // Whether to run DICOM C-Move operations synchronously. If set to
   // "false" (the default), each incoming C-Move request results in
   // creating a new background job. Up to Orthanc 1.3.2, the implicit
-  // behavior was to use synchronous C-Move.
-  "SynchronousCMove" : false,
+  // behavior was to use synchronous C-Move. Between Orthanc 1.4.0 and
+  // 1.4.2, the default behavior was set to asynchronous C-Move. Since
+  // Orthanc 1.4.3, the default behavior is synchronous C-Move
+  // (backward compatibility with Orthanc <= 1.3.2).
+  "SynchronousCMove" : true,
 
   // Maximum number of completed jobs that are kept in memory. A
   // processing job is considered as complete once it is tagged as