diff options
author | Benjamin Tissoires <benjamin.tissoires@redhat.com> | 2019-01-25 12:44:27 +0100 |
---|---|---|
committer | Benjamin Tissoires <benjamin.tissoires@redhat.com> | 2019-01-25 12:44:27 +0100 |
commit | 6c3dbc3c6f52a42ddbeeaec9bbfcc82c1c839f7d (patch) | |
tree | 29834be1c70610ea3373e03c64891a6b35569fd1 /models | |
parent | b5411841cf6aa33b2c0d29d85cbc795e3faa7f4f (diff) | |
download | wekan-6c3dbc3c6f52a42ddbeeaec9bbfcc82c1c839f7d.tar.gz wekan-6c3dbc3c6f52a42ddbeeaec9bbfcc82c1c839f7d.tar.bz2 wekan-6c3dbc3c6f52a42ddbeeaec9bbfcc82c1c839f7d.zip |
api: new_card: add the card at the end of the list
If we keep the `0` value, the card might be inserted in the middle of the
list, making it hard to find it later on.
Always append the card at the end of the list by setting a sort value
based on the number of cards in the list.
Diffstat (limited to 'models')
-rw-r--r-- | models/cards.js | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/models/cards.js b/models/cards.js index 9b32e89a..ff19a9a0 100644 --- a/models/cards.js +++ b/models/cards.js @@ -1526,6 +1526,10 @@ if (Meteor.isServer) { Authentication.checkUserId(req.userId); const paramBoardId = req.params.boardId; const paramListId = req.params.listId; + const currentCards = Cards.find({ + listId: paramListId, + archived: false, + }, { sort: ['sort'] }); const check = Users.findOne({ _id: req.body.authorId, }); @@ -1538,7 +1542,7 @@ if (Meteor.isServer) { description: req.body.description, userId: req.body.authorId, swimlaneId: req.body.swimlaneId, - sort: 0, + sort: currentCards.count(), members, }); JsonRoutes.sendResult(res, { |