express-toolkit 0.8.0 → 0.8.3

package/README.md

@@ -19,6 +19,7 @@ Tiny little utilities for reducing expressjs boilerplate code when building simp
   - [Default endpoints](#default-endpoints)
   - [Disable endpoints](#disable-endpoints)
   - [Sorting](#sorting)
+  - [Pagination](#pagination) 
   - [Projection](#projection)
   - [Custom primary key](#custom-primary-key)
   - [Hooks](#hooks)
@@ -237,6 +238,12 @@ For example, the following request would sort dinosaurs by age, oldest to younge
 ```http
 GET /dinosaurs?sortby=age&sortorder=DESC
 ```
+## Pagination
+GET endpoints support result pagination through `skip` and `limit` parameters:
+- `skip <Number>` tells the endpoint how many results to skip
+- `limit <Number` tells the endpoint how many results to include in the response
+
+To implement a pagination scheme, you can leverage these two parameters in the following way: Suppose you want to return `R` results per page and you want to return page number `P`, you just need to set limit to `R` and skip to `(P-1)*R`
 
 ## Projection
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "express-toolkit",
-  "version": "0.8.0",
+  "version": "0.8.3",
   "files": [
     "index.js",
     "src/"
@@ -16,6 +16,7 @@
     "test": "nyc --reporter=text ava test/*.spec.js --verbose",
     "test-ci": "npx nyc --reporter=lcov ava test/*.spec.js --verbose",
     "commit": "npx git-cz",
+    "documentation": "docsify init ./docs",
     "pre-commit": "npm run lint && npm run test",
     "changelog": "npx easy-changelog --out=./CHANGELOG.md",
     "release": "sh scripts/release.sh"
@@ -34,13 +35,13 @@
     "express"
   ],
   "dependencies": {
-    "express": "^4.17.3",
-    "json8-patch": "^1.0.4",
-    "mongoose": "^5.12.3",
+    "express": "^4.18.1",
+    "json8-patch": "^1.0.6",
+    "mongoose": "^6.5.1",
     "throwable-http-errors": "^2.1.0"
   },
   "devDependencies": {
-    "ava": "^3.8.1",
+    "ava": "^4.3.1",
     "body-parser": "^1.19.0",
     "eslint": "^7.32.0",
     "eslint-config-standard": "^16.0.3",
@@ -52,6 +53,7 @@
     "mongodb-memory-server": "^8.4.0",
     "nyc": "^15.1.0",
     "sinon": "^7.5.0",
-    "supertest": "^4.0.2"
+    "supertest": "^4.0.2",
+    "uuid": "^8.3.2"
   }
 }

package/src/controller.js

@@ -20,11 +20,13 @@ class Controller {
    * @param {String} [config.id] The attribute to use as primary key for findById, updateById and deleteById. Defaults to _id.
    * @param {Number} [config.defaultSkipValue] The default skip value to be used in find() queries
    * @param {Number} [config.defaultLimitValue] The default skip value to be used in find() queries
+   * @param {Number} [config.useUpdateOne] If set to true, updateById method will use mongoose's Model.updateOne() instead of Document.save()
    */
   constructor (config) {
     this.Model = config.model
     this.id = config.id || '_id'
     this.name = config.name
+    this.useUpdateOne = Boolean(config.useUpdateOne) || false
     this.defaultSkipValue = config.defaultSkipValue || DEFAULT_SKIP_VALUE
     this.defaultLimitValue = config.defaultLimitValue || DEFAULT_LIMIT_VALUE
     this.__hooks = {}
@@ -37,10 +39,10 @@ class Controller {
   async find (query) {
     let skip = this.defaultSkipValue
     let limit = this.defaultLimitValue
-    if (query.hasOwnProperty('limit')) {
+    if (Object.prototype.hasOwnProperty.call(query, 'limit')) {
       limit = Number(query.limit)
     }
-    if (query.hasOwnProperty('skip')) {
+    if (Object.prototype.hasOwnProperty.call(query, 'skip')) {
       skip = Number(query.skip)
     }
 
@@ -180,7 +182,7 @@ class Controller {
     const instances = await this.Model.find(query)
 
     for (const instance of instances) {
-      for (var k in update) {
+      for (const k in update) {
         instance.set(k, update[k])
       }
 
@@ -207,7 +209,7 @@ class Controller {
       throw new Errors.NotFound()
     }
 
-    for (var k in update) {
+    for (const k in update) {
       instance.set(k, update[k])
     }
 
@@ -216,7 +218,13 @@ class Controller {
       throw new Errors.BadRequest(validationError.message)
     }
 
-    return instance.save()
+    if (this.useUpdateOne) {
+      await this.Model.updateOne({ [this.id]: id }, update)
+
+      return instance.toJSON()
+    } else {
+      return instance.save()
+    }
   }
 
   /**
@@ -233,7 +241,7 @@ class Controller {
       throw new Errors.NotFound()
     }
     // Primary identifiers cannot be replaced
-    replacement['_id'] = instance._id
+    replacement._id = instance._id
     replacement[this.id] = instance[this.id]
     instance.overwrite(replacement)