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.

Sign up for GitHub

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

Jump to bottom

glTF importer does not check for invalid property name on blendshapes #94740

Closed
TokageItLab opened this issue Jul 25, 2024 · 0 comments · Fixed by #94783
Closed

glTF importer does not check for invalid property name on blendshapes #94740

TokageItLab opened this issue Jul 25, 2024 · 0 comments · Fixed by #94783
Labels
bug topic:animation topic:import topic:3d
Milestone
4.4

Comments

@TokageItLab
Copy link
Member

TokageItLab commented Jul 25, 2024 •
edited
Loading

Tested versions

4.1, 4.2, 4.3

System information

Any

Issue description

Colon : is a reserved word used for subpath separators and type annotation. It never possible to have trackpath and it will fail to create a blendshape track, cannot be animating.

image

Steps to reproduce

Import glTF contain : character as blendshape name

Minimal reproduction project (MRP)

blend_shape_test.zip
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug topic:animation topic:import topic:3d
Projects
None yet
Milestone
4.4
Development

Successfully merging a pull request may close this issue.

Add validation to glTF importer for Blendshape and Animation TokageItLab/godot
2 participants
@akien-mga @TokageItLab