@react-native-documents/picker 10.1.4 → 10.1.6

package/android/src/main/java/com/reactnativedocumentpicker/RNDocumentPickerModule.kt

@@ -83,12 +83,8 @@ class RNDocumentPickerModule(reactContext: ReactApplicationContext) :
 
   @ReactMethod
   override fun pick(opts: ReadableMap, promise: Promise) {
-    val currentActivity = currentActivity
+    val currentActivity = reactApplicationContext.currentActivity ?: return rejectWithNullActivity(promise)
 
-    if (currentActivity == null) {
-      rejectWithNullActivity(promise)
-      return
-    }
     if (!promiseWrapper.trySetPromiseRejectingIncoming(promise, "pick")) {
       return
     }
@@ -145,11 +141,8 @@ class RNDocumentPickerModule(reactContext: ReactApplicationContext) :
 
   @ReactMethod
   override fun pickDirectory(opts: ReadableMap, promise: Promise) {
-    val currentActivity = currentActivity
-    if (currentActivity == null) {
-      rejectWithNullActivity(promise)
-      return
-    }
+    val currentActivity = reactApplicationContext.currentActivity ?: return rejectWithNullActivity(promise)
+
     if (!promiseWrapper.trySetPromiseRejectingIncoming(promise, "pickDirectory")) {
       return
     }

package/ios/swift/DocPicker.swift

@@ -28,10 +28,10 @@ import MobileCoreServices
   }
 
   public func getMetadataFor(url: URL) throws -> DocumentMetadataBuilder {
-    if (currentOptions?.isOpenMode() == true) {
-      return try self.getOpenedDocumentInfo(url: url, requestLongTermAccess: currentOptions?.requestLongTermAccess ?? false)
+    return if (currentOptions?.isOpenMode() == true) {
+      try self.getOpenedDocumentInfo(url: url, requestLongTermAccess: currentOptions?.requestLongTermAccess ?? false)
     } else {
-      return try self.getAnyModeMetadata(url: url)
+      try self.getAnyModeMetadata(url: url)
     }
   }
 

package/ios/swift/FileOperations.swift

@@ -4,45 +4,47 @@ import Foundation
 
 @objc public class FileOperations: NSObject {
   
-  @objc public static func keepLocalCopyAtUniqueDestination(from: Array<Dictionary<String, String>>, destinationPreset: String, resolve: @escaping RNDPPromiseResolveBlock) {
-    Task {
-      let results = await moveFiles(from: from, destinationPreset: destinationPreset)
+  @objc public static func keepLocalCopyAtUniqueDestination(from: [[String: String]], destinationPreset: String, resolve: @escaping RNDPPromiseResolveBlock) {
+    DispatchQueue.global(qos: .utility).async {
+      let results = moveFiles(from: from, destinationPreset: destinationPreset)
       resolve(results)
     }
   }
   
-  static func moveFiles(from: Array<Dictionary<String, String>>, destinationPreset: String) async -> [[String: String?]] {
+  static func moveFiles(from: [[String: String]], destinationPreset: String) -> [[String: String?]] {
     let destinationRootDir = getDirectoryForFileDestination(destinationPreset)
     let uniqueSubDirName = UUID().uuidString
-    let destinationDir: URL = destinationRootDir.appendingPathComponent("\(uniqueSubDirName)/", isDirectory: true)
-    // TODO do we need all of this Task dance?
+    let destinationDir = destinationRootDir.appendingPathComponent(uniqueSubDirName, isDirectory: true)
     
-    return await withTaskGroup(of: LocalCopyResponse.self) { group in
-      var results: Array<Dictionary<String, String?>> = [[String: String?]]()
-      
-      for dictionary in from {
-        group.addTask {
-          do {
-            guard let uriString = dictionary["uri"], let uri = URL(string: uriString) else {
-              return LocalCopyResponse.error(sourceUri: dictionary["uri"], copyError: "Invalid URI")
-            }
-            guard let fileName = dictionary["fileName"] else {
-              return LocalCopyResponse.error(sourceUri: uri.absoluteString, copyError: "Invalid fileName")
-            }
-            
-            let destinationUrl = try moveToDestination(from: uri, usingFilename: fileName, destinationDir: destinationDir)
-            return LocalCopyResponse.success(sourceUri: uri.absoluteString, localUri: destinationUrl.absoluteString)
-          } catch {
-            return LocalCopyResponse.error(sourceUri: dictionary["uri"]!, copyError: error.localizedDescription)
-          }
-        }
+    do {
+      try FileManager.default.createDirectory(at: destinationDir, withIntermediateDirectories: true, attributes: nil)
+    } catch {
+      return from.map { dictionary in
+        LocalCopyResponse.error(sourceUri: dictionary["uri"], copyError: "Failed to create destination directory: \(error.localizedDescription)").dictionaryRepresentation
       }
-      
-      for await result in group {
-        results.append(result.dictionaryRepresentation)
-      }
-      
-      return results
+    }
+    
+    // move files
+    return from.map { dictionary in
+      moveSingleFile(dictionary: dictionary, destinationDir: destinationDir).dictionaryRepresentation
+    }
+  }
+  
+  private static func moveSingleFile(dictionary: [String: String], destinationDir: URL) -> LocalCopyResponse {
+    guard let uriString = dictionary["uri"],
+          let uri = URL(string: uriString),
+          let fileName = dictionary["fileName"] else {
+      return LocalCopyResponse.error(
+        sourceUri: dictionary["uri"],
+        copyError: "Invalid URI or fileName"
+      )
+    }
+    
+    do {
+      let destinationUrl = try moveToDestination(from: uri, usingFilename: fileName, destinationDir: destinationDir)
+      return LocalCopyResponse.success(sourceUri: uri.absoluteString, localUri: destinationUrl.absoluteString)
+    } catch {
+      return LocalCopyResponse.error(sourceUri: uriString, copyError: error.localizedDescription)
     }
   }
   
@@ -50,19 +52,20 @@ import Foundation
     let destinationFile = destinationDir.appendingPathComponent(fileName).standardized
     
     guard destinationFile.path.hasPrefix(destinationDir.standardized.path) else {
-      throw NSError(domain: "PathTraversalPrevention", code: 400, userInfo: [NSLocalizedDescriptionKey: "The copied file is attempting to write outside of the target directory."])
+      throw NSError(
+        domain: "PathTraversalPrevention",
+        code: 400,
+        userInfo: [NSLocalizedDescriptionKey: "The copied file is attempting to write outside of the target directory."]
+      )
     }
     
-    try FileManager.default.createDirectory(at: destinationDir, withIntermediateDirectories: true, attributes: nil)
     try FileManager.default.moveItem(at: from, to: destinationFile)
     
     return destinationFile
   }
   
   static func getDirectoryForFileDestination(_ copyToDirectory: String) -> URL {
-    if copyToDirectory == "documentDirectory" {
-      return FileManager.default.urls(for: .documentDirectory, in: .userDomainMask).first!
-    }
-    return FileManager.default.urls(for: .cachesDirectory, in: .userDomainMask).first!
+    let searchPath: FileManager.SearchPathDirectory = copyToDirectory == "documentDirectory" ? .documentDirectory : .cachesDirectory
+    return FileManager.default.urls(for: searchPath, in: .userDomainMask).first!
   }
 }

package/ios/swift/IsKnownTypeImpl.swift

@@ -28,14 +28,10 @@ import UniformTypeIdentifiers
 
   static func createUTType(kind: String, value: String) -> UTType? {
     switch kind {
-    case "UTType":
-      return UTType(value)
-    case "mimeType":
-      return UTType(mimeType: value)
-    case "extension":
-      return UTType(filenameExtension: value)
-    default:
-      return nil
+    case "UTType": UTType(value)
+    case "mimeType": UTType(mimeType: value)
+    case "extension": UTType(filenameExtension: value)
+    default: nil
     }
   }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@react-native-documents/picker",
-  "version": "10.1.4",
+  "version": "10.1.6",
   "description": "A react native interface to access documents from dropbox, google drive, iCloud...",
   "main": "lib/commonjs/index",
   "module": "lib/module/index",