Skip to content

Commit caf70b4

Browse files
authored
Merge pull request Expensify#87995 from hungvu193/fix-664420-part5
2 parents 4acb25b + cfab5e9 commit caf70b4

7 files changed

Lines changed: 11 additions & 10 deletions

File tree

src/pages/RestrictedAction/Workspace/WorkspaceRestrictedActionPage.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -66,10 +66,10 @@ function WorkspaceRestrictedActionPage({
6666
if (isLoadingSubscriptionData !== false) {
6767
return;
6868
}
69-
if (!shouldRestrictUserBillableActions(policy, ownerBillingGracePeriodEnd, userBillingGracePeriods, amountOwed)) {
69+
if (!shouldRestrictUserBillableActions(policy, ownerBillingGracePeriodEnd, userBillingGracePeriods, amountOwed, session?.accountID)) {
7070
Navigation.goBack();
7171
}
72-
}, [policy, isLoadingSubscriptionData, userBillingGracePeriods, ownerBillingGracePeriodEnd, amountOwed]);
72+
}, [policy, isLoadingSubscriptionData, userBillingGracePeriods, ownerBillingGracePeriodEnd, amountOwed, session?.accountID]);
7373

7474
// Show a loading indicator while waiting for fresh billing data from the server,
7575
// instead of flashing the restriction UI which may no longer apply.

src/pages/inbox/sidebar/FABPopoverContent/useScanActions.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ function useScanActions() {
5353

5454
const startQuickScan = () => {
5555
interceptAnonymousUser(() => {
56-
if (policyChatPolicyID && shouldRestrictUserBillableActions(policyChatPolicyID, ownerBillingGracePeriodEnd, userBillingGracePeriodEnds, amountOwed)) {
56+
if (policyChatPolicyID && shouldRestrictUserBillableActions(policyChatPolicyID, ownerBillingGracePeriodEnd, userBillingGracePeriodEnds, amountOwed, session?.accountID)) {
5757
Navigation.navigate(ROUTES.RESTRICTED_ACTION.getRoute(policyChatPolicyID));
5858
return;
5959
}

src/pages/iou/request/ParticipantSearchResults.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -418,7 +418,7 @@ function ParticipantSearchResults({
418418
option.isPolicyExpenseChat &&
419419
option.policyID &&
420420
optionPolicy &&
421-
shouldRestrictUserBillableActions(optionPolicy, ownerBillingGracePeriodEnd, userBillingGracePeriodEnds, amountOwed)
421+
shouldRestrictUserBillableActions(optionPolicy, ownerBillingGracePeriodEnd, userBillingGracePeriodEnds, amountOwed, currentUserAccountID)
422422
) {
423423
Navigation.navigate(ROUTES.RESTRICTED_ACTION.getRoute(option.policyID));
424424
return;

src/pages/iou/request/step/BaseRequestStepWorkspace.tsx

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ function BaseRequestStepWorkspace({transaction, getPolicies, onSelectWorkspace}:
4242
const styles = useThemeStyles();
4343
const {translate, localeCompare} = useLocalize();
4444

45-
const {login: currentUserLogin} = useCurrentUserPersonalDetails();
45+
const {login: currentUserLogin, accountID: currentUserAccountID} = useCurrentUserPersonalDetails();
4646
const [allPolicies] = useOnyx(ONYXKEYS.COLLECTION.POLICY);
4747
const [userBillingGracePeriodEnds] = useOnyx(ONYXKEYS.COLLECTION.SHARED_NVP_PRIVATE_USER_BILLING_GRACE_PERIOD_END);
4848
const [ownerBillingGracePeriodEnd] = useOnyx(ONYXKEYS.NVP_PRIVATE_OWNER_BILLING_GRACE_PERIOD_END);
@@ -81,11 +81,12 @@ function BaseRequestStepWorkspace({transaction, getPolicies, onSelectWorkspace}:
8181

8282
const selectWorkspace = (item: WorkspaceListItem) => {
8383
const policyID = item.policyID;
84-
if (shouldRestrictUserBillableActions(policyID, ownerBillingGracePeriodEnd, userBillingGracePeriodEnds, amountOwed)) {
84+
const policy = allPolicies?.[`${ONYXKEYS.COLLECTION.POLICY}${policyID}`];
85+
if (shouldRestrictUserBillableActions(policyID, ownerBillingGracePeriodEnd, userBillingGracePeriodEnds, amountOwed, currentUserAccountID)) {
8586
Navigation.navigate(ROUTES.RESTRICTED_ACTION.getRoute(policyID));
8687
return;
8788
}
88-
onSelectWorkspace(allPolicies?.[`${ONYXKEYS.COLLECTION.POLICY}${policyID}`]);
89+
onSelectWorkspace(policy);
8990
};
9091

9192
return (

src/pages/iou/request/step/IOURequestEditReportCommon.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -228,7 +228,7 @@ function IOURequestEditReportCommon({
228228
return;
229229
}
230230

231-
if (item?.policyID && shouldRestrictUserBillableActions(item.policyID, ownerBillingGracePeriodEnd, userBillingGracePeriodEnds, amountOwed)) {
231+
if (item?.policyID && shouldRestrictUserBillableActions(item.policyID, ownerBillingGracePeriodEnd, userBillingGracePeriodEnds, amountOwed, currentUserPersonalDetails.accountID)) {
232232
Navigation.navigate(ROUTES.RESTRICTED_ACTION.getRoute(item.policyID));
233233
return;
234234
}

src/pages/iou/request/step/IOURequestStepDestination.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -110,7 +110,7 @@ function IOURequestStepDestination({
110110
};
111111

112112
const updateDestination = (destination: ListItem & {currency: string}) => {
113-
if (openedFromStartPage && policy?.id && shouldRestrictUserBillableActions(policy.id, ownerBillingGracePeriodEnd, userBillingGracePeriodEnds, amountOwed)) {
113+
if (openedFromStartPage && policy?.id && shouldRestrictUserBillableActions(policy.id, ownerBillingGracePeriodEnd, userBillingGracePeriodEnds, amountOwed, accountID)) {
114114
Navigation.navigate(ROUTES.RESTRICTED_ACTION.getRoute(policy.id));
115115
return;
116116
}

src/pages/iou/request/step/IOURequestStepReport.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -263,7 +263,7 @@ function IOURequestStepReport({route, transaction}: IOURequestStepReportProps) {
263263

264264
const createReport = () => {
265265
const restrictionPolicyID = isPerDiemTransaction ? perDiemOriginalPolicy?.id : policyForMovingExpensesID;
266-
if (restrictionPolicyID && shouldRestrictUserBillableActions(restrictionPolicyID, ownerBillingGracePeriodEnd, userBillingGracePeriodEnds, amountOwed)) {
266+
if (restrictionPolicyID && shouldRestrictUserBillableActions(restrictionPolicyID, ownerBillingGracePeriodEnd, userBillingGracePeriodEnds, amountOwed, session?.accountID)) {
267267
Navigation.navigate(ROUTES.RESTRICTED_ACTION.getRoute(restrictionPolicyID));
268268
return;
269269
}

0 commit comments

Comments
 (0)