diff --git a/src/components/MoneyReportHeader.tsx b/src/components/MoneyReportHeader.tsx index 95e18a22c323..f74a26af9cc2 100644 --- a/src/components/MoneyReportHeader.tsx +++ b/src/components/MoneyReportHeader.tsx @@ -844,7 +844,7 @@ function MoneyReportHeader({ } const currentTransaction = transactions.at(0); - initSplitExpense(currentTransaction, moneyRequestReport?.reportID ?? String(CONST.DEFAULT_NUMBER_ID)); + initSplitExpense(currentTransaction); }, }, [CONST.REPORT.SECONDARY_ACTIONS.CHANGE_WORKSPACE]: { diff --git a/src/components/MoneyRequestHeader.tsx b/src/components/MoneyRequestHeader.tsx index 5943018af1e5..0d1aa00f62c5 100644 --- a/src/components/MoneyRequestHeader.tsx +++ b/src/components/MoneyRequestHeader.tsx @@ -240,7 +240,7 @@ function MoneyRequestHeader({report, parentReportAction, policy, onBackButtonPre icon: Expensicons.ArrowSplit, value: CONST.REPORT.SECONDARY_ACTIONS.SPLIT, onSelected: () => { - initSplitExpense(transaction, reportID ?? String(CONST.DEFAULT_NUMBER_ID)); + initSplitExpense(transaction); }, }, [CONST.REPORT.TRANSACTION_SECONDARY_ACTIONS.VIEW_DETAILS]: { diff --git a/src/libs/actions/IOU.ts b/src/libs/actions/IOU.ts index 2cf7c3150e79..95ee380775a9 100644 --- a/src/libs/actions/IOU.ts +++ b/src/libs/actions/IOU.ts @@ -11661,11 +11661,13 @@ function getSearchOnyxUpdate({ /** * Create a draft transaction to set up split expense details for the split expense flow */ -function initSplitExpense(transaction: OnyxEntry, reportID: string, isOpenCreatedSplit?: boolean) { +function initSplitExpense(transaction: OnyxEntry, isOpenCreatedSplit?: boolean) { if (!transaction) { return; } + const reportID = transaction?.reportID ?? String(CONST.DEFAULT_NUMBER_ID); + if (isOpenCreatedSplit) { const originalTransactionID = transaction.comment?.originalTransactionID; const originalTransaction = allTransactions[`${ONYXKEYS.COLLECTION.TRANSACTION}${originalTransactionID}`]; diff --git a/tests/actions/IOUTest.ts b/tests/actions/IOUTest.ts index 8cf87fd84688..e82dcc98c8ae 100644 --- a/tests/actions/IOUTest.ts +++ b/tests/actions/IOUTest.ts @@ -6072,9 +6072,7 @@ describe('actions/IOU', () => { reportID: '456', }; - const reportID = '456'; - - initSplitExpense(transaction, reportID); + initSplitExpense(transaction); await waitForBatchedUpdates(); const draftTransaction = await getOnyxValue(`${ONYXKEYS.COLLECTION.SPLIT_TRANSACTION_DRAFT}${transaction.transactionID}`); @@ -6086,7 +6084,7 @@ describe('actions/IOU', () => { expect(draftTransaction?.amount).toBe(100); expect(draftTransaction?.currency).toBe('USD'); expect(draftTransaction?.merchant).toBe('Test Merchant'); - expect(draftTransaction?.reportID).toBe(reportID); + expect(draftTransaction?.reportID).toBe(transaction.reportID); expect(splitExpenses?.[0].amount).toBe(50); expect(splitExpenses?.[0].description).toBe('Test comment'); @@ -6100,9 +6098,8 @@ describe('actions/IOU', () => { }); it('should not initialize split expense for null transaction', async () => { const transaction: Transaction | undefined = undefined; - const reportID = '456'; - initSplitExpense(transaction, reportID); + initSplitExpense(transaction); await waitForBatchedUpdates(); expect(transaction).toBeFalsy();