Merge pull request #643 from rgoldberg/493-json-parse-error

Fix JSON parsing error by making `formattedPrice` & `price` optional for `SearchResult`
mas-cli · Nov 20, 2024 · 63dc7f1 · 63dc7f1
2 parents ca921c0 + 309cfec
commit 63dc7f1
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 4 deletions.
diff --git a/Sources/mas/Formatters/AppInfoFormatter.swift b/Sources/mas/Formatters/AppInfoFormatter.swift
@@ -18,7 +18,7 @@ enum AppInfoFormatter {
         let headline = [
             "\(app.trackName)",
             "\(app.version)",
-            "[\(app.formattedPrice)]",
+            "[\(app.displayPrice)]",
         ]
         .joined(separator: " ")
 

diff --git a/Sources/mas/Formatters/SearchResultFormatter.swift b/Sources/mas/Formatters/SearchResultFormatter.swift
@@ -29,7 +29,7 @@ enum SearchResultFormatter {
             let version = result.version
 
             if includePrice {
-                output += String(format: "%12lu  %@  (%@)  %@\n", appID, appName, version, result.formattedPrice)
+                output += String(format: "%12lu  %@  (%@)  %@\n", appID, appName, version, result.displayPrice)
             } else {
                 output += String(format: "%12lu  %@  (%@)\n", appID, appName, version)
             }

diff --git a/Sources/mas/Models/SearchResult.swift b/Sources/mas/Models/SearchResult.swift
@@ -10,16 +10,20 @@ struct SearchResult: Decodable {
     var bundleId: String
     var currentVersionReleaseDate: String
     var fileSizeBytes: String
-    var formattedPrice: String
+    var formattedPrice: String?
     var minimumOsVersion: String
-    var price: Double
+    var price: Double?
     var sellerName: String
     var sellerUrl: String?
     var trackId: AppID
     var trackName: String
     var trackViewUrl: String
     var version: String
 
+    var displayPrice: String {
+        formattedPrice ?? "Unknown"
+    }
+
     init(
         bundleId: String = "",
         currentVersionReleaseDate: String = "",