Skip to content

Commit

Permalink
Fix issue where AppOps from App info did not work
Browse files Browse the repository at this point in the history
  • Loading branch information
jclehner committed Jan 6, 2014
1 parent b50b429 commit 358d379
Showing 1 changed file with 11 additions and 8 deletions.
19 changes: 11 additions & 8 deletions src/at/jclehner/appopsxposed/ApkVariant.java
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,7 @@

import android.app.Fragment;
import android.content.Context;
import android.content.Intent;
import android.os.Build;
import android.os.Bundle;
import android.preference.PreferenceActivity;
Expand All @@ -38,11 +39,9 @@
import at.jclehner.appopsxposed.variants.HTC;
import at.jclehner.appopsxposed.variants.Samsung;
import at.jclehner.appopsxposed.variants.Sony;
import de.robv.android.xposed.IXposedHookInitPackageResources;
import de.robv.android.xposed.IXposedHookLoadPackage;
import de.robv.android.xposed.XposedBridge;
import de.robv.android.xposed.XposedHelpers;
import de.robv.android.xposed.callbacks.XC_InitPackageResources.InitPackageResourcesParam;
import de.robv.android.xposed.callbacks.XC_LoadPackage.LoadPackageParam;

/**
Expand All @@ -58,7 +57,7 @@
*/
public abstract class ApkVariant implements IXposedHookLoadPackage
{
private static final boolean USE_INDICATOR_CLASSES = false;
private static final boolean USE_INDICATOR_CLASSES = true;

protected final String ANY = "";

Expand Down Expand Up @@ -291,10 +290,8 @@ public boolean onMenuItemClick(MenuItem item)
final Fragment f = (Fragment) param.thisObject;
final PreferenceActivity pa = (PreferenceActivity) f.getActivity();

debug("onMenuItemClick:" + "\n intent=" + pa.getIntent() +
"\n extras=" + pa.getIntent().getExtras());

final Bundle args = f.getArguments() == null ? new Bundle() : f.getArguments();

if(!args.containsKey("package"))
{
String pkg;
Expand All @@ -312,6 +309,10 @@ public boolean onMenuItemClick(MenuItem item)

if(pkg == null || pkg.isEmpty())
{
log("No package in intent or args:\n" +
" intent=" + pa.getIntent() + "\n" +
" extras=" + pa.getIntent().getExtras() + "\n" +
" args=" + args);
Toast.makeText(f.getActivity(), "Error!", Toast.LENGTH_SHORT).show();
return true;
}
Expand All @@ -321,9 +322,11 @@ public boolean onMenuItemClick(MenuItem item)
else
log("Package obtained from Fragment args: " + args.getString("package"));

final Intent intent = new Intent("android.settings.SETTINGS");
intent.putExtra(PreferenceActivity.EXTRA_SHOW_FRAGMENT, AppOpsXposed.APP_OPS_DETAILS_FRAGMENT);
intent.putExtra(PreferenceActivity.EXTRA_SHOW_FRAGMENT_ARGUMENTS, args);

pa.startPreferencePanel(AppOpsXposed.APP_OPS_DETAILS_FRAGMENT, args,
Util.getSettingsIdentifier("string/app_ops_settings"), null, f, 1);
pa.startActivity(intent);
return true;
}
});
Expand Down

0 comments on commit 358d379

Please sign in to comment.