Merge pull request #39 from t27/master

fix to make the URL and data keys optional during update
edwardhotchkiss · Oct 23, 2015 · 9d4796b · 9d4796b
2 parents 1a17c14 + bcc823e
commit 9d4796b
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/lib/short.js b/lib/short.js
@@ -90,8 +90,13 @@ exports.retrieve = function(hash) {
 exports.update = function(hash, updates) {
   var promise = new Promise();
   ShortURL.findOne({hash: hash}, function(err, doc) {
-    doc.URL = updates.URL;
-    doc.data = extend(doc.data, updates.data);
+    if (updates.URL) {
+      doc.URL = updates.URL;
+    }
+    if (updates.data) {
+      doc.data = extend(doc.data, updates.data);
+      doc.markModified('data'); //Required by mongoose, as data is of Mixed type
+    }
     doc.save(function(err, updatedObj, numAffected) {
       if (err) {
         promise.reject(new Error('MongoDB - Cannot save updates'), true);