Fix handling of missing bin file in gltf separate #94506
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fix #85177
The missing
.bin
file case was not handled very well, because the error check after attempting to load it checked the wrong variable, so the function returnedOK
and errors popped up elsewhere.Since it's very possible for the user to miss moving the
.bin
when moving gltf separate in the editor filesystem (because it isn't visible), adding another check to more explicitly complain that the file is missing.Errors before:
Errors after: