WIP: VuidManager move to client

Author: muzahidul-opti

Sources/ODP/OdpManager.swift

@@ -18,7 +18,7 @@ import Foundation
 
 public class OdpManager {
     var enabled: Bool
-    var enableVuid: Bool
+//    var enableVuid: Bool
     var vuidManager: OdpVuidManager
 
     var odpConfig: OdpConfig!
@@ -27,9 +27,9 @@ public class OdpManager {
 
     let logger = OPTLoggerFactory.getLogger()
 
-    var vuid: String {
-        return vuidManager.vuid
-    }
+//    var vuid: String {
+//        return vuidManager.vuid
+//    }
 
     /// OdpManager init
     /// - Parameters:
@@ -52,7 +52,7 @@ public class OdpManager {
                 eventManager: OdpEventManager? = nil) {
 
         self.enabled = !disable
-        self.enableVuid = enableVuid
+//        self.enableVuid = enableVuid
         self.vuidManager = OdpVuidManager(enabled: enableVuid)
 
         guard enabled else {
@@ -69,9 +69,9 @@ public class OdpManager {
         self.segmentManager.odpConfig = odpConfig
         self.eventManager.odpConfig = odpConfig
 
-        if enableVuid {
-            self.eventManager.registerVUID(vuid: vuidManager.vuid)
-        }
+//        if enableVuid {
+//            self.eventManager.registerVUID(vuid: vuidManager.vuid)
+//        }
     }
 
     func fetchQualifiedSegments(userId: String,
@@ -91,7 +91,7 @@ public class OdpManager {
                                                completionHandler: completionHandler)
     }
 
-    func identifyUser(userId: String) {
+    func identifyUser(userId: String, vuid: String) {
         guard enabled else {
             logger.d("ODP identify event is not dispatched (ODP disabled).")
             return
@@ -102,14 +102,14 @@ public class OdpManager {
             return
         }
 
-        var vuid = vuidManager.vuid
+        var _vuid = vuid
         var fsUserId: String? = userId
         if OdpVuidManager.isVuid(userId) {
             // overwrite if userId is vuid (when userContext is created with vuid)
-            vuid = userId
+            _vuid = userId
             fsUserId = nil
         }
-        eventManager.identifyUser(vuid: vuid, userId: fsUserId)
+        eventManager.identifyUser(vuid: _vuid, userId: fsUserId)
     }
 
     /// Send an event to the ODP server.

Sources/Optimizely+Decide/OptimizelyUserContext.swift

@@ -88,7 +88,7 @@ public class OptimizelyUserContext {
         if identify {
             // async call so event building overhead is not blocking context creation
             lock.async {
-                self.optimizely?.identifyUserToOdp(userId: userId)
+                self.optimizely?.identifyUserToOdp(userId: userId, vuid: optimizely.vuid)
             }
         }
     }

Sources/Optimizely/OptimizelyClient.swift

@@ -60,6 +60,7 @@ open class OptimizelyClient: NSObject {
     var decisionService: OPTDecisionService!
     public var notificationCenter: OPTNotificationCenter?
     public var odpManager: OdpManager!
+    private var vuidManager: OdpVuidManager!
     let sdkSettings: OptimizelySdkSettings
 
     // MARK: - Public interfaces
@@ -117,6 +118,10 @@ open class OptimizelyClient: NSObject {
         self.decisionService = HandlerRegistryService.shared.injectDecisionService(sdkKey: self.sdkKey)
         self.notificationCenter = HandlerRegistryService.shared.injectNotificationCenter(sdkKey: self.sdkKey)
 
+        self.vuidManager = OdpVuidManager(enabled: sdkSettings.enableVuid)
+        if self.vuidManager.enabled {
+            self.odpManager.eventManager.registerVUID(vuid: self.vuidManager.vuid)
+        }
         logger.d("SDK Version: \(version)")
     }
 
@@ -974,15 +979,15 @@ extension OptimizelyClient {
 
     /// the device vuid (read only)
     public var vuid: String {
-        return odpManager.vuid
+        return self.vuidManager.vuid
     }
 
     public var enableVuid: Bool {
-        return odpManager.enableVuid
+        return self.vuidManager.enabled
     }
 
-    func identifyUserToOdp(userId: String) {
-        odpManager.identifyUser(userId: userId)
+    func identifyUserToOdp(userId: String, vuid: String) {
+        odpManager.identifyUser(userId: userId, vuid: self.vuid)
     }
 
     func fetchQualifiedSegments(userId: String,