Fixed error in relay identification from MQTT messages (issue #31)

CHANGELOG.md

@@ -3,6 +3,10 @@
 The format is based on [Keep a Changelog](http://keepachangelog.com/)
 and this project adheres to [Semantic Versioning](http://semver.org/).
 
+## [1.4.2] 2017-01-09
+### Fixed
+- Fixed error in relay identification from MQTT messages (issue #31)
+
 ## [1.4.1] 2017-01-05
 ### Added
 - Alexa support by default on all devices

README.md

@@ -4,7 +4,7 @@ ESPurna ("spark" in Catalan) is a custom firmware for ESP8266 based smart switch
 It was originally developed with the **[IteadStudio Sonoff](https://www.itead.cc/sonoff-wifi-wireless-switch.html)** in mind but now it supports a growing number of ESP8266-based boards.
 It uses the Arduino Core for ESP8266 framework and a number of 3rd party libraries.
 
-**Current Release Version is 1.4.1**, read the [changelog](CHANGELOG.md).
+**Current Release Version is 1.4.2**, read the [changelog](CHANGELOG.md).
 
 ## Features
 

code/espurna/config/version.h

@@ -1,4 +1,4 @@
 #define APP_NAME                "ESPurna"
-#define APP_VERSION             "1.4.1"
+#define APP_VERSION             "1.4.2"
 #define APP_AUTHOR              "xose.perez@gmail.com"
 #define APP_WEBSITE             "http://tinkerman.cat"

code/espurna/led.ino

@@ -79,8 +79,11 @@ void ledMQTTCallback(unsigned int type, const char * topic, const char * payload
         if (!t.endsWith(mqttSetter)) return;
 
         // Get led ID
-        unsigned int ledID = topic[strlen(MQTT_LED_TOPIC)+1] - '0';
-        if (ledID >= ledCount()) ledID = 0;
+        unsigned int ledID = topic[strlen(topic) - mqttSetter.length() - 1] - '0';
+        if (ledID >= ledCount()) {
+            DEBUG_MSG("[LED] Wrong ledID (%d)\n", ledID);
+            return;
+        }
 
         // get value
         unsigned int value =  (char)payload[0] - '0';

code/espurna/relay.ino

@@ -203,8 +203,11 @@ void relayMQTTCallback(unsigned int type, const char * topic, const char * paylo
         if (!t.endsWith(mqttSetter)) return;
 
         // Get relay ID
-        unsigned int relayID = topic[strlen(MQTT_RELAY_TOPIC)+1] - '0';
-        if (relayID >= relayCount()) relayID = 0;
+        unsigned int relayID = topic[strlen(topic) - mqttSetter.length() - 1] - '0';
+        if (relayID >= relayCount()) {
+            DEBUG_MSG("[RELAY] Wrong relayID (%d)\n", relayID);
+            return;
+        }
 
         // Action to perform
         unsigned int value = (char)payload[0] - '0';