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

Polygon entity with callback property hierarchy crashes #8225

Closed
hpinkos opened this issue Sep 27, 2019 · 3 comments
Closed

Polygon entity with callback property hierarchy crashes #8225

hpinkos opened this issue Sep 27, 2019 · 3 comments

Comments

@hpinkos
Copy link
Contributor

hpinkos commented Sep 27, 2019

This works in 1.59 but not in 1.60
Likely related to this PR: #7991

Marking this 'next release' since it is a regression, but I don't expect it to be fixed for the October release.

var viewer = new Cesium.Viewer('cesiumContainer');

viewer.entities.add({
    polygon : {
        hierarchy : new Cesium.CallbackProperty(function() { 
            return Cesium.Cartesian3.fromDegreesArray([-115.0, 37.0,
                                                        -115.0, 32.0,
                                                        -107.0, 33.0,
                                                        -102.0, 31.0,
                                                        -102.0, 35.0]);
        }, false)
    }
});
@hpinkos
Copy link
Contributor Author

hpinkos commented Sep 27, 2019

@shunter do you have time to look into this?

@shunter
Copy link
Contributor

shunter commented Sep 27, 2019

Returning a position array instead of a PolygonHierarchy was always undocumented behavior. You can fix your code snippet by calling the PolygonHierarchy constructor.

@OmarShehata
Copy link
Contributor

Closing since this is intentional, and also it's a duplicate of #8135.

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

No branches or pull requests

3 participants