From 79eda0dd237bebcd632d0c05649c677514b3c673 Mon Sep 17 00:00:00 2001 From: Kyle Hickinson Date: Thu, 24 Oct 2019 10:50:24 -0400 Subject: [PATCH] Fix #1786: Don't use probi value for USD conversion on one-time tips --- .../Settings/Tips/Details/TipsDetailViewController.swift | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/BraveRewardsUI/Settings/Tips/Details/TipsDetailViewController.swift b/BraveRewardsUI/Settings/Tips/Details/TipsDetailViewController.swift index f1af1c115..340982555 100644 --- a/BraveRewardsUI/Settings/Tips/Details/TipsDetailViewController.swift +++ b/BraveRewardsUI/Settings/Tips/Details/TipsDetailViewController.swift @@ -195,16 +195,19 @@ extension TipsDetailViewController: UITableViewDataSource, UITableViewDelegate { let contribution = tip.weight switch tip.rewardsCategory { case .oneTimeTip: + let value = BATValue(probi: "\(contribution)") cell.typeNameLabel.text = Strings.OneTimeText + Date.stringFrom(reconcileStamp: tip.reconcileStamp) - cell.tokenView.batContainer.amountLabel.text = BATValue(probi: "\(contribution)")?.displayString + cell.tokenView.batContainer.amountLabel.text = value?.displayString + cell.tokenView.usdContainer.amountLabel.text = state.ledger.dollarStringForBATAmount(value?.doubleValue ?? 0, includeCurrencyCode: false) case .recurringTip: cell.tokenView.batContainer.amountLabel.text = BATValue(contribution).displayString + cell.tokenView.usdContainer.amountLabel.text = state.ledger.dollarStringForBATAmount(contribution, includeCurrencyCode: false) cell.typeNameLabel.text = Strings.RecurringText default: cell.tokenView.batContainer.amountLabel.text = "" + cell.tokenView.usdContainer.amountLabel.text = "" cell.typeNameLabel.text = "" } - cell.tokenView.usdContainer.amountLabel.text = state.ledger.dollarStringForBATAmount(contribution, includeCurrencyCode: false) return cell } }