Leaflet: Feature: add a 'error' event to ImageOverlay

[IvanSanchez]

As per https://stackoverflow.com/questions/41841381/leaflet-imagelayer-seturl-error-handling , it should be useful to have a error event in L.ImageOverlay, akin to the tileerror event in L.TileLayer.

[APwhitehat]

Looking into this issue.
The problem is in
https://github.com/Leaflet/Leaflet/blob/master/src/layer/ImageOverlay.js
I suppose ... ?

[IvanSanchez]

Yeah. See:

https://github.com/Leaflet/Leaflet/tree/ad75456fa54aee4d38e92dd7654128819cf8e31a/src/layer/ImageOverlay.js#L167
and also:
https://github.com/Leaflet/Leaflet/tree/ad75456fa54aee4d38e92dd7654128819cf8e31a/src/layer/tile/TileLayer.js#L144

[perliedman]

@AP-whitehat yes, exactly. From a quick look, an error handler should probably be added in _initImage.

(Apparently @IvanSanchez wrote a response quicker than I did :smile: )

[APwhitehat]

Issue not yet Fixed ,
am I going in the right direction ?

please point out my mistakes 😛

[IvanSanchez]

@AP-whitehat Please open up a pull request, so we can comment on specific lines of your code :-)

[perliedman]

Closed by #5307!