@react-native-documents/picker 11.0.4 → 12.0.0

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

@@ -6,7 +6,6 @@ enum class CopyDestination(val preset: String) {
   DOCUMENT_DIRECTORY("documentDirectory");
 
   companion object {
-    // keep values() for RN 73 compatibility
-    fun fromPath(path: String): CopyDestination = values().find { it.preset == path } ?: CACHES_DIRECTORY
+    fun fromPath(path: String): CopyDestination = entries.find { it.preset == path } ?: CACHES_DIRECTORY
   }
 }

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

@@ -52,10 +52,11 @@ class DocumentMetadataBuilder(forUri: Uri) {
     openableMimeTypes?.let {
       val arrayOfExtensionsAndMime = Arguments.createArray()
       it.forEach { mimeType ->
-        val virtualFileDetails = Arguments.createMap()
         val maybeExtension = MimeTypeMap.getSingleton().getExtensionFromMimeType(mimeType)
-        virtualFileDetails.putString("mimeType", mimeType)
-        virtualFileDetails.putString("extension", maybeExtension)
+        val virtualFileDetails = Arguments.createMap().apply {
+          putString("mimeType", mimeType)
+          putString("extension", maybeExtension)
+        }
         arrayOfExtensionsAndMime.pushMap(virtualFileDetails)
       }
       map.putArray("convertibleToMimeTypes", arrayOfExtensionsAndMime)

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

@@ -189,8 +189,8 @@ class FileOperations(private val uriMap: MutableMap<String, Uri>) {
   }
 
   val copyStreamToAnother: (InputStream, OutputStream) -> Long = { inputStream, outputStream ->
-    inputStream.use { input ->
-      outputStream.use { output ->
+    inputStream.use { _ ->
+      outputStream.use { _ ->
         val bytesCopied = if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.Q) {
           FileUtils.copy(inputStream, outputStream)
         } else {

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

@@ -4,37 +4,35 @@ import android.webkit.MimeTypeMap
 import com.facebook.react.bridge.Arguments
 import com.facebook.react.bridge.WritableMap
 
-class IsKnownTypeImpl {
-  companion object {
-    fun isKnownType(kind: String, value: String): WritableMap {
-      return when (kind) {
-        "mimeType" -> {
-          val extensionForMime = MimeTypeMap.getSingleton().getExtensionFromMimeType(value)
-          createMap(
-            isKnown = extensionForMime != null,
-            preferredFilenameExtension = extensionForMime,
-            mimeType = if (extensionForMime != null) value else null
-          )
-        }
-        "extension" -> {
-          val mimeForExtension = MimeTypeMap.getSingleton().getMimeTypeFromExtension(value)
-          createMap(
-            isKnown = mimeForExtension != null,
-            preferredFilenameExtension = if (mimeForExtension != null) value else null,
-            mimeType = mimeForExtension
-          )
-        }
-        else -> createMap(isKnown = false, preferredFilenameExtension = null, mimeType = null)
+object IsKnownTypeImpl {
+  fun isKnownType(kind: String, value: String): WritableMap {
+    return when (kind) {
+      "mimeType" -> {
+        val extensionForMime = MimeTypeMap.getSingleton().getExtensionFromMimeType(value)
+        createMap(
+          isKnown = extensionForMime != null,
+          preferredFilenameExtension = extensionForMime,
+          mimeType = if (extensionForMime != null) value else null
+        )
       }
+      "extension" -> {
+        val mimeForExtension = MimeTypeMap.getSingleton().getMimeTypeFromExtension(value)
+        createMap(
+          isKnown = mimeForExtension != null,
+          preferredFilenameExtension = if (mimeForExtension != null) value else null,
+          mimeType = mimeForExtension
+        )
+      }
+      else -> createMap(isKnown = false, preferredFilenameExtension = null, mimeType = null)
     }
+  }
 
-    private fun createMap(isKnown: Boolean, preferredFilenameExtension: String?, mimeType: String?): WritableMap {
-      return Arguments.createMap().apply {
-        putNull("UTType")
-        putBoolean("isKnown", isKnown)
-        putString("preferredFilenameExtension", preferredFilenameExtension)
-        putString("mimeType", mimeType)
-      }
+  private fun createMap(isKnown: Boolean, preferredFilenameExtension: String?, mimeType: String?): WritableMap {
+    return Arguments.createMap().apply {
+      putNull("UTType")
+      putBoolean("isKnown", isKnown)
+      putString("preferredFilenameExtension", preferredFilenameExtension)
+      putString("mimeType", mimeType)
     }
   }
 }

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

@@ -75,17 +75,39 @@ class MetadataGetter(private val uriMap: MutableMap<String, Uri>) {
       contentResolver: ContentResolver,
       metadataBuilder: DocumentMetadataBuilder,
       couldBeVirtualFile: Boolean
+  ) {
+    try {
+      queryContentResolverMetadataInternal(contentResolver, metadataBuilder, couldBeVirtualFile)
+    } catch (e: Exception) {
+      val suppressedSummary =
+        e.suppressed.joinToString(separator = "; ") { suppressed ->
+            "${suppressed.javaClass.simpleName}: ${suppressed.message ?: "no message"}"
+          }
+      metadataBuilder.metadataReadingError(
+        "Could not read file metadata: ${e.javaClass.simpleName}: ${e.message ?: "no message"}" +
+          (" (suppressed summary: [$suppressedSummary])")
+      )
+    }
+  }
+
+  private fun queryContentResolverMetadataInternal(
+      contentResolver: ContentResolver,
+      metadataBuilder: DocumentMetadataBuilder,
+      couldBeVirtualFile: Boolean
   ) {
     val forUri = metadataBuilder.getUri()
+    val hasNoMime = !metadataBuilder.hasMime()
 
     val projection = mutableListOf(
-      DocumentsContract.Document.COLUMN_MIME_TYPE,
       OpenableColumns.DISPLAY_NAME,
       OpenableColumns.SIZE,
     ).apply {
       if (couldBeVirtualFile) {
         add(DocumentsContract.Document.COLUMN_FLAGS)
       }
+      if (hasNoMime) {
+        add(DocumentsContract.Document.COLUMN_MIME_TYPE)
+      }
     }.toTypedArray()
 
     contentResolver
@@ -97,39 +119,41 @@ class MetadataGetter(private val uriMap: MutableMap<String, Uri>) {
         null
       )
       .use { cursor ->
-        if (cursor != null && cursor.moveToFirst()) {
-          metadataBuilder.name(
-            getCursorValue(cursor, OpenableColumns.DISPLAY_NAME, String::class.java)
-          )
-
-          if (!metadataBuilder.hasMime()) {
-            metadataBuilder.mimeType(
-              getCursorValue(
-                cursor, DocumentsContract.Document.COLUMN_MIME_TYPE, String::class.java
-              )
+        if (cursor == null) {
+          metadataBuilder.metadataReadingError("Could not read file metadata because cursor was null. This is likely an issue with the underlying ContentProvider.")
+          return
+        }
+        if (!cursor.moveToFirst()) {
+          metadataBuilder.metadataReadingError("Could not read file metadata because cursor could not move to the first result row. This is likely an issue with the underlying ContentProvider. Row count: ${cursor.count}, columns: ${cursor.columnNames.joinToString(",")}")
+          return
+        }
+        metadataBuilder.name(
+          getCursorValue(cursor, OpenableColumns.DISPLAY_NAME, String::class.java)
+        )
+        metadataBuilder.size(getCursorValue(cursor, OpenableColumns.SIZE, Long::class.java))
+
+        if (hasNoMime) {
+          metadataBuilder.mimeType(
+            getCursorValue(
+              cursor, DocumentsContract.Document.COLUMN_MIME_TYPE, String::class.java
             )
-          }
+          )
+        }
 
-          if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.N) {
-            // https://developer.android.com/training/data-storage/shared/documents-files#open-virtual-file
-            val isVirtual =
-              if (couldBeVirtualFile) {
-                val cursorValue: Int =
-                  getCursorValue(
-                    cursor, DocumentsContract.Document.COLUMN_FLAGS, Int::class.java
-                  )
-                    ?: 0
-                cursorValue and DocumentsContract.Document.FLAG_VIRTUAL_DOCUMENT != 0
-              } else {
-                false
-              }
-            metadataBuilder.virtual(isVirtual)
-          }
-          metadataBuilder.size(getCursorValue(cursor, OpenableColumns.SIZE, Long::class.java))
-        } else {
-          // metadataBuilder only contains the uri, type and error in this unlikely case
-          // there's nothing more we can do
-          metadataBuilder.metadataReadingError("Could not read file metadata")
+        if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.N) {
+          // https://developer.android.com/training/data-storage/shared/documents-files#open-virtual-file
+          val isVirtual =
+            if (couldBeVirtualFile) {
+              val cursorValue: Int =
+                getCursorValue(
+                  cursor, DocumentsContract.Document.COLUMN_FLAGS, Int::class.java
+                )
+                  ?: 0
+              cursorValue and DocumentsContract.Document.FLAG_VIRTUAL_DOCUMENT != 0
+            } else {
+              false
+            }
+          metadataBuilder.virtual(isVirtual)
         }
       }
   }
@@ -137,19 +161,19 @@ class MetadataGetter(private val uriMap: MutableMap<String, Uri>) {
   @Suppress("UNCHECKED_CAST")
   private fun <T> getCursorValue(cursor: Cursor, columnName: String, valueType: Class<T>): T? {
     val columnIndex = cursor.getColumnIndex(columnName)
-    if (columnIndex != -1 && !cursor.isNull(columnIndex)) {
-      return runCatching {
-        when (valueType) {
-          String::class.java -> cursor.getString(columnIndex) as T
-          Int::class.java -> cursor.getInt(columnIndex) as T
-          Long::class.java -> cursor.getLong(columnIndex) as T
-          Double::class.java -> cursor.getDouble(columnIndex) as T
-          Float::class.java -> cursor.getFloat(columnIndex) as T
-          else -> null
-        }
-      // throw should not happen but if it does, we return null
-      }.getOrNull()
+    if (columnIndex == -1 || cursor.isNull(columnIndex)) {
+      return null
     }
-    return null
+    return runCatching {
+      when (valueType) {
+        String::class.java -> cursor.getString(columnIndex) as T
+        Int::class.java -> cursor.getInt(columnIndex) as T
+        Long::class.java -> cursor.getLong(columnIndex) as T
+        Double::class.java -> cursor.getDouble(columnIndex) as T
+        Float::class.java -> cursor.getFloat(columnIndex) as T
+        else -> null
+      }
+      // throw should not happen but if it does, we return null
+    }.getOrNull()
   }
 }

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

@@ -2,6 +2,8 @@
 package com.reactnativedocumentpicker
 
 import android.content.Intent
+import android.os.Build
+import android.provider.DocumentsContract
 import com.facebook.react.bridge.ReadableArray
 import com.facebook.react.bridge.ReadableMap
 
@@ -14,6 +16,20 @@ data class PickOptions(
   val requestLongTermAccess: Boolean,
   val allowVirtualFiles: Boolean,
 ) {
+  constructor(readableMap: ReadableMap) : this(
+    mode = readableMap.getString("mode"),
+    mimeTypes = if (readableMap.hasKey("type") && !readableMap.isNull("type")) {
+      readableMap.getArray("type")?.let { readableArrayToStringArray(it) } ?: arrayOf("*/*")
+    } else {
+      arrayOf("*/*")
+    },
+    initialDirectoryUrl = if (readableMap.hasKey("initialDirectoryUrl")) readableMap.getString("initialDirectoryUrl") else null,
+    localOnly = readableMap.hasKey("localOnly") && readableMap.getBoolean("localOnly"),
+    multiple = readableMap.hasKey("allowMultiSelection") && readableMap.getBoolean("allowMultiSelection"),
+    requestLongTermAccess = readableMap.hasKey("requestLongTermAccess") && readableMap.getBoolean("requestLongTermAccess"),
+    allowVirtualFiles = readableMap.hasKey("allowVirtualFiles") && readableMap.getBoolean("allowVirtualFiles")
+  )
+
   val action: String
     get() = if ("open" == mode) Intent.ACTION_OPEN_DOCUMENT else Intent.ACTION_GET_CONTENT
 
@@ -26,33 +42,37 @@ data class PickOptions(
       mimeTypes.joinToString("|")
     }
   }
-}
 
-fun parsePickOptions(readableMap: ReadableMap): PickOptions {
-  val mode = readableMap.getString("mode")
+  fun getPickIntent(): Intent {
+    // TODO option for extra task on stack?
+    // reminder - flags are for granting rights to others
 
-  val mimeTypes = if (readableMap.hasKey("type") && !readableMap.isNull("type")) {
-    readableMap.getArray("type")?.let { readableArrayToStringArray(it) } ?: arrayOf("*/*")
-  } else {
-    arrayOf("*/*")
-  }
+    return Intent(action).apply {
+      val types = mimeTypes
 
-  val initialDirectoryUrl = if (readableMap.hasKey("initialDirectoryUrl")) readableMap.getString("initialDirectoryUrl") else null
-  val localOnly = readableMap.hasKey("localOnly") && readableMap.getBoolean("localOnly")
-  val multiple = readableMap.hasKey("allowMultiSelection") && readableMap.getBoolean("allowMultiSelection")
-  val requestLongTermAccess = readableMap.hasKey("requestLongTermAccess") && readableMap.getBoolean("requestLongTermAccess")
-  val allowVirtualFiles = readableMap.hasKey("allowVirtualFiles") && readableMap.getBoolean("allowVirtualFiles")
-
-  return PickOptions(
-    mode = mode,
-    mimeTypes = mimeTypes,
-    initialDirectoryUrl = initialDirectoryUrl,
-    localOnly = localOnly,
-    multiple = multiple,
-    requestLongTermAccess = requestLongTermAccess,
-    allowVirtualFiles = allowVirtualFiles,
-  )
+      type =
+        if (types.size > 1) {
+          putExtra(Intent.EXTRA_MIME_TYPES, types)
+          intentFilterTypes
+        } else {
+          types[0]
+        }
+      if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O &&
+        initialDirectoryUrl != null
+      ) {
+        // only works for ACTION_OPEN_DOCUMENT
+        // TODO must be URI
+        putExtra(DocumentsContract.EXTRA_INITIAL_URI, initialDirectoryUrl)
+      }
+      if (!allowVirtualFiles) {
+        addCategory(Intent.CATEGORY_OPENABLE)
+      }
+      putExtra(Intent.EXTRA_LOCAL_ONLY, localOnly)
+      putExtra(Intent.EXTRA_ALLOW_MULTIPLE, multiple)
+    }
+  }
 }
+
 fun readableArrayToStringArray(readableArray: ReadableArray): Array<String> {
   /**
    * MIME type and Uri scheme matching in the

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

@@ -0,0 +1,83 @@
+// LICENSE: see License.md in the package root
+package com.reactnativedocumentpicker
+
+import android.util.Log
+import com.facebook.react.bridge.Promise
+
+class PromiseWrapper(private val MODULE_NAME: String) {
+  private var promise: Promise? = null
+  private var nameOfCallInProgress: String? = null
+
+  fun trySetPromiseRejectingIncoming(promise: Promise, fromCallsite: String): Boolean {
+    val previousPromise = this.promise
+    if (previousPromise != null) {
+      rejectNewPromiseBecauseOldOneIsInProgress(promise, fromCallsite)
+      return false
+    }
+    this.promise = promise
+    nameOfCallInProgress = fromCallsite
+    return true
+  }
+
+  fun resolve(value: Any?) {
+    val resolver = promise
+    if (resolver == null) {
+      Log.e(MODULE_NAME, "cannot resolve promise because it's null")
+      return
+    }
+
+    resetMembers()
+    resolver.resolve(value)
+  }
+
+  fun reject(code: String, e: Exception) {
+    val message =
+      e.localizedMessage ?: e.message ?: "unknown error"
+
+    this.reject(code, message, e)
+  }
+
+  fun reject(e: Exception) {
+    val message =
+      e.localizedMessage ?: e.message ?: "unknown error"
+
+    this.reject(nameOfCallInProgress ?: "unknown call in progress", message, e)
+  }
+
+  fun rejectAsUserCancelledOperation() {
+    this.reject(E_DOCUMENT_PICKER_CANCELED, "user canceled the document picker")
+  }
+
+  fun reject(code: String, message: String, e: Exception? = null) {
+    val rejecter = promise
+    if (rejecter == null) {
+      Log.e(MODULE_NAME, "cannot reject promise because it's null")
+      return
+    }
+
+    resetMembers()
+    rejecter.reject(code, message, e)
+  }
+
+  private fun resetMembers() {
+    nameOfCallInProgress = null
+    promise = null
+  }
+
+  private fun rejectNewPromiseBecauseOldOneIsInProgress(
+    promise: Promise,
+    requestedOperation: String?
+  ) {
+    // TODO better message
+    promise.reject(
+      ASYNC_OP_IN_PROGRESS,
+      "Warning: previous promise did not settle and you attempted to overwrite it. " +
+              "You've called \"" + requestedOperation + "\" while \"" + this.nameOfCallInProgress + "\" was already in progress and has not completed yet."
+    )
+  }
+
+  companion object {
+    const val ASYNC_OP_IN_PROGRESS: String = "ASYNC_OP_IN_PROGRESS"
+    const val E_DOCUMENT_PICKER_CANCELED: String = "OPERATION_CANCELED"
+  }
+}

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

@@ -86,11 +86,11 @@ class RNDocumentPickerModule(reactContext: ReactApplicationContext) :
     if (!promiseWrapper.trySetPromiseRejectingIncoming(promise, "pick")) {
       return
     }
-    val options = parsePickOptions(opts)
+    val options = PickOptions(opts)
     currentPickOptions = options
 
     try {
-      val intent = IntentFactory.getPickIntent(options)
+      val intent = options.getPickIntent()
       currentActivity.startActivityForResult(intent, PICK_FILES_REQUEST_CODE)
     } catch (e: ActivityNotFoundException) {
       promiseWrapper.reject(UNABLE_TO_OPEN_FILE_TYPE, e)
@@ -140,7 +140,7 @@ class RNDocumentPickerModule(reactContext: ReactApplicationContext) :
     if (!promiseWrapper.trySetPromiseRejectingIncoming(promise, "pickDirectory")) {
       return
     }
-    val options = parsePickOptions(opts)
+    val options = PickOptions(opts)
     currentPickOptions = options
     try {
       val intent =
@@ -316,13 +316,13 @@ class RNDocumentPickerModule(reactContext: ReactApplicationContext) :
 
   companion object {
     fun rejectWithNullActivity(promise: Promise) {
-      promise.reject(PRESENTER_IS_NULL, PRESENTER_IS_NULL)
+      promise.reject(NULL_PRESENTER, "Current activity is null. Cannot present sign-in UI. Make sure there are no modal windows being presented or dismissed.")
     }
 
     private const val PICK_FILES_REQUEST_CODE = 41
     private const val PICK_DIR_REQUEST_CODE = 42
     private const val SAVE_DOC_REQUEST_CODE = 43
-    private const val PRESENTER_IS_NULL = "NULL_PRESENTER"
+    private const val NULL_PRESENTER = "NULL_PRESENTER"
     private const val UNABLE_TO_OPEN_FILE_TYPE = "UNABLE_TO_OPEN_FILE_TYPE"
     private const val E_OTHER_PRESENTING_ERROR = "OTHER_PRESENTING_ERROR"
     private const val E_INVALID_DATA_RETURNED = "INVALID_DATA_RETURNED"

package/ios/RNDocumentPicker.mm

@@ -18,20 +18,23 @@
 @end
 
 @implementation RNDocumentPicker {
-  DocPicker *docPicker;
-  DocSaver *docSaver;
+  DocPicker *_docPicker;
+  DocSaver *_docSaver;
 }
 
-- (instancetype)init {
-  if ((self = [super init])) {
-    docPicker = [DocPicker new];
-    docSaver = [DocSaver new];
+// initialization happens on serial queue so there are no races
+- (DocPicker *)docPicker {
+  if (!_docPicker) {
+    _docPicker = [DocPicker new];
   }
-  return self;
+  return _docPicker;
 }
 
-+ (BOOL)requiresMainQueueSetup {
-  return NO;
+- (DocSaver *)docSaver {
+  if (!_docSaver) {
+    _docSaver = [DocSaver new];
+  }
+  return _docSaver;
 }
 
 RCT_EXPORT_MODULE()
@@ -43,26 +46,9 @@ RCT_EXPORT_METHOD(pick:
                   reject:
                   (RCTPromiseRejectBlock) reject)
 {
-  // https://stackoverflow.com/questions/5270519/what-is-difference-between-uimodaltransitionstyle-and-uimodalpresentationstyle
-  UIModalPresentationStyle presentationStyle = [RCTConvert UIModalPresentationStyle:options[@"presentationStyle"]];
-  UIModalTransitionStyle transitionStyle = [RCTConvert UIModalTransitionStyle:options[@"transitionStyle"]];
-  NSArray *allowedUTIs = [RCTConvert NSArray:options[@"type"]];
-  BOOL allowMultiple = [RCTConvert BOOL:options[@"allowMultiSelection"]];
-  BOOL showExtensions = [RCTConvert BOOL:options[@"showFileExtensions"]];
-  NSString *mode = options[@"mode"];
-  NSString *initialDir = options[@"initialDirectoryUrl"];
-  BOOL requestLongTermAccess = [RCTConvert BOOL:options[@"requestLongTermAccess"]];
-  
-  PickerOptions *pickerOptions = [[PickerOptions alloc] initWithTypes:allowedUTIs
-                                                                 mode:mode
-                                                  initialDirectoryUrl:initialDir
-                                                  allowMultiSelection:allowMultiple
-                                             shouldShowFileExtensions:showExtensions
-                                                      transitionStyle:transitionStyle
-                                                    presentationStyle:presentationStyle
-                                                requestLongTermAccess:requestLongTermAccess];
-  
-  [docPicker presentWithOptions:pickerOptions resolve:resolve reject:reject];
+  dispatch_async(dispatch_get_main_queue(), ^{
+    [self.docPicker presentWithOptionsDict:options resolve:resolve reject:reject];
+  });
 }
 
 RCT_EXPORT_METHOD(pickDirectory:(NSDictionary *)options resolve:(RCTPromiseResolveBlock)resolve reject:(RCTPromiseRejectBlock)reject) {
@@ -92,21 +78,13 @@ RCT_EXPORT_SYNCHRONOUS_TYPED_METHOD(NSDictionary *, isKnownType:(NSString *)kind
 }
 
 RCT_EXPORT_METHOD(writeDocuments:(NSDictionary *)options resolve:(RCTPromiseResolveBlock)resolve reject:(RCTPromiseRejectBlock)reject) {
-  UIModalPresentationStyle presentationStyle = [RCTConvert UIModalPresentationStyle:options[@"presentationStyle"]];
-  UIModalTransitionStyle transitionStyle = [RCTConvert UIModalTransitionStyle:options[@"transitionStyle"]];
-  BOOL showExtensions = [RCTConvert BOOL:options[@"showFileExtensions"]];
-  BOOL asCopy = [RCTConvert BOOL:options[@"copy"]];
-
-  NSString *initialDir = options[@"initialDirectoryUri"];
-  NSArray<NSString*> *documentUrl = options[@"sourceUris"];
-  
-  SaverOptions* saverOptions = [[SaverOptions alloc] initWithSourceUrlStrings:documentUrl asCopy:asCopy initialDirectoryUrl:initialDir shouldShowFileExtensions:showExtensions transitionStyle:transitionStyle presentationStyle:presentationStyle];
-  
-  [docSaver presentWithOptions:saverOptions resolve:resolve reject:reject];
+  dispatch_async(dispatch_get_main_queue(), ^{
+    [self.docSaver presentWithOptionsDict:options resolve:resolve reject:reject];
+  });
 }
 
 RCT_EXPORT_METHOD(releaseSecureAccess:(NSArray *)uris resolve:(RCTPromiseResolveBlock)resolve reject:(RCTPromiseRejectBlock)reject) {
-  [docPicker stopAccessingOpenedUrls:uris];
+  [self.docPicker stopAccessingOpenedUrls:uris];
   resolve([NSNull null]);
 }