Document::breakDependency should handle exceptions to avoid to leave …

…document in an inconsistent state
FreeCAD · Mar 15, 2019 · ca7770b · ca7770b
1 parent 45a0427
commit ca7770b
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/src/App/Document.cpp b/src/App/Document.cpp
@@ -3031,7 +3031,13 @@ void Document::breakDependency(DocumentObject* pcObject, bool clear)
             std::vector<App::ObjectIdentifier> paths;
             pcObject->ExpressionEngine.getPathsToDocumentObject(it->second, paths);
             for (std::vector<App::ObjectIdentifier>::iterator jt = paths.begin(); jt != paths.end(); ++jt) {
-                pcObject->ExpressionEngine.setValue(*jt, nullptr);
+                // When nullifying the expression handle case where an identifier lacks of the property
+                try {
+                    pcObject->ExpressionEngine.setValue(*jt, nullptr);
+                }
+                catch (const Base::Exception& e) {
+                    e.ReportException();
+                }
             }
         }
     }