Skip to content

Commit d9974a9

Browse files
chore: cleanup ClientBuilder.build method
1 parent 2848b5c commit d9974a9

File tree

1 file changed

+7
-9
lines changed

1 file changed

+7
-9
lines changed

DevCycle/DevCycleClient.swift

Lines changed: 7 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -166,7 +166,7 @@ public class DevCycleClient {
166166
if let config = config,
167167
self.checkIfEdgeDBEnabled(config: config, enableEdgeDB: self.enableEdgeDB)
168168
{
169-
if !user.isAnonymous {
169+
if !(user.isAnonymous ?? false) {
170170
self.service?.saveEntity(
171171
user: user,
172172
completion: { data, response, error in
@@ -399,14 +399,14 @@ public class DevCycleClient {
399399
}
400400

401401
public func identifyUser(user: DevCycleUser, callback: IdentifyCompletedHandler? = nil) throws {
402-
guard let currentUser = self.user, !currentUser.userId.isEmpty,
403-
!user.userId.isEmpty
402+
guard let currentUser = self.user, let userId = currentUser.userId,
403+
let incomingUserId = user.userId
404404
else {
405405
throw ClientError.InvalidUser
406406
}
407407
self.flushEvents()
408408
var updateUser: DevCycleUser = currentUser
409-
if currentUser.userId == user.userId {
409+
if userId == incomingUserId {
410410
updateUser.update(with: user)
411411
} else {
412412
updateUser = user
@@ -626,14 +626,12 @@ public class DevCycleClient {
626626
throw ClientError.MissingSDKKeyOrUser
627627
}
628628

629-
let result = self.client
630629
if let service = service {
631-
result.initialize(service: service, callback: onInitialized)
630+
self.client.initialize(service: service, callback: onInitialized)
632631
} else {
633-
result.initialize(callback: onInitialized)
632+
self.client.initialize(callback: onInitialized)
634633
}
635-
self.client = DevCycleClient()
636-
return result
634+
return self.client
637635
}
638636

639637
@available(iOS 13.0, macOS 10.15, tvOS 13.0, watchOS 6.0, *)

0 commit comments

Comments
 (0)