prevent undefined property from removing all click events when removed from map

[robertleeplummerjr]

If a class does not have a click event, and it is removed, it causes all click events to be removed from map. This prevents that from happening.

[robertleeplummerjr]

@mourner That was the fastest I've ever seen someone interact with a pull request. Are you sure you are a human? 👍

[robertleeplummerjr]

The other way to solve this is to properly set the option, or: L.Class.prototype._onClick = function () { };. The most logical for stability seemed like what was included in the pr.

[mourner]

Thanks! Actually that label was added by a bot (waffle.io). I'm mostly off for holidays. :)

[robertleeplummerjr]

👍

[robertleeplummerjr]

Also, waffle.io rocks.

[z0mt3c]

Actually i think this change/merge is not correct. According to robertleeplummerjr@3cec7fc i would suggest it should just be renamed to _fireMouseEvent? Your proposed change will allow you to register handlers but will not de-register the listener on removal.

@mourner please have another look... i guess this one shouldn't have passed its review.

[robertleeplummerjr]

The origin: 3cec7fc of the above. @z0mt3c You use "it" a few times here, can you clarify which forks/revisions/libraries you are referring to when you mention them above?