[openstreetmap/openstreetmap-website] Deprecated include of L.Mixin.Events: this property will be removed in future releases (#1598)

Andy Allan notifications at github.com
Wed Mar 28 03:24:43 UTC 2018


I'm going to reopen this, since the warnings appear dozens of times in each test run, and it's a needless distraction for our developers.

They originate from the [leaflet-locationfilter](https://github.com/kajic/leaflet-locationfilter) plugin that we include at [vendor/assets/leaflet-locationfilter.js](https://github.com/openstreetmap/openstreetmap-website/blob/master/vendor/assets/leaflet/leaflet.locationfilter.js). The plugin appears to be unmaintained, with the last update 4 years ago. We're running a patched version already (see https://github.com/kajic/leaflet-locationfilter/pull/23) so there's precedent for us patching the plugin further. 

However, it would be better to either get upstream to accept patches, or find a maintained fork, or find a replacement plugin. I'd like to reduce the amount of code we maintain, rather than adding further workarounds to other people's code, whenever possible.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/issues/1598#issuecomment-376748597
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20180327/b32f8143/attachment.html>


More information about the rails-dev mailing list