Removed Ticker-based pulses. This allows to change pulse time on the go (#816)

code/espurna/relay.ino

@@ -23,6 +23,7 @@ typedef struct {
     unsigned long delay_off;    // Delay to turn relay OFF
     unsigned char pulse;        // RELAY_PULSE_NONE, RELAY_PULSE_OFF or RELAY_PULSE_ON
     unsigned long pulse_ms;     // Pulse length in millis
+    unsigned long pulse_start;  // Current pulse start (millis), 0 means no pulse
 
     // Status variables
 
@@ -34,10 +35,6 @@ typedef struct {
     bool report;                // Whether to report to own topic
     bool group_report;          // Whether to report to group topic
 
-    // Helping objects
-
-    Ticker pulseTicker;         // Holds the pulse back timer
-
 } relay_t;
 std::vector<relay_t> _relays;
 bool _relayRecursive = false;
@@ -203,6 +200,21 @@ void _relayProcess(bool mode) {
 
 }
 
+/**
+ * Walks the relay vector check if any relay has to pulse back
+ */
+void _relayPulseCheck() {
+    unsigned long current_time = millis();
+    for (unsigned char id = 0; id < _relays.size(); id++) {
+        if (_relays[id].pulse_start > 0) {
+            if (current_time - _relays[id].pulse_start > _relays[id].pulse_ms) {
+                _relays[id].pulse_start = 0;
+                relayToggle(id);
+            }
+        }
+    }
+}
+
 // -----------------------------------------------------------------------------
 // RELAY
 // -----------------------------------------------------------------------------
@@ -218,10 +230,10 @@ void relayPulse(unsigned char id) {
     bool pulseStatus = (mode == RELAY_PULSE_ON);
 
     if (pulseStatus == status) {
-        _relays[id].pulseTicker.detach();
+        _relays[id].pulse_start = 0;
     } else {
         DEBUG_MSG_P(PSTR("[RELAY] Scheduling relay #%d back in %lums (pulse)\n"), id, ms);
-        _relays[id].pulseTicker.once_ms(ms, relayToggle, id);
+        _relays[id].pulse_start = millis();
     }
 
 }
@@ -464,6 +476,7 @@ void _relayBoot() {
         }
         _relays[i].current_status = !status;
         _relays[i].target_status = status;
+        _relays[i].pulse_start = 0;
         #if RELAY_PROVIDER == RELAY_PROVIDER_STM
             _relays[i].change_time = millis() + 3000 + 1000 * i;
         #else
@@ -821,6 +834,7 @@ void _relayInitCommands() {
 //------------------------------------------------------------------------------
 
 void _relayLoop() {
+    _relayPulseCheck();
     _relayProcess(false);
     _relayProcess(true);
 }

code/html/index.html

@@ -1336,7 +1336,7 @@
             </div>
             <div class="pure-g">
                 <div class="pure-u-1 pure-u-lg-1-4"><label>Pulse time (s)</label></div>
-                <div class="pure-u-1 pure-u-lg-1-4"><input name="relayTime" class="pure-u-1" type="number" min="0" step="0.1" max="3600" /></div>
+                <div class="pure-u-1 pure-u-lg-1-4"><input name="relayTime" class="pure-u-1" type="number" min="0" step="0.1" max="86400" /></div>
             </div>
             <div class="pure-g module module-mqtt">
                 <div class="pure-u-1 pure-u-lg-1-4"><label>MQTT group</label></div>