Fix resource property type hint

chippmann commented 1 year ago

Fixes #394

This fixes the type hints for resource types. The issue we were having was two fold:

For once we did not check the supertypes recursively to detect whether the property was a resource type (in fact the relevant type is Reference). We also had that problem for list types. I fixed it there too as well.
Secondly, since we initially implemented this, two new type hints were added, which meant that the ordinal was not correct anymore. By adding the missing typehints, the ordinal we send to the cpp code is correct again. Note: We should investigate if we can either automatically generate these typehint enums, or automatically test whether they are still correct. But I think this is out of the scope for this fix.

piiertho commented 1 year ago

Did not see but we seem to have a problem in our tests on array

chippmann commented 1 year ago

Did not see but we seem to have a problem in our tests on array

WDYM? I don't see anything in the actions in this PR?

utopia-rise / godot-kotlin-jvm