Merge pull request #2839 from HSLdevcom/minor-mqtt-fix

Don't unsubscribe unless there are existing topics
HSLdevcom · Jun 19, 2019 · a45f419 · a45f419
2 parents 7828de7 + 3752c4d
commit a45f419
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/app/util/mqttClient.js b/app/util/mqttClient.js
@@ -79,7 +79,9 @@ export function parseMessage(topic, message, agency) {
 export function changeTopics(settings, actionContext) {
   const { client, oldTopics } = settings;
 
-  client.unsubscribe(oldTopics);
+  if (Array.isArray(oldTopics) && oldTopics.length > 0) {
+    client.unsubscribe(oldTopics);
+  }
   // remove existing vehicles/topics
   actionContext.dispatch('RealTimeClientReset');
   const topics = settings.options.map(option => getTopic(option, settings));