@zeniai/client-epic-state 5.0.53-betaML3 → 5.0.53-betaML4

package/lib/esm/view/expenseAutomationView/reducers/transactionsViewReducer.js

@@ -397,6 +397,17 @@ const expenseAutomationTransactionsView = createSlice({
                 const transactionLocalData = recordGet(draft.transactionCategorizationView[selectedTab]
                     .transactionReviewLocalDataById, transactionId.id, undefined);
                 if (transactionLocalData != null) {
+                    // Snapshot the pre-fetch selection. setEntityRecommendationForLineIdInLocalData
+                    // mutates selectedCheckBoxTransactionIds internally (auto-add when
+                    // all-filled, auto-remove otherwise) but does not enforce
+                    // MAX_SELECTION_LIMIT. We restore from this snapshot below and let
+                    // the reducer-level gate own the selection contract so the cap is
+                    // honored and the gate mirrors the original auto-add gate from
+                    // updateSelectedVendor/CustomerForTransaction.
+                    const selectionBeforeFetch = [
+                        ...draft.transactionCategorizationView[selectedTab]
+                            .selectedCheckBoxTransactionIds,
+                    ];
                     const newLocalData = setEntityRecommendationForLineIdInLocalData(draft.transactionCategorizationView[selectedTab], transaction, transactionLocalData.transactionReviewLocalData, entity, lineIds, uncategorizedAccounts, action.payload.recommendationWithCOTByLineId, isUncategorizedExpenseCategoryEnabled, isAccountingClassesEnabled);
                     // Flip the per-line "fetching" flag to Completed for the lines this
                     // fetch covered. Reads as the success arm of the
@@ -417,6 +428,22 @@ const expenseAutomationTransactionsView = createSlice({
                         transactionId: transaction.id,
                         transactionReviewLocalData: newLocalData,
                     };
+                    // Auto-add the transaction to the bulk-save selection now that
+                    // fresh recs have landed and all required fields are filled. This
+                    // is the success-arm of the deferral noted in
+                    // updateSelectedVendor/CustomerForTransaction: their auto-add was
+                    // suppressed because pre-fetch category/class were stale; we run
+                    // the same gate here against newLocalData so the checkbox flips
+                    // on against fresh recs.
+                    draft.transactionCategorizationView[selectedTab].selectedCheckBoxTransactionIds = selectionBeforeFetch;
+                    if (checkIfAllLineItemsAreCategoryClassFilled(transaction, newLocalData, uncategorizedAccounts, isUncategorizedExpenseCategoryEnabled, isAccountingClassesEnabled) === true &&
+                        selectionBeforeFetch.length < MAX_SELECTION_LIMIT &&
+                        selectionBeforeFetch.includes(transaction.id) === false) {
+                        draft.transactionCategorizationView[selectedTab].selectedCheckBoxTransactionIds = [
+                            ...selectionBeforeFetch,
+                            transaction.id,
+                        ];
+                    }
                     // Mark transaction as dirty (autofill happened)
                     if (!draft.transactionCategorizationView[selectedTab].transactionIdsWithUnsavedData.includes(transaction.id)) {
                         draft.transactionCategorizationView[selectedTab].transactionIdsWithUnsavedData = [