tdInit only available through getDevices() about node-telldus HOT 13 CLOSED

And tdClose as well then I guess or how is that solved today?

from node-telldus.

Correct! That's not handled at all today

from node-telldus.

I've read somewhere, can't find where now, that tdInit() only should be called once. So I guess that there should be an function that checks a boolean that is indicating if the Tellstick is initialized or not, and that function should be called in each of the "command functions".

Not sure when/where we should call tdClose(), I guess you normally never "shut down" node app.

from node-telldus.

http://www.telldus.com/forum/viewtopic.php?f=7&t=2188#p14262

from node-telldus.

Yeah that's a problem, we need to decide if we wan't to leave the tdInit call to the user, via telldus.tdInit(callback) or if we want to make logic in the js-wrapper that run tdInit on the first call and never again. The latter version is imho the cleanest option, and the first option is the most "library-agnostic".

from node-telldus.

I'm all for running it automatically in the wrapper.

from node-telldus.

I agree.

from node-telldus.

Agreed :) I'll add the core c++ stuff later, after that you'll be able to call telldus.tdInit(callback); (and close, if needed)

from node-telldus.

Untested as i dont have access to my tellstick right now. Short documentation in commit description, did not change telldus.js to avoid merge conflicts.

from node-telldus.

I have now tested, using my own changes to telldus.cc and .js, with and without tdInit/tdClose and I really can't make out any difference...
Anyone got anything.

from node-telldus.

I know nothing, seems like some people use init and close to successfully restart the library on failures. Telldus documentation says that tdInit "initializes the library", so i guess the best bet is to run it for compability with future versions.

from node-telldus.

tdInit is now called on module load.
tdClose on process exit.
Don't know if it's the best way of doing it but it seems to work.

from node-telldus.

That should be entirely fine, great 👍

from node-telldus.