Add bool flag and functions to handle properties appended to a CBOR message that may be discarded

Author: pennam

src/property/Property.cpp

@@ -39,6 +39,7 @@ Property::Property()
 , _update_policy{UpdatePolicy::OnChange}
 , _has_been_updated_once{false}
 , _has_been_modified_in_callback{false}
+, _has_been_appended_but_not_sended{false}
 , _last_updated_millis{0}
 , _update_interval_millis{0}
 , _last_local_change_timestamp{0}
@@ -106,6 +107,10 @@ bool Property::shouldBeUpdated() {
     return true;
   }
 
+  if (_has_been_appended_but_not_sended) {
+    return true;
+  }
+
   if (_has_been_modified_in_callback) {
     return true;
   }
@@ -126,6 +131,13 @@ void Property::requestUpdate()
   _update_requested = true;
 }
 
+void Property::appendCompleted()
+{
+  if (_has_been_appended_but_not_sended) {
+    _has_been_appended_but_not_sended = false;
+  }
+}
+
 void Property::execCallbackOnChange() {
   if (_update_callback_func != nullptr) {
     _update_callback_func();

src/property/Property.h

@@ -174,6 +174,7 @@ class Property
     void setTimestamp(unsigned long const timestamp);
     bool shouldBeUpdated();
     void requestUpdate();
+    void appendCompleted();
     void execCallbackOnChange();
     void execCallbackOnSync();
     void setLastCloudChangeTimestamp(unsigned long cloudChangeTime);
@@ -229,7 +230,8 @@ class Property
 
     UpdatePolicy       _update_policy;
     bool               _has_been_updated_once,
-                       _has_been_modified_in_callback;
+                       _has_been_modified_in_callback,
+                       _has_been_appended_but_not_sended;
     /* Variables used for UpdatePolicy::TimeInterval */
     unsigned long      _last_updated_millis,
              _update_interval_millis;