fix: Fixed bug - wrong card placement after creation, cleanup

client/src/containers/ListContainer.js

@@ -12,7 +12,7 @@ const makeMapStateToProps = () => {
 
   return (state, { id, index }) => {
     const { name, isPersisted } = selectListById(state, id);
-    const [cardIds, cardIdsFull, isFiltered] = selectCardIdsByListId(state, id);
+    const { cardIds, cardIdsFull, isFiltered } = selectCardIdsByListId(state, id);
     const currentUserMembership = selectors.selectCurrentUserMembershipForCurrentBoard(state);
 
     const isCurrentUserEditor =

client/src/models/List.js

@@ -91,6 +91,7 @@ export default class extends BaseModel {
     const filterUserIds = this.board.filterUsers.toRefArray().map((user) => user.id);
     const filterLabelIds = this.board.filterLabels.toRefArray().map((label) => label.id);
     let isFiltered = false;
+
     if (filterUserIds.length > 0) {
       cardModels = cardModels.filter((cardModel) => {
         const users = cardModel.users.toRefArray();
@@ -107,7 +108,7 @@ export default class extends BaseModel {
       });
     }
 
-    return [cardModels, cardModelsFull, isFiltered];
+    return { cardModels, cardModelsFull, isFiltered };
   }
 
   deleteRelated() {

client/src/selectors/core.js

@@ -93,7 +93,8 @@ export const selectNextCardPosition = createSelector(
       return listModel;
     }
 
-    return nextPosition(listModel.getFilteredOrderedCardsModelArray(), index, excludedId);
+    // eslint-disable-next-line prettier/prettier
+    return nextPosition(listModel.getFilteredOrderedCardsModelArray().cardModels, index, excludedId);
   },
 );
 

client/src/selectors/lists.js

@@ -33,12 +33,13 @@ export const makeSelectCardIdsByListId = () =>
       if (!listModel) {
         return listModel;
       }
+
       const cardsModelArray = listModel.getFilteredOrderedCardsModelArray();
-      return [
+      return {
-        cardsModelArray[0].map((cardModel) => cardModel.id),
+        cardIds: cardsModelArray.cardModels.map((cardModel) => cardModel.id),
-        cardsModelArray[1].map((cardModel) => cardModel.id),
+        cardIdsFull: cardsModelArray.cardModelsFull.map((cardModel) => cardModel.id),
-        cardsModelArray[2],
+        isFiltered: cardsModelArray.isFiltered,
-      ];
+      };
     },
   );