Update to latest jsonapi-server with parsed filter.

Update to support latest version of jsonapi-server, check that we have a
minimum version that provides the `processedFilter` property and make
use of that to avoid the duplicated effort.

Author: pmcnr-hx

lib/mongoHandler.js

@@ -6,21 +6,30 @@ var async = require("async");
 var debug = require("./debugging");
 var mongodb = require("mongodb");
 var Joi = require("joi");
+var semver = require("semver");
+
+var MIN_SERVER_VERSION = "1.10.0";
 
 var MongoStore = module.exports = function MongoStore(config) {
+  MongoStore._checkMinServerVersion();
   this._config = config;
 };
 
-
-var FILTER_OPERATORS = ["<", ">", "~", ":"];
-
-
 /**
   Handlers readiness status. This should be set to `true` once all handlers are ready to process requests.
  */
 MongoStore.prototype.ready = false;
 
 
+MongoStore._checkMinServerVersion = function() {
+  var serverVersion = require('jsonapi-server')._version;
+  if (!serverVersion) return;
+  if (semver.lt(serverVersion, MIN_SERVER_VERSION)) {
+    throw new Error("This version of jsonapi-store-mongodb requires jsonapi-server>=" + MIN_SERVER_VERSION + ".");
+  }
+};
+
+
 MongoStore._mongoUuid = function(uuid) {
   return new mongodb.Binary(uuid, mongodb.Binary.SUBTYPE_UUID);
 };
@@ -51,26 +60,9 @@ MongoStore._getRelationshipAttributeNames = function(attributes) {
 };
 
 
-MongoStore._splitFilterElement = function(filterElementStr) {
-  if (FILTER_OPERATORS.indexOf(filterElementStr[0]) !== -1) {
-    return { operator: filterElementStr[0], value: filterElementStr.substring(1) };
-  }
-  return { operator: null, value: filterElementStr };
-};
-
-
-MongoStore._filterElementToMongoExpr = function(attributeConfig, filterElementStr) {
-  var filterElement = MongoStore._splitFilterElement(filterElementStr);
+MongoStore._filterElementToMongoExpr = function(filterElement) {
   var value = filterElement.value;
-  if (!attributeConfig._settings) {  // not a relationship attribute
-    var validationResult = attributeConfig.validate(filterElement.value);
-    if (validationResult.error) return null;
-    value = validationResult.value;
-  }
   if (!filterElement.operator) return value;
-  if (["~", ":"].indexOf(filterElement.operator) !== -1 && typeof value !== "string") {
-    return null;
-  }
   var mongoExpr = {
     ">": { $gt: value },
     "<": { $lt: value },
@@ -83,24 +75,15 @@ MongoStore._filterElementToMongoExpr = function(attributeConfig, filterElementSt
 
 MongoStore.prototype._getSearchCriteria = function(request) {
   var self = this;
-  if (!request.params.filter) return { };
-
-  var criteria = Object.keys(request.params.filter).map(function(attribute) {
+  var filter = request.processedFilter;
+  if (!filter) return { };
+  var criteria = Object.keys(filter).map(function(attribute) {
+    var values = filter[attribute].map(MongoStore._filterElementToMongoExpr);
     var attributeConfig = self.resourceConfig.attributes[attribute];
-    // If the filter attribute doesn't exist, skip it
-    if (!attributeConfig) return null;
-
-    var values = request.params.filter[attribute];
     // Relationships need to be queried via .id
     if (attributeConfig._settings) {
       attribute += ".id";
-      // Filters on nested resources should be skipped
-      if (values instanceof Object) return null;
     }
-
-    values = [].concat(values); // Coerce values to an array to simplify the logic
-    var filterElemForAttrToMongoExpr = MongoStore._filterElementToMongoExpr.bind(null, attributeConfig);
-    values = values.map(filterElemForAttrToMongoExpr);
     values = values.reduce(function(mongoExpressions, mongoExpr) {
       if (mongoExpr !== null) {
         var mongoExprForAttr = { };

package.json

@@ -30,12 +30,13 @@
     "joi": "^6.10.1",
     "lodash.omitby": "^4.4.0",
     "mongodb": "^2.1.20",
+    "semver": "^5.1.0"
   },
   "devDependencies": {
-    "jsonapi-server": "1.4.0",
     "blanket": "^1.2.3",
     "coveralls": "^2.11.9",
     "eslint": "^2.11.0",
+    "jsonapi-server": "holidayextras/jsonapi-server#2ba05c6",
     "mocha": "^2.5.3",
     "mocha-lcov-reporter": "^1.2.0",
     "mocha-performance": "^0.1.1",