address comment

ProtonMail · Oct 16, 2023 · 18b0e8f · 18b0e8f
1 parent 13e8a80
commit 18b0e8f
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/libraries/PaymentsUI/Sources/Extensions/PriceFormatter.swift b/libraries/PaymentsUI/Sources/Extensions/PriceFormatter.swift
@@ -22,12 +22,12 @@
 import Foundation
 
 enum PriceFormatter {
-    static func formatPlanPrice(price: Double, locale: Locale = Locale.current, currency: String? = nil, maximumFractionDigits: Int = 2) -> String {
+    static func formatPlanPrice(price: Double, locale: Locale = Locale.current, currencyCode: String? = nil, maximumFractionDigits: Int = 2) -> String {
         let formatter = NumberFormatter()
         formatter.numberStyle = .currency
         formatter.locale = locale
-        if let currency {
-            formatter.currencyCode = currency
+        if let currencyCode {
+            formatter.currencyCode = currencyCode
         }
         formatter.maximumFractionDigits = maximumFractionDigits
         let priceString = formatter.string(from: NSNumber(value: price)) ?? ""

diff --git a/libraries/PaymentsUI/Sources/ViewModels/PlanPresentation/V5/CurrentPlanDetailsV5.swift b/libraries/PaymentsUI/Sources/ViewModels/PlanPresentation/V5/CurrentPlanDetailsV5.swift
@@ -73,8 +73,8 @@ struct CurrentPlanDetailsV5 {
         }
 
         var price: String
-        if let amount = details.amount {
-            price = PriceFormatter.formatPlanPrice(price: Double(amount) / 100, currency: details.currency)
+        if let amount = details.amount, amount != 0 {
+            price = PriceFormatter.formatPlanPrice(price: Double(amount) / 100, currencyCode: details.currency)
         } else {
             price = PUITranslations.plan_details_free_price.l10n
         }