Merge pull request #8 from rcpfuchs/master

fixed a bug that only one permission was shown to accept when multipl…
k0shk0sh · Mar 28, 2016 · 29300ce · 29300ce
2 parents 8a40c54 + 5a93aa0
commit 29300ce
Showing 1 changed file with 9 additions and 2 deletions.
diff --git a/permission/src/main/java/com/fastaccess/permission/base/PermissionHelper.java b/permission/src/main/java/com/fastaccess/permission/base/PermissionHelper.java
@@ -195,13 +195,20 @@ public void requestAfterExplanation(@NonNull String permissionName) {
      * to be called when explanation is presented to the user
      */
     public void requestAfterExplanation(@NonNull String[] permissions) {
+
+        ArrayList<String> permissionsToRequest = new ArrayList<>();
+
         for (String permissionName : permissions) {
             if (isPermissionDeclined(permissionName)) {
-                ActivityCompat.requestPermissions(context, new String[]{permissionName}, REQUEST_PERMISSIONS);
+                permissionsToRequest.add(permissionName); // add permission to request
             } else {
-                permissionCallback.onPermissionPreGranted(permissionName);
+                permissionCallback.onPermissionPreGranted(permissionName); // do not request, since it is already granted
             }
         }
+
+        permissions = permissionsToRequest.toArray(new String[permissionsToRequest.size()]);
+
+        ActivityCompat.requestPermissions(context, permissions, REQUEST_PERMISSIONS);
     }
 
     /**