search

moscajs / mosca

MQTT broker as a module
mosca.io
3.2k stars 509 forks source link

Invalid this access #484

Closed behrad closed 8 years ago

behrad commented 8 years ago

this.clients in https://github.com/mcollina/mosca/blob/master/lib/server.js#L247 & https://github.com/mcollina/mosca/blob/master/lib/server.js#L299 shouldn't be that.clients?

mcollina commented 8 years ago

that and this are the same there, it's set to the EE instance, feel free to send a pr to make it more readable if you like.

behrad commented 8 years ago

it's set to the EE instance

So is this happening by the runtime for callbacks passed to EE#on ?

mcollina commented 8 years ago

yes, it's part of the EE interface. Il giorno dom 29 mag 2016 alle 14:09 Behrad notifications@github.com ha scritto:

it's set to the EE instance

So is this happening by the runtime for callbacks passed to EE#on ?

— You are receiving this because you modified the open/close state.

Reply to this email directly, view it on GitHub https://github.com/mcollina/mosca/issues/484#issuecomment-222357271, or mute the thread https://github.com/notifications/unsubscribe/AADL4462zFhQXWz1uQSRiWy6sZ7AFFRUks5qGYIVgaJpZM4IpKNx .