♻️ (Analytics): Refactor setDefaultEventParameter to use enum

leka · Nov 22, 2024 · 24749c7 · 24749c7
1 parent 640fa9e
commit 24749c7
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 9 deletions.
diff --git a/Modules/AnalyticsKit/Sources/Manager/AnalyticsManager+DefaultParameters.swift b/Modules/AnalyticsKit/Sources/Manager/AnalyticsManager+DefaultParameters.swift
@@ -5,19 +5,15 @@
 import FirebaseAnalytics
 
 public extension AnalyticsManager {
-    static func setDefaultEventParameterRootOwnerUid(_ id: String?) {
-        self.setDefaultEventParameters(["lk_default_root_owner_uid": id ?? "(lk_not_set)"])
+    static func setDefaultEventParameterRootOwnerUid(_ uid: String?) {
+        setDefaultEventParameter(for: .rootOwnerUid(uid))
     }
 
-    static func setDefaultEventParameterCaregiverUid(_ id: String?) {
-        self.setDefaultEventParameters(["lk_default_caregiver_uid": id ?? "(lk_not_set)"])
+    static func setDefaultEventParameterCaregiverUid(_ uid: String?) {
+        setDefaultEventParameter(for: .caregiverUid(uid))
     }
 
     static func clearDefaultEventParameters() {
-        self.setDefaultEventParameters(nil)
-    }
-
-    static func setDefaultEventParameters(_ parameters: [String: Any]?) {
-        Analytics.setDefaultEventParameters(parameters)
+        Analytics.setDefaultEventParameters(nil)
     }
 }
diff --git a/Modules/AnalyticsKit/Sources/Manager/AnalyticsManager.swift b/Modules/AnalyticsKit/Sources/Manager/AnalyticsManager.swift
@@ -14,6 +14,11 @@ public class AnalyticsManager {
 
     // MARK: Internal
 
+    enum DefaultEventParamters {
+        case rootOwnerUid(String?)
+        case caregiverUid(String?)
+    }
+
     enum Event {
         case activityStart
         case activityEnd
@@ -91,4 +96,13 @@ public class AnalyticsManager {
     static func setUserProperty(value: String, name: String) {
         Analytics.setUserProperty(value, forName: name)
     }
+
+    static func setDefaultEventParameter(for parameter: DefaultEventParamters) {
+        switch parameter {
+            case let .rootOwnerUid(uid):
+                Analytics.setDefaultEventParameters(["root_owner_uid": uid ?? "(lk_not_set)"])
+            case let .caregiverUid(uid):
+                Analytics.setDefaultEventParameters(["caregiver_uid": uid ?? "(lk_not_set)"])
+        }
+    }
 }