Events: listener without context can be added twice

var m = new L.Marker();
var x = function() {console.log("event fired")};

/* register without context */
m.on("test", x);
m.on("test", x);

/* fires twice */
m.fire("test");

/* register with context */
context = {}
m.on("test2", x, context);
m.on("test2", x, context);

/* fires once*/
m.fire("test2");

https://playground-leaflet.rhcloud.com/wum/1/edit?html,js,output

[yohanboniface]

Addendum: If you supply the second listener with two different contexts it also fires twice (once for each context).

We had a small chat about this, and it seems it’s a feature, not a bug 😉 Basically, this is what you are doing when you do something like:

this._map.on('click', this._onClick, layer1)
this._map.on('click', this._onClick, layer2)

So what needs to be unique is the combination of listener + context.

[fab1an]

This is fixed now, is it?