Merge branch '377-error-countbyenumeratingwithstate-objects-count' into 'master'

Resolve "Error: countByEnumeratingWithState:objects:count: >"

Closes #377

See merge request pace/mobile/ios/pace-cloud-sdk!432

Author: talipatpace

PACECloudSDK/AppKit/Extensions/UIViewController+Extension.swift

@@ -9,36 +9,62 @@ import UIKit
 
 extension UIViewController {
     var isGettingDismissed: Bool {
-        isBeingDismissed ||
-        isMovingFromParent ||
-        navigationController?.isBeingDismissed == true ||
-        isUnderlyingViewControllerBeingDismissed(of: self)
+        return isBeingDismissed
+        || isMovingFromParent
+        || navigationController?.isBeingDismissed == true
+        || checkUnderlyingDismissal()
     }
 
-    private func isUnderlyingViewControllerBeingDismissed(of viewController: UIViewController?) -> Bool {
-        let presentingViewController = viewController?.presentingViewController
-        let presentedViewController = viewController?.presentedViewController
+    private func checkUnderlyingDismissal() -> Bool {
+        var visited = Set<ObjectIdentifier>()
+        return isUnderlyingViewControllerBeingDismissed(of: self, visited: &visited)
+    }
+
+    private func isUnderlyingViewControllerBeingDismissed(
+        of viewController: UIViewController?,
+        visited: inout Set<ObjectIdentifier>
+    ) -> Bool {
+        guard let vc = viewController else { return false }
+        let id = ObjectIdentifier(vc)
+        guard !visited.contains(id) else { return false }
+        visited.insert(id)
 
-        if presentingViewController == nil && presentedViewController == nil {
-            return false
+        if isParentDismissed(vc, visited: &visited) {
+            return true
         }
 
-        if presentingViewController == nil {
-            if presentedViewController?.isBeingDismissed == true {
-                return true
-            }
+        return isChildDismissed(vc, visited: &visited)
+    }
+
+    private func isParentDismissed(
+        _ vc: UIViewController,
+        visited: inout Set<ObjectIdentifier>
+    ) -> Bool {
+        guard let presenting = vc.presentingViewController else { return false }
 
-            return isUnderlyingViewControllerBeingDismissed(of: presentedViewController)
+        if presenting.isBeingDismissed {
+            return true
         }
 
-        if presentedViewController == nil {
-            if presentingViewController?.isBeingDismissed == true {
-                return true
-            }
+        return isUnderlyingViewControllerBeingDismissed(
+            of: presenting,
+            visited: &visited
+        )
+    }
+
+    private func isChildDismissed(
+        _ vc: UIViewController,
+        visited: inout Set<ObjectIdentifier>
+    ) -> Bool {
+        guard let presented = vc.presentedViewController else { return false }
 
-            return isUnderlyingViewControllerBeingDismissed(of: presentingViewController)
+        if presented.isBeingDismissed {
+            return true
         }
 
-        return isUnderlyingViewControllerBeingDismissed(of: presentingViewController) || isUnderlyingViewControllerBeingDismissed(of: presentedViewController)
+        return isUnderlyingViewControllerBeingDismissed(
+            of: presented,
+            visited: &visited
+        )
     }
 }