From 5dcb68e3c7706718b11beb4ab9dad9ab8757917c Mon Sep 17 00:00:00 2001
From: hobbyquaker <bastiraff@googlemail.com>
Date: Sat, 12 Jan 2019 20:40:41 +0100
Subject: [PATCH] try to fix scene usage (#71)

---
 homematic-devices/hm-lc-ja1pbu-fm.js | 45 ++++++++++++++++++++++++----
 1 file changed, 40 insertions(+), 5 deletions(-)

diff --git a/homematic-devices/hm-lc-ja1pbu-fm.js b/homematic-devices/hm-lc-ja1pbu-fm.js
index 69c7aaa..c0f449c 100644
--- a/homematic-devices/hm-lc-ja1pbu-fm.js
+++ b/homematic-devices/hm-lc-ja1pbu-fm.js
@@ -2,16 +2,32 @@ const Accessory = require('./lib/accessory');
 
 module.exports = class HmLcJa1 extends Accessory {
     init(config) {
-        this.addService('WindowCovering', config.name)
+        let timeout;
+        let level;
+        let levelSlats;
+
+        const that = this;
+
+        const service = this.addService('WindowCovering', config.name);
+
+        service
             .get('CurrentPosition', config.deviceAddress + ':1.LEVEL', value => {
+                if (typeof level === 'undefined') {
+                    level = value;
+                }
                 return value * 100;
             })
 
             .get('TargetPosition', config.deviceAddress + ':1.LEVEL', value => {
                 return value * 100;
             })
-            .set('TargetPosition', config.deviceAddress + ':1.LEVEL', value => {
-                return value / 100;
+            .set('TargetPosition', (value, callback) => {
+                level = value / 100;
+                clearTimeout(timeout);
+                timeout = setTimeout(() => {
+                    setCombined();
+                }, 250);
+                callback();
             })
 
             .get('PositionState', config.deviceAddress + ':1.DIRECTION', (value, c) => {
@@ -26,14 +42,33 @@ module.exports = class HmLcJa1 extends Accessory {
             })
 
             .get('CurrentVerticalTiltAngle', config.deviceAddress + ':1.LEVEL_SLATS', value => {
+                if (typeof levelSlats === 'undefined') {
+                    levelSlats = value;
+                }
                 return (value * 180) - 90;
             })
 
             .get('TargetVerticalTiltAngle', config.deviceAddress + ':1.LEVEL_SLATS', value => {
                 return (value * 180) - 90;
             })
-            .set('TargetVerticalTiltAngle', config.deviceAddress + ':1.LEVEL_SLATS', value => {
-                return (value + 90) / 180;
+            .set('TargetVerticalTiltAngle', (value, callback) => {
+                levelSlats = (value + 90) / 180;
+                clearTimeout(timeout);
+                timeout = setTimeout(() => {
+                    setCombined();
+                }, 250);
+                callback();
+            });
+
+        function setCombined() {
+            const b1 = ('0' + ((level || 0) * 200).toString(16)).slice(-2);
+            const b2 = ('0' + ((levelSlats || 0) * 200).toString(16)).slice(-2);
+            const value = '0x' + b1 + ',0x' + b2;
+            that.ccuSetValue(config.deviceAddress + ':1.LEVEL_COMBINED', value, error => {
+                if (error) {
+                    service.updateCharacteristic('TargetPosition', error);
+                }
             });
+        }
     }
 };