Merge pull request mroderick#98 from mroderick/stop-using-jquery-error

mroderick · web-flow · commit 2c3c878dada6 · 2017-02-28T22:52:20.000+01:00
Fix mroderick#94, prefer console.error instead, fallback to $.error when present
diff --git a/wrappers/jquery/pubsub.js.post.txt b/wrappers/jquery/pubsub.js.post.txt
@@ -1,15 +1,26 @@
 
+	function reportError(message) {
+		if (typeof console !== 'undefined'){
+			console.error(new Error(errorMessage));
+			return;
+		}
+
+		if ($.error) {
+			$.error(errorMessage);
+		}
+	}
+
 	if ( $.pubsub ){
-		$.error( 'pubsub is already defined on jQuery, skipping integration of PubSubJS' );
+		reportError('pubsub is already defined on jQuery, skipping integration of PubSubJS');
 		return false;
 	}
 
 	function pubsub( method ){
 		if ( PubSub.hasOwnProperty( method ) && typeof PubSub[ method ] === 'function' ){
 			return PubSub[ method ].apply( self, Array.prototype.slice.call( arguments, 1 ) );
-		} else {
-			$.error( 'Method ' +  method + ' does not exist on jQuery.pubsub' );
 		}
+
+		reportError('Method ' +  method + ' does not exist on jQuery.pubsub');
 	}
 
 	// export into jQuery
