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

Signal binding in editor to C# method with extra call arguments invalid #84256

Closed
aogden opened this issue Oct 31, 2023 · 4 comments
Closed

Signal binding in editor to C# method with extra call arguments invalid #84256

aogden opened this issue Oct 31, 2023 · 4 comments

Comments

@aogden
Copy link

aogden commented Oct 31, 2023

Godot version

v4.1.2.stable.mono.official [399c9dc]

System information

MacOS Ventura 13.6

Issue description

Incompatibility trying to attach a signal to a C# receiver method with extra call arguments. Receiver method will not show up in the "select a method" picker box. Method name can be manually entered and functions properly but signal connections occasionally lost by reimport/scene cleanup.
image
image

Steps to reproduce

Reproduction project included. Several buttons with signal connections to the UI.cs script that has a method public void OnButtonPress(int index). Signal connection uses extra call arguments to pass index into method but method name not populated in method picker.

Minimal reproduction project

GodotSignalBindingBug.zip

@zaevi
Copy link
Contributor

zaevi commented Nov 1, 2023

Not available for GDScript too. Seems the picker don't consider the extra arguments.

@haslingerm
Copy link

Not available for GDScript too. Seems the picker don't consider the extra arguments.

If that is true then maybe title and labels should be updated?
I just ran into this issue today and think it deserves higher priority, because it is a clear blocker for a common use case. And C# only issue are treated as second-class compared to an issue that affect everyone.

@os-architect
Copy link

os-architect commented Sep 2, 2024

still happening. I'm new to Godot, are there any workarounds today?

@raulsntos
Copy link
Member

@os-architect The fix is included in the 4.3 release.

@raulsntos raulsntos added this to the 4.3 milestone Sep 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants