esp_rmaker_params: Fixed an issue with esp_rmaker_param_update_and_notify()

shahpiyushv · shahpiyushv · commit 252b47c2ccc9 · 2021-11-17T23:14:44.000+05:30
Wrong flags were getting modified, thereby causing the param update
publish message to get skipped after sending the notification message.
diff --git a/components/esp_rainmaker/src/core/esp_rmaker_param.c b/components/esp_rainmaker/src/core/esp_rmaker_param.c
@@ -230,7 +230,7 @@ static esp_err_t esp_rmaker_allocate_and_populate_params(uint8_t flags, bool res
 
 static esp_err_t esp_rmaker_report_param_internal(uint8_t flags)
 {
-    esp_err_t err = esp_rmaker_allocate_and_populate_params(RMAKER_PARAM_FLAG_VALUE_CHANGE, true);
+    esp_err_t err = esp_rmaker_allocate_and_populate_params(flags, true);
     if (err == ESP_OK) {
         /* Just checking if there are indeed any params to report by comparing with a decent enough
          * length as even the smallest possible data, Eg. '{"d":{"p":0}}' will be > 10 bytes.

Original file line number	Diff line number	Diff line change
`@@ -230,7 +230,7 @@ static esp_err_t esp_rmaker_allocate_and_populate_params(uint8_t flags, bool res`
`230`	`230`
`231`	`231`	`static esp_err_t esp_rmaker_report_param_internal(uint8_t flags)`
`232`	`232`	`{`
`233`		`- esp_err_t err = esp_rmaker_allocate_and_populate_params(RMAKER_PARAM_FLAG_VALUE_CHANGE, true);`
	`233`	`+ esp_err_t err = esp_rmaker_allocate_and_populate_params(flags, true);`
`234`	`234`	`if (err == ESP_OK) {`
`235`	`235`	`/* Just checking if there are indeed any params to report by comparing with a decent enough`
`236`	`236`	`* length as even the smallest possible data, Eg. '{"d":{"p":0}}' will be > 10 bytes.`