On broadcast loop over a copied array of channels

uxder · Jul 21, 2015 · b1332c9 · b1332c9
1 parent 0c2d2f2
commit b1332c9
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/radio.js b/radio.js
@@ -68,7 +68,8 @@
      *    radio('channel2').broadcast(param1, param2, param3 ... );
      */
     broadcast: function() {
-      var i, c = this.channels[this.channelName],
+      // use slice() so as to loop on a copy of channels list as it might be changed while looping
+      var i, c = this.channels[this.channelName].slice(),
         l = c.length,
         subscriber, callback, context;
       //iterate through current channel and run each subscriber

diff --git a/radio.min.js b/radio.min.js