Merge pull request #563 from emiloberg/master

Fixes bug that prevented unsubscribe from WebSockets, browser side. Fixes node-red/node-red/#562
This commit is contained in:
Nick O'Leary 2015-02-15 21:54:48 +00:00
commit a808cb44c2
1 changed files with 6 additions and 6 deletions

View File

@ -95,15 +95,15 @@ RED.comms = (function() {
}
function unsubscribe(topic,callback) {
if (subscriptions.topic) {
for (var i=0;i<subscriptions.topic.length;i++) {
if (subscriptions.topic[i] === callback) {
subscriptions.topic.splice(i,1);
if (subscriptions[topic]) {
for (var i=0;i<subscriptions[topic].length;i++) {
if (subscriptions[topic][i] === callback) {
subscriptions[topic].splice(i,1);
break;
}
}
if (subscriptions.topic.length === 0) {
delete subscriptions.topic;
if (subscriptions[topic].length === 0) {
delete subscriptions[topic];
}
}
}