Card edit API has duplicated variable names for board move functionality instead of using existing ones

[C0rn3j]

#5090 added ability to move cards to other boards lists but it added three new variables for Board, List and Swimlane making the API much more confusing.

move card list using listId: https://github.com/DimDz/wekan/blob/32e50b869dbbdd1fc23fcd8607ea25098208c9d9/models/cards.js#L3807-L3821

move card swimlane using swimlaneId: https://github.com/DimDz/wekan/blob/32e50b869dbbdd1fc23fcd8607ea25098208c9d9/models/cards.js#L3795-L3806

move card board using newBoardId, newSwimlaneId and newListId https://github.com/DimDz/wekan/blob/32e50b869dbbdd1fc23fcd8607ea25098208c9d9/models/cards.js#L3833-L3860:

So now the 7.44 API docs have:

list	path	string	true	the list ID of the card
» listId	body	string	false	the new list ID of the card (move operation)
» newListId	body	string	true	the newListId value

and now for example, passing newListId won't do anything because the functionality is there just for board moves.

I suggest to just add singular boardId instead and not dupe the other two variables.
That means the body.listId and body.swimlaneId checks need an additional && ! BoardId check, so they're skipped in case the entire board is being moved.

@DimDz

[xet7]

@C0rn3j

Please send PR to make it less confusing. Thanks!