fix permission check

This commit is contained in:
Dominik Schürmann 2013-09-06 14:02:35 +02:00
parent 51cf7939d8
commit 9611236bf1

View File

@ -120,12 +120,19 @@ public class OtherHelper {
public static void checkPackagePermissionForActions(Activity activity, String pkgName, public static void checkPackagePermissionForActions(Activity activity, String pkgName,
String permName, String action, String[] restrictedActions) { String permName, String action, String[] restrictedActions) {
if (action != null) { if (action != null) {
// PackageManager pkgManager = activity.getPackageManager(); PackageManager pkgManager = activity.getPackageManager();
for (int i = 0; i < restrictedActions.length; i++) {
if (restrictedActions[i].equals(action)) {
// TODO: currently always cancels! THis is the old API
// end activity
activity.setResult(Activity.RESULT_CANCELED, null);
activity.finish();
// for (int i = 0; i < restrictedActions.length; i++) {
// if (restrictedActions[i].equals(action)) {
// if (pkgName != null // if (pkgName != null
// && (pkgManager.checkPermission(permName, pkgName) == PackageManager.PERMISSION_GRANTED || pkgName // && (pkgManager.checkPermission(permName, pkgName) ==
// PackageManager.PERMISSION_GRANTED || pkgName
// .equals(Constants.PACKAGE_NAME))) { // .equals(Constants.PACKAGE_NAME))) {
// Log.d(Constants.TAG, pkgName + " has permission " + permName + ". Action " // Log.d(Constants.TAG, pkgName + " has permission " + permName + ". Action "
// + action + " was granted!"); // + action + " was granted!");
@ -140,13 +147,9 @@ public class OtherHelper {
// activity.setResult(Activity.RESULT_CANCELED, null); // activity.setResult(Activity.RESULT_CANCELED, null);
// activity.finish(); // activity.finish();
// } // }
// } }
// } }
// TODO: currently always cancels! THis is the old API
// end activity
activity.setResult(Activity.RESULT_CANCELED, null);
activity.finish();
} }
} }