Merge pull request #64 from tonylukasavage/issue-59

fix issue with falsy attribute values
SeyZ · Feb 26, 2016 · 9de6a80 · 9de6a80
2 parents 927c31a + 1bc3dea
commit 9de6a80
Show file tree

Hide file tree

Showing 2 changed files with 27 additions and 1 deletion.
diff --git a/lib/serializer-utils.js b/lib/serializer-utils.js
@@ -233,7 +233,7 @@ module.exports = function (collectionName, record, payload, opts) {
     }
 
     _.each(opts.attributes, function (attribute) {
-      if (record[attribute]) {
+      if (attribute in record) {
         if (!data.attributes) { data.attributes = {}; }
         that.serialize(data, record, attribute, opts[attribute]);
       } else {

diff --git a/test/serializer.js b/test/serializer.js
@@ -1662,4 +1662,30 @@ describe('JSON API Serializer', function () {
       expect(json.data.attributes).to.have.property('last-name');
     });
   });
+
+  describe('falsy attribute values', function() {
+    it('properly attach falsy attributes', function() {
+      var dataSet = {
+        id: '1',
+        count: 0,
+        bool: false,
+        dbNull: null,
+        emptyString: ''
+      };
+
+      var json = new JSONAPISerializer('tester', {
+        attributes: ['count', 'bool', 'dbNull', 'emptyString']
+      }).serialize(dataSet);
+
+      expect(json.data.attributes).to.have.property('count');
+      expect(json.data.attributes.count).to.equal(0);
+      expect(json.data.attributes).to.have.property('bool');
+      expect(json.data.attributes.bool).to.equal(false);
+      expect(json.data.attributes).to.have.property('db-null');
+      expect(json.data.attributes['db-null']).to.equal(null);
+      expect(json.data.attributes).to.have.property('empty-string');
+      expect(json.data.attributes['empty-string']).to.equal('');
+    });
+  });
+
 });