Accounting for if docs exist before calling 'dataByKey'

History.md

@@ -1,5 +1,10 @@
 # Change History
 
+## v1.3.9 2022-06-27
+
+- Fixed errors being thrown when unsetting if documents do not exist
+- Ref: https://github.com/robfallows/tunguska-reactive-aggregate/issues/61
+
 ## v1.3.8 2022-06-06
 
 - Added [options.specificWarnings](https://github.com/robfallows/tunguska-reactive-aggregate/pull/65)

aggregate.js

@@ -250,7 +250,9 @@ export const ReactiveAggregate = (sub, collection = null, pipeline = [], options
         } else {
           if (sub._session.collectionViews instanceof Map) {
             // Since the pipeline fields might have been removed, we need to find the differences and define them as 'undefined' so the sub removes them.
-            const previousFields = [...sub._session.collectionViews.get(localOptions.clientCollection).documents.get(doc_id).dataByKey.keys()];
+
+            const previousDocs = sub._session.collectionViews.get(localOptions.clientCollection).documents;
+            const previousFields = [previousDocs && previousDocs.get(doc_id).dataByKey.keys()];
             previousFields.forEach(field => {
               // At this point they are undefined because they no longer exist in the new doc, they're not literally set as undefined
               if (doc[field] === undefined) {

package.js

@@ -1,6 +1,6 @@
 Package.describe({
   name: 'tunguska:reactive-aggregate',
-  version: '1.3.8',
+  version: '1.3.9',
   summary: 'Publish aggregations reactively',
   git: 'https://github.com/robfallows/tunguska-reactive-aggregate',
   documentation: 'README.md'