Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

null _handleLine and _rotationMarker not handled properly #43

Open
alcalin opened this issue Feb 15, 2019 · 1 comment
Open

null _handleLine and _rotationMarker not handled properly #43

alcalin opened this issue Feb 15, 2019 · 1 comment

Comments

@alcalin
Copy link

alcalin commented Feb 15, 2019

In the example you have provided, if I disable 'rotate' and keep the rest on, I will get some errors in_.nScaleStart and _onScaleEnd

Util.js:59 Uncaught TypeError: Cannot read property '_leaflet_id' of null
    at stamp (Util.js:59)
    at NewClass.removeLayer (Layer.js:180)
    at NewClass._onScaleStart (L.Path.Transform.js:formatted:804)
    at NewClass.fire (Events.js:190)
    at NewClass._fireDOMEvent (Map.js:1433)
    at NewClass._handleDOMEvent (Map.js:1390)
    at HTMLDivElement.handler (DomEvent.js:79)
stamp @ Util.js:59
removeLayer @ Layer.js:180
_onScaleStart @ L.Path.Transform.js:formatted:804
fire @ Events.js:190
_fireDOMEvent @ Map.js:1433
_handleDOMEvent @ Map.js:1390
handler @ DomEvent.js:79
Layer.js:158 Uncaught TypeError: Cannot read property '_layerAdd' of null
    at NewClass.addLayer (Layer.js:158)
    at NewClass._onScaleEnd (L.Path.Transform.js:formatted:827)
    at NewClass.fire (Events.js:190)
    at NewClass._fireDOMEvent (Map.js:1433)
    at NewClass._handleDOMEvent (Map.js:1390)
    at HTMLDivElement.handler (DomEvent.js:79)

Seems that the this._handleLine and this._rotationMarker are null when rotation is disabled. Can anyone else confirm this ?

@dktue
Copy link

dktue commented Feb 28, 2019

I just ran into the same bug. I will enable rotation as a workaround now, thanks for the hint.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants