From 9602338ce80fd2960c3f26959133a2e7464e69e5 Mon Sep 17 00:00:00 2001
From: hjk <hjk@qt.io>
Date: Wed, 17 Mar 2021 13:08:45 +0100
Subject: [PATCH] Utils: Fix manual apply of BoolAspects

Amends b7acf45c1. The primary connection was already there, and
correctly if'd, and the patch added accidentally a second connection
which was harmless in the auto-apply case, but effectively made the
manual-apply case also auto-apply.

Change-Id: I5ee72ff4715fb0077a47f71e8d45a5b6251c4121
Reviewed-by: Christian Stenger <christian.stenger@qt.io>
---
 src/libs/utils/aspects.cpp | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/src/libs/utils/aspects.cpp b/src/libs/utils/aspects.cpp
index 7e8fadaddb1..c05c5c3679f 100644
--- a/src/libs/utils/aspects.cpp
+++ b/src/libs/utils/aspects.cpp
@@ -1225,9 +1225,6 @@ void BoolAspect::addToLayout(LayoutBuilder &builder)
         break;
     }
     d->m_checkBox->setChecked(value());
-    connect(d->m_checkBox.data(), &QAbstractButton::clicked, this, [this] {
-        setValue(d->m_checkBox->isChecked());
-    });
     if (isAutoApply()) {
         connect(d->m_checkBox.data(), &QAbstractButton::clicked,
                 this, [this](bool val) { setValue(val); });