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

Animation: Fix reset value when adding new Bezier track #82012

Merged
merged 1 commit into from
Jul 8, 2024

Conversation

BrinerLovo
Copy link
Contributor

In Float and Integer types, there is no subindex – only the primary value. Currently, trying to retrieve a subindex from these types in the Variant leads to a return value of null. To address this, the proposed change ensures that the default value is returned for these types instead of attempting an invalid subindex retrieval.

@fire
Copy link
Member

fire commented Feb 12, 2024

@TokageItLab I am not sure, but can you take a look at this animation reset value functionality for bezier tracks?

@akien-mga akien-mga modified the milestones: 4.x, 4.3 Jul 8, 2024
@akien-mga akien-mga changed the title Fix Animation: reset value when adding new Bezier track Animation: Fix reset value when adding new Bezier track Jul 8, 2024
Fixes godotengine#81929.

In Float and Integer types, there is no subindex – only the primary value.
Currently, trying to retrieve a subindex from these types in the Variant leads
to a return value of null. To address this, the proposed change ensures that
the default value is returned for these types instead of attempting an invalid
subindex retrieval.
@akien-mga akien-mga merged commit 8bf4a15 into godotengine:master Jul 8, 2024
18 checks passed
@akien-mga
Copy link
Member

Thanks! And congrats for your first merged Godot contribution 🎉

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

Successfully merging this pull request may close these issues.

Animation: when adding new Bezier track, first key value is not recorded (is always 0)
5 participants