fix: use count or estimatedDocumentCount based on the filters applied (#78)(@imjiten)

imjiten · web-flow · commit 92fc85ff12c0 · 2022-06-30T19:38:16.000+02:00
* Update resource.ts

* Update resource.ts

* Update resource.ts

* Update resource.ts
diff --git a/src/resource.ts b/src/resource.ts
@@ -67,7 +67,10 @@ class Resource extends BaseResource {
     }
 
     async count(filters = null) {
-      return this.MongooseModel.find(convertFilter(filters)).estimatedDocumentCount()
+      if (Object.keys(convertFilter(filters)).length > 0) {
+          return this.MongooseModel.count(convertFilter(filters))
+      }
+      return this.MongooseModel.estimatedDocumentCount()
     }
 
     async find(filters = {}, { limit = 20, offset = 0, sort = {} }: FindOptions) {

Original file line number	Diff line number	Diff line change
`@@ -67,7 +67,10 @@ class Resource extends BaseResource {`
`67`	`67`	`}`
`68`	`68`
`69`	`69`	`async count(filters = null) {`
`70`		`- return this.MongooseModel.find(convertFilter(filters)).estimatedDocumentCount()`
	`70`	`+ if (Object.keys(convertFilter(filters)).length > 0) {`
	`71`	`+ return this.MongooseModel.count(convertFilter(filters))`
	`72`	`+ }`
	`73`	`+ return this.MongooseModel.estimatedDocumentCount()`
`71`	`74`	`}`
`72`	`75`
`73`	`76`	`async find(filters = {}, { limit = 20, offset = 0, sort = {} }: FindOptions) {`