From 9cf3b3bdd650b9673af81b9462bd37f96df91df5 Mon Sep 17 00:00:00 2001 From: faanskit Date: Thu, 20 May 2021 07:24:22 +0200 Subject: [PATCH] Fix for issue discussed im #198. Scene device triggers were not unique from Scenes and device_trigger /config topic was too long --- plejd/MqttClient.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/plejd/MqttClient.js b/plejd/MqttClient.js index 47c7d92..5be9434 100644 --- a/plejd/MqttClient.js +++ b/plejd/MqttClient.js @@ -36,7 +36,7 @@ const getTopicName = ( ) => `${getBaseTopic(uniqueId, mqttDeviceType)}/${topicType}`; const getButtonEventTopic = (/** @type {string} */ deviceId) => `${getTopicName(deviceId, MQTT_TYPES.DEVICE_AUTOMATION, TOPIC_TYPES.STATE)}`; -const getTriggerUniqueId = (/** @type { string } */ uniqueId) => `${uniqueId}_trigger`; +const getTriggerUniqueId = (/** @type { string } */ uniqueId) => `${uniqueId}_trig`; const getSceneEventTopic = (/** @type {string} */ sceneId) => `${getTopicName(getTriggerUniqueId(sceneId), MQTT_TYPES.DEVICE_AUTOMATION, TOPIC_TYPES.STATE)}`; const getSubscribePath = () => `${discoveryPrefix}/+/${nodeId}/#`; @@ -110,13 +110,13 @@ const getSceneDeviceTriggerhDiscoveryPayload = ( /** @type {import('./types/DeviceRegistry').OutputDevice} */ sceneDevice, ) => ({ automation_type: 'trigger', - '~': getBaseTopic(sceneDevice.uniqueId, MQTT_TYPES.DEVICE_AUTOMATION), + '~': getBaseTopic(`${sceneDevice.uniqueId}_trig`, MQTT_TYPES.DEVICE_AUTOMATION), qos: 1, topic: `~/${TOPIC_TYPES.STATE}`, type: 'scene', subtype: 'trigger', device: { - identifiers: `${sceneDevice.uniqueId}`, + identifiers: `${sceneDevice.uniqueId}_trigger`, manufacturer: 'Plejd', model: sceneDevice.typeName, name: sceneDevice.name,