Merge pull request #15230 from barbudor/timer_var_mode_2_3

Fix %TIMERx% variables for negative values
This commit is contained in:
Theo Arends 2022-03-26 16:13:41 +01:00 committed by GitHub
commit d427220ee4
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 18 additions and 2 deletions

View File

@ -235,6 +235,22 @@ uint16_t SunMinutes(uint32_t dawn)
#endif // USE_SUNRISE
uint16_t TimerGetTimeOfDay(uint8_t index)
{
Timer xtimer = Settings->timer[index];
int16_t xtime = xtimer.time;
#ifdef USE_SUNRISE
if (xtimer.mode) {
if (xtime >= 12*60) xtime = 12*60 - xtime;
xtime += (int16_t)SunMinutes(xtimer.mode-1);
if (xtime < 0) xtime += 24*60;
if (xtime >= 24*60) xtime -= 24*60;
}
#endif
return xtime;
}
/*******************************************************************************************/
void TimerSetRandomWindow(uint32_t index)

View File

@ -477,7 +477,7 @@ bool RulesRuleMatch(uint8_t rule_set, String &event, String &rule, bool stop_all
if ((index > 0) && (index <= MAX_TIMERS)) {
snprintf_P(stemp, sizeof(stemp), PSTR("%%TIMER%d%%"), index);
if (rule_param.startsWith(stemp)) {
rule_param = String(Settings->timer[index -1].time);
rule_param = String(TimerGetTimeOfDay(index -1));
}
}
}
@ -796,7 +796,7 @@ bool RuleSetProcess(uint8_t rule_set, String &event_saved)
#if defined(USE_TIMERS)
for (uint32_t i = 0; i < MAX_TIMERS; i++) {
snprintf_P(stemp, sizeof(stemp), PSTR("%%TIMER%d%%"), i +1);
RulesVarReplace(commands, stemp, String(Settings->timer[i].time));
RulesVarReplace(commands, stemp, String(TimerGetTimeOfDay(i)));
}
#if defined(USE_SUNRISE)
RulesVarReplace(commands, F("%SUNRISE%"), String(SunMinutes(0)));