Workaround for expo/expo#39480.

Author: jameswilddev

react-native/services/PermissionHelper/index.tsx

@@ -17,12 +17,25 @@ export class PermissionHelper implements PermissionHelperInterface {
    *                    might only grant access to a small subset of resources.
    */
   async acquire (
-    permissions: ReadonlyArray<() => Promise<PermissionResponse>>,
+    permissions: ReadonlyArray<{
+      get: () => Promise<PermissionResponse>
+      request: () => Promise<PermissionResponse>
+    }>,
     onFailure: (showSettingsScreen: () => Promise<void>) => Promise<void>,
     onSuccess: () => Promise<void>
   ): Promise<void> {
-    for (const permission of permissions) {
-      if (!(await permission()).granted) {
+    for (const { get, request } of permissions) {
+      const got = await get()
+
+      if (!got.granted) {
+        if (got.canAskAgain) {
+          const requested = await request()
+
+          if (requested.granted) {
+            continue
+          }
+        }
+
         await onFailure(showSettingsScreen)
         return
       }

react-native/services/PermissionHelper/unit.tsx

@@ -4,21 +4,36 @@ test('executes the success callback when permissions are granted', async () => {
   const onFailure = jest.fn().mockResolvedValue(undefined)
   const onSuccess = jest.fn().mockResolvedValue(undefined)
 
-  const firstPermission = jest.fn().mockResolvedValue({ granted: true })
-  const secondPermission = jest.fn().mockResolvedValue({ granted: true })
-  const thirdPermission = jest.fn().mockResolvedValue({ granted: true })
+  const getFirstPermission = jest.fn().mockResolvedValue({ granted: false })
+  const requestFirstPermission = jest.fn().mockResolvedValue({ granted: true })
+  const getSecondPermission = jest.fn().mockResolvedValue({ granted: false })
+  const requestSecondPermission = jest.fn().mockResolvedValue({ granted: true })
+  const getThirdPermission = jest.fn().mockResolvedValue({ granted: false })
+  const requestThirdPermission = jest.fn().mockResolvedValue({ granted: true })
 
   const permissionHelper = new PermissionHelper()
 
   await permissionHelper.acquire(
-    [firstPermission, secondPermission, thirdPermission],
+    [{
+      get: getFirstPermission,
+      request: requestFirstPermission
+    }, {
+      get: getSecondPermission,
+      request: requestSecondPermission
+    }, {
+      get: getThirdPermission,
+      request: requestThirdPermission
+    }],
     onFailure,
     onSuccess
   )
 
-  expect(firstPermission).toHaveBeenCalledTimes(1)
-  expect(secondPermission).toHaveBeenCalledTimes(1)
-  expect(thirdPermission).toHaveBeenCalledTimes(1)
+  expect(getFirstPermission).toHaveBeenCalledTimes(1)
+  expect(requestFirstPermission).toHaveBeenCalledTimes(1)
+  expect(getSecondPermission).toHaveBeenCalledTimes(1)
+  expect(requestSecondPermission).toHaveBeenCalledTimes(1)
+  expect(getThirdPermission).toHaveBeenCalledTimes(1)
+  expect(requestThirdPermission).toHaveBeenCalledTimes(1)
   expect(onFailure).not.toHaveBeenCalled()
   expect(onSuccess).toBeCalledTimes(1)
 })
@@ -27,21 +42,36 @@ test('executes the failure callback when permissions are denied', async () => {
   const onFailure = jest.fn().mockResolvedValue(undefined)
   const onSuccess = jest.fn().mockResolvedValue(undefined)
 
-  const firstPermission = jest.fn().mockResolvedValue({ granted: true })
-  const secondPermission = jest.fn().mockResolvedValue({ granted: false })
-  const thirdPermission = jest.fn()
+  const getFirstPermission = jest.fn().mockResolvedValue({ granted: false })
+  const requestFirstPermission = jest.fn().mockResolvedValue({ granted: true })
+  const getSecondPermission = jest.fn().mockResolvedValue({ granted: false })
+  const requestSecondPermission = jest.fn().mockResolvedValue({ granted: false })
+  const getThirdPermission = jest.fn().mockResolvedValue({ granted: false })
+  const requestThirdPermission = jest.fn()
 
   const permissionHelper = new PermissionHelper()
 
   await permissionHelper.acquire(
-    [firstPermission, secondPermission, thirdPermission],
+    [{
+      get: getFirstPermission,
+      request: requestFirstPermission
+    }, {
+      get: getSecondPermission,
+      request: requestSecondPermission
+    }, {
+      get: getThirdPermission,
+      request: requestThirdPermission
+    }],
     onFailure,
     onSuccess
   )
 
-  expect(firstPermission).toHaveBeenCalledTimes(1)
-  expect(secondPermission).toHaveBeenCalledTimes(1)
-  expect(thirdPermission).not.toHaveBeenCalled()
+  expect(getFirstPermission).toHaveBeenCalledTimes(1)
+  expect(requestFirstPermission).toHaveBeenCalledTimes(1)
+  expect(getSecondPermission).toHaveBeenCalledTimes(1)
+  expect(requestSecondPermission).toHaveBeenCalledTimes(1)
+  expect(getThirdPermission).not.toHaveBeenCalled()
+  expect(requestThirdPermission).not.toHaveBeenCalled()
   expect(onFailure).toBeCalledTimes(1)
   expect(onFailure).toBeCalledWith(showSettingsScreen)
   expect(onSuccess).not.toHaveBeenCalled()

react-native/services/PictureHelper/index.tsx

@@ -28,7 +28,13 @@ export class PictureHelper implements PictureHelperInterface {
     onSuccess: (uuid: string) => Promise<void>
   ): Promise<void> {
     await this.permissionHelper.acquire(
-      [ImagePicker.requestCameraPermissionsAsync, ImagePicker.requestMediaLibraryPermissionsAsync],
+      [{
+        get: ImagePicker.getCameraPermissionsAsync,
+        request: ImagePicker.requestCameraPermissionsAsync
+      }, {
+        get: ImagePicker.getMediaLibraryPermissionsAsync,
+        request: ImagePicker.requestMediaLibraryPermissionsAsync
+      }],
       onPermissionDenied,
       async () => {
         const result = await ImagePicker.launchCameraAsync({
@@ -68,7 +74,10 @@ export class PictureHelper implements PictureHelperInterface {
     onSuccess: (uuid: string) => Promise<void>
   ): Promise<void> {
     await this.permissionHelper.acquire(
-      [ImagePicker.requestMediaLibraryPermissionsAsync],
+      [{
+        get: ImagePicker.getMediaLibraryPermissionsAsync,
+        request: ImagePicker.requestMediaLibraryPermissionsAsync
+      }],
       onPermissionDenied,
       async () => {
         const result = await ImagePicker.launchImageLibraryAsync({
@@ -101,7 +110,10 @@ export class PictureHelper implements PictureHelperInterface {
     onSuccess: (uuids: readonly string[]) => Promise<void>
   ): Promise<void> {
     await this.permissionHelper.acquire(
-      [ImagePicker.requestMediaLibraryPermissionsAsync],
+      [{
+        get: ImagePicker.getMediaLibraryPermissionsAsync,
+        request: ImagePicker.requestMediaLibraryPermissionsAsync
+      }],
       onPermissionDenied,
       async () => {
         const result = await ImagePicker.launchImageLibraryAsync({

react-native/types/PermissionHelperInterface/index.tsx

@@ -15,7 +15,10 @@ export interface PermissionHelperInterface {
    *                    might only grant access to a small subset of resources.
    */
   acquire: (
-    permissions: ReadonlyArray<() => Promise<PermissionResponse>>,
+    permissions: ReadonlyArray<{
+      get: () => Promise<PermissionResponse>
+      request: () => Promise<PermissionResponse>
+    }>,
     onFailure: (showSettingsScreen: () => Promise<void>) => Promise<void>,
     onSuccess: () => Promise<void>
   ) => Promise<void>