CleverTap Push fixes (#569)

* Fix isEqualToHandledNotification for CT push

* CleverTap open deeplinks on push open

Author: nzagorchev

LeanplumSDK/LeanplumSDK/Classes/Internal/Leanplum.m

@@ -3,7 +3,7 @@
 //  Leanplum
 //
 //  Created by Andrew First on 4/30/12.
-//  Copyright (c) 2022 Leanplum, Inc. All rights reserved.
+//  Copyright (c) 2023 Leanplum, Inc. All rights reserved.
 //
 //  Licensed to the Apache Software Foundation (ASF) under one
 //  or more contributor license agreements.  See the NOTICE file
@@ -1625,7 +1625,8 @@ + (void)setShouldOpenNotificationHandler:(LeanplumShouldHandleNotificationBlock)
 + (void)setCleverTapOpenDeepLinksInForeground:(BOOL)openDeepLinksInForeground
 {
     LPCTNotificationsManager *manager = (LPCTNotificationsManager *)[Leanplum notificationsManager];
-    [manager setOpenDeepLinksInForeground:openDeepLinksInForeground];
+    NSNumber *value = [NSNumber numberWithBool:openDeepLinksInForeground];
+    [manager setOpenDeepLinksInForeground:value];
 }
 
 + (void)setHandleCleverTapNotification:(_Nullable LeanplumHandleCleverTapNotificationBlock)block

LeanplumSDK/LeanplumSDK/ClassesSwift/Migration/LPCTNotificationsManager.swift

@@ -3,14 +3,15 @@
 //  LeanplumSDK
 //
 //  Created by Nikola Zagorchev on 12.10.22.
-//  Copyright © 2022 Leanplum. All rights reserved.
+//  Copyright © 2023 Leanplum. All rights reserved.
 
 import Foundation
 // Use @_implementationOnly to *not* expose CleverTapSDK to the Leanplum-Swift header
 @_implementationOnly import CleverTapSDK
 
 @objc public class LPCTNotificationsManager: NotificationsManager {
-    @objc public var openDeepLinksInForeground = false
+    // Using NSNumber since Optional Bool cannot be represented in Objective-C
+    @objc public var openDeepLinksInForeground: NSNumber?
     @objc public var handleCleverTapNotificationBlock: LeanplumHandleCleverTapNotificationBlock?
 
 
@@ -74,7 +75,8 @@ import Foundation
             return
         }
 
-        handleNotification(openDeepLinksInForeground)
+        // If `openDeepLinksInForeground` is not set explicitly, use `true` for notification Opens.
+        handleNotification(openDeepLinksInForeground?.boolValue ?? (event == .opened))
     }
 
     @objc public func handleWithCleverTapInstance(action: @escaping () -> ()) {

LeanplumSDK/LeanplumSDK/ClassesSwift/Notifications/Proxy/NotificationsProxy+Utilities.swift

@@ -3,7 +3,7 @@
 //  LeanplumSDK
 //
 //  Created by Nikola Zagorchev on 23.12.21.
-//  Copyright © 2021 Leanplum. All rights reserved.
+//  Copyright © 2023 Leanplum. All rights reserved.
 
 import Foundation
 
@@ -13,7 +13,8 @@ extension NotificationsProxy {
         if let fromStart = notificationHandledFromStart {
             let idA = Leanplum.notificationsManager().getNotificationId(fromStart)
             let idB = Leanplum.notificationsManager().getNotificationId(userInfo)
-            return idA == idB
+            // CleverTap notifications do not have notification Id
+            return idA == idB && idA != "-1"
         }
         return false
     }