currentValue of contact not supported for contact sensor?

šŸš§ Developers
1

code:

if (contactSensor)      ind = (contactSensor.currentValue("contact").contains('closed') ? 1 : 0);
else                    ind = -1

log:

app:16 2018-03-06 13:09:55.696:errorCommand ā€˜currentValueā€™ is not support by device. on line 1668

2

Any reason why contactSensor.currentContact == 'closed' canā€™t be used?

3

no particular reason but this seems to work:

if (motionSensors)      ind = (motionSensors.currentValue("motion").contains('active') ? 1 : 0);

any reason the same style would not work for contact sensor?

4

Gotcha. I thought for a second that might be working since it seems to be a map of multiple motion sensors, but after testing it quickly it still doesnā€™t work. Iā€™d just go with what works for now since thereā€™s no official documentation. currentValue() and current[attribute name] seem to have the same functionality.

5

Iā€™ll test this tomorrow, as it makes no sense why this doesnt workā€¦
In this instance, contactSensor is multiple:true?, or falseā€¦

1 Like
6

thanks. multiple:true

7

they do have the same functionality. but i use currentValue(ā€œattributeā€) all over the place and too lazy to write a regex to replace everything :grinning:

8

were you able to confirm why this one fails and when a fix would be available?

thanks.

9

its failing with every occurrence so cant get the app to work.

code:

image
image.png964Ɨ46 8.17 KB

log:

app:16 2018-03-08 18:37:27.610:errorCommand 'currentValue' is not support by device. on line 2411

10

This only fails against an input where multiple is true, which is a list, has nothing to do with the attribute name. For now just use an iterator.

11

thanks. yes had figured out that part already.

i am trying to keep the codebase the same for ST and hubitat. any word on when there will be a fix?

thanks.

12

Please submit an issue to support on this, weā€™ll then get it into our issues list.
For the time being you can use an iteration (find, findAll, each, collect ect), this will also work on your other platformā€¦

13

Hereā€™s a little javascript to regex it. Itā€™ll convert .currentValue("contact") to .currentContact or any other attribute. I havenā€™t tested if current[Attribute] works on a map though.

1 Like
14

thank you. unfortunately this would not solve the issue though. see @mike.maxwell response above.

15

I just did a quick test and it works on a map. I know it doesnā€™t solve the currentValue bug but might be an easier transition until a fix is in place. Run it through the regex and copy the updated code to Hubitat. Or Mikeā€™s suggestion is probably better to make a change thatā€™ll work on both platforms.

16

didnt get this part :slight_smile:

whether its sensor.currentValue(ā€œcontactā€) or sensor.currentContact both are going to fail when sensor is multiple unless i use an iterator.

17

Hm, I might be missing something. It should work in the context of your first post:

if (contactSensor)      ind = (contactSensor.currentContact.contains('closed') ? 1 : 0);
else                    ind = -1

I tested this section and had no errors and returned true/false correctly. Is there another context where it wonā€™t work?

1 Like
18

thought it didnt work without an iterator when contact sensors specified is multiple?

19

Worked for me testing with switches:

2018-03-08 21:57:44.864:debug switches.currentSwitch.contains('on'): false
2018-03-08 21:57:44.863:debug switches.currentSwitch.contains('off'): true
2018-03-08 21:57:44.861:debug switches.currentSwitch: [off, off]
2018-03-08 21:57:44.855:debug switches: [Backyard Light, Office Lamp]
1 Like
20

ahh ā€¦ ok. wasnt expecting that based on that post. will check with contact sensors. if it works ā€¦ i use atom for github personal stuff so these should work for me:

search: (\.currentValue\(")([a-z])(.+)("\))
replace: .current\U$2\L$3

will give it a try after i have confirmed.

thanks again.

1 Like