From 36f4b99638af24fc9d64563f4111ec8c13243cae Mon Sep 17 00:00:00 2001 From: David Snopek Date: Fri, 9 Jun 2023 12:30:39 -0500 Subject: [PATCH] Fix wrapping Object's in GDExtension that aren't exposed --- core/object/object.cpp | 24 ++++++++++++++++++++++++ core/object/object.h | 12 +----------- 2 files changed, 25 insertions(+), 11 deletions(-) diff --git a/core/object/object.cpp b/core/object/object.cpp index d6937539c776..c76188a2cd32 100644 --- a/core/object/object.cpp +++ b/core/object/object.cpp @@ -1719,6 +1719,30 @@ uint32_t Object::get_edited_version() const { } #endif +StringName Object::get_class_name_for_extension(const GDExtension *p_library) const { + // Only return the class name per the extension if it matches the given p_library. + if (_extension && _extension->library == p_library) { + return _extension->class_name; + } + + // Extensions only have wrapper classes for classes exposed in ClassDB. + const StringName *class_name = _get_class_namev(); + if (ClassDB::is_class_exposed(*class_name)) { + return *class_name; + } + + // Find the nearest parent class that's exposed. + StringName parent_class = ClassDB::get_parent_class(*class_name); + while (parent_class != StringName()) { + if (ClassDB::is_class_exposed(parent_class)) { + return parent_class; + } + parent_class = ClassDB::get_parent_class(parent_class); + } + + return SNAME("Object"); +} + void Object::set_instance_binding(void *p_token, void *p_binding, const GDExtensionInstanceBindingCallbacks *p_callbacks) { // This is only meant to be used on creation by the binder. ERR_FAIL_COND(_instance_bindings != nullptr); diff --git a/core/object/object.h b/core/object/object.h index 6f626b0ed0f7..a3e9d025ea8e 100644 --- a/core/object/object.h +++ b/core/object/object.h @@ -800,17 +800,7 @@ class Object { return *_class_name_ptr; } - _FORCE_INLINE_ const StringName &get_class_name_for_extension(const GDExtension *p_library) const { - // Only return the class name per the extension if it matches the given p_library. - if (_extension && _extension->library == p_library) { - return _extension->class_name; - } - // Otherwise, return the name of the built-in class. - if (unlikely(!_class_name_ptr)) { - return *_get_class_namev(); - } - return *_class_name_ptr; - } + StringName get_class_name_for_extension(const GDExtension *p_library) const; /* IAPI */