KM-16629: PR review comments

Author: kp-mario-nachbaur

LocalPackages/PIALibrary/Sources/PIALibrary/Account/DefaultAccountProvider.swift

@@ -419,8 +419,12 @@ public final class DefaultAccountProvider: AccountProvider, ConfigurationAccess,
                 return .failure(error)
             case .success(let products):
                 log.debug("Available products from store: \(products)")
+                let planProducts = self.planProducts
+                if planProducts == nil {
+                    log.debug("\(#function) returning empty products")
+                }
                 Macros.postNotification(.__InAppDidFetchProducts, [.products: planProducts ?? [:]])
-                return .success(planProducts!)
+                return .success(planProducts ?? [:])
             }
         }
 

LocalPackages/PIALibrary/Sources/PIALibrary/InApp/AppStoreProvider.swift

@@ -75,14 +75,14 @@ final class AppStoreProvider: NSObject, InAppProvider {
         log.debug("Start observing unfinished transactions")
 
         // On startup we check for unfinished transactions and finish them. With
-        // out current architecture is difficult to handle them. Users can always
+        // our current architecture is difficult to handle them. Users can always
         // use the restore flow to claim transactions.
         transactionObserverTask = Task {
             for await result in Transaction.updates {
                 if Task.isCancelled { break }
                 switch result {
                 case .unverified(let transaction, let error):
-                    log.warning("Unverified transaction: \(transaction.id) (\(error)")
+                    log.warning("Unverified transaction: \(transaction.id) \(error)")
                     await transaction.finish()
                 case .verified(let transaction):
                     await transaction.finish()