Change Android Permissions module to use promises

Author: cmcewen

Libraries/PermissionsAndroid/PermissionsAndroid.js

@@ -12,7 +12,7 @@
 'use strict';
 
 const DialogManagerAndroid = require('NativeModules').DialogManagerAndroid;
-const AndroidPermissions = require('NativeModules').AndroidPermissions;
+const Permissions = require('NativeModules').PermissionsAndroid;
 
 type Rationale = {
   title: string;
@@ -99,13 +99,7 @@ class PermissionsAndroid {
    * permissions has been granted
    */
   checkPermission(permission: string) : Promise<boolean> {
-    return new Promise((resolve, reject) => {
-      AndroidPermissions.checkPermission(
-        permission,
-        function(perm: string, result: boolean) { resolve(result); },
-        function(error: string) { reject(error); },
-      );
-    });
+    return Permissions.checkPermission(permission)
   }
 
   /**
@@ -119,36 +113,25 @@ class PermissionsAndroid {
    * and then shows the system permission dialog
    */
   requestPermission(permission: string, rationale?: Rationale) : Promise<boolean> {
-    return new Promise((resolve, reject) => {
-      const requestPermission = () => {
-        AndroidPermissions.requestPermission(
-          permission,
-          function(perm: string, result: boolean) { resolve(result); },
-          function(error: string) { reject(error); },
-        );
-      };
+    return new Promise(async (resolve, reject) => {
+      const requestPermission = () => resolve(Permissions.requestPermission(permission));
 
       if (rationale) {
-        AndroidPermissions.shouldShowRequestPermissionRationale(
-          permission,
-          function(perm: string, shouldShowRationale: boolean) {
-            if (shouldShowRationale) {
-              DialogManagerAndroid.showAlert(
-                rationale,
-                () => {
-                  DialogManagerAndroid.showAlert({
-                    message: 'Error Requesting Permissions'
-                  }, {}, {});
-                  reject();
-                },
-                requestPermission
-              );
-            } else {
-              requestPermission();
-            }
-          },
-          function(error: string) { reject(error); },
-        );
+        const shouldShowRationale = await Permissions.shouldShowRequestPermissionRationale(permission)
+        if (shouldShowRationale) {
+          DialogManagerAndroid.showAlert(
+            rationale,
+            () => {
+              DialogManagerAndroid.showAlert({
+                message: 'Error Requesting Permissions'
+              }, {}, {});
+              reject();
+            },
+            requestPermission
+          );
+        } else {
+          requestPermission();
+        }
       } else {
         requestPermission();
       }

ReactAndroid/src/main/java/com/facebook/react/modules/permissions/PermissionsModule.java

@@ -16,6 +16,7 @@
 import android.util.SparseArray;
 
 import com.facebook.react.bridge.Callback;
+import com.facebook.react.bridge.Promise;
 import com.facebook.react.bridge.ReactApplicationContext;
 import com.facebook.react.bridge.ReactContextBaseJavaModule;
 import com.facebook.react.bridge.ReactMethod;
@@ -37,29 +38,22 @@ public PermissionsModule(ReactApplicationContext reactContext) {
 
   @Override
   public String getName() {
-    return "AndroidPermissions";
+    return "PermissionsAndroid";
   }
 
   /**
    * Check if the app has the permission given. successCallback is called with true if the
    * permission had been granted, false otherwise. See {@link Activity#checkSelfPermission}.
    */
   @ReactMethod
-  public void checkPermission(
-      final String permission,
-      final Callback successCallback,
-      final Callback errorCallback) {
+  public void checkPermission(final String permission, final Promise promise) {
     PermissionAwareActivity activity = getPermissionAwareActivity();
     if (Build.VERSION.SDK_INT < Build.VERSION_CODES.M) {
-      successCallback.invoke(
-          permission,
-          activity.checkPermission(permission, Process.myPid(), Process.myUid()) ==
-              PackageManager.PERMISSION_GRANTED);
+      promise.resolve(activity.checkPermission(permission, Process.myPid(), Process.myUid()) ==
+        PackageManager.PERMISSION_GRANTED);
       return;
     }
-    successCallback.invoke(
-        permission,
-        activity.checkSelfPermission(permission) == PackageManager.PERMISSION_GRANTED);
+    promise.resolve(activity.checkSelfPermission(permission) == PackageManager.PERMISSION_GRANTED);
   }
 
   /**
@@ -71,17 +65,12 @@ public void checkPermission(
    * See {@link Activity#shouldShowRequestPermissionRationale}.
    */
   @ReactMethod
-  public void shouldShowRequestPermissionRationale(
-      final String permission,
-      final Callback successCallback,
-      final Callback errorCallback) {
+  public void shouldShowRequestPermissionRationale(final String permission, final Promise promise) {
     if (Build.VERSION.SDK_INT < Build.VERSION_CODES.M) {
-      successCallback.invoke(permission, false);
+      promise.resolve(false);
       return;
     }
-    successCallback.invoke(
-        permission,
-        getPermissionAwareActivity().shouldShowRequestPermissionRationale(permission));
+    promise.resolve(getPermissionAwareActivity().shouldShowRequestPermissionRationale(permission));
   }
 
   /**
@@ -91,28 +80,23 @@ public void shouldShowRequestPermissionRationale(
    * See {@link Activity#checkSelfPermission}.
    */
   @ReactMethod
-  public void requestPermission(
-      final String permission,
-      final Callback successCallback,
-      final Callback errorCallback) {
+  public void requestPermission(final String permission, final Promise promise) {
     PermissionAwareActivity activity = getPermissionAwareActivity();
     if (Build.VERSION.SDK_INT < Build.VERSION_CODES.M) {
-      successCallback.invoke(
-          permission,
-          activity.checkPermission(permission, Process.myPid(), Process.myUid()) ==
+      promise.resolve(activity.checkPermission(permission, Process.myPid(), Process.myUid()) ==
               PackageManager.PERMISSION_GRANTED);
       return;
     }
     if (activity.checkSelfPermission(permission) == PackageManager.PERMISSION_GRANTED) {
-      successCallback.invoke(permission, true);
+      promise.resolve(true);
       return;
     }
 
     mCallbacks.put(
         mRequestCode, new Callback() {
           @Override
           public void invoke(Object... args) {
-            successCallback.invoke(permission, args[0].equals(PackageManager.PERMISSION_GRANTED));
+            promise.resolve(args[0].equals(PackageManager.PERMISSION_GRANTED));
           }
         });