Merge pull request #501 from derTuca/add-null-callback-option

Added null check for updateCallback on query

Author: EddyVerbruggen

firebase.android.js

@@ -1677,12 +1677,12 @@ firebase.query = function (updateCallback, path, options) {
         var listener = new com.google.firebase.database.ValueEventListener({
           onDataChange: function (snapshot) {
             var data = firebase.getCallbackData('ValueChanged', snapshot);
-            updateCallback(data);
+            if(updateCallback) updateCallback(data);
             // resolve promise with data in case of single event, see https://github.com/EddyVerbruggen/nativescript-plugin-firebase/issues/126
             resolve(data);
           },
           onCancelled: function (databaseError) {
-            updateCallback({
+            if(updateCallback) updateCallback({
               error: databaseError.getMessage()
             });
             // see comment at 'onDataChange'

firebase.ios.js

@@ -1728,7 +1728,7 @@ firebase.query = function (updateCallback, path, options) {
 
       if (options.singleEvent) {
         query.observeSingleEventOfTypeWithBlock(FIRDataEventType.FIRDataEventTypeValue, function (snapshot) {
-          updateCallback(firebase.getCallbackData('ValueChanged', snapshot));
+          if(updateCallback) updateCallback(firebase.getCallbackData('ValueChanged', snapshot));
           // resolve promise with data in case of single event, see https://github.com/EddyVerbruggen/nativescript-plugin-firebase/issues/126
           resolve(firebase.getCallbackData('ValueChanged', snapshot));
         });