Merge pull request #96 from supernotes/fix/openURL

Fix deprecated UIApplication.openURL(_:) call
carsten-klaffke · Nov 29, 2024 · c7b4534 · c7b4534
2 parents 3a017bf + d3a8f29
commit c7b4534
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/Example/SendIntentExample/ios/App/Share/ShareViewController.swift b/Example/SendIntentExample/ios/App/Share/ShareViewController.swift
@@ -203,15 +203,15 @@ class ShareViewController: UIViewController {
         }
     }
 
-    @objc func openURL(_ url: URL) -> Bool {
+    @objc func openURL(_ url: URL) {
         var responder: UIResponder? = self
         while responder != nil {
             if let application = responder as? UIApplication {
-                return application.perform(#selector(openURL(_:)), with: url) != nil
+                application.open(url, options: [:], completionHandler: nil)
+                return
             }
             responder = responder?.next
         }
-        return false
     }
 
 }
diff --git a/README.md b/README.md
@@ -179,7 +179,7 @@ class ShareViewController: UIViewController {
         let fileManager = FileManager.default
 
         let copyFileUrl =
-        fileManager.containerURL(forSecurityApplicationGroupIdentifier: "group.SendIntentExample")!
+        fileManager.containerURL(forSecurityApplicationGroupIdentifier: "YOUR_APP_GROUP_ID")!
             .absoluteString.addingPercentEncoding(withAllowedCharacters: .urlQueryAllowed)!
         + "/screenshot_\(index).png"
         do {
@@ -297,15 +297,15 @@ class ShareViewController: UIViewController {
         }
     }
 
-    @objc func openURL(_ url: URL) -> Bool {
+    @objc func openURL(_ url: URL) {
         var responder: UIResponder? = self
         while responder != nil {
             if let application = responder as? UIApplication {
-                return application.perform(#selector(openURL(_:)), with: url) != nil
+                application.open(url, options: [:], completionHandler: nil)
+                return
             }
             responder = responder?.next
         }
-        return false
     }
 
 }