feat(gen:endpoint): change PATCH to do a patch

add fast-json-patch dep

Author: Awk34

Diff for: templates/app/_package.json

@@ -30,6 +30,7 @@
     "errorhandler": "^1.4.2",
     "compression": "^1.5.2",
     "composable-middleware": "^0.3.0",
+    "fast-json-patch": "^1.0.0",
     "lodash": "^4.6.1",
     "lusca": "^1.3.0",
     "babel-runtime": "^6.6.1",

Diff for: templates/endpoint/basename.controller.js

@@ -3,13 +3,14 @@
  * GET     <%= route %>              ->  index<% if(filters.models) { %>
  * POST    <%= route %>              ->  create
  * GET     <%= route %>/:id          ->  show
- * PUT     <%= route %>/:id          ->  update
+ * PUT     <%= route %>/:id          ->  upsert
+ * PATCH   <%= route %>/:id          ->  patch
  * DELETE  <%= route %>/:id          ->  destroy<% } %>
  */
 
 'use strict';<% if(filters.models) { %>
 
-import _ from 'lodash';<% if(filters.mongooseModels) { %>
+import jsonpatch from 'fast-json-patch';<% if(filters.mongooseModels) { %>
 import <%= classedName %> from './<%= basename %>.model';<% } if(filters.sequelizeModels) { %>
 import {<%= classedName %>} from '<%= relativeRequire(config.get('registerModelsFile')) %>';<% } %>
 
@@ -22,27 +23,15 @@ function respondWithResult(res, statusCode) {
   };
 }
 
-function saveUpsert(new) {
+function patchUpdates(patches) {
   return function(entity) {
-    <%_ if(filters.mongooseModels) { -%>
-    var updated = _.assignIn(entity, updates);
-    return updated.save();
-    <%_ } -%>
-    <%_ if(filters.sequelizeModels) { -%>
-    return entity.updateAttributes(updates);
-    <%_ } -%>
-  };
-}
+    try {
+      jsonpatch.apply(entity, patches, /*validate*/ true);
+    } catch(err) {
+      return Promise.reject(err);
+    }
 
-function saveUpdates(updates) {
-  return function(entity) {
-    <%_ if(filters.mongooseModels) { -%>
-    var updated = _.merge(entity, updates);
-    return updated.save();
-    <%_ } -%>
-    <%_ if(filters.sequelizeModels) { -%>
-    return entity.updateAttributes(updates);
-    <%_ } -%>
+    return entity.save();
   };
 }
 
@@ -123,7 +112,7 @@ export function upsert(req, res) {
 }
 
 // Updates an existing <%= classedName %> in the DB
-export function update(req, res) {
+export function patch(req, res) {
   if(req.body._id) {
     delete req.body._id;
   }
@@ -134,7 +123,7 @@ export function update(req, res) {
     }
   })<% } %>
     .then(handleEntityNotFound(res))
-    .then(saveUpdates(req.body))
+    .then(patchUpdates(req.body))
     .then(respondWithResult(res))
     .catch(handleError(res));
 }

Diff for: templates/endpoint/index.js

@@ -8,8 +8,8 @@ var router = express.Router();
 router.get('/', controller.index);<% if (filters.models) { %>
 router.get('/:id', controller.show);
 router.post('/', controller.create);
-router.patch('/:id', controller.update);
 router.put('/:id', controller.upsert);
+router.patch('/:id', controller.patch);
 router.delete('/:id', controller.destroy);<% } %>
 
 module.exports = router;

Diff for: templates/endpoint/index.spec.js

@@ -7,6 +7,7 @@ var <%= cameledName %>CtrlStub = {
   show: '<%= cameledName %>Ctrl.show',
   create: '<%= cameledName %>Ctrl.create',
   upsert: '<%= cameledName %>Ctrl.upsert',
+  patch: '<%= cameledName %>Ctrl.patch',
   destroy: '<%= cameledName %>Ctrl.destroy'<% } %>
 };
 
@@ -66,9 +67,9 @@ describe('<%= classedName %> API Router:', function() {
   });
 
   describe('PATCH <%= route %>/:id', function() {
-    it('should route to <%= cameledName %>.controller.update', function() {
+    it('should route to <%= cameledName %>.controller.patch', function() {
       <%= expect() %>routerStub.patch
-        .withArgs('/:id', '<%= cameledName %>Ctrl.update')
+        .withArgs('/:id', '<%= cameledName %>Ctrl.patch')
         <%= to() %>.have.been.calledOnce;
     });
   });