Merge PR #516: Nuki: Fix bad if clause guards

bose/soundtouchtypes.h

@@ -139,7 +139,7 @@ struct ContentItemObject {
     QString source;             //Attribute. The type or name of the service playing. To determine if the product is in standby mode, check if source == STANDBY.
     QString location;           //Attribute. READ-ONLY. This attribute is used by Bose to point to the content the user is accessing. You can save it to access content later, but do not attempt to generate your own locations.
     QString sourceAccount;      //Attribute. The user-account associated with this source.
-    bool isPresetable;          //Attribute. TRUE if the source can be set as one of the six (6) SoundTouch presets.
+    bool isPresetable = false;  //Attribute. TRUE if the source can be set as one of the six (6) SoundTouch presets.
     QString itemName;           //Element. The album, station, playlist, song, phone, etc. name depending on the source.
     QString containerArt;       //URL
 };

nuki/bluez/bluetoothmanager.cpp

@@ -180,10 +180,10 @@ void BluetoothManager::processInterfaceList(const QDBusObjectPath &objectPath, c
             if (properties.contains("Service")) {
                 QDBusObjectPath serviceObjectPath = qvariant_cast<QDBusObjectPath>(properties.value("Service"));
                 BluetoothGattService *service = findService(serviceObjectPath);
-                if (service)
+                if (service) {
                     qCDebug(dcBluez()) << "Add characteristic" << serviceObjectPath.path() << properties << service;
                     service->addCharacteristicInternally(objectPath, properties);
-
+                }
             }
         }
     }
@@ -196,10 +196,10 @@ void BluetoothManager::processInterfaceList(const QDBusObjectPath &objectPath, c
             if (properties.contains("Characteristic")) {
                 QDBusObjectPath characterisitcObjectPath = qvariant_cast<QDBusObjectPath>(properties.value("Characteristic"));
                 BluetoothGattCharacteristic *characteristic = findCharacteristic(characterisitcObjectPath);
-                if (characteristic)
+                if (characteristic) {
                     qCDebug(dcBluez()) << "Add descriptor" << characterisitcObjectPath.path() << properties << characteristic;
                     characteristic->addDescriptorInternally(objectPath, properties);
-
+                }
             }
         }
     }

sonos/integrationpluginsonos.cpp

@@ -151,7 +151,6 @@ void IntegrationPluginSonos::confirmPairing(ThingPairingInfo *info, const QStrin
         if (!sonos) {
             qWarning(dcSonos()) << "No sonos connection found for thing:" << info->thingName();
             m_setupSonosConnections.remove(info->thingId());
-            sonos->deleteLater();
             info->finish(Thing::ThingErrorHardwareFailure);
             return;
         }

tado/tado.h

@@ -67,20 +67,20 @@ public:
     };
 
     struct ZoneState {
-        bool connected;
-        bool power;
+        bool connected = false;
+        bool power = false;
         QString tadoMode;
         QString settingType;
-        double settingTemperature;
-        bool settingPower;
-        double temperature;
-        double humidity;
-        bool windowOpen;
-        double heatingPowerPercentage;
+        double settingTemperature = false;
+        bool settingPower = false;
+        double temperature = false;
+        double humidity = false;
+        bool windowOpen = false;
+        double heatingPowerPercentage = false;
         QString heatingPowerType;
-        bool overlayIsSet;
-        bool overlaySettingPower;
-        double overlaySettingTemperature;
+        bool overlayIsSet = false;
+        bool overlaySettingPower = false;
+        double overlaySettingTemperature = false;
         QString overlayType;
     };