Salome HOME
Fix shape dependencies leading to regressions in test cases.
authorazv <azv@opencascade.com>
Fri, 26 Apr 2019 22:02:28 +0000 (01:02 +0300)
committerazv <azv@opencascade.com>
Fri, 26 Apr 2019 22:02:28 +0000 (01:02 +0300)
src/FeaturesPlugin/FeaturesPlugin_Boolean.cpp

index d466d1a1348e046c44d7350dc6027645309ebfd9..6602ebccb7f47d2afb87d403a9d6c80b96d09703 100644 (file)
@@ -211,6 +211,9 @@ bool FeaturesPlugin_Boolean::processObject(
     std::shared_ptr<ModelAPI_ResultBody> aResultBody =
         document()->createBody(data(), theResultIndex);
 
+    // tools should be added to the list to fulfill the correct history of modification
+    aListWithObject.insert(aListWithObject.end(), theTools.begin(), theTools.end());
+
     ListOfShape aUsedTools = theTools;
     aUsedTools.insert(aUsedTools.end(), thePlanes.begin(), thePlanes.end());
 
@@ -307,6 +310,8 @@ bool FeaturesPlugin_Boolean::processCompsolid(
 
     ListOfShape aCompSolidList;
     aCompSolidList.push_back(theCompsolid);
+    // tools should be added to the list to fulfill the correct history of modification
+    aCompSolidList.insert(aCompSolidList.end(), theTools.begin(), theTools.end());
 
     ListOfShape aUsedTools = theTools;
     aUsedTools.insert(aUsedTools.end(), thePlanes.begin(), thePlanes.end());