Skip to content

Commit 9424642

Browse files
committed
update submit primary action
1 parent fae8558 commit 9424642

2 files changed

Lines changed: 3 additions & 6 deletions

File tree

src/components/MoneyReportHeaderPrimaryAction/SubmitPrimaryAction.tsx

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ import useTransactionsAndViolationsForReport from '@hooks/useTransactionsAndViol
1616
import {search} from '@libs/actions/Search';
1717
import getNonEmptyStringOnyxID from '@libs/getNonEmptyStringOnyxID';
1818
import {getFilteredReportActionsForReportView} from '@libs/ReportActionsUtils';
19-
import {getNextApproverAccountID, hasViolations as hasViolationsReportUtils, isReportOwner, shouldBlockSubmitDueToStrictPolicyRules} from '@libs/ReportUtils';
19+
import {hasViolations as hasViolationsReportUtils, shouldBlockSubmitDueToPreventSelfApproval, shouldBlockSubmitDueToStrictPolicyRules} from '@libs/ReportUtils';
2020
import {hasAnyPendingRTERViolation as hasAnyPendingRTERViolationTransactionUtils} from '@libs/TransactionUtils';
2121
import {submitReport} from '@userActions/IOU/ReportWorkflow';
2222
import {markPendingRTERTransactionsAsCash} from '@userActions/Transaction';
@@ -57,10 +57,7 @@ function SubmitPrimaryAction({reportID}: SubmitPrimaryActionProps) {
5757
};
5858
const confirmPendingRTERAndProceed = useConfirmPendingRTERAndProceed(hasAnyPendingRTERViolation, handleMarkPendingRTERTransactionsAsCash);
5959

60-
const nextApproverAccountID = getNextApproverAccountID(moneyRequestReport);
61-
const isSubmitterSameAsNextApprover =
62-
isReportOwner(moneyRequestReport) && (nextApproverAccountID === moneyRequestReport?.ownerAccountID || moneyRequestReport?.managerID === moneyRequestReport?.ownerAccountID);
63-
const isBlockSubmitDueToPreventSelfApproval = isSubmitterSameAsNextApprover && policy?.preventSelfApproval;
60+
const isBlockSubmitDueToPreventSelfApproval = shouldBlockSubmitDueToPreventSelfApproval(moneyRequestReport, policy);
6461
const isBlockSubmitDueToStrictPolicyRules = shouldBlockSubmitDueToStrictPolicyRules(
6562
moneyRequestReport?.reportID,
6663
violations,

tests/unit/ReportUtilsTest.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -92,9 +92,9 @@ import {
9292
getIOUReportActionDisplayMessage,
9393
getLinkedIOUTransaction,
9494
getMostRecentlyVisitedReport,
95-
getNextApproverAccountID,
9695
getMovedActionMessage,
9796
getMovedTransactionMessage,
97+
getNextApproverAccountID,
9898
getOriginalReportID,
9999
getOutstandingChildRequest,
100100
getParentNavigationSubtitle,

0 commit comments

Comments
 (0)