Merge pull request #40 from nicksellen/master

victor73 · victor73 · commit 3cfe13d41d4e · 2015-08-07T22:43:28.000-04:00
fix check for missing subscription
diff --git a/lib/stomp.js b/lib/stomp.js
@@ -295,7 +295,7 @@ Stomp.prototype.is_a_message = function(this_frame) {
 //
 Stomp.prototype.should_run_message_callback = function(this_frame) {
     var subscription = this._subscribed_to[this_frame.headers.destination];
-    if (this_frame.headers.destination !== null && subscription !== null) {
+    if (this_frame.headers.destination !== null && subscription) {
         if (subscription.enabled && subscription.callback !== null && typeof(subscription.callback) == 'function') {
             subscription.callback(this_frame.body, this_frame.headers);
         }

Original file line number	Diff line number	Diff line change
`@@ -295,7 +295,7 @@ Stomp.prototype.is_a_message = function(this_frame) {`
`295`	`295`	`//`
`296`	`296`	`Stomp.prototype.should_run_message_callback = function(this_frame) {`
`297`	`297`	`var subscription = this._subscribed_to[this_frame.headers.destination];`
`298`		`- if (this_frame.headers.destination !== null && subscription !== null) {`
	`298`	`+ if (this_frame.headers.destination !== null && subscription) {`
`299`	`299`	`if (subscription.enabled && subscription.callback !== null && typeof(subscription.callback) == 'function') {`
`300`	`300`	`subscription.callback(this_frame.body, this_frame.headers);`
`301`	`301`	`}`