Fix checkbox search widget wrapper for boolean field types

Fixes #60152
qgis · Feb 12, 2025 · bd53243 · bd53243
1 parent ea0e8f9
commit bd53243
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 1 deletion.
diff --git a/src/gui/editorwidgets/qgscheckboxsearchwidgetwrapper.cpp b/src/gui/editorwidgets/qgscheckboxsearchwidgetwrapper.cpp
@@ -43,8 +43,19 @@ QVariant QgsCheckboxSearchWidgetWrapper::value() const
 {
   QVariant v;
 
+  const QMetaType::Type fieldType = layer()->fields().at( mFieldIdx ).type();
+
   if ( mCheckBox )
-    v = mCheckBox->isChecked() ? config( QStringLiteral( "CheckedState" ), true ) : config( QStringLiteral( "UncheckedState" ), false );
+  {
+    if ( fieldType == QMetaType::Type::Bool )
+    {
+      v = mCheckBox->isChecked();
+    }
+    else
+    {
+      v = mCheckBox->isChecked() ? config( QStringLiteral( "CheckedState" ), true ) : config( QStringLiteral( "UncheckedState" ), false );
+    }
+  }
 
   return v;
 }

diff --git a/tests/src/python/test_qgssearchwidgetwrapper.py b/tests/src/python/test_qgssearchwidgetwrapper.py
@@ -518,6 +518,9 @@ def testCreateExpression(self):
         parent = QWidget()
         w = QgsCheckboxSearchWidgetWrapper(layer, 2)
         w.initWidget(parent)
+        # boolean fields should ignore CheckedState/UncheckedState config
+        config = {"CheckedState": 5, "UncheckedState": 9}
+        w.setConfig(config)
         c = w.widget()
         c.setChecked(True)
         self.assertEqual(