OpenZWave / node-openzwave-shared

OpenZWave addon for Node.js (all versions) including management and security functions
Other
199 stars 113 forks source link

Fix in EnablePoll method, it instead of getting a value from index 1,… #202

Closed rumata28 closed 6 years ago

rumata28 commented 6 years ago

It does look like a bug.

API supports "old" and "new" ways of passing value_id - as an object or as 4 arguments. Each such "dual-API" method does a check of input arguments to determine if that is "old" or "new" style of invocation.

And in "EnablePoll" method, it does look like a bug to me - instead of getting value from 1-st position, it just gets 1.

ekarak commented 6 years ago

thank you!