From 10ec1b20390ac69931e5603a1219bb421f2b280e Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Xose=20P=C3=A9rez?= <xose.perez@gmail.com>
Date: Thu, 6 Sep 2018 10:48:34 +0200
Subject: [PATCH] Fix HA names (#1183)

---
 code/espurna/homeassistant.ino | 17 ++++++++++++++---
 1 file changed, 14 insertions(+), 3 deletions(-)

diff --git a/code/espurna/homeassistant.ino b/code/espurna/homeassistant.ino
index d0b6ffe8..52fe5c6c 100644
--- a/code/espurna/homeassistant.ino
+++ b/code/espurna/homeassistant.ino
@@ -13,6 +13,17 @@ Copyright (C) 2017-2018 by Xose Pérez <xose dot perez at gmail dot com>
 bool _haEnabled = false;
 bool _haSendFlag = false;
 
+// -----------------------------------------------------------------------------
+// UTILS
+// -----------------------------------------------------------------------------
+
+String _haFixName(String name) {
+    for (unsigned char i=0; i<name.length(); i++) {
+        if (!isalnum(name.charAt(i))) name.setCharAt(i, '_');
+    }
+    return name;
+}
+
 // -----------------------------------------------------------------------------
 // SENSORS
 // -----------------------------------------------------------------------------
@@ -22,7 +33,7 @@ bool _haSendFlag = false;
 void _haSendMagnitude(unsigned char i, JsonObject& config) {
 
     unsigned char type = magnitudeType(i);
-    config["name"] = getSetting("hostname") + String(" ") + magnitudeTopic(type);
+    config["name"] = _haFixName(getSetting("hostname") + String(" ") + magnitudeTopic(type));
     config.set("platform", "mqtt");
     config["state_topic"] = mqttTopic(magnitudeTopicIndex(i).c_str(), false);
     config["unit_of_measurement"] = magnitudeUnits(type);
@@ -64,10 +75,10 @@ void _haSendSwitch(unsigned char i, JsonObject& config) {
 
     String name = getSetting("hostname");
     if (relayCount() > 1) {
-        name += String(" #") + String(i);
+        name += String("_") + String(i);
     }
 
-    config.set("name", name);
+    config.set("name", _haFixName(name));
     config.set("platform", "mqtt");
 
     if (relayCount()) {