Add fix suggested by Vit

Author: JakubKorytko

src/components/ReportActionItem/TransactionPreview/TransactionPreviewContent.tsx

@@ -19,7 +19,7 @@ import {getAvatarsForAccountIDs} from '@libs/OptionsListUtils';
 import Parser from '@libs/Parser';
 import {getCleanedTagName} from '@libs/PolicyUtils';
 import {getThumbnailAndImageURIs} from '@libs/ReceiptUtils';
-import {getOriginalMessage, isMoneyRequestAction} from '@libs/ReportActionsUtils';
+import {getOriginalMessage, getReportActions, isMoneyRequestAction} from '@libs/ReportActionsUtils';
 import type {TransactionDetails} from '@libs/ReportUtils';
 import {canEditMoneyRequest, getTransactionDetails, getWorkspaceIcon, isIOUReport, isPolicyExpenseChat, isReportApproved, isSettled} from '@libs/ReportUtils';
 import StringUtils from '@libs/StringUtils';
@@ -60,6 +60,7 @@ function TransactionPreviewContent({
     const ownerAccountID = iouReport?.ownerAccountID ?? reportPreviewAction?.childOwnerAccountID ?? CONST.DEFAULT_NUMBER_ID;
     const isReportAPolicyExpenseChat = isPolicyExpenseChat(chatReport);
     const {amount: requestAmount, comment: requestComment, merchant, tag, category, currency: requestCurrency} = transactionDetails;
+    const reportActions = useMemo(() => (iouReport ? getReportActions(iouReport) ?? {} : {}), [iouReport]);
 
     const transactionPreviewCommonArguments = useMemo(
         () => ({
@@ -96,8 +97,9 @@ function TransactionPreviewContent({
                 ...transactionPreviewCommonArguments,
                 shouldShowRBR,
                 violationMessage,
+                reportActions,
             }),
-        [transactionPreviewCommonArguments, shouldShowRBR, violationMessage],
+        [transactionPreviewCommonArguments, shouldShowRBR, violationMessage, reportActions],
     );
     const getTranslatedText = (item: TranslationPathOrText) => (item.translationPath ? translate(item.translationPath) : item.text ?? '');
 

src/libs/TransactionPreviewUtils.ts

@@ -10,7 +10,7 @@ import {convertToDisplayString} from './CurrencyUtils';
 import DateUtils from './DateUtils';
 import type {PlatformStackRouteProp} from './Navigation/PlatformStackNavigation/types';
 import type {TransactionDuplicateNavigatorParamList} from './Navigation/types';
-import {getOriginalMessage, getReportAction, getReportActions, isMessageDeleted, isMoneyRequestAction} from './ReportActionsUtils';
+import {getOriginalMessage, getReportAction, isMessageDeleted, isMoneyRequestAction} from './ReportActionsUtils';
 import {hasActionsWithErrors, hasReportViolations, isPaidGroupPolicy, isPaidGroupPolicyExpenseReport, isReportApproved, isReportOwner, isSettled} from './ReportUtils';
 import type {TransactionDetails} from './ReportUtils';
 import StringUtils from './StringUtils';
@@ -140,10 +140,8 @@ function getViolationTranslatePath(violations: OnyxTypes.TransactionViolations,
  * it returns an empty array. It identifies the latest error in each action and filters out duplicates to
  * ensure only unique error messages are returned.
  */
-function getUniqueActionErrors(report: OnyxEntry<OnyxTypes.Report>) {
-    const reportActions = Object.values(report ? getReportActions(report) ?? {} : {});
-
-    const reportErrors = reportActions.map((reportAction) => {
+function getUniqueActionErrors(reportActions: OnyxTypes.ReportActions) {
+    const reportErrors = Object.values(reportActions).map((reportAction) => {
         const errors = reportAction.errors ?? {};
         const key = Object.keys(errors).sort().reverse().at(0) ?? '';
         const error = errors[key];
@@ -162,6 +160,7 @@ function getTransactionPreviewTextAndTranslationPaths({
     isBillSplit,
     shouldShowRBR,
     violationMessage,
+    reportActions,
 }: {
     iouReport: OnyxEntry<OnyxTypes.Report>;
     transaction: OnyxEntry<OnyxTypes.Transaction>;
@@ -171,6 +170,7 @@ function getTransactionPreviewTextAndTranslationPaths({
     isBillSplit: boolean;
     shouldShowRBR: boolean;
     violationMessage?: string;
+    reportActions?: OnyxTypes.ReportActions;
 }) {
     const isFetchingWaypoints = isFetchingWaypointsFromServer(transaction);
     const isTransactionOnHold = isOnHold(transaction);
@@ -216,8 +216,8 @@ function getTransactionPreviewTextAndTranslationPaths({
         }
     }
 
-    if (RBRMessage === undefined && hasActionWithErrors) {
-        const actionsWithErrors = getUniqueActionErrors(iouReport);
+    if (RBRMessage === undefined && hasActionWithErrors && !!reportActions) {
+        const actionsWithErrors = getUniqueActionErrors(reportActions);
         RBRMessage = actionsWithErrors.length > 1 ? {translationPath: 'violations.reviewRequired'} : {text: actionsWithErrors.at(0)};
     }
 

tests/unit/TransactionPreviewUtils.test.ts

@@ -2,10 +2,8 @@ import {buildOptimisticIOUReport, buildOptimisticIOUReportAction} from '@libs/Re
 import {createTransactionPreviewConditionals, getTransactionPreviewTextAndTranslationPaths, getUniqueActionErrors, getViolationTranslatePath} from '@libs/TransactionPreviewUtils';
 import {buildOptimisticTransaction} from '@libs/TransactionUtils';
 import CONST from '@src/CONST';
-import * as ReportActionUtils from '@src/libs/ReportActionsUtils';
 import * as ReportUtils from '@src/libs/ReportUtils';
-import type {Report, ReportActions} from '@src/types/onyx';
-import {iouReportR14932 as mockedReport} from '../../__mocks__/reportData/reports';
+import type {ReportActions} from '@src/types/onyx';
 
 const basicProps = {
     iouReport: buildOptimisticIOUReport(123, 234, 1000, '1', 'USD'),
@@ -258,14 +256,8 @@ describe('TransactionPreviewUtils', () => {
     });
 
     describe('getUniqueActionErrors', () => {
-        test('returns an empty array if there is no report or it is empty', () => {
-            expect(getUniqueActionErrors(undefined)).toEqual([]);
-            expect(getUniqueActionErrors({} as Report)).toEqual([]);
-        });
-
-        test('returns an empty array if there are no actions in the report', () => {
-            jest.spyOn(ReportActionUtils, 'getReportActions').mockReturnValue({});
-            expect(getUniqueActionErrors(mockedReport)).toEqual([]);
+        test('returns an empty array if there are no actions', () => {
+            expect(getUniqueActionErrors({})).toEqual([]);
         });
 
         test('returns unique error messages from report actions', () => {
@@ -276,10 +268,9 @@ describe('TransactionPreviewUtils', () => {
                 3: {errors: {a: 'Error A', d: 'Error D'}},
                 /* eslint-enable @typescript-eslint/naming-convention */
             } as unknown as ReportActions;
-            jest.spyOn(ReportActionUtils, 'getReportActions').mockReturnValue(actions);
 
             const expectedErrors = ['Error B', 'Error C', 'Error D'];
-            expect(getUniqueActionErrors(mockedReport).sort()).toEqual(expectedErrors.sort());
+            expect(getUniqueActionErrors(actions).sort()).toEqual(expectedErrors.sort());
         });
 
         test('returns the latest error message if multiple errors exist under a single action', () => {
@@ -288,9 +279,8 @@ describe('TransactionPreviewUtils', () => {
                 1: {errors: {z: 'Error Z2', a: 'Error A', f: 'Error Z'}},
                 /* eslint-enable @typescript-eslint/naming-convention */
             } as unknown as ReportActions;
-            jest.spyOn(ReportActionUtils, 'getReportActions').mockReturnValue(actions);
 
-            expect(getUniqueActionErrors(mockedReport)).toEqual(['Error Z2']);
+            expect(getUniqueActionErrors(actions)).toEqual(['Error Z2']);
         });
 
         test('filters out non-string error messages', () => {
@@ -300,9 +290,8 @@ describe('TransactionPreviewUtils', () => {
                 2: {errors: {c: null, d: 'Error D'}},
                 /* eslint-enable @typescript-eslint/naming-convention */
             } as unknown as ReportActions;
-            jest.spyOn(ReportActionUtils, 'getReportActions').mockReturnValue(actions);
 
-            expect(getUniqueActionErrors(mockedReport)).toEqual(['Error B', 'Error D']);
+            expect(getUniqueActionErrors(actions)).toEqual(['Error B', 'Error D']);
         });
     });
 });