Issue #24206: No plane for sketch

Invalidate values of the sketch normal if the plane was not selected.

diff --git a/src/GeomData/GeomData_Dir.cpp b/src/GeomData/GeomData_Dir.cpp
index 5d738511aa5d8889398780e505bcbff908176cb9..9d87844a3b4d154d2a088bfadcee88ed89dda2b7 100644 (file)
--- a/src/GeomData/GeomData_Dir.cpp
+++ b/src/GeomData/GeomData_Dir.cpp
@@ -83,12 +83,28 @@ void GeomData_Dir::reinit()
   }
 }
 
+void GeomData_Dir::reset()
+{
+  if (myLab.FindAttribute(TDataStd_RealArray::GetID(), myCoords)) {
+    // invalidate values of the direction
+    myCoords->SetValue(0, 0.0);
+    myCoords->SetValue(1, 0.0);
+    myCoords->SetValue(2, 0.0);
+  }
+  GeomDataAPI_Dir::reset();
+}
+
 bool GeomData_Dir::isInitialized()
 {
   // Check once again the direction is initialized.
   // Use case (bos #18905): draw a sketch, click "Change sketch plane", then abort it.
   //                        myIsInitialized value is dropped to false, thus recheck.
-  if (!myIsInitialized)
-    myIsInitialized = myLab.FindAttribute(TDataStd_RealArray::GetID(), myCoords) == Standard_True;
+  // Additionally verify the values are valid.
+  // Use case (bos #24206): start a sketch, highlight a coordinate plane, then click anywhere
+  //                        in empty space. Sketch features in menu appears and can be selected.
+  if (!myIsInitialized) {
+    myIsInitialized = myLab.FindAttribute(TDataStd_RealArray::GetID(), myCoords) == Standard_True
+        && myCoords->Value(0) != 0.0 && myCoords->Value(1) != 0.0 && myCoords->Value(2) != 0.0;
+  }
   return ModelAPI_Attribute::isInitialized();
 }

diff --git a/src/GeomData/GeomData_Dir.h b/src/GeomData/GeomData_Dir.h
index 91b6cd086b7b19f1f5590359f8008528e77f5bbd..1c53bb8c108739db2e9f8ece7a1adfb22eb931f6 100644 (file)
--- a/src/GeomData/GeomData_Dir.h
+++ b/src/GeomData/GeomData_Dir.h
@@ -57,6 +57,9 @@ class GeomData_Dir : public GeomDataAPI_Dir
   /// Returns \c ture if the direction is initialized
   GEOMDATA_EXPORT virtual bool isInitialized();
 
+  /// Resets attribute to default state.
+  GEOMDATA_EXPORT virtual void reset();
+
  protected:
   /// Initializes attributes
   GEOMDATA_EXPORT GeomData_Dir(TDF_Label& theLabel);