-
-
Notifications
You must be signed in to change notification settings - Fork 160
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
Add get_property_list
to gdextension classes
#665
Labels
c: register
Register classes, functions and other symbols to GDScript
feature
Adds functionality to the library
Comments
lilizoey
added
feature
Adds functionality to the library
status: upstream
Depending on upstream fix (typically Godot)
c: register
Register classes, functions and other symbols to GDScript
labels
Apr 16, 2024
lilizoey
changed the title
Add
Add Apr 16, 2024
get_property_list
to gdextensionsget_property_list
to gdextension classes
Upstream PR to unblock: godotengine/godot#91179 |
Upstream has been merged. |
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
c: register
Register classes, functions and other symbols to GDScript
feature
Adds functionality to the library
To dynamically update properties in Godot you must override
get_property_list
(_get_property_list
in gdscript). This is a special-cased virtual method that doesn't show up along normal virtual methods, so we do not support this currently.This method however does require returning a pointer to a property list, and there isn't a good way to get the length back when godot asks us to free it. See godotengine/godot-proposals#9462 for an upstream issue that proposes a fix to this feature.
#621 is a PR that tried to implement this initially, however im closing that for now and opening this issue instead to track that.
Additionally
property_can_revert
andproperty_get_revert
should also be implemented so people can set custom reverts for properties.The text was updated successfully, but these errors were encountered: