Add explicit getter and setter for datas (#509)

* Add explicit getter and setter

* Remove mistake

Author: bakpaul

bindings/Sofa/src/SofaPython3/Sofa/Core/Binding_BaseData.cpp

@@ -112,9 +112,13 @@ std::unique_ptr<DataContainerContext> writeableArray(BaseData* self)
     return nullptr;
 }
 
-void __setattr__(py::object self, const std::string& s, py::object value)
+py::object getValue(py::object self)
+{
+    return PythonFactory::valueToPython_ro(py::cast<BaseData*>(self));
+}
+
+void setValue(py::object self, py::object value)
 {
-    SOFA_UNUSED(s);
     BaseData* selfdata = py::cast<BaseData*>(self);
 
     if(py::isinstance<DataContainer>(value))
@@ -132,14 +136,20 @@ void __setattr__(py::object self, const std::string& s, py::object value)
     BindingBase::SetAttr(py::cast(selfdata->getOwner()),selfdata->getName(),value);
 }
 
+void __setattr__(py::object self, const std::string& s, py::object value)
+{
+    SOFA_UNUSED(s);
+    setValue(self, value);
+}
+
 py::object __getattr__(py::object self, const std::string& s)
 {
     /// If this is data.value we returns the content value of the data field converted into
     /// a python object that is easy to manipulate. The conversion is done with the toPython
     /// function.
     if(s == "value")
     {
-        return PythonFactory::valueToPython_ro(py::cast<BaseData*>(self));
+        return getValue(self);
     }
 
     if(s == "linkpath")
@@ -150,6 +160,8 @@ py::object __getattr__(py::object self, const std::string& s)
     throw py::attribute_error("There is no attribute '"+s+"'");
 }
 
+
+
 void setParent(BaseData* self, BaseData* parent)
 {
     self->setParent(parent);
@@ -211,6 +223,8 @@ void moduleAddBaseData(py::module& m)
     data.def("getName", [](BaseData& b){ return b.getName(); }, sofapython3::doc::baseData::getName);
     data.def("setName", [](BaseData& b, const std::string& s){ b.setName(s); }, sofapython3::doc::baseData::setName);
     data.def("getCounter", [](BaseData& self) { return self.getCounter(); }, sofapython3::doc::baseData::getCounter);
+    data.def("setValue", setValue);
+    data.def("getValue", getValue);
     data.def("getHelp", &BaseData::getHelp, sofapython3::doc::baseData::getHelp);
     data.def("unset", [](BaseData& b){ b.unset(); }, sofapython3::doc::baseData::unset);
     data.def("getOwner", &getOwner, sofapython3::doc::baseData::getOwner);