Release 2.4.2

Author: Fuchs

CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog 
 
+## 2.4.2 (2017-01-30) 
+
+* Existing action commands can now be edited
+* Fix a bug that prevented weekly schedules to be filled correctly
+* Fix a bug that prevented action editor from loading an "off" state correctly
+* Workaround a plasma bug that makes configuration unavailable
+* Update tooltip as soon as data is initially available
+
 ## 2.2.0 (2017-01-28)
 
 * ActionEditor now can load existing actions, thus you can modify existing schedule commands

README.md

@@ -15,15 +15,14 @@ It allows you to
   * Set the brightness
   * Set the colour or white temperature
   * Control single lights belonging to the group
-* Actions
-  * Use some global actions, such as turning all lights on or off
-* Set an alternative connection, including http basic auth
-  * Control your lights from outside your home network 
-  * You need to set up a proxy (e.g. apache httpd) reachable from the outside
+* Use quick actions for your favourite or most used commands
+  * Create, modify and use actions, such as turning all lights off or dimming all lights of a room
 * Manage your stuff
   * Search new lights, modify and delete existing ones
   * Create new Groups and Rooms, modify and delete existing ones
   * Create new schedules, modify and delete existing ones
+* Set an alternative connection, including http basic auth
+  * Control your lights from outside your home network 
 
 ## License 
 

TODO.md

@@ -1,5 +1,6 @@
 # Nice to have features for next release (3.*)
 
-* Auto-discover Hue bridge without relying on Philips (ssdp discovery needs to be implemented, can't be done in pure qml/js)
+* Auto-discover Hue bridge without relying on Philips (ssdp discovery can't be done in pure qml/js)
 * Scenes support
 * Sensors support
+* Better error handling (unauthorized / unavailable mid-command)

package/contents/code/hue.js

@@ -20,15 +20,17 @@ var useAltConnection = false;
 var initialized = false;
 var altConnectionEnabled;
 var noConnection = false;
-var debug = false;
+var debugMode = false;
 
 // INIT
 
 function dbgPrint(msg) {
-    if(!debug) {
+    if(!debugMode) {
         return;
     }
-    print('[Hoppla-Hue] ' + msg)
+    else {
+        print('[Hoppla-Hue] ' + msg)
+    }
 }
 
 
@@ -70,6 +72,7 @@ function getHueConfigured() {
     var auth = plasmoid.configuration.authToken
     if(!base || !auth) {
         // something went terribly wrong here
+        dbgPrint("Can't access configuration");
         return false;
     }
     if (!base.trim() || !auth.trim()) {
@@ -2142,9 +2145,6 @@ function getHueIp (callback) {
     request.send();
 }
 
-function dbgPrint(msg) {
-    print('[Hoppla] ' + msg)
-}
 
 /**
  * Ignore me, I am here because extracting strings for i18n is stupid

package/contents/ui/FullRepresentation.qml

@@ -348,6 +348,8 @@ FocusScope {
 
     function fetchAllDone() {
         busyOverlay.opacity = 0;
+        // Done to initially set the tooltip
+        checkHueConnection(updatedConnection, true);
     }
 
     /**
@@ -429,7 +431,7 @@ FocusScope {
             var actionItems = JSON.parse(plasmoid.configuration.actionlist);
         }
         catch(e) {
-            debugPrint("Failed to parse actionlist json: " + json);
+            debugPrint("Failed to parse actionlist json: " + actionItems);
             return;
         }
 

package/contents/ui/configActions.qml

@@ -196,6 +196,9 @@ Item {
 
         Button {
             id: btnAddAction
+            anchors {
+                right: parent.right
+            }
             text: i18n("Add new action")
             onClicked: addAction()
         }

package/contents/ui/configGroups.qml

@@ -221,6 +221,9 @@ Item {
 
         Button {
             id: btnAddGroup
+            anchors {
+                right: parent.right
+            }
             text: i18n("Add new group")
             onClicked: addGroup()
         }

package/contents/ui/configLights.qml

@@ -192,6 +192,9 @@ Item {
 
         Button {
             id: btnAddLight
+            anchors {
+                right: parent.right
+            }
             text: i18n("Add new light")
             onClicked: addLight()
             enabled: true

package/contents/ui/configSchedules.qml

@@ -197,6 +197,9 @@ Item {
 
         Button {
             id: btnAddSchedule
+            anchors {
+                right: parent.right
+            }
             text: i18n("Add new schedule")
             onClicked: addSchedule();
         }

package/contents/ui/main.qml

@@ -25,19 +25,28 @@ import "hue.js" as Hue
 
 Item {
     id: hopplaApplet
-    property bool debug: false
+    property bool debugMode: false
 
     Plasmoid.toolTipMainText: i18n("Hue Light Control")
     Plasmoid.icon: "im-jabber"
 
     Plasmoid.compactRepresentation: CompactRepresentation { }
     Plasmoid.fullRepresentation: FullRepresentation { }
 
+    Component.onCompleted: {
+        // We need to init here, otherwise there is a strange plasma bug
+        // which can lead to plasmoid.configuration being unavailable, 
+        // thus the plasmoid failing badly.
+        initHueConfig();
+    }
+    
     function debugPrint(msg) {
-        if(!debug) {
+        if(!debugMode) {
             return;
         }
-        print('[Hoppla] ' + msg)
+        else {
+            print('[Hoppla] ' + msg)
+        }
     }
 
     function initHueConfig() {

package/metadata.desktop

@@ -18,6 +18,6 @@ X-KDE-PluginInfo-Name=ch.fuchsnet.plasma.hoppla-sa
 X-KDE-PluginInfo-Category=System Information
 X-KDE-PluginInfo-Author=Christian Loosli
 X-KDE-PluginInfo-Email[email protected]
-X-KDE-PluginInfo-Version=2.0.0
+X-KDE-PluginInfo-Version=2.4.2
 X-KDE-PluginInfo-Website=https://github.com/Fuchs/hoppla-sa
 X-KDE-PluginInfo-License=LGPL