From 52187f027c9a8fce4030491c19bd96890f61b7e0 2017-10-25 09:12:00
From: Thibaud Rabillard <trabillard@itlink.fr>
Date: 2017-10-25 09:12:00
Subject: [PATCH] Comments variable cancellation after a drop as it is not sure if it is really needed.

---

diff --git a/gui/src/Visualization/VisualizationTabWidget.cpp b/gui/src/Visualization/VisualizationTabWidget.cpp
index 2f4176f..148fbcb 100644
--- a/gui/src/Visualization/VisualizationTabWidget.cpp
+++ b/gui/src/Visualization/VisualizationTabWidget.cpp
@@ -171,6 +171,13 @@ void VisualizationTabWidget::dropMimeData(int index, const QMimeData *mimeData)
 
         if (!variables.isEmpty())
         {
+            //Abort the requests for the variables (if any)
+            //Commented, because it's not sure if it's needed or not
+            //for (const auto& var : variables)
+            //{
+            //    sqpApp->variableController().onAbortProgressRequested(var);
+            //}
+
             if (nbGraph == 1)
             {
                 //This is the only graph in the previous zone, close the zone
diff --git a/gui/src/Visualization/VisualizationZoneWidget.cpp b/gui/src/Visualization/VisualizationZoneWidget.cpp
index b1634a7..3cb3499 100644
--- a/gui/src/Visualization/VisualizationZoneWidget.cpp
+++ b/gui/src/Visualization/VisualizationZoneWidget.cpp
@@ -364,6 +364,13 @@ void VisualizationZoneWidget::dropMimeData(int index, const QMimeData *mimeData)
         {
             //The drop didn't occur in the same zone
 
+            //Abort the requests for the variables (if any)
+            //Commented, because it's not sure if it's needed or not
+            //for (const auto& var : variables)
+            //{
+            //    sqpApp->variableController().onAbortProgressRequested(var);
+            //}
+
             auto previousParentZoneWidget = graphWidget->parentZoneWidget();
             auto nbGraph = parentDragDropContainer->countDragWidget();
             if (nbGraph == 1)