Calling setData on Map Series throws an error with pattern-fill.js

[SyedTayyab99]

Expected behaviour

When series[0].setData() function is called, it should update the data of the series.

Actual behaviour

The call throws the following error

Uncaught TypeError: Cannot read properties of undefined (reading 'chart')
at r.calculatePatternDimensions (pattern-fill.src.js:169:36)
at R. (pattern-fill.src.js:417:21)
at highmaps.src.js:1748:25
at Array.forEach ()
at e (highmaps.src.js:1745:17)
at H.complexColor (highmaps.src.js:9664:17)
at H.fillSetter (highmaps.src.js:9993:21)
at H. (highmaps.src.js:9586:29)
at C (highmaps.src.js:1413:21)
at H.attr (highmaps.src.js:9564:21)

Steps to reproduce

I am attaching the files being used. On line 577, there is a call to "hchart.series[0].setData(data)" which results in error.
live-data.zip

Product version

Latest Versions

Affected browser(s)

All browsers

[karolkolodziej]

Hi @SyedTayyab99!

I'm unable to recreate that locally. Will you be able to prepare a minimal demo in some online code editor?

[SyedTayyab99]

Hi @karolkolodziej Thanks a lot for your response. Let me try that

[karolkolodziej]

There is a lot of custom code in your example shared initially. It would be great if you can reduce it ;)

[SyedTayyab99]

@karolkolodziej Please check the following fiddle. Once you click on the "Reload Data" button, you'll be able to see the issue.

https://jsfiddle.net/SyedTayyab/an4ocdsp/

Please feel free to let me know if you need anything else from my side.

[karolkolodziej]

Thank you @SyedTayyab99 for the demo.

Internal note:
I managed to simplify it even more.
@KacperMadej @hubertkozik Could you please take a look? It seems like this line might be the issue:

highcharts/ts/Extensions/PatternFill.ts

Lines 573 to 579 in 82e2e12

	// If we don't have a width/height yet, handle it. Try faking a point
	// and running the algorithm again.
	if (pattern._width === 'defer' || pattern._height === 'defer') {
	Point.prototype.calculatePatternDimensions.call(
	{ graphic: { element: element } }, pattern
	);
	}

[SyedTayyab99]

Hi @karolkolodziej please let me know if there is a work around or a patch that I can use in the meantime?

[mabuhaddah]

Dear @karolkolodziej , Hope this message finds you well.
Your kind support is highly appreciated.
Please we need any hotfix for that issue urgently.

Thanks

[mabuhaddah]

@karolkolodziej your quick action and feedback are highly appreciated!

[hubertkozik]

Hello guys!

I have prepared a workaround for that issue by overwriting drawPoints method for MapSeries. It is inside the IIFE function in the below demo. If you want to use it, just copy-paste it into your project and everything should work as intended. A similar solution will be used in the Highcharts Maps code.

Live demo: https://jsfiddle.net/BlackLabel/w8z3fv4k/