Nnnn fix invalid viewmodel properties

some fixes for data binding by commit:
- fix double callback when properties are synced between dart and cpp
- fix range mapper missing interpolator
- fix validating group converters
- fix duplication of view model properties and added validations

extra comments in PR

Diffs=
6e30b9bb43 Nnnn fix invalid viewmodel properties (#8922)

Co-authored-by: hernan <[email protected]>

.rive_head

@@ -1 +1 @@
-6d8854ec047020794d0d4babbcad8343d216256a
+6e30b9bb430926b63f0ed662b89250b033993c87

include/rive/data_bind/converters/data_converter_range_mapper.hpp

@@ -24,9 +24,11 @@ class DataConverterRangeMapper : public DataConverterRangeMapperBase
 
 public:
     void interpolator(KeyFrameInterpolator* interpolator);
+    KeyFrameInterpolator* interpolator() const { return m_interpolator; };
     DataType outputType() override { return DataType::number; };
     DataValue* convert(DataValue* value, DataBind* dataBind) override;
     DataValue* reverseConvert(DataValue* value, DataBind* dataBind) override;
+    void copy(const DataConverterRangeMapperBase& object);
 
 private:
     DataValueNumber m_output;

src/data_bind/converters/data_converter_range_mapper.cpp

@@ -5,6 +5,12 @@
 
 using namespace rive;
 
+void DataConverterRangeMapper::copy(const DataConverterRangeMapperBase& object)
+{
+    interpolator(object.as<DataConverterRangeMapper>()->interpolator());
+    DataConverterRangeMapperBase::copy(object);
+}
+
 DataValueNumber* DataConverterRangeMapper::calculateRange(DataValue* input,
                                                           float minInput,
                                                           float maxInput,

src/importers/backboard_importer.cpp

@@ -90,6 +90,20 @@ StatusCode BackboardImporter::resolve()
         auto asset = m_FileAssets[index];
         referencer->setAsset(asset);
     }
+
+    for (auto converter : m_DataConverters)
+    {
+        if (converter->is<DataConverterRangeMapper>())
+        {
+            size_t converterId =
+                converter->as<DataConverterRangeMapper>()->interpolatorId();
+            if (converterId != -1 && converterId < m_interpolators.size())
+            {
+                converter->as<DataConverterRangeMapper>()->interpolator(
+                    m_interpolators[converterId]);
+            }
+        }
+    }
     for (auto referencer : m_DataConverterGroupItemReferencers)
     {
         auto index = (size_t)referencer->converterId();
@@ -109,19 +123,6 @@ StatusCode BackboardImporter::resolve()
         referencer->converter(
             m_DataConverters[index]->clone()->as<DataConverter>());
     }
-    for (auto converter : m_DataConverters)
-    {
-        if (converter->is<DataConverterRangeMapper>())
-        {
-            size_t converterId =
-                converter->as<DataConverterRangeMapper>()->interpolatorId();
-            if (converterId != -1 && converterId < m_interpolators.size())
-            {
-                converter->as<DataConverterRangeMapper>()->interpolator(
-                    m_interpolators[converterId]);
-            }
-        }
-    }
     return StatusCode::Ok;
 }