fix: Add container ground overlays in KML/KMZ

[jeffdgr8]

Pass the correct set of ground overlays from a nested container to properly add to the map.

Fixes #740 🦕

[OlavAa]

Pass the correct set of ground overlays from a nested container to properly add to the map.

Fixes #740 🦕

Confirmed, issue is fixed.

[wangela]

@jeffdgr8 Since this no longer uses GetGroundOverlayMap(), should this be treated as a breaking change?

[jeffdgr8]

@wangela no, it shouldn't break anything. The current behavior uses getGroundOverlayMap() to get the root container's ground overlays and passes them to addGroundOverlays(), along with the child containers. This function iterates over the child containers and adds their ground overlays as well. But the bug is that this function doesn't use these child containers' ground overlays from the parameter, but instead calls getGroundOverlayMap() to get the root container's ground overlays that have already been added.

So current behavior works for ground overlays in the root of the XML, but ground overlays in nested containers are not rendered as they should be. All ground overlays should be rendered correctly now with the change.

[kikoso]

This seems to fix the issue, great job @jeffdgr8 !

[wangela]

Thank you @jeffdgr8 for contributing a fix to an issue reported by @OlavAa. Much appreciated!

[googlemaps-bot]

🎉 This PR is included in version 3.5.1 🎉

The release is available on:

• v3.5.1
• GitHub release

Your semantic-release bot 📦🚀