Skip to content

Conversation

@Treer
Copy link

@Treer Treer commented Jan 31, 2023

also addresses #104

This PR might age like milk and need to be dropped as refinements continue in Godot 4, but today it works 😄

Godot 4.0-stable only took minor tweaks from the initial Godot 4 Beta-16 pull request

@quentincaffeino
Copy link
Owner

Thank you, definitely better than nothing

/// This is a hack until we can fix the code generated by GodotPluginsInitializerGenerator to specify it means System.Console not this Console
/// https://github.com/godotengine/godot/pull/72434
/// </summary>
public static TextWriter Error => System.Console.Error;
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Damn that was quick, the Godot devs have fixed it already!

@Treer Treer changed the title Godot 4 Beta-16 / .net6 port Godot 4.0 / .net6 port Mar 2, 2023
Treer added 2 commits January 21, 2024 14:37
FuncRefs are replaced by Callables, and there's a change to array syntax

The demo projects will still need attention, but the console works in Godot 4.
Renames call() to invoke_call() to avoid "The function signature doesn't match the parent" errors.

Since gdscript doesn't support function overloading, classes can no longer have a method named "call" because the Object class has one.
Treer added a commit to Treer/WorldWanderer that referenced this pull request Jan 21, 2024
Pulls 4.2 fixes for quentincaffeino/godot-console from quentincaffeino/godot-console#108
@Treer Treer changed the title Godot 4.0 / .net6 port Godot 4.2 / .net6 port Jan 21, 2024
@Ringo5103
Copy link

This crashes for me due to Iterator.gd still using the .call() function

commit missing changes that were supposed to be part of a7d9232
@Treer
Copy link
Author

Treer commented Mar 26, 2025

thanks, my files locally had the change but I somehow hadn't pushed it here, and wouldn't have noticed

I'm not sure whether it's due to changes in Godot version, but the console text edit field kept failing to have focus unless you clicked in it. These changes seem to be sufficient to fix that.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants