Updating countInStore to fix issues with passing params incorrectly. Also added the ability to pass query parameters to both count() and countInStore()

Author: tylermmorton

docs/model-classes.md

@@ -85,15 +85,15 @@ created () {
 }
 ```
 
-### count()
+### count(params)
 
-Model classes have a `count` method, which is a proxy to the [`find` action] with the param `$limit: 0` (./service-plugin.html#find-params). <Badge text="3.10.5+" />
+Model classes have a `count` method, which is a proxy to the [`find` action] with the param `$limit: 0` automaticalluy appended. On the Feathers server, `$limit: 0` results in a fast count query. (./service-plugin.html#find-params). <Badge text="3.10.5+" />
 
 ```js
 // In your Vue component
 async created () {
   const { Todo } = this.$FeathersVuex.api
-  const todosCount = await Todo.count()
+  const todosCount = await Todo.count({ query: { priority: 'critical' }})
   // or
   Todo.count().then((total) => { this.todoCount = total })
 }
@@ -107,7 +107,7 @@ Model classes have a `countInStore` method, which is a proxy to the [`count` get
 // In your Vue component
 created () {
   const { Todo } = this.$FeathersVuex.api
-  const todosCount = Todo.countInStore()
+  const todosCount = Todo.countInStore({ query: { priority: 'critical' }})
 }
 ```
 

src/service-module/make-base-model.ts

@@ -176,17 +176,18 @@ export default function makeBaseModel(options: FeathersVuexOptions) {
       return this._getters('find', params)
     }
 
-    public static count() {
-      const params = {
-        $limit: 0 // <- limit 0 in feathers is a fast count query
+    public static count(params) {
+      params = params || {
+        query: {} 
       }
+      params.query.$limit = 0; // <- limit 0 in feathers is a fast count query
       return this._dispatch('find', params).then((res) => {
         return res.total
       })
     }
 
-    public static countInStore() {
-      return this._getters('count')
+    public static countInStore(params) {
+      return this._getters('count', params)
     }
 
     public static get(id, params) {

src/service-module/service-module.getters.ts

@@ -21,8 +21,34 @@ export default function makeServiceGetters() {
     list(state) {
       return state.ids.map(id => state.keyedById[id])
     },
-    count: state => () => {
-      return state.ids.length;
+    count: state => params => {
+      if (isRef(params)) {
+        params = params.value
+      }
+      params = { ...params } || {}
+
+      // Set params.temps to true to include the tempsById records
+      params.temps = params.hasOwnProperty('temps') ? params.temps : false
+
+      const { paramsForServer, whitelist, keyedById } = state
+      const q = _omit(params.query || {}, paramsForServer)
+      const customOperators = Object.keys(q).filter(
+        k => k[0] === '$' && !defaultOps.includes(k)
+      )
+      const cleanQuery = _omit(q, customOperators)
+
+      const { query, filters } = filterQuery(cleanQuery, {
+        operators: additionalOperators.concat(whitelist)
+      })
+      let values = _.values(keyedById)
+
+      if (params.temps) {
+        values = values.concat(_.values(state.tempsById))
+      }
+
+      values = values.filter(sift(query))
+
+      return values.length
     },
     find: state => params => {
       if (isRef(params)) {