From 1d4a57a393f7c822bd61494756bd3fbfc2814207 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Sebastian=20K=C3=BCrten?= <sebastian@mobanisto.de> Date: Tue, 22 Feb 2022 12:57:43 +0100 Subject: [PATCH] Rename string resources and clean up dnkf build file --- dont-kill-me-lib/build.gradle | 10 ++---- .../android/dontkillmelib/PowerUtils.java | 18 +++++----- .../src/main/res/values/strings.xml | 36 +++++++++---------- .../AbstractDoNotKillMeFragment.java | 5 ++- .../android/dontkillme/DoNotKillMeUtils.java | 3 +- .../mailbox/android/dontkillme/DozeView.java | 7 ++-- .../dontkillme/HuaweiAppLaunchView.java | 7 ++-- .../dontkillme/HuaweiProtectedAppsView.java | 8 ++--- .../android/dontkillme/XiaomiView.java | 15 ++++---- .../main/res/layout/fragment_dont_kill_me.xml | 2 +- .../src/main/res/layout/power_view.xml | 8 ++--- 11 files changed, 55 insertions(+), 64 deletions(-) diff --git a/dont-kill-me-lib/build.gradle b/dont-kill-me-lib/build.gradle index a2af387e..8e3efb0b 100644 --- a/dont-kill-me-lib/build.gradle +++ b/dont-kill-me-lib/build.gradle @@ -1,16 +1,14 @@ plugins { id 'com.android.library' - id 'checkstyle' } android { compileSdkVersion 31 defaultConfig { - minSdkVersion 16 - targetSdkVersion 30 + minSdk 16 + targetSdk 30 - vectorDrawables.useSupportLibrary = true consumerProguardFiles "consumer-rules.pro" } @@ -27,7 +25,5 @@ android { } dependencies { - implementation "androidx.fragment:fragment:$androidx_fragment_version" - implementation "androidx.constraintlayout:constraintlayout:$androidx_constraintlayout_version" - implementation "com.google.android.material:material:$google_material_version" + implementation "androidx.annotation:annotation:1.3.0" } diff --git a/dont-kill-me-lib/src/main/java/org/briarproject/android/dontkillmelib/PowerUtils.java b/dont-kill-me-lib/src/main/java/org/briarproject/android/dontkillmelib/PowerUtils.java index 36cab27c..639ac03d 100644 --- a/dont-kill-me-lib/src/main/java/org/briarproject/android/dontkillmelib/PowerUtils.java +++ b/dont-kill-me-lib/src/main/java/org/briarproject/android/dontkillmelib/PowerUtils.java @@ -1,5 +1,12 @@ package org.briarproject.android.dontkillmelib; +import static android.content.Context.POWER_SERVICE; +import static android.content.pm.PackageManager.MATCH_DEFAULT_ONLY; +import static android.os.Build.BRAND; +import static android.os.Build.VERSION.SDK_INT; +import static android.provider.Settings.ACTION_REQUEST_IGNORE_BATTERY_OPTIMIZATIONS; +import static java.lang.Runtime.getRuntime; + import android.annotation.SuppressLint; import android.annotation.TargetApi; import android.content.Context; @@ -9,19 +16,12 @@ import android.content.pm.ResolveInfo; import android.net.Uri; import android.os.PowerManager; +import androidx.annotation.Nullable; + import java.io.IOException; import java.util.List; import java.util.Scanner; -import androidx.annotation.Nullable; - -import static android.content.Context.POWER_SERVICE; -import static android.content.pm.PackageManager.MATCH_DEFAULT_ONLY; -import static android.os.Build.BRAND; -import static android.os.Build.VERSION.SDK_INT; -import static android.provider.Settings.ACTION_REQUEST_IGNORE_BATTERY_OPTIMIZATIONS; -import static java.lang.Runtime.getRuntime; - public class PowerUtils { private final static String PACKAGE_NAME_HUAWEI = diff --git a/dont-kill-me-lib/src/main/res/values/strings.xml b/dont-kill-me-lib/src/main/res/values/strings.xml index 262102af..27f5961c 100644 --- a/dont-kill-me-lib/src/main/res/values/strings.xml +++ b/dont-kill-me-lib/src/main/res/values/strings.xml @@ -1,24 +1,24 @@ <?xml version="1.0" encoding="utf-8"?> <resources xmlns:tools="http://schemas.android.com/tools" tools:ignore="MissingTranslation"> + <string name="dnkm_doze_title">Background Connections</string> + <string name="dnkm_doze_intro">To work properly, this app needs to run in the background.</string> + <string name="dnkm_doze_explanation">Please disable battery optimizations so this app can run in the background.</string> + <string name="dnkm_doze_button">Allow Connections</string> - <string name="setup_doze_title">Background Connections</string> - <string name="setup_doze_intro">To work properly, this app needs to run in the background.</string> - <string name="setup_doze_explanation">Please disable battery optimizations so this app can run in the background.</string> - <string name="setup_doze_button">Allow Connections</string> - <string name="create_account_button">Continue</string> + <string name="dnkm_huawei_protected_text">Please tap the button below and make sure this app is protected in the \"Protected Apps\" screen.</string> + <string name="dnkm_huawei_protected_button">Protect this app</string> + <string name="dnkm_huawei_protected_help">If this app is not added to the protected apps list, it will be unable to run in the background.</string> - <string name="setup_huawei_text">Please tap the button below and make sure this app is protected in the \"Protected Apps\" screen.</string> - <string name="setup_huawei_button">Protect this app</string> - <string name="setup_huawei_help">If this app is not added to the protected apps list, it will be unable to run in the background.</string> - <string name="setup_huawei_app_launch_text">Please tap the button below, open the \"App launch\" screen and make sure this app is set to \"Manage manually\".</string> - <string name="setup_huawei_app_launch_button">Open Battery Settings</string> - <string name="setup_huawei_app_launch_help">If this app is not set to \"Manage manually\" in the \"App launch\" screen, it will not be able to run in the background.</string> - <string name="setup_xiaomi_text">To run in the background, this app needs to be locked to the recent apps list.</string> - <string name="setup_xiaomi_button">Protect this app</string> - <string name="setup_xiaomi_help">If this app is not locked to the recent apps list, it will be unable to run in the background.</string> - <string name="setup_xiaomi_dialog_body_old">1. Open the recent apps list (also called the app switcher)\n\n2. Swipe down on the image of this app to show the padlock icon\n\n3. If the padlock is not locked, tap to lock it</string> - <string name="setup_xiaomi_dialog_body_new">1. Open the recent apps list (also called the app switcher)\n\n2. Press and hold the image of this app until the padlock button appears\n\n3. If the padlock is not locked, tap to lock it</string> + <string name="dnkm_huawei_app_launch_text">Please tap the button below, open the \"App launch\" screen and make sure this app is set to \"Manage manually\".</string> + <string name="dnkm_huawei_app_launch_button">Open Battery Settings</string> + <string name="dnkm_huawei_app_launch_help">If this app is not set to \"Manage manually\" in the \"App launch\" screen, it will not be able to run in the background.</string> - <string name="got_it">Got it</string> - <string name="help">Help</string> + <string name="dnkm_xiaomi_text">To run in the background, this app needs to be locked to the recent apps list.</string> + <string name="dnkm_xiaomi_button">Protect this app</string> + <string name="dnkm_xiaomi_help">If this app is not locked to the recent apps list, it will be unable to run in the background.</string> + <string name="dnkm_xiaomi_dialog_body_old">1. Open the recent apps list (also called the app switcher)\n\n2. Swipe down on the image of this app to show the padlock icon\n\n3. If the padlock is not locked, tap to lock it</string> + <string name="dnkm_xiaomi_dialog_body_new">1. Open the recent apps list (also called the app switcher)\n\n2. Press and hold the image of this app until the padlock button appears\n\n3. If the padlock is not locked, tap to lock it</string> + + <string name="dnkm_got_it">Got it</string> + <string name="dnkm_help">Help</string> </resources> diff --git a/mailbox-android/src/main/java/org/briarproject/mailbox/android/dontkillme/AbstractDoNotKillMeFragment.java b/mailbox-android/src/main/java/org/briarproject/mailbox/android/dontkillme/AbstractDoNotKillMeFragment.java index b99c26ba..fb988d86 100644 --- a/mailbox-android/src/main/java/org/briarproject/mailbox/android/dontkillme/AbstractDoNotKillMeFragment.java +++ b/mailbox-android/src/main/java/org/briarproject/mailbox/android/dontkillme/AbstractDoNotKillMeFragment.java @@ -44,7 +44,7 @@ public abstract class AbstractDoNotKillMeFragment extends Fragment public View onCreateView(LayoutInflater inflater, @Nullable ViewGroup container, @Nullable Bundle savedInstanceState) { - requireActivity().setTitle(getString(R.string.setup_doze_title)); + requireActivity().setTitle(getString(R.string.dnkm_doze_title)); setHasOptionsMenu(false); View v = inflater.inflate(R.layout.fragment_dont_kill_me, container, false); @@ -93,8 +93,7 @@ public abstract class AbstractDoNotKillMeFragment extends Fragment dozeView.setChecked(true); } else if (getContext() != null) { secondAttempt = true; - String s = getString( - org.briarproject.android.dontkillmelib.R.string.setup_doze_explanation); + String s = getString(R.string.dnkm_doze_explanation); showOnboardingDialog(getContext(), s); } } diff --git a/mailbox-android/src/main/java/org/briarproject/mailbox/android/dontkillme/DoNotKillMeUtils.java b/mailbox-android/src/main/java/org/briarproject/mailbox/android/dontkillme/DoNotKillMeUtils.java index f3f0b799..8fd695f9 100644 --- a/mailbox-android/src/main/java/org/briarproject/mailbox/android/dontkillme/DoNotKillMeUtils.java +++ b/mailbox-android/src/main/java/org/briarproject/mailbox/android/dontkillme/DoNotKillMeUtils.java @@ -12,8 +12,7 @@ public class DoNotKillMeUtils { new AlertDialog.Builder(ctx, R.style.OnboardingDialogTheme) .setMessage(text) - .setNeutralButton( - org.briarproject.android.dontkillmelib.R.string.got_it, + .setNeutralButton(R.string.dnkm_got_it, (dialog, which) -> dialog.cancel()) .show(); } diff --git a/mailbox-android/src/main/java/org/briarproject/mailbox/android/dontkillme/DozeView.java b/mailbox-android/src/main/java/org/briarproject/mailbox/android/dontkillme/DozeView.java index 00cfefe3..beb7da11 100644 --- a/mailbox-android/src/main/java/org/briarproject/mailbox/android/dontkillme/DozeView.java +++ b/mailbox-android/src/main/java/org/briarproject/mailbox/android/dontkillme/DozeView.java @@ -28,10 +28,9 @@ public class DozeView extends PowerView { public DozeView(Context context, @Nullable AttributeSet attrs, int defStyleAttr) { super(context, attrs, defStyleAttr); - setText(org.briarproject.android.dontkillmelib.R.string.setup_doze_intro); + setText(R.string.dnkm_doze_intro); setIcon(R.drawable.ic_battery_alert_white); - setButtonText( - org.briarproject.android.dontkillmelib.R.string.setup_doze_button); + setButtonText(R.string.dnkm_doze_button); } @Override @@ -45,7 +44,7 @@ public class DozeView extends PowerView { @Override protected int getHelpText() { - return org.briarproject.android.dontkillmelib.R.string.setup_doze_explanation; + return R.string.dnkm_doze_explanation; } @Override diff --git a/mailbox-android/src/main/java/org/briarproject/mailbox/android/dontkillme/HuaweiAppLaunchView.java b/mailbox-android/src/main/java/org/briarproject/mailbox/android/dontkillme/HuaweiAppLaunchView.java index 0b5e3b2c..36dd76b8 100644 --- a/mailbox-android/src/main/java/org/briarproject/mailbox/android/dontkillme/HuaweiAppLaunchView.java +++ b/mailbox-android/src/main/java/org/briarproject/mailbox/android/dontkillme/HuaweiAppLaunchView.java @@ -26,10 +26,9 @@ public class HuaweiAppLaunchView extends PowerView { public HuaweiAppLaunchView(Context context, @Nullable AttributeSet attrs, int defStyleAttr) { super(context, attrs, defStyleAttr); - setText(org.briarproject.android.dontkillmelib.R.string.setup_huawei_app_launch_text); + setText(R.string.dnkm_huawei_app_launch_text); setIcon(R.drawable.ic_restore_mirrored_white); - setButtonText( - org.briarproject.android.dontkillmelib.R.string.setup_huawei_app_launch_button); + setButtonText(R.string.dnkm_huawei_app_launch_button); } @Override @@ -40,7 +39,7 @@ public class HuaweiAppLaunchView extends PowerView { @Override @StringRes protected int getHelpText() { - return org.briarproject.android.dontkillmelib.R.string.setup_huawei_app_launch_help; + return R.string.dnkm_huawei_app_launch_help; } @Override diff --git a/mailbox-android/src/main/java/org/briarproject/mailbox/android/dontkillme/HuaweiProtectedAppsView.java b/mailbox-android/src/main/java/org/briarproject/mailbox/android/dontkillme/HuaweiProtectedAppsView.java index e218dc05..5ac80e1c 100644 --- a/mailbox-android/src/main/java/org/briarproject/mailbox/android/dontkillme/HuaweiProtectedAppsView.java +++ b/mailbox-android/src/main/java/org/briarproject/mailbox/android/dontkillme/HuaweiProtectedAppsView.java @@ -4,6 +4,7 @@ import android.content.Context; import android.util.AttributeSet; import org.briarproject.android.dontkillmelib.PowerUtils; +import org.briarproject.mailbox.R; import androidx.annotation.Nullable; import androidx.annotation.StringRes; @@ -27,9 +28,8 @@ public class HuaweiProtectedAppsView extends PowerView { @Nullable AttributeSet attrs, int defStyleAttr) { super(context, attrs, defStyleAttr); - setText(org.briarproject.android.dontkillmelib.R.string.setup_huawei_text); - setButtonText( - org.briarproject.android.dontkillmelib.R.string.setup_huawei_button); + setText(R.string.dnkm_huawei_protected_text); + setButtonText(R.string.dnkm_huawei_protected_button); } @Override @@ -40,7 +40,7 @@ public class HuaweiProtectedAppsView extends PowerView { @Override @StringRes protected int getHelpText() { - return org.briarproject.android.dontkillmelib.R.string.setup_huawei_help; + return R.string.dnkm_huawei_protected_help; } @Override diff --git a/mailbox-android/src/main/java/org/briarproject/mailbox/android/dontkillme/XiaomiView.java b/mailbox-android/src/main/java/org/briarproject/mailbox/android/dontkillme/XiaomiView.java index 623f5d41..27768ba8 100644 --- a/mailbox-android/src/main/java/org/briarproject/mailbox/android/dontkillme/XiaomiView.java +++ b/mailbox-android/src/main/java/org/briarproject/mailbox/android/dontkillme/XiaomiView.java @@ -3,6 +3,8 @@ package org.briarproject.mailbox.android.dontkillme; import android.content.Context; import android.util.AttributeSet; +import org.briarproject.mailbox.R; + import androidx.annotation.Nullable; import androidx.annotation.StringRes; import androidx.annotation.UiThread; @@ -25,9 +27,8 @@ public class XiaomiView extends PowerView { public XiaomiView(Context context, @Nullable AttributeSet attrs, int defStyleAttr) { super(context, attrs, defStyleAttr); - setText(org.briarproject.android.dontkillmelib.R.string.setup_xiaomi_text); - setButtonText( - org.briarproject.android.dontkillmelib.R.string.setup_xiaomi_button); + setText(R.string.dnkm_xiaomi_text); + setButtonText(R.string.dnkm_xiaomi_button); } @Override @@ -38,16 +39,14 @@ public class XiaomiView extends PowerView { @Override @StringRes protected int getHelpText() { - return org.briarproject.android.dontkillmelib.R.string.setup_xiaomi_help; + return R.string.dnkm_xiaomi_help; } @Override protected void onButtonClick() { int bodyRes = isMiuiTenOrLater() - ? - org.briarproject.android.dontkillmelib.R.string.setup_xiaomi_dialog_body_new - : - org.briarproject.android.dontkillmelib.R.string.setup_xiaomi_dialog_body_old; + ? R.string.dnkm_xiaomi_dialog_body_new + : R.string.dnkm_xiaomi_dialog_body_old; showOnboardingDialog(getContext(), getContext().getString(bodyRes)); setChecked(true); } diff --git a/mailbox-android/src/main/res/layout/fragment_dont_kill_me.xml b/mailbox-android/src/main/res/layout/fragment_dont_kill_me.xml index a7a7cba6..a5c2ab2f 100644 --- a/mailbox-android/src/main/res/layout/fragment_dont_kill_me.xml +++ b/mailbox-android/src/main/res/layout/fragment_dont_kill_me.xml @@ -78,7 +78,7 @@ android:layout_height="wrap_content" android:layout_marginTop="24dp" android:enabled="false" - android:text="@string/create_account_button" + android:text="@string/button_continue" app:layout_constraintBottom_toBottomOf="parent" app:layout_constraintEnd_toEndOf="parent" app:layout_constraintStart_toStartOf="parent" diff --git a/mailbox-android/src/main/res/layout/power_view.xml b/mailbox-android/src/main/res/layout/power_view.xml index ed868eb9..01b7cdf6 100644 --- a/mailbox-android/src/main/res/layout/power_view.xml +++ b/mailbox-android/src/main/res/layout/power_view.xml @@ -14,7 +14,7 @@ app:layout_constraintEnd_toEndOf="parent" app:layout_constraintStart_toEndOf="@+id/icon" app:layout_constraintTop_toTopOf="parent" - tools:text="@string/setup_huawei_text" /> + tools:text="@string/dnkm_huawei_protected_text" /> <androidx.appcompat.widget.AppCompatImageView android:id="@+id/icon" @@ -23,7 +23,7 @@ android:layout_height="48dp" android:layout_marginEnd="12dp" android:layout_marginRight="12dp" - android:contentDescription="@string/help" + android:contentDescription="@string/dnkm_help" android:padding="12dp" app:layout_constraintBottom_toBottomOf="@+id/textView" app:layout_constraintEnd_toStartOf="@+id/textView" @@ -53,7 +53,7 @@ app:layout_constraintEnd_toStartOf="@+id/helpButton" app:layout_constraintStart_toEndOf="@+id/icon" app:layout_constraintTop_toBottomOf="@+id/textView" - tools:text="@string/setup_huawei_button" /> + tools:text="@string/dnkm_huawei_protected_button" /> <ImageButton android:id="@+id/helpButton" @@ -61,7 +61,7 @@ android:layout_width="48dp" android:layout_height="48dp" android:background="?attr/selectableItemBackgroundBorderless" - android:contentDescription="@string/help" + android:contentDescription="@string/dnkm_help" app:layout_constraintBottom_toBottomOf="@+id/button" app:layout_constraintEnd_toEndOf="parent" app:layout_constraintTop_toTopOf="@+id/button" -- GitLab