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

ObjectLoader: Call onError() if no metadata found in JSON. #24891

Merged

Conversation

frading
Copy link
Contributor

@frading frading commented Nov 1, 2022

Description

While the ObjectLoader calls onError if the json parsing has failed, it does not if no metadata has been found in the json.

It does print out an error in the console, but that cannot still be caught at the application level.

So for instance in Polygonjs's node editor, if someone tries loads a .json that was not exported from a threejs scene, no error can be displayed.

@Mugen87 Mugen87 added this to the r147 milestone Nov 2, 2022
@Mugen87 Mugen87 merged commit 1674b5b into mrdoob:dev Nov 2, 2022
@Mugen87 Mugen87 changed the title ObjectLoader: call onError if no metadata found in json ObjectLoader: Call onError() if no metadata found in JSON. Nov 2, 2022
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

Successfully merging this pull request may close these issues.

2 participants