Skip to content

Commit

Permalink
Update getDistance util and use it more
Browse files Browse the repository at this point in the history
  • Loading branch information
neil-marcellini committed Apr 30, 2024
1 parent 9c2f556 commit 2859ec2
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion src/components/MoneyRequestConfirmationList.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -267,7 +267,7 @@ function MoneyRequestConfirmationList({

const currency = (mileageRate as MileageRate)?.currency ?? policyCurrency;

const distance = transaction?.comment?.customUnit?.quantity ?? 0;
const distance = TransactionUtils.getDistance(transaction);
const taxRates = policy?.taxRates ?? null;

// A flag for showing the categories field
Expand Down
2 changes: 1 addition & 1 deletion src/components/ReportActionItem/MoneyRequestView.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -199,7 +199,7 @@ function MoneyRequestView({
const {unit} = mileageRate;
const rate = transaction?.comment?.customUnit?.defaultP2PRate ?? mileageRate.rate;

const distance = DistanceRequestUtils.convertToDistanceInMeters(transaction?.comment?.customUnit?.quantity ?? 0, unit);
const distance = DistanceRequestUtils.convertToDistanceInMeters(TransactionUtils.getDistance(transaction), unit);
const rateToDisplay = DistanceRequestUtils.getRateForDisplay(unit, rate, currency, translate, toLocaleDigit, isOffline);
const distanceToDisplay = DistanceRequestUtils.getDistanceForDisplay(hasRoute, distance, unit, rate, translate);
let merchantTitle = isEmptyMerchant ? '' : transactionMerchant;
Expand Down
4 changes: 2 additions & 2 deletions src/libs/TransactionUtils.ts
Original file line number Diff line number Diff line change
Expand Up @@ -356,8 +356,8 @@ function getMerchant(transaction: OnyxEntry<Transaction>): string {
return transaction?.modifiedMerchant ? transaction.modifiedMerchant : transaction?.merchant ?? '';
}

function getDistance(transaction: Transaction): number {
return transaction?.routes?.route0?.distance ?? 0;
function getDistance(transaction: Transaction | null): number {
return transaction?.comment?.customUnit?.quantity ?? 0;
}

/**
Expand Down

0 comments on commit 2859ec2

Please sign in to comment.