Fix optional wrapping issue in mapToStrings function by explicitedly …

…allowing Any? values then unwrapping them.
segment-integrations · Dec 20, 2023 · f2e2df0 · f2e2df0
1 parent fd06a43
commit f2e2df0
Showing 1 changed file with 7 additions and 5 deletions.
diff --git a/Sources/SegmentFirebase/FirebaseDestination.swift b/Sources/SegmentFirebase/FirebaseDestination.swift
@@ -201,13 +201,15 @@ extension FirebaseDestination {
     }
 
     // Makes sure all traits are string based for Firebase API
-    func mapToStrings(_ mapDictionary: [String: Any], finalize: (String, String) -> Void) {
+    func mapToStrings(_ mapDictionary: [String: Any?], finalize: (String, String) -> Void) {
 
         for (key, data) in mapDictionary {
-            var dataString = data as? String ?? "\(data)"
-            let keyString = key.replacingOccurrences(of: " ", with: "_")
-            dataString = dataString.trimmingCharacters(in: .whitespacesAndNewlines)
-            finalize(keyString, dataString)
+            if let d = data {
+                var dataString = d as? String ?? "\(d)"
+                let keyString = key.replacingOccurrences(of: " ", with: "_")
+                dataString = dataString.trimmingCharacters(in: .whitespacesAndNewlines)
+                finalize(keyString, dataString)
+            }
         }
     }
 }