Merge pull request #951 from newsiberian/fix-productGrid-load-products

Fix for the `productGrid` products loading

packages/reaction-product-variant/client/templates/products/productGrid/productGrid.js

@@ -142,7 +142,20 @@ Template.productGrid.helpers({
       return 0;
     }
 
-    let gridProducts = ReactionCore.Collections.Products.find({}).fetch();
+    // we are passing `ancestors: []`, because in case when we turn back from PDP
+    // for a moment we still subscribed to variants too, and we will get an error
+    // because of it, because our `productGrid` component can't work with variants
+    // objects.
+    //
+    // Also, we it is possible to change this selector to the following:
+    // `type: { $in: ["simple"] }`, but I found this way is not kind to package
+    // creators, because to specify they new product type, they will need to change
+    // this file, which broke another piece of compatibility with `reaction`
+    let gridProducts = ReactionCore.Collections.Products.find({
+      ancestors: []
+      // keep this, as an example
+      // type: { $in: ["simple"] }
+    }).fetch();
     const products = gridProducts.sort(compare);
     Template.instance().products = products;
     return products;