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

Placeholder script instance uses VariantUtils::evaluate incorrectly #794

Closed
Naros opened this issue Sep 7, 2024 · 0 comments · Fixed by #795
Closed

Placeholder script instance uses VariantUtils::evaluate incorrectly #794

Naros opened this issue Sep 7, 2024 · 0 comments · Fixed by #795
Labels
bug/confirmed Bug/regression has been confirmed. kind/bug A bug or regression in expected behavior.
Milestone

Comments

@Naros
Copy link
Member

Naros commented Sep 7, 2024

Describe the bug

The OScriptPlaceHolderInstance::set uses the VariantUtils::evaluate method; however, it bases its logic on the validity result rather than the equality/in-equality result. It should be based on the operator result instead.

Expected behavior

No response

Actual behavior

No response

How to Reproduce?

No response

Godot full version

4.3.stable

Orchestrator version

2.1.1.stable / 2.0.3.stable

Additional information

No response

@Naros Naros added kind/bug A bug or regression in expected behavior. bug/confirmed Bug/regression has been confirmed. labels Sep 7, 2024
@Naros Naros added this to the 2.2 milestone Sep 7, 2024
Naros added a commit to Naros/godot-orchestrator that referenced this issue Sep 7, 2024
Naros added a commit to Naros/godot-orchestrator that referenced this issue Sep 7, 2024
@Naros Naros closed this as completed in #795 Sep 7, 2024
Naros added a commit to Naros/godot-orchestrator that referenced this issue Sep 7, 2024
Naros added a commit to Naros/godot-orchestrator that referenced this issue Sep 7, 2024
Naros added a commit to Naros/godot-orchestrator that referenced this issue Sep 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug/confirmed Bug/regression has been confirmed. kind/bug A bug or regression in expected behavior.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant