Do no expect relationships when serializing a record with no associations

Author: dynamiccast

lib/api/services/JsonApiService.js

@@ -10,6 +10,21 @@ var createRecord = require('../blueprints/create');
 var destroyOneRecord = require('../blueprints/destroy');
 var updateOneRecord = require('../blueprints/update');
 
+// Copied from http://stackoverflow.com/questions/13523951/how-to-check-the-depth-of-an-object
+var depthOf = function(object) {
+  var level = 1;
+  var key;
+  for (key in object) {
+    if (!object.hasOwnProperty(key)) continue;
+
+    if (typeof object[key] == 'object') {
+      var depth = depthOf(object[key]) + 1;
+      level = Math.max(depth, level);
+    }
+  }
+  return level;
+};
+
 module.exports = {
 
   findRecords: findRecords,
@@ -23,9 +38,10 @@ module.exports = {
     for (let name in sails.models) {
 
       let model = sails.models[name];
+      let identity = pluralize(model.identity);
       let associations = {};
 
-      sails.log.verbose("Registering model " + name + " as " + pluralize(model.identity));
+      sails.log.verbose("Registering model " + name + " as " + identity);
       model.associations.forEach((association) => {
 
         let alias = association['alias'];
@@ -37,11 +53,21 @@ module.exports = {
         };
       });
 
-      Serializer.register(pluralize(model.identity), {
+      Serializer.register(identity, {
         id: model.primaryKey || 'id',
         convertCase: this.getAttributesSerializedCaseSetting(),
         relationships: associations
       });
+
+      /*
+       * Most of the time, no relationships are provided with the record.
+       * Which is why we need a custom schema in this case
+       */
+      Serializer.register(identity, 'noRelationships-' + identity, {
+        id: model.primaryKey || 'id',
+        convertCase: this.getAttributesSerializedCaseSetting(),
+        relationships: {}
+      });
     };
   },
 
@@ -128,7 +154,12 @@ module.exports = {
 
   serialize: function(modelName, data) {
 
-    var returnedValue = Serializer.serialize(modelName, data);
+    var returnedValue = null;
+    if ((_.isArray(data) && depthOf(data) > 2) || (_.isObjectLike(data) && _.isArray(data) === false && depthOf(data) > 1)) {
+      returnedValue = Serializer.serialize(modelName, data);
+    } else {
+      returnedValue = Serializer.serialize(modelName, data, 'noRelationships-' + modelName);
+    }
 
     /*
      * To avoid the situation where many to many relationships are not described both ways