#1, #2 and #3 issues bug fixes

Author: iyerrama29

example/index.js

@@ -14,6 +14,7 @@ function connect () {
 function find (contentType = 'blog') {
   return Stack.contentType(contentType)
     .entries()
+    .includeReferences()
     .find()
 }
 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "datasync-filesystem-sdk",
-  "version": "1.0.1",
+  "version": "1.0.2",
   "description": "JavaScript filesystem SDK to query data synced via @contentstack/datasync-content-store-filesystem",
   "main": "dist/index.js",
   "scripts": {

src/stack.ts

@@ -512,7 +512,7 @@ export class Stack {
     }
 
     if (uid && typeof uid === 'string') {
-      this.q.query[uid] = uid
+      this.q.query.uid = uid
     }
 
     return this
@@ -535,7 +535,7 @@ export class Stack {
     stack.q.isSingle = true
     stack.q.content_type_uid = stack.types.assets
     if (uid && typeof uid === 'string') {
-      stack.q.query[uid] = uid
+      stack.q.query.uid = uid
     }
 
     return stack
@@ -605,7 +605,7 @@ export class Stack {
     stack.q.isSingle = true
     stack.q.content_type_uid = stack.types.content_types
     if (uid && typeof uid === 'string') {
-      stack.q.query[uid] = uid
+      stack.q.query.uid = uid
     }
 
     return stack
@@ -726,8 +726,14 @@ export class Stack {
    */
   public tags(values) {
     if (values && typeof values === 'object' && values instanceof Array) {
-      this.q.query.tags = {
-        $in: values,
+      if (values.length === 0) {
+        this.q.query.tags = {
+          $size: 0,
+        }
+      } else {
+        this.q.query.tags = {
+          $in: values,
+        }
       }
 
       return this
@@ -1323,7 +1329,11 @@ export class Stack {
 
   private async includeReferenceIteration(eQuery: any, ctQuery: any, locale: string, include: string[], oldShelf:
     IShelf[]) {
-    if (oldShelf.length === 0 || ctQuery.$or.length === 0) {
+    if (oldShelf.length === 0) {
+      return
+    } else if (ctQuery.$or.length === 0 && eQuery.$or.length > 0) {
+      await this.bindLeftoverAssets(eQuery, locale, oldShelf)
+
       return
     }
 
@@ -1680,11 +1690,41 @@ export class Stack {
     return this.includeAllReferencesIteration(queries, ctQueries, locale, objectPointerList)
   }
 
+  private async bindLeftoverAssets(queries: IQuery, locale: string, pointerList: IShelf[]) {
+    const contents = await readFile(getAssetsPath(locale) + '.json')
+    const filteredAssets = contents.filter(sift(queries))
+
+    filteredAssets.forEach((doc) => {
+      this.projections.forEach((key) => {
+        if (doc.hasOwnProperty(key)) {
+          delete doc[key]
+        }
+      })
+    })
+
+    for (let l = 0, m = pointerList.length; l < m; l++) {
+      for (let n = 0, o = filteredAssets.length; n < o; n++) {
+        if (pointerList[l].uid === filteredAssets[n].uid) {
+          pointerList[l].path[pointerList[l].position] = filteredAssets[n]
+          break
+        }
+      }
+    }
+
+    return
+  }
+
   // tslint:disable-next-line: max-line-length
   private async includeAllReferencesIteration(oldEntryQueries: IQuery, oldCtQueries: IQuery, locale: string, oldObjectPointerList: IShelf[], depth = 0) {
-    if (depth > this.q.referenceDepth || oldObjectPointerList.length === 0 || oldCtQueries.$or.length === 0) {
+    if (depth > this.q.referenceDepth || oldObjectPointerList.length === 0) {
+      return
+    } else if (oldCtQueries.$or.length === 0 && oldObjectPointerList.length > 0 && oldEntryQueries.$or.length > 0) {
+      // its most likely only assets
+      await this.bindLeftoverAssets(oldEntryQueries, locale, oldObjectPointerList)
+
       return
     }
+
     const {
       ctQueries,
       paths,

typings/stack.d.ts

@@ -527,6 +527,7 @@ export declare class Stack {
     private fetchDocuments;
     private includeAssetsOnly;
     private bindReferences;
+    private bindLeftoverAssets;
     private includeAllReferencesIteration;
     private subIncludeAllReferencesIteration;
     private getAllReferencePaths;