Stacked set not updating device state

[JonHamilton]

I'm running 4.1; checked master and didn't see any changes which looked relevant to this issue.

I have a "countdown timer" working so that if so configured, my devices will shut off after the configured duration. Next I was looking to implement a "hey, I'm going to shut off soon" warning similar to what is found on digital countdown timers (where it flashes the light off for a second, then back on for a minute, then off for good).

I tried doing this with a stacked set command, and it does cycle through the device settings as expected -- but it doesn't update the ->state at any point, so to my code it looks like the device is always just "on" even though it is transitioning state a few times. Am I misunderstanding or missing something?

$unit->set("OFF2ON60OFF", "countdown_timer");

This does trigger an event as I'd expect, but in my tie_event code, if I check the state, it always says that it's on.

Thanks for any pointers.

[JonHamilton]

I found the problem, and it's me :) Quoted the string and left ON and OFF uppercase in the quoted string, so the state updates didn't behave right. Lowercase worked fine/triggers the $state updates as expected.

I'm not sure - this still seems like it should be updating state and/or the behavior when handed an uppercase value should be consistent between the internally represented $state and the behavior of the system - but at least it makes sense and has a sane workaround. If this is expected behavior, please feel free to close as such.