Cleanup SuLogger

This commit is contained in:
topjohnwu 2019-04-10 18:09:41 -04:00
parent 679db97209
commit 53c5ca59b6
2 changed files with 12 additions and 20 deletions

View File

@ -8,9 +8,7 @@ import com.topjohnwu.magisk.App;
import com.topjohnwu.magisk.ClassMap; import com.topjohnwu.magisk.ClassMap;
import com.topjohnwu.magisk.Config; import com.topjohnwu.magisk.Config;
import com.topjohnwu.magisk.Const; import com.topjohnwu.magisk.Const;
import com.topjohnwu.magisk.R;
import com.topjohnwu.magisk.SuRequestActivity; import com.topjohnwu.magisk.SuRequestActivity;
import com.topjohnwu.magisk.container.Policy;
import com.topjohnwu.magisk.uicomponents.Notifications; import com.topjohnwu.magisk.uicomponents.Notifications;
import com.topjohnwu.magisk.uicomponents.Shortcuts; import com.topjohnwu.magisk.uicomponents.Shortcuts;
import com.topjohnwu.magisk.utils.DownloadApp; import com.topjohnwu.magisk.utils.DownloadApp;
@ -19,14 +17,6 @@ import com.topjohnwu.superuser.Shell;
public class GeneralReceiver extends BroadcastReceiver { public class GeneralReceiver extends BroadcastReceiver {
private static SuLogger SU_LOGGER = new SuLogger() {
@Override
public String getMessage(Policy policy) {
return App.self.getString(policy.policy == Policy.ALLOW ?
R.string.su_allow_toast : R.string.su_deny_toast, policy.appName);
}
};
private String getPkg(Intent i) { private String getPkg(Intent i) {
return i.getData() == null ? "" : i.getData().getEncodedSchemeSpecificPart(); return i.getData() == null ? "" : i.getData().getEncodedSchemeSpecificPart();
} }
@ -54,10 +44,10 @@ public class GeneralReceiver extends BroadcastReceiver {
app.startActivity(i); app.startActivity(i);
break; break;
case "log": case "log":
SU_LOGGER.handleLogs(intent); SuLogger.handleLogs(intent);
break; break;
case "notify": case "notify":
SU_LOGGER.handleNotify(intent); SuLogger.handleNotify(intent);
break; break;
case "boot_complete": case "boot_complete":
default: default:

View File

@ -8,14 +8,15 @@ import android.widget.Toast;
import com.topjohnwu.magisk.App; import com.topjohnwu.magisk.App;
import com.topjohnwu.magisk.Config; import com.topjohnwu.magisk.Config;
import com.topjohnwu.magisk.R;
import com.topjohnwu.magisk.container.Policy; import com.topjohnwu.magisk.container.Policy;
import com.topjohnwu.magisk.container.SuLogEntry; import com.topjohnwu.magisk.container.SuLogEntry;
import java.util.Date; import java.util.Date;
public abstract class SuLogger { public class SuLogger {
public void handleLogs(Intent intent) { public static void handleLogs(Intent intent) {
int fromUid = intent.getIntExtra("from.uid", -1); int fromUid = intent.getIntExtra("from.uid", -1);
if (fromUid < 0) return; if (fromUid < 0) return;
@ -64,13 +65,16 @@ public abstract class SuLogger {
app.mDB.addLog(log); app.mDB.addLog(log);
} }
private void handleNotify(Policy policy) { private static void handleNotify(Policy policy) {
if (policy.notification && if (policy.notification &&
(int) Config.get(Config.Key.SU_NOTIFICATION) == Config.Value.NOTIFICATION_TOAST) (int) Config.get(Config.Key.SU_NOTIFICATION) == Config.Value.NOTIFICATION_TOAST) {
Utils.toast(getMessage(policy), Toast.LENGTH_SHORT); Utils.toast(App.self.getString(policy.policy == Policy.ALLOW ?
R.string.su_allow_toast : R.string.su_deny_toast, policy.appName),
Toast.LENGTH_SHORT);
}
} }
public void handleNotify(Intent intent) { public static void handleNotify(Intent intent) {
int fromUid = intent.getIntExtra("from.uid", -1); int fromUid = intent.getIntExtra("from.uid", -1);
if (fromUid < 0) return; if (fromUid < 0) return;
if (fromUid == Process.myUid()) return; if (fromUid == Process.myUid()) return;
@ -81,6 +85,4 @@ public abstract class SuLogger {
handleNotify(policy); handleNotify(policy);
} catch (PackageManager.NameNotFoundException ignored) {} } catch (PackageManager.NameNotFoundException ignored) {}
} }
public abstract String getMessage(Policy policy);
} }