Merge pull request #86 from digitalsadhu/add/option_to_scope_method

Add/option to scope method

Author: digitalsadhu

README.md

@@ -67,7 +67,14 @@ Example:
   "loopback-component-jsonapi": {
     "restApiRoot": "/api",
     "enable": true,
-    "handleErrors": true
+    "handleErrors": true,
+    "exclude": [
+      {"model": "comment"},
+      {"methods": "find"},
+      {"model": "post", "methods": "find"},
+      {"model": "person", "methods": ["find", "create"]}
+    ],
+    "hideIrrelevantMethods": true
   }
 }
 ```
@@ -87,6 +94,49 @@ format. Validation errors include the correct properties in order to work
 out of the box with ember.
 Default: true
 
+### exclude
+Allows blacklisting of models and methods. (See example above)
+Define an array of blacklist objects. Blacklist objects can contain "model" key
+"methods" key or both. If just "model" is defined then all methods for the
+specified model will not use jsonapi. If just the "methods" key is defined then
+all methods specified on all models will be not use jsonapi. If a combination of
+"model" and "methods" keys are used then the specific combination of model and methods
+specified will not use jsonapi.
+
+#### Please note
+The default component behavior currently is to only modify the output of the following
+methods on all models to be json api compliant:
+- find
+- create
+- updateAttributes
+- deleteById
+- findById
+- __get__.*
+- __findRelationships__.*
+
+And the default current behavior for modifying input only applies to the following methods on
+all models:
+- create
+- updateAttributes
+
+Type: array
+Default: null
+
+### hideIrrelevantMethods
+By default, loopback-component-jsonapi disables a number of methods from each endpoint
+that are not jsonapi relevant. These methods are:
+- upsert
+- exists
+- findOne
+- count
+- createChangeStream
+- updateAll
+You can use this option to reenable these methods.
+Please note, these methods will not be modified by the component and so their output
+will not be in a jsonapi compliant format.
+Type: boolean
+Default: true
+
 ## Debugging
 You can enable debug logging by setting an environment variable:
 DEBUG=loopback-component-jsonapi

lib/create.js

@@ -1,4 +1,5 @@
 var url = require('url');
+var utils = require('./utils');
 
 module.exports = function (app, options) {
   //get remote methods.
@@ -9,6 +10,10 @@ module.exports = function (app, options) {
   //register after remote method hook on all methods
   remotes.after('**', function (ctx, next) {
 
+    if (utils.shouldNotApplyJsonApi(ctx, options)) {
+      return next();
+    };
+
     //in this case we are only interested in handling create operations.
     if (ctx.method.name === 'create') {
       //JSON API specifies that created resources should have the

lib/delete.js

@@ -1,3 +1,5 @@
+var utils = require('./utils');
+
 module.exports = function (app, options) {
   //get remote methods.
   //set strong-remoting for more information
@@ -7,6 +9,10 @@ module.exports = function (app, options) {
   //register after remote method hook on all methods
   remotes.after('**', function (ctx, next) {
 
+    if (utils.shouldNotApplyJsonApi(ctx, options)) {
+      return next();
+    };
+
     //in this case we are only interested in handling create operations.
     if (ctx.method.name === 'deleteById') {
       //JSON API specifies that successful

lib/deserialize.js

@@ -3,23 +3,26 @@ var deserializer = require('./deserializer');
 var RelUtils = require('./utilities/relationship-utils');
 var utils = require('./utils');
 
-module.exports = function (app) {
+module.exports = function (app, options) {
   /**
    * Register a handler to run before all remote methods so that we can
    * transform JSON API structured JSON payload into something loopback
    * can work with.
    */
   app.remotes().before('**', function (ctx, next) {
     var data, serverRelations, errors;
+
+    if (utils.shouldNotApplyJsonApi(ctx, options)) {
+      return next();
+    };
+
     var regexs = [
-      /\.create/,
-      /prototype\.updateAttributes/,
-      /prototype\.__createRelationships__/,
-      /prototype\.__updateRelationships__/
+      /^create$/,
+      /^updateAttributes$/
     ];
 
     var matches = regexs.filter(function (regex) {
-      return ctx.methodString.match(regex);
+      return regex.test(ctx.method.name);
     });
 
     if (matches.length > 0) {

lib/headers.js

@@ -1,11 +1,17 @@
 'use strict';
 var is = require('type-is');
 var _ = require('lodash');
+var utils = require('./utils');
 
 module.exports = function (app, options) {
   var remotes = app.remotes();
 
   remotes.before('**', function (ctx, next) {
+
+    if (utils.shouldNotApplyJsonApi(ctx, options)) {
+      return next();
+    };
+
     // We must force `application/json` until we can override it through strong remoting
     if (ctx.req.accepts('application/vnd.api+json')) {
       ctx.req.headers.accept = 'application/json';

lib/relationships.js

@@ -12,6 +12,10 @@ module.exports = function (app, options) {
 
   remotes.before('**', function (ctx, next) {
 
+    if (utils.shouldNotApplyJsonApi(ctx, options)) {
+      return next();
+    };
+
     id = ctx.req.params.id;
     data = ctx.args.data;
     model = utils.getModelFromContext(ctx, app);
@@ -23,6 +27,10 @@ module.exports = function (app, options) {
   // for create
   remotes.after('**', function (ctx, next) {
 
+    if (utils.shouldNotApplyJsonApi(ctx, options)) {
+      return next();
+    };
+
     if (ctx.result) {
       id = ctx.result.id;
       data = ctx.req.body;

lib/removeRemoteMethods.js

@@ -5,9 +5,11 @@ module.exports = function (app, options) {
   //does not support.
   var isStatic = true;
 
-  Object.keys(models).forEach(function (model) {
-    ['upsert', 'exists', 'findOne', 'count', 'createChangeStream', 'updateAll'].forEach(function (method) {
-      models[model].disableRemoteMethod(method, isStatic);
+  if (options.hideIrrelevantMethods !== false) {
+    Object.keys(models).forEach(function (model) {
+      ['upsert', 'exists', 'findOne', 'count', 'createChangeStream', 'updateAll'].forEach(function (method) {
+        models[model].disableRemoteMethod(method, isStatic);
+      });
     });
-  });
+  }
 };

lib/serialize.js

@@ -3,15 +3,13 @@ var serializer = require('./serializer');
 var utils = require('./utils');
 var _ = require('lodash');
 var regexs = [
-  /\.find/,
-  /\.create/,
-  /\.deleteById/,
-  /\.findById/,
-  /prototype\.__get__/,
-  /prototype\.updateAttributes/,
-  /prototype\.__findRelationships__/,
-  /prototype\.__createRelationships__/,
-  /prototype\.__updateRelationships__/
+  /^find$/,
+  /^create$/,
+  /^deleteById$/,
+  /^findById$/,
+  /^__get__.*/,
+  /^updateAttributes$/,
+  /^__findRelationships__.*/
 ];
 
 module.exports = function (app, defaults) {
@@ -26,8 +24,12 @@ module.exports = function (app, defaults) {
       model,
       requestedIncludes;
 
+    if (utils.shouldNotApplyJsonApi(ctx, defaults)) {
+      return next();
+    };
+
     var matches = regexs.filter(function (regex) {
-      return ctx.methodString.match(regex);
+      return regex.test(ctx.method.name);
     });
 
     if (!matches.length) {

lib/update.js

@@ -1,7 +1,14 @@
+var utils = require('./utils');
+
 module.exports = function (app, options) {
   var remotes = app.remotes();
 
   remotes.after('**', function (ctx, next) {
+
+    if (utils.shouldNotApplyJsonApi(ctx, options)) {
+      return next();
+    };
+
     if (ctx.method.name === 'updateAttributes') {
       // remote links object from resource response
       delete ctx.result.links;

lib/utils.js

@@ -1,5 +1,6 @@
 var url = require('url');
 var inflection = require('inflection');
+var _ = require('lodash');
 
 /**
  * Public API
@@ -13,7 +14,8 @@ module.exports = {
   modelNameFromContext: modelNameFromContext,
   pluralForModel: pluralForModel,
   urlFromContext: urlFromContext,
-  primaryKeyForModel: primaryKeyForModel
+  primaryKeyForModel: primaryKeyForModel,
+  shouldNotApplyJsonApi: shouldNotApplyJsonApi
 };
 
 function primaryKeyForModel (model) {
@@ -164,3 +166,22 @@ function buildModelUrl (protocol, host, apiRoot, modelName, id) {
 
   return result;
 }
+
+function shouldNotApplyJsonApi (ctx, options) {
+  //handle options.exclude
+  if (!options.exclude) return false;
+
+  var modelName = ctx.method.sharedClass.name;
+  var methodName = ctx.method.name;
+  var model;
+  var methods;
+  for (var i = 0; i < options.exclude.length; i++) {
+    model = options.exclude[i].model;
+    methods = options.exclude[i].methods;
+    if (model === modelName && !methods) return true;
+    if (!model && methods === methodName) return true;
+    if (model === modelName && methods === methodName) return true;
+    if (model === modelName && _.contains(methods, methodName)) return true;
+  }
+  return false;
+}