Merge pull request #249 from shiraji/use_PermissionChecker

hotchemi · web-flow · commit b9cbc0f5e3eb · 2016-12-26T00:08:40.000+09:00
Use PermissionChecker's checkSelfPermission
diff --git a/library/src/main/android/permissions/dispatcher/PermissionUtils.java b/library/src/main/android/permissions/dispatcher/PermissionUtils.java
@@ -110,7 +110,7 @@ private static boolean hasSelfPermission(Context context, String permission) {
     private static boolean hasSelfPermissionForXiaomi(Context context, String permission) {
         AppOpsManager appOpsManager = (AppOpsManager) context.getSystemService(Context.APP_OPS_SERVICE);
         int checkOp = appOpsManager.checkOp(AppOpsManager.permissionToOp(permission), Process.myUid(), context.getPackageName());
-        return checkOp == AppOpsManager.MODE_ALLOWED && ActivityCompat.checkSelfPermission(context, permission) == PackageManager.PERMISSION_GRANTED;
+        return checkOp == AppOpsManager.MODE_ALLOWED && checkSelfPermission(context, permission) == PackageManager.PERMISSION_GRANTED;
     }
 
     /**

Original file line number	Diff line number	Diff line change
`@@ -110,7 +110,7 @@ private static boolean hasSelfPermission(Context context, String permission) {`
`110`	`110`	`private static boolean hasSelfPermissionForXiaomi(Context context, String permission) {`
`111`	`111`	`AppOpsManager appOpsManager = (AppOpsManager) context.getSystemService(Context.APP_OPS_SERVICE);`
`112`	`112`	`int checkOp = appOpsManager.checkOp(AppOpsManager.permissionToOp(permission), Process.myUid(), context.getPackageName());`
`113`		`- return checkOp == AppOpsManager.MODE_ALLOWED && ActivityCompat.checkSelfPermission(context, permission) == PackageManager.PERMISSION_GRANTED;`
	`113`	`+ return checkOp == AppOpsManager.MODE_ALLOWED && checkSelfPermission(context, permission) == PackageManager.PERMISSION_GRANTED;`
`114`	`114`	`}`
`115`	`115`
`116`	`116`	`/**`