fix: when pagination is false, skip the execution of count

[Seitk]

Based on the documentation

If pagination is set to false, it will return all docs without adding limit condition

When we set pagination is false we do expect to all documents are returned, we can simply use the length of docs as countDocuments is considered costly in general

yarn test
yarn run v1.22.19
$ npm run build

> mongoose-paginate-v2@1.7.1 build
> babel src -d dist

Successfully compiled 3 files with Babel (226ms).
$ ./node_modules/.bin/mocha tests/*.js -R spec --ui bdd --timeout 9999999


  mongoose-paginate
(node:7214) DeprecationWarning: collection.ensureIndex is deprecated. Use createIndexes instead.
(Use `node --trace-deprecation ...` to show where the warning was created)
    ✓ promise return test
    ✓ callback test
    ✓ with page and limit
    ✓ first page with page and limit, limit > doc.length
    ✓ first page with page and limit, limit > doc.length, offset by one
    ✓ first page with page and limit
    ✓ last page with page and limit
    ✓ with offset and limit (not page)
    ✓ with offset and limit (not page) condition: offset > 0 < limit
    ✓ with limit=0 (metadata only)
    ✓ with empty custom labels
    ✓ with custom labels
    ✓ with custom Meta label
    ✓ Sub documents pagination
    ✓ all data (without pagination)
    ✓ estimated count works
    ✓ count Custom Fn works
    ✓ count Custom Fn with Promise return works
    ✓ pagination=false, limit/page=undefined -> return all docs
    ✓ PaginationParameters-helper -> number type options work
    ✓ PaginationParameters-helper -> (JSON) object type options work
    ✓ PaginationParameters-helper -> boolean options work
    ✓ PaginationParameters-helper -> string options work


  23 passing (228ms)

✨  Done in 1.42s.

[aravindnc]

Looks good to me.

[Seitk]

Hi @aravindnc could you publish the fix to npm 🙏🏻

[aravindnc]

Hi @aravindnc could you publish the fix to npm 🙏🏻

@Seitk
1.7.2 is published. Sorry for the delay. And thanks for the fix.