MagicMirror: this.sendSocketNotification is not a function

I have just about finished my Traffic module to get current commute time, but I’m having difficulty sending a socket notification other than the initial one in my start function. The project can be found here and the issue lies in MMM-Traffic.js.

I send a socket notification in start which queries the google maps api in node_helper.js then sends a socket notification back to MMM-Traffic.js which will then call updateDom(). I’m using updateCommute() as a proxy for setInterval that will send the initial socket notification at the user defined interval, but I am getting an error saying that this.sendSocketNotification is not a function. Any ideas?

About this issue

Original URL
State: closed
Created 8 years ago
Comments: 81 (20 by maintainers)

Commits related to this issue

Fix socket reference bug. #196 — committed to MagicMirrorOrg/MagicMirror by MichMich 8 years ago

Most upvoted comments

Don’t forget to add your module to the wiki, so that others can find it! Will give it a try later, thanks for your work!

Am 20.04.2016 um 09:58 schrieb Michael Teeuw notifications@github.com:

Great work @SamLewis0602! I’m using your module on my mirror. My girlfriend asked me for this features a few days ago! 😃

I ended you a pull request with multi language support. This way it can be used in other languages as well.

I’ll continue further question/issues in your repro te keep everything in one place. 😃

— You are receiving this because you were mentioned. Reply to this email directly or view it on GitHub

paviro on Apr 20, 2016

@paviro: If in the handler itself you don’t need the original this you can override it like this.

setTimeout(
  this.handler.bind(this),
  this.config.updateIntervall
);

setTimeout(
  function(){
    //doSomething
  }.bind(this),
  this.config.updateIntervall
);

CFenner on Apr 20, 2016

Great! Everything is working now. Thanks again for your help!

saml-dev on Apr 19, 2016

(Until someone opens the interface twice in multiple browsers. Then the shit hits the fan. 😉)

MichMich on Apr 18, 2016

The Log.log helper does not work in the node helper as you might have noticed. 😃

console.log(aVariable);

Should work.

Note: to send a notification from the node helper to the front end, the frontend first needs to send a message (to open the connection.)

MichMich on Apr 18, 2016