Expose soft body pin methods to GDScript

[bengtsts]

This change was merged into master #51364 but requires class name changes for 3.x in code and documentation.
Node3D changed to Spatial and SoftBody3D to SoftBody.

[pouleyKetchoupp]

Looks good! I've just re-run the linux job, I don't think it failed for a good reason.

[pouleyKetchoupp]

My bad, there's a legitimate error in mono:

/home/runner/work/godot/godot/modules/mono/glue/GodotSharp/GodotSharp/Generated/GodotObjects/SoftBody.cs(634,74): error CS1750: A value of type 'string' cannot be used as a default parameter because there are no standard conversions to type 'NodePath' [/home/runner/work/godot/godot/modules/mono/glue/GodotSharp/GodotSharp/GodotSharp.csproj]

I'm not sure why this is happening. Maybe something wrong with an empty NodePath as default parameter.

[pouleyKetchoupp]

I've just tested locally, it looks like it could be a bug in mono (maybe fixed only on master since this error happens on 3.x only).

The generated mono glue for the new exposed method is:

[DebuggerBrowsable(DebuggerBrowsableState.Never)]
private static IntPtr method_bind_34 = Object.godot_icall_Object_ClassDB_get_method(nativeName, "set_point_pinned");

[GodotMethod("set_point_pinned")]
public void SetPointPinned(int pointIndex, bool pinned, NodePath attachmentPath = "")
{
    NativeCalls.godot_icall_3_802(method_bind_34, Object.GetPtr(this), pointIndex, pinned, NodePath.GetPtr(attachmentPath));
}

Although the node path argument should be NodePath attachmentPath = NodePath() or NodePath attachmentPath = NodePath("")

@neikeq Do you know what could cause this issue?

[neikeq]

@pouleyKetchoupp

This:

godot/modules/mono/editor/bindings_generator.cpp

Lines 2578 to 2581 in ef1c509

	case Variant::STRING:
	case Variant::NODE_PATH:
	r_iarg.default_argument = "\"" + r_iarg.default_argument + "\"";
	break;

should be:

if (r_iarg.type.cname == name_cache.type_NodePath) {
	r_iarg.default_argument = "(%s)\"" + r_iarg.default_argument + "\"";
	r_iarg.def_param_mode = ArgumentInterface::NULLABLE_REF;
} else {
	CRASH_COND(r_iarg.type.cname != name_cache.type_String);
	r_iarg.default_argument = "\"" + r_iarg.default_argument + "\"";
}
break;

to match current master.

[akien-mga]

Rebasing on latest 3.x should fix the Mono build issue.

[pouleyKetchoupp]

Please squash your commits (see interactive rebase) and it should be good to go!