Merge pull request #44 from firebase/jw-geofire-302

Fixed Node.js bug

Author: Jacob Wenger

build/header

@@ -12,9 +12,6 @@
 
 // Include RSVP if this is being run in node
 if (typeof module !== "undefined" && typeof process !== "undefined") {
-  // We need this `Firebase || require("firebase")` hack to get GeoFire to work properly in Node.js
-  // since otherwise, checking for `instanceof Firebase` fails.
-  var Firebase = Firebase || require("firebase");
   var RSVP = require("rsvp");
 }
 

changelog.txt

@@ -0,0 +1 @@
+fixed - Fixed typo in 3.0.1 release which caused it to be unusable.

examples/fish1/js/vendor/geofire.min.js

@@ -99,7 +99,7 @@ var GeoFire = function(firebaseRef) {
   /*****************/
   /*  CONSTRUCTOR  */
   /*****************/
-  if (firebaseRef instanceof Firebase === false) {
+  if (Object.prototype.toString.call(firebaseRef) !== "[object Object]") {
     throw new Error("firebaseRef must be an instance of Firebase");
   }
 

examples/fish2/js/vendor/geofire.min.js

@@ -489,7 +489,7 @@ var GeoQuery = function (firebaseRef, queryCriteria) {
   /*  CONSTRUCTOR  */
   /*****************/
   // Firebase reference of the GeoFire which created this query
-  if (firebaseRef instanceof Firebase === false) {
+  if (Object.prototype.toString.call(firebaseRef) !== "[object Object]") {
     throw new Error("firebaseRef must be an instance of Firebase");
   }
   var _firebaseRef = firebaseRef;