search

toggledbits / Reactor

Reactor (for Vera and openLuup) is a Vera Home Automation plugin that provides advanced programmable logic.
17 stars 0 forks source link

Group Status Issue #27

Closed Tarkus-NJ closed 5 years ago

Tarkus-NJ commented 5 years ago

Patrick,

I am using group state to pick up the true or false status of a conditional group in a separate sensor. I am getting an unexpected false when the conditional group is showing true in originating sensor. Summary and snippet attached.

Group_Status_Conflict

Snippet_Group_Condition.txt Logic_Summary_Group_Condition.txt

toggledbits commented 5 years ago

OK. This is a simple fix. Rather than roll another beta for it just yet, I'll give you what should be the "less than one line" fix for it:

L_Reactor.lua line 2217 reads: local vv = getVarNumeric( varname, 0, tdev, GRPSID ) ~= 0 -- boolean!

Change the tdev to cond.device: local vv = getVarNumeric( varname, 0, cond.device, GRPSID ) ~= 0 -- boolean!

Then reload Luup.

Tarkus-NJ commented 5 years ago

Using Notepad++ it came up on line 2261. I searched for the string and it was the only line it was found on. The latest beta is reactor-beta3.0-19087, correct? Anyway I changed that line and problem is solved.

Thanks

toggledbits commented 5 years ago

Latest beta is 19090... miss the email? SPAM folder?

Tarkus-NJ commented 5 years ago

Exactly,, in spam folder. Will apply your fix before installing.

Thanks

toggledbits commented 5 years ago

In 19093