Changeset 747 for XIOS/trunk/src/node/axis.cpp

Timestamp:

10/21/15 11:50:11 (9 years ago)

Author:

rlacroix

Message:

Fix the inherited name for axis/domains with reference in some cases.

Correctly account for transformations and non default ids.

File:

• XIOS/trunk/src/node/axis.cpp (modified) (1 diff)

XIOS/trunk/src/node/axis.cpp

@@ -872 +872 @@
   }
 
+  /*!
+   * Go through the hierarchy to find the axis from which the transformations must be inherited
+   */
   void CAxis::solveInheritanceTransformation()
   {
-    if (this->hasTransformation()) return;
-
+    if (hasTransformation() || !hasDirectAxisReference())
+      return;
+
+    CAxis* axis = this;
     std::vector<CAxis*> refAxis;
-    CAxis* refer_sptr;
-    CAxis* refer_ptr = this;
-    while (refer_ptr->hasDirectAxisReference())
-    {
-      refAxis.push_back(refer_ptr);
-      refer_sptr = refer_ptr->getDirectAxisReference();
-      refer_ptr  = refer_sptr;
-      if (refer_ptr->hasTransformation()) break;
-    }
-
-    if (refer_ptr->hasTransformation())
-      for (int idx = 0; idx < refAxis.size(); ++idx)
-        refAxis[idx]->setTransformations(refer_ptr->getAllTransformations());
+    while (!axis->hasTransformation() && axis->hasDirectAxisReference())
+    {
+      refAxis.push_back(axis);
+      axis = axis->getDirectAxisReference();
+    }
+
+    if (axis->hasTransformation())
+      for (size_t i = 0; i < refAxis.size(); ++i)
+        refAxis[i]->setTransformations(axis->getAllTransformations());
+
+    // Try to inherit the id of the referenced object as the axis name
+    // when no name was been defined and a defaut id is used.
+    if (name.isEmpty())
+    {
+      static const std::string defId("__axis_undef_id_");
+      const std::string& id = getId();
+      if (id.size() > defId.size() && id.compare(0, defId.size(), defId) == 0)
+        name = axis->getId();
+    }
   }