Merge pull request #97 from jbutz/master

Unsubscribe not removing hierarchy
mroderick · Oct 5, 2016 · 4e6d888 · 4e6d888
2 parents 7382c29 + cd405ea
commit 4e6d888
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/pubsub.js b/src/pubsub.js
@@ -21,7 +21,7 @@ https://github.com/mroderick/PubSubJS
     var PubSub = {};
     root.PubSub = PubSub;
     factory(PubSub);
-    
+
 }(( typeof window === 'object' && window ) || this, function (PubSub){
 	'use strict';
 
@@ -179,7 +179,7 @@ https://github.com/mroderick/PubSubJS
 	/*Public: Clear subscriptions by the topic
 	*/
 	PubSub.clearSubscriptions = function clearSubscriptions(topic){
-		var m; 
+		var m;
 		for (m in messages){
 			if (messages.hasOwnProperty(m) && m.indexOf(topic) === 0){
 				delete messages[m];
@@ -214,7 +214,7 @@ https://github.com/mroderick/PubSubJS
 			m, message, t;
 
 		if (isTopic){
-			delete messages[value];
+			PubSub.clearSubscriptions(value);
 			return;
 		}
 

diff --git a/test/test-unsubscribe.js b/test/test-unsubscribe.js
@@ -115,7 +115,7 @@
 
 			PubSub.unsubscribe(topicB);
 
-			PubSub.publishSync(topicA, TestHelper.getUniqueString());
+			PubSub.publishSync(topicC, TestHelper.getUniqueString());
 
 			assert(spyA.called);
 			refute(spyB.called);