fix content media bug

Author: welkinwong

core/controllers/contents.controller.js

@@ -271,13 +271,13 @@ exports.update = function (req, res) {
         optional: true,
         custom: {
           options: [function (value) {
-            if (value === 'draft' || value === 'pushed' || value === 'deleted') {
+            if (value === 'draft' || value === 'pushed') {
               return true;
             } else {
               return false;
             }
           }],
-          errorMessage: 'status 需为 draft 或 pushed 或 deleted'
+          errorMessage: 'status 需为 draft 或 pushed'
         }
       },
       'deleted': {
@@ -340,7 +340,7 @@ exports.update = function (req, res) {
         },
         custom: {
           options: [function (value) {
-            if (value === 'draft' || value === 'pushed' || value === 'deleted') {
+            if (value === 'draft' || value === 'pushed') {
               return true;
             } else {
               return false;
@@ -442,11 +442,11 @@ exports.update = function (req, res) {
     data.alias = req.body.alias;
     data.date = req.body.date;
     data.thumbnail = req.body.thumbnail;
-    data.media = req.body.media;
+    data.media = req.body.media || [];
     data.abstract = req.body.abstract;
     data.content = req.body.content;
-    data.tags = req.body.tags;
-    data.extensions = req.body.extensions;
+    data.tags = req.body.tags || [];
+    data.extensions = req.body.extensions || {};
   }
 
   var query = {

core/services/contents.service.js

@@ -330,7 +330,7 @@ exports.save = function (options, callback) {
         });
       }],
       pullMedia: ['updateContent', function (callback, results) {
-        if (_.isEmpty(data.media)) data.media = [];
+        if (!data.media) return callback();
 
         var oldMedia = results.updateContent.media;
         var newMedia = data.media;
@@ -355,15 +355,15 @@ exports.save = function (options, callback) {
           _.pull(pullMedia, newThumbnail);
         }
 
-        mediaModel.update({_id: {$in: pullMedia}}, {$pull: {quotes: _id}}, {
+        mediaModel.update({ _id: { $in: pullMedia } }, { $pull: { quotes: _id } }, {
           multi: true,
           runValidators: true
         }, function (err) {
           callback(err);
         });
       }],
       addMedia: ['updateContent', function (callback, results) {
-        if (_.isEmpty(data.media)) data.media = [];
+        if (!data.media) return callback();
 
         var oldMedia = results.updateContent.media;
         var newMedia = data.media;
@@ -382,7 +382,7 @@ exports.save = function (options, callback) {
           addMedia.push(newThumbnail);
         }
 
-        mediaModel.update({_id: {$in: addMedia}}, {$addToSet: {quotes: _id}}, {
+        mediaModel.update({ _id: { $in: addMedia } }, { $addToSet: { quotes: _id } }, {
           multi: true,
           runValidators: true
         }, function (err) {
@@ -400,7 +400,7 @@ exports.save = function (options, callback) {
   } else {
     async.auto({
       checkAlias: function (callback) {
-        exports.checkAlias({alias: data.alias}, function (err, alias) {
+        exports.checkAlias({ alias: data.alias }, function (err, alias) {
           if (err) callback(err);
 
           data.alias = alias;