From: jfa <jfa@opencascade.com>
Date: Mon, 6 Nov 2023 13:54:07 +0000 (+0000)
Subject: [bos #37570] [CEA] SHAPER hdf tests are failing. Fix Sphere feature attributes order.
X-Git-Tag: V9_12_0b1~5^2
X-Git-Url: http://git.salome-platform.org/gitweb/?a=commitdiff_plain;h=4a800cf754a8ba8485f5320ca1b4b6f308b622a7;p=modules%2Fshaper.git

[bos #37570] [CEA] SHAPER hdf tests are failing. Fix Sphere feature attributes order.
---

diff --git a/src/PrimitivesPlugin/PrimitivesPlugin_Sphere.cpp b/src/PrimitivesPlugin/PrimitivesPlugin_Sphere.cpp
index 0c4b38f40..cf88ca191 100644
--- a/src/PrimitivesPlugin/PrimitivesPlugin_Sphere.cpp
+++ b/src/PrimitivesPlugin/PrimitivesPlugin_Sphere.cpp
@@ -44,26 +44,56 @@ PrimitivesPlugin_Sphere::PrimitivesPlugin_Sphere()
 //=================================================================================================
 void PrimitivesPlugin_Sphere::initAttributes()
 {
-  data()->addAttribute(PrimitivesPlugin_Sphere::CREATION_METHOD(),
-                       ModelAPI_AttributeString::typeId());
+  // Attention! A fix for 37570 Tuleap issue.
+  // We could have studies with aCenterPoint at the first position (old studies)
+  // and studies with CREATION_METHOD() at the first position (new studies)
 
   // data for the first mode : by a point and a radius
-  data()->addAttribute(PrimitivesPlugin_Sphere::CENTER_POINT_ID(),
-                       ModelAPI_AttributeSelection::typeId());
-
-  data()->addAttribute(PrimitivesPlugin_Sphere::RADIUS_ID(),
-                       ModelAPI_AttributeDouble::typeId());
-
-  // Initialize the center point of the sphere at the origin if the center point is not filled.
-  AttributeSelectionPtr aCenterPoint =
-    data()->selection(PrimitivesPlugin_Sphere::CENTER_POINT_ID());
-  if (!aCenterPoint->isInitialized()) {
-    ObjectPtr aPointObj = ModelAPI_Session::get()->moduleDocument()
-      ->objectByName(ModelAPI_ResultConstruction::group(), L"Origin");
-    if (aPointObj.get()) {
-      ResultPtr aPointRes = std::dynamic_pointer_cast<ModelAPI_Result>(aPointObj);
-      aCenterPoint->setValue(aPointRes, std::shared_ptr<GeomAPI_Shape>());
+  AttributeSelectionPtr aCenterPoint = std::dynamic_pointer_cast<ModelAPI_AttributeSelection>
+    (data()->addAttribute(PrimitivesPlugin_Sphere::CENTER_POINT_ID(),
+                          ModelAPI_AttributeSelection::typeId())); // #1 (old studies)
+  if (aCenterPoint->isInitialized()) {
+    // we are opening an old study:
+    // #1 Center Point
+    // #2 Radius
+    // #3 Creation method (initialize now)
+
+    data()->addAttribute(PrimitivesPlugin_Sphere::RADIUS_ID(),
+                         ModelAPI_AttributeDouble::typeId()); // #2 (old studies)
+
+    AttributeStringPtr aMethod = std::dynamic_pointer_cast<ModelAPI_AttributeString>
+      (data()->addAttribute(PrimitivesPlugin_Sphere::CREATION_METHOD(),
+                            ModelAPI_AttributeString::typeId())); // #3
+    aMethod->setValue(CREATION_METHOD_BY_PT_RADIUS());
+  }
+  else {
+    // we are opening a new study or creating Sphere from scratch
+    // #1 Creation method
+    // #2 Center Point
+    // #3 Radius
+
+    data()->addAttribute(PrimitivesPlugin_Sphere::CREATION_METHOD(),
+                         ModelAPI_AttributeString::typeId(),
+                         1); // #1 new studies or from scratch
+
+    aCenterPoint = std::dynamic_pointer_cast<ModelAPI_AttributeSelection>
+      (data()->addAttribute(PrimitivesPlugin_Sphere::CENTER_POINT_ID(),
+                            ModelAPI_AttributeSelection::typeId(),
+                            2)); // #2
+
+    // Initialize the center point of the sphere at the origin if the center point is not filled.
+    if (!aCenterPoint->isInitialized()) {
+      ObjectPtr aPointObj = ModelAPI_Session::get()->moduleDocument()
+        ->objectByName(ModelAPI_ResultConstruction::group(), L"Origin");
+      if (aPointObj.get()) {
+        ResultPtr aPointRes = std::dynamic_pointer_cast<ModelAPI_Result>(aPointObj);
+        aCenterPoint->setValue(aPointRes, std::shared_ptr<GeomAPI_Shape>());
+      }
     }
+
+    data()->addAttribute(PrimitivesPlugin_Sphere::RADIUS_ID(),
+                         ModelAPI_AttributeDouble::typeId(),
+                         3); // #3
   }
 
   // data for the second mode : by dimensions