z3ta/tellcage

A wrapper/middleman for the Node.JS telldus module that will restart telldus when it crashes

tellcage's Issues

Failing on missing callbacks

I have not used callback scripts in my application on telldus. (Maybe I should have)

Tellcage died on me the first time I used turnon command because question[msg.id](msg.err, msg.answer); was not a function. I solved this by modifying all the relevant places to have a dummy function such as: question[id] = callback?callback:function (){};

That solution is a bit ugly, but it was a quick fix.

Always subscribing to events

Currently it seems that this wrapper always subscribes to raw and device events. I do not need raw events, sensor and device events are all that I need. I think it could be smarter to start the corresponding event subscription in the worker only when the first subscription that needs is requested from the tellcage.js.

I have your current code working just fine, and I have commented out the raw event subscription from source. This idea just came to my mind and if other people are interested in this tellcage, we could improve it. If not, I'm happy as is. :)