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

Error on sankey update while hovered #10624

Closed
futpib opened this issue Apr 25, 2019 · 1 comment · Fixed by #10643
Closed

Error on sankey update while hovered #10624

futpib opened this issue Apr 25, 2019 · 1 comment · Fixed by #10643

Comments

@futpib
Copy link

futpib commented Apr 25, 2019

Expected behaviour

No error.

Actual behaviour

A call to chart.update while a sankey edge is hovered results in error.

highcharts.src.js:45176 Uncaught TypeError: Cannot read property 'options' of null
    at object.setState (highcharts.src.js:45176)
    at sankey.src.js:204
    at Array.forEach (<anonymous>)
    at object.setNodeState [as setState] (sankey.src.js:203)
    at object.destroy (highcharts.src.js:30243)
    at object.destroy (highcharts.src.js:34938)
    at object.destroy (sankey.src.js:193)
    at remove (highcharts.src.js:37793)
    at object.remove (highcharts.src.js:37809)
    at object.remove (highcharts.src.js:40165)

Live demo with steps to reproduce

https://jsfiddle.net/L6gq7y54/

  1. Hover any edge (hovering nodes does not lead to error)
  2. Wait for a random update to hit while hovered (up to 3 seconds)
  3. Observe the broken chart and an error in the console

Product version

Highcharts JS v7.1.1 (2019-04-09)

Affected browser(s)

Probably all (but only latest Chrome and Firefox were tested).

Workaround

			if (chart.hoverPoint) {
				chart.hoverPoint.onMouseOut();
			}

			chart.update(...);
@sebastianbochan
Copy link
Contributor

sebastianbochan commented Apr 26, 2019

Hi @futpib,
Thank you for reporting about the problem. The issue is related with the new option - inactive state.

Internal note
Its regression since v7.0.3.

Demo:

FYI @pawelfus

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

Successfully merging a pull request may close this issue.

3 participants