v6.0.9

seydx · Jun 27, 2021 · 87ad14e · 87ad14e
1 parent fdb414e
commit 87ad14e
Show file tree

Hide file tree

Showing 5 changed files with 24 additions and 6 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,10 @@
 # Changelog
 
+# v6.0.9 - 2021-06-28
+
+## Other Changes
+- Fixed an issue where changing lightbulb color didnt work
+
 # v6.0.8 - 2021-06-27
 
 ## Other Changes

diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "homebridge-fritz-platform",
-  "version": "6.0.8",
+  "version": "6.0.9",
   "description": "Homebridge Plugin to control FritzBox router, smarthome devices and more.",
   "main": "index.js",
   "funding": [

diff --git a/src/accessories/smarthome/smarthome.handler.js b/src/accessories/smarthome/smarthome.handler.js
@@ -1217,9 +1217,15 @@ class Handler {
                 }
               } else {
                 //color
-                let validHueSat = getValidColor(accessory, false, state);
+                //state = saturation
+                const hue = accessory
+                  .getService(this.api.hap.Service.Lightbulb)
+                  .getCharacteristic(this.api.hap.Characteristic.Hue).value;
+
+                const validHueSat = getValidColor(hue, state);
+
                 logger.info(
-                  `Hue ${validHueSat.hue} - Saturation ${validHueSat.sat}`,
+                  `Hue ${hue} (${validHueSat.hue}) - Saturation ${state} (${validHueSat.sat})`,
                   `${accessory.displayName} (${subtype})`
                 );
 
@@ -1337,9 +1343,15 @@ class Handler {
                 }
               } else {
                 //color
-                let validHueSat = getValidColor(accessory, false, state);
+                //state = saturation
+                const hue = accessory
+                  .getService(this.api.hap.Service.Lightbulb)
+                  .getCharacteristic(this.api.hap.Characteristic.Hue).value;
+
+                const validHueSat = getValidColor(hue, state);
+
                 logger.info(
-                  `Hue ${validHueSat.hue} - Saturation ${validHueSat.sat}`,
+                  `Hue ${hue} (${validHueSat.hue}) - Saturation ${state} (${validHueSat.sat})`,
                   `${accessory.displayName} (${subtype})`
                 );
 

diff --git a/src/accessories/smarthome/smarthome.utils.js b/src/accessories/smarthome/smarthome.utils.js
@@ -13,6 +13,7 @@ exports.colortemp2api = (param) => {
 };
 
 exports.getValidColor = (hue, sat) => {
+  hue = Math.round(hue);
   sat = Math.round(sat);
 
   let hues = [35, 52, 92, 120, 160, 195, 212, 225, 266, 296, 335, 358];