fix: error handling on collections

Author: codediodeio

.gitignore

@@ -3,4 +3,7 @@ node_modules
 yarn.lock
 package-lock.json
 index.js
-index.mjs
+index.mjs
+dist/
+firebase-debug.log
+firestore-debug.log

example/.gitignore

@@ -1,3 +1,5 @@
 .DS_Store
 node_modules
 public/bundle.*
+firestore-debug.log
+firebase-debug.log

example/firebase-debug.log

@@ -1,5 +1,5 @@
 {
-  "version": "0.0.3",
+  "version": "0.0.4",
   "name": "sveltefire",
   "svelte": "src/index.js",
   "main": "dist/index.js",
@@ -17,6 +17,7 @@
     "type": "git",
     "url": "https://github.com/codediodeio/sveltefire.git"
   },
+  "publishConfig": { "registry": "https://npm.pkg.github.com/" },
   "devDependencies": {
     "concurrently": "^5.0.0",
     "cypress": "^3.6.1",

example/firestore-debug.log

@@ -110,29 +110,28 @@ export function collectionStore(path, queryFn, opts) {
 
     _teardown = (query || ref).onSnapshot(
       snapshot => {
+
+        // Will always return an array
         const data = snapshot.docs.map(docSnap => ({
           ...docSnap.data(),
           // Allow end user override fields mapped for ID and Ref
           ...(idField ? { [idField]: docSnap.id } : null),
           ...(refField ? { [refField]: docSnap.ref } : null)
         }));
-        const len = data.length;
+
         if (log) {
           const type = _loading ? 'New Query' : 'Updated Query';
-          console.groupCollapsed(`${type} ${ref.id} | ${len} hits`);
+          console.groupCollapsed(`${type} ${ref.id} | ${data.length} hits`);
           console.log(`${ref.path}`);
           console.log(`Snapshot: `, snapshot);
           console.groupEnd();
         }
-
-        _error = null;
         next(data);
       },
 
       error => {
         console.error(error);
-        _error = error;
-        next(null);
+        next(null, error);
       }
     );