gtk/templates: Add missing traits for TemplateChild

Fixes #1842

Author: bilelmoussaoui

gtk4-macros/tests/templates.rs

@@ -303,3 +303,50 @@ glib::wrapper! {
 fn blueprint_file() {
     let _: MyWidget5 = glib::Object::new();
 }
+
+mod imp6 {
+    use super::*;
+
+    #[derive(Default, glib::Properties, gtk::CompositeTemplate)]
+    #[template(string = r#"
+<?xml version="1.0" encoding="UTF-8"?>
+<interface>
+<template class="TestWidget" parent="GtkWidget">
+<child>
+<object class="GtkWidget" id="widget" />
+</child>
+</template>
+</interface>
+"#)]
+    #[properties(wrapper_type = super::TestWidget)]
+    pub struct TestWidget {
+        #[property(get)]
+        #[template_child]
+        widget: gtk::TemplateChild<gtk::Widget>,
+    }
+
+    #[glib::object_subclass]
+    impl ObjectSubclass for TestWidget {
+        const NAME: &'static str = "TestWidget";
+        type Type = super::TestWidget;
+        type ParentType = gtk::Widget;
+
+        fn class_init(klass: &mut Self::Class) {
+            klass.bind_template();
+        }
+
+        fn instance_init(obj: &glib::subclass::InitializingObject<Self>) {
+            obj.init_template();
+        }
+    }
+
+    #[glib::derived_properties]
+    impl ObjectImpl for TestWidget {}
+    impl WidgetImpl for TestWidget {}
+    impl TestWidget {}
+}
+
+glib::wrapper! {
+    pub struct TestWidget(ObjectSubclass<imp6::TestWidget>)
+    @extends gtk::Widget;
+}

gtk4/src/subclass/widget.rs

@@ -1211,12 +1211,12 @@ pub unsafe trait WidgetClassExt: ClassStruct {
 unsafe impl<T: ClassStruct> WidgetClassExt for T where T::Type: WidgetImpl {}
 
 #[derive(Debug, PartialEq, Eq)]
-#[repr(transparent)]
 pub struct TemplateChild<T>
 where
     T: ObjectType + FromGlibPtrNone<*mut <T as ObjectType>::GlibType>,
 {
     ptr: *mut <T as ObjectType>::GlibType,
+    should_drop: bool,
 }
 
 impl<T> Default for TemplateChild<T>
@@ -1228,6 +1228,7 @@ where
 
         Self {
             ptr: std::ptr::null_mut(),
+            should_drop: false,
         }
     }
 }
@@ -1245,6 +1246,44 @@ where
     }
 }
 
+impl<T> ToValue for TemplateChild<T>
+where
+    T: ToValue + ObjectType + FromGlibPtrNone<*mut <T as ObjectType>::GlibType>,
+{
+    #[inline]
+    fn to_value(&self) -> glib::Value {
+        T::to_value(&self.get())
+    }
+
+    #[inline]
+    fn value_type(&self) -> glib::Type {
+        T::static_type()
+    }
+}
+
+impl<T> glib::value::ValueType for TemplateChild<T>
+where
+    T: glib::value::ValueType + ObjectType + FromGlibPtrNone<*mut <T as ObjectType>::GlibType>,
+{
+    type Type = <T as glib::value::ValueType>::Type;
+}
+
+unsafe impl<'a, T> glib::value::FromValue<'a> for TemplateChild<T>
+where
+    T: ObjectType + FromGlibPtrNone<*mut <T as ObjectType>::GlibType>,
+{
+    type Checker = glib::value::GenericValueTypeChecker<T>;
+
+    #[inline]
+    unsafe fn from_value(value: &'a glib::Value) -> Self {
+        skip_assert_initialized!();
+        TemplateChild {
+            ptr: T::from_value(value).into_glib_ptr(),
+            should_drop: true,
+        }
+    }
+}
+
 impl<T> std::ops::Deref for TemplateChild<T>
 where
     T: ObjectType + FromGlibPtrNone<*mut <T as ObjectType>::GlibType>,
@@ -1274,6 +1313,19 @@ where
     }
 }
 
+impl<T> Drop for TemplateChild<T>
+where
+    T: ObjectType + FromGlibPtrNone<*mut <T as ObjectType>::GlibType>,
+{
+    fn drop(&mut self) {
+        if self.should_drop {
+            unsafe {
+                crate::glib::gobject_ffi::g_object_unref(self.ptr as *mut _);
+            }
+        }
+    }
+}
+
 impl<T> TemplateChild<T>
 where
     T: ObjectType + FromGlibPtrNone<*mut <T as ObjectType>::GlibType>,