Merge pull request #2 from joelalejandro/fix/issue-1

Adds support for GET methods

Author: Joel Alejandro Villarreal Bertoldi

index.js

@@ -2,19 +2,29 @@
 const JSONAPISerializer = require('jsonapi-serializer').Serializer;
 
 function jsonapify(data, model, selfUrl) {
+  const idAttribute = Object.keys(model.attributes).filter(function(attribute) {
+    return model.attributes[attribute].primaryKey;
+  })[0];
+
+  const attributes = Object.keys(model.attributes).filter(function(attribute) { return attribute !== idAttribute; });
+
+  const idParameter = !Array.isArray(data) ? ('/' + data[idAttribute]) : '';
+
   return new JSONAPISerializer(model.name, data, {
     topLevelLinks: {
-      self: selfUrl
+      self: '/' + selfUrl + idParameter
     },
-    attributes: Object.keys(model.attributes).filter(function(attribute) {
-      return !model.attributes[attribute].primaryKey;
-    })
+    attributes: attributes
   });
 }
 
 module.exports = function (options = {}) { // eslint-disable-line no-unused-vars
   return function (hook) {
-    hook.result = jsonapify(hook.result.data, hook.service.Model, hook.path);
+    if (hook.method === 'find') {
+      hook.result = jsonapify(hook.result.data, hook.service.Model, hook.path);
+    } else if (hook.method === 'get') {
+      hook.result = jsonapify(hook.result.toJSON(), hook.service.Model, hook.path);
+    }
     return Promise.resolve(hook);
   };
 };

package.json

@@ -1,6 +1,6 @@
 {
   "name": "feathers-hooks-jsonapify",
-  "version": "0.1.0",
+  "version": "0.1.3",
   "description": "Feathers hook for outputting data in a JSON-API-compliant way.",
   "main": "index.js",
   "repository": {