Add support for non-plain objects

Fixes #36
substantial · Sep 15, 2015 · d2d0703 · d2d0703
1 parent 0d0c56b
commit d2d0703
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,6 +1,7 @@
 # Change Log
 
 ## [unreleased]
+* Fix support for Date objects and other non-plain objects (https://github.com/substantial/updeep/issues/36)
 
 ## [0.10.0]
 * Add support for wildcards (`*`) to `u.updateIn`. (https://github.com/substantial/updeep/issues/27)

diff --git a/lib/update.js b/lib/update.js
@@ -1,6 +1,7 @@
 import wrap from './wrap';
 import isEmpty from './util/isEmpty';
 import defaultObject from './util/defaultObject';
+import isPlainObject from 'lodash/lang/isPlainObject';
 
 function reduce(object, callback, initialValue) {
   return Object.keys(object).reduce((acc, key) => {
@@ -58,7 +59,7 @@ function update(updates, object, ...args) {
     return updates(object, ...args);
   }
 
-  if (updates === null || typeof updates !== 'object') {
+  if (!isPlainObject(updates)) {
     return updates;
   }
 

diff --git a/test/updeep-spec.js b/test/updeep-spec.js
@@ -146,4 +146,10 @@ describe('updeep', () => {
     const result = u({ a: {} }, {});
     expect(result).to.eql({ a: {} });
   });
+
+  it('works with date objects', () => {
+    const date = new Date();
+    const result = u({ created: date }, {});
+    expect(result).to.eql({ created: date });
+  });
 });