Fix conditional report issues (#430)

Author: user2684

README.md

@@ -502,7 +502,7 @@ The following methods are available for all the child:
 #if NODEMANAGER_CONDITIONAL_REPORT == ON
 	// force to send an update after the configured number of minutes
 	void setForceUpdateTimerValue(unsigned long value);
-	// never report values below this threshold (default: FLT_MIN)
+	// never report values below this threshold (default: -FLT_MAX)
 	void setMinThreshold(float value);
 	// never report values above this threshold (default: FLT_MAX)
 	void setMaxThreshold(float value);

nodemanager/Child.cpp

@@ -141,7 +141,8 @@ void Child::sendValue(bool force) {
 	// do not send if no samples have been collected, unless instructed otherwise
 	if (! _send_without_value && _samples == 0) return;
 #if NODEMANAGER_CONDITIONAL_REPORT == ON
-	if (! force) {
+	// ignore conditional reporting settings if forced to report or if it is the first run
+	if (! force || ! _sensor->getFirstRun()) {
 		// if the value is a number
 		if (_format != STRING) {
 			// if below or above the thresholds, do not send the value

nodemanager/Child.h

@@ -82,7 +82,7 @@ class Child {
 #if NODEMANAGER_CONDITIONAL_REPORT == ON
 	// force to send an update after the configured number of minutes
 	void setForceUpdateTimerValue(unsigned long value);
-	// never report values below this threshold (default: FLT_MIN)
+	// never report values below this threshold (default: -FLT_MAX)
 	void setMinThreshold(float value);
 	// never report values above this threshold (default: FLT_MAX)
 	void setMaxThreshold(float value);
@@ -122,7 +122,7 @@ class Child {
 	double _last_value = 0;
 	const char* _last_value_string = "";
 	Timer* _force_update_timer = new Timer();
-	float _min_threshold = FLT_MIN;
+	float _min_threshold = -FLT_MAX;
 	float _max_threshold = FLT_MAX;
 	float _value_delta = 0;
 #endif

nodemanager/Node.cpp

@@ -558,6 +558,7 @@ void NodeManager::loop() {
 		// send the message, multiple times if requested
 		for (int i = 0; i < _retries; i++) {
 			if (mGetPayloadType(_message) == P_INT16) debug_verbose(PSTR(LOG_MSG "SEND(%d) t=%d p=%d\n"),_message.sensor,_message.type,_message.getInt());
+			if (mGetPayloadType(_message) == P_LONG32) debug_verbose(PSTR(LOG_MSG "SEND(%d) t=%d p=%ld\n"),_message.sensor,_message.type,_message.getLong());
 			if (mGetPayloadType(_message) == P_FLOAT32) debug_verbose(PSTR(LOG_MSG "SEND(%d) t=%d p=%d.%02d\n"),_message.sensor,_message.type,(unsigned int)_message.getFloat(), (unsigned int)(_message.getFloat()*100)%100);
 			if (mGetPayloadType(_message) == P_STRING) debug_verbose(PSTR(LOG_MSG "SEND(%d) t=%d p=%s\n"),_message.sensor,_message.type,_message.getString());
 			send(_message, _ack);

nodemanager/Sensor.cpp

@@ -108,6 +108,11 @@ void Sensor::setReportIntervalDays(uint8_t value) {
 	setReportIntervalSeconds(value*86400UL);
 }
 
+// return true if it is the first execution of loop on this sensor
+bool Sensor::getFirstRun() {
+	return _first_run;
+}
+
 // register a child
 void Sensor::registerChild(Child* child) {
 	children.push(child);

nodemanager/Sensor.h

@@ -58,6 +58,8 @@ class Sensor {
 	void setMeasureTimerMode(timer_mode value);
 	// [27] Set the value for the reporting timer's mode which has been set with setReportTimerMode() If not set explicitely, will be set with the same value as the reporting timer
 	void setMeasureTimerValue(unsigned long value);
+	// return true if it is the first execution of loop on this sensor
+	bool getFirstRun();
 	// list of configured child
 	List<Child*> children;
 	// return the child object based on the provided child_id