godot-clyde-dialogue icon indicating copy to clipboard operation
godot-clyde-dialogue copied to clipboard

ClydeDialogueResourceLoader parse error with load virtual function on Godot 3.6

Open Riordan-DC opened this issue 1 year ago • 1 comments

Hi,

Love this plugin! Amazing work. It's small, maintainable, self contained. Perfect.

I've noticed that in the ClydeDialogueResourceLoader you will receive a parse error if using Godot 3.6 because the load virtual function requires an additional parameter.

Currently:

func load(path, original_path):
	return ResourceLoader.load(path)

To fix:

func load(path, original_path, use_sub_res_cache):
	return ResourceLoader.load(path)

You can see they have changed it in the docs: https://docs.godotengine.org/en/3.5/classes/class_resourceformatloader.html

https://docs.godotengine.org/en/3.6/classes/class_resourceformatloader.html

Not sure if it will still override the virtual function in 3.5 if the parameter is added.

Riordan-DC avatar Feb 19 '24 20:02 Riordan-DC

Thanks @Riordan-DC . Yeah, the override needs to match the parent signature, so I guess this will be a breaking change in 3.6. I'll keep this noted till the release candidate. Cheers

viniciusgerevini avatar Feb 24 '24 15:02 viniciusgerevini