[CORE] Refactoring add functions from BaseLink and Link

Sofa/framework/Core/simutest/objectmodel/BaseLink_simutest.cpp

@@ -114,6 +114,30 @@ TEST_F(BaseLink_test, remove)
     ASSERT_FALSE(owner.l_target.remove(nullptr));
 }
 
+TEST_F(BaseLink_test, add)
+{
+    FakeComponent Component1;
+    Component1.setName("Component1");
+    FakeComponent Component2;
+    Component2.setName("Component2");
+    FakeComponent Component3;
+    Component3.setName("Component3");
+
+    FakeComponent* ptr;
+    ptr = &Component2;
+
+    EXPECT_EQ(Component1.l_target.getValueString(), "");
+
+    Component1.l_target.add(ptr);
+    EXPECT_EQ(Component1.l_target.getValueString(), "@Component2");
+
+    ptr = &Component3;
+
+    Component1.l_target.add(ptr);
+    EXPECT_EQ(Component1.l_target.getValueString(), "@Component2 @Component3");
+}
+
+
 //////////////////////// Testing valid path //////////////////////////////////////
 class MultiLink_simutest : public BaseLink_test {};
 

Sofa/framework/Core/src/sofa/core/objectmodel/BaseLink.h

@@ -186,6 +186,7 @@ class SOFA_CORE_API BaseLink
     virtual void _doClear_() = 0;
     virtual std::string _doGetLinkedPath_(const size_t=0) const = 0;
     virtual bool _doRemove_(Base* target) = 0;
+    virtual bool addRawPtr(Base* baseptr) = 0;
 
     unsigned int m_flags;
     std::string m_name;

Sofa/framework/Core/src/sofa/core/objectmodel/Link.h

@@ -506,6 +506,24 @@ class TLink : public BaseLink
         return TLink::add(destptr, path);
     }
 
+    bool addRawPtr(Base* baseptr) override
+    {
+        /// If the pointer is null and the path empty we do nothing
+        if(!baseptr)
+            return false;
+
+        /// Downcast the pointer to a compatible type and
+        /// If the types are not compatible with the Link we returns false
+        auto destptr = castTo<DestType*>(baseptr);
+        if(!destptr)
+        {
+            return false;
+        }
+
+        /// TLink:adding accepts nullptr (for a not yet resolved link).
+        return TLink::add(destptr);
+    }
+
     /// Returns false on type mismatch
     bool _doSet_(Base* baseptr, const size_t index) override
     {