Merge pull request #98 from sdetweil/multiinstance

change to support multiple instances even with same url

Author: timdows

MMM-JsonTable.js

@@ -31,14 +31,14 @@ Module.register("MMM-JsonTable", {
 
   // Request node_helper to get json from url
   getJson () {
-    this.sendSocketNotification("MMM-JsonTable_GET_JSON", this.config.url);
+    this.sendSocketNotification("MMM-JsonTable_GET_JSON", {url:this.config.url,id:this.identifier});
   },
 
   socketNotificationReceived (notification, payload) {
     if (notification === "MMM-JsonTable_JSON_RESULT") {
       // Only continue if the notification came from the request we made
       // This way we can load the module more than once
-      if (payload.url === this.config.url) {
+      if (payload.id === this.identifier) {
         this.jsonData = payload.data;
         this.updateDom(this.config.animationSpeed);
       }

node_helper.js

@@ -6,24 +6,24 @@ module.exports = NodeHelper.create({
     Log.log("MMM-JsonTable helper started...");
   },
 
-  getJson (url) {
+  getJson (payload) {
     const self = this;
 
-    fetch(url)
+    fetch(payload.url)
       .then((response) => response.json())
       .then((json) => {
         // Send the json data back with the url to distinguish it on the receiving part
         self.sendSocketNotification("MMM-JsonTable_JSON_RESULT", {
-          url,
+          id:payload.id,
           data: json
         });
       });
   },
 
   // Subclass socketNotificationReceived received.
-  socketNotificationReceived (notification, url) {
+  socketNotificationReceived (notification, payload ) {
     if (notification === "MMM-JsonTable_GET_JSON") {
-      this.getJson(url);
+      this.getJson(payload);
     }
   }
 });