Bläddra i källkod

Merge branch 'zarnifoulette-devel' into devel

Fix: REST API: Add PUT method to update a card.
Thanks to zarnifoulette ! Related to #1037
Lauri Ojansivu 8 år sedan
förälder
incheckning
9ccffe5342
2 ändrade filer med 5 tillägg och 4 borttagningar
  1. 2 1
      CHANGELOG.md
  2. 3 3
      models/cards.js

+ 2 - 1
CHANGELOG.md

@@ -2,7 +2,8 @@
 
 This release adds the following new features:
 
-* [REST API: Add PUT method to update a card](https://github.com/wekan/wekan/pull/1095).
+* [REST API: Add PUT method to update a card](https://github.com/wekan/wekan/pull/1095) and
+  [related fix](https://github.com/wekan/wekan/pull/1097).
 
 Thanks to GitHub user zarnifoulette for contributions!
 

+ 3 - 3
models/cards.js

@@ -425,17 +425,17 @@ if (Meteor.isServer) {
     const paramBoardId = req.params.boardId;
     const paramCardId = req.params.cardId;
     const paramListId = req.params.listId;
-    if(req.body.title !== undefined){
+    if(req.body.hasOwnProperty('title')){
       const newTitle = req.body.title;
       Cards.update({ _id: paramCardId, listId: paramListId, boardId: paramBoardId, archived: false },
                 {$set:{title:newTitle}});
     }
-    if(req.body.listId !== undefined){
+    if(req.body.hasOwnProperty('listId')){
       const newParamListId = req.body.listId;
       Cards.update({ _id: paramCardId, listId: paramListId, boardId: paramBoardId, archived: false },
                 {$set:{listId:newParamListId}});
     }
-    if(req.body.description !== undefined){
+    if(req.body.hasOwnProperty('description')){
       const newDescription = req.body.description;
       Cards.update({ _id: paramCardId, listId: paramListId, boardId: paramBoardId, archived: false },
                 {$set:{description:newDescription}});