+ fix crash on abort of adding primitives

src/Mod/PartDesign/Gui/TaskDatumParameters.cpp

@@ -29,6 +29,7 @@
 # include <QTextStream>
 # include <QMessageBox>
 # include <Precision.hxx>
+# include <boost/bind.hpp>
 #endif
 
 #include <Base/Console.h>
@@ -214,8 +215,8 @@ TaskDatumParameters::TaskDatumParameters(ViewProviderDatum *DatumView,QWidget *p
     updateListOfModes(eMapMode(pcDatum->MapMode.getValue()));
 
     //temporary show coordinate systems for selection
-    PartDesign::Body * body = PartDesign::Body::findBodyOf ( DatumView->getObject() );
+    PartDesign::Body * body = PartDesign::Body::findBodyOf(DatumView->getObject());
-    if(body) {
+    if (body) {
         try {
             App::Origin *origin = body->getOrigin();
             ViewProviderOrigin* vpOrigin;
@@ -225,14 +226,52 @@ TaskDatumParameters::TaskDatumParameters(ViewProviderDatum *DatumView,QWidget *p
             Base::Console().Error ("%s\n", ex.what () );
         }
     }
+
     if (pcDatum->Support.getSize() == 0)
         autoNext = true;
     else
         autoNext = false;
 
     DatumView->setPickable(false);
+
+    // connect object deletion with slot
+    auto bnd = boost::bind(&TaskDatumParameters::objectDeleted, this, _1);
+    Gui::Document* document = Gui::Application::Instance->getDocument(DatumView->getObject()->getDocument());
+    connectDelObject = document->signalDeletedObject.connect(bnd);
 }
 
+TaskDatumParameters::~TaskDatumParameters()
+{
+    connectDelObject.disconnect();
+    if (DatumView)
+        resetViewMode();
+    delete ui;
+}
+
+void TaskDatumParameters::resetViewMode()
+{
+    //end temporary view mode of coordinate system
+    PartDesign::Body * body = PartDesign::Body::findBodyOf(DatumView->getObject());
+    if (body) {
+        try {
+            App::Origin *origin = body->getOrigin();
+            ViewProviderOrigin* vpOrigin;
+            vpOrigin = static_cast<ViewProviderOrigin*>(Gui::Application::Instance->getViewProvider(origin));
+            vpOrigin->resetTemporaryVisibility();
+        }
+        catch (const Base::Exception &ex) {
+            Base::Console().Error("%s\n", ex.what());
+        }
+    }
+
+    DatumView->setPickable(true);
+}
+
+void TaskDatumParameters::objectDeleted(const Gui::ViewProviderDocumentObject& view)
+{
+    if (DatumView == &view)
+        DatumView = nullptr;
+}
 
 const QString makeHintText(std::set<eRefType> hint)
 {
@@ -737,26 +776,6 @@ bool   TaskDatumParameters::getFlip() const
     return ui->checkBoxFlip->isChecked();
 }
 
-TaskDatumParameters::~TaskDatumParameters()
-{
-    //end temporary view mode of coordinate system
-    PartDesign::Body * body = PartDesign::Body::findBodyOf ( DatumView->getObject() );
-    if(body) {
-        try {
-            App::Origin *origin = body->getOrigin();
-            ViewProviderOrigin* vpOrigin;
-            vpOrigin = static_cast<ViewProviderOrigin*>(Gui::Application::Instance->getViewProvider(origin));
-            vpOrigin->resetTemporaryVisibility();
-        } catch (const Base::Exception &ex) {
-            Base::Console().Error ("%s\n", ex.what () );
-        }
-    }
-
-    DatumView->setPickable(true);
-    
-    delete ui;
-}
-
 void TaskDatumParameters::changeEvent(QEvent *e)
 {
     TaskBox::changeEvent(e);

src/Mod/PartDesign/Gui/TaskDatumParameters.h

@@ -89,6 +89,8 @@ protected:
     void changeEvent(QEvent *e);
 
 private:
+    void resetViewMode();
+    void objectDeleted(const Gui::ViewProviderDocumentObject&);
     void onSelectionChanged(const Gui::SelectionChanges& msg);
     void updateUI(std::string message = std::string(), bool isWarning = false);
 
@@ -117,6 +119,8 @@ private:
     std::vector<Attacher::eMapMode> modesInList; //this list is synchronous to what is populated into listOfModes widget.
     bool completed;
 
+    typedef boost::BOOST_SIGNALS_NAMESPACE::connection Connection;
+    Connection connectDelObject;
 };
 
 /// simulation dialog for the TaskView

src/Mod/PartDesign/Gui/TaskPrimitiveParameters.cpp

@@ -29,7 +29,7 @@
 # include <QTextStream>
 # include <QMessageBox>
 # include <Precision.hxx>
-#include <boost/bind/bind.hpp>
+# include <boost/bind/bind.hpp>
 #endif
 
 #include "TaskPrimitiveParameters.h"
@@ -658,11 +658,12 @@ bool TaskPrimitiveParameters::accept()
     vp->setVisible(cs_visibility);
 
     connection.disconnect();
+
     return true;
 }
 
-bool TaskPrimitiveParameters::reject() {
+bool TaskPrimitiveParameters::reject()
-    
+{
     // roll back the done things
     Gui::Command::abortCommand();
     Gui::Command::doCommand(Gui::Command::Gui,"Gui.activeDocument().resetEdit()");
@@ -671,10 +672,11 @@ bool TaskPrimitiveParameters::reject() {
     ViewProviderDatumCoordinateSystem* vp = static_cast<ViewProviderDatumCoordinateSystem*>(
         Gui::Application::Instance->activeDocument()->getViewProvider(cs));
 
-    if(vp)
+    if (vp)
         vp->setVisible(cs_visibility);
 
     connection.disconnect();
+
     return true;
 }