From 270c5a3c794238b3bb342b27895308b007c8650b Mon Sep 17 00:00:00 2001 From: Tyler Nijmeh Date: Sun, 4 Apr 2021 20:29:43 -0700 Subject: Refactor reset to be more accurate Signed-off-by: Tyler Nijmeh --- .../draco/buoy/fragments/MainPreferenceFragment.kt | 4 ++-- .../com/draco/buoy/services/TileResetService.kt | 2 +- .../com/draco/buoy/utils/BatterySaverManager.kt | 26 +++++----------------- 3 files changed, 9 insertions(+), 23 deletions(-) (limited to 'app/src/main/java/com') diff --git a/app/src/main/java/com/draco/buoy/fragments/MainPreferenceFragment.kt b/app/src/main/java/com/draco/buoy/fragments/MainPreferenceFragment.kt index 554cc84..1a65293 100644 --- a/app/src/main/java/com/draco/buoy/fragments/MainPreferenceFragment.kt +++ b/app/src/main/java/com/draco/buoy/fragments/MainPreferenceFragment.kt @@ -106,8 +106,8 @@ class MainPreferenceFragment : PreferenceFragmentCompat(), SharedPreferences.OnS override fun onPreferenceTreeClick(preference: Preference): Boolean { when (preference.key) { - getString(R.string.pref_profile_key_reset) -> { - batterySaverManager.reset() + getString(R.string.pref_profile_key_default) -> { + batterySaverManager.resetToDefault() refreshSettings() } getString(R.string.pref_profile_key_light) -> { diff --git a/app/src/main/java/com/draco/buoy/services/TileResetService.kt b/app/src/main/java/com/draco/buoy/services/TileResetService.kt index 21f22d7..14c568d 100644 --- a/app/src/main/java/com/draco/buoy/services/TileResetService.kt +++ b/app/src/main/java/com/draco/buoy/services/TileResetService.kt @@ -13,6 +13,6 @@ class TileResetService : TileService() { override fun onClick() { super.onClick() - batterySaverManager.reset() + batterySaverManager.resetToDefault() } } \ No newline at end of file diff --git a/app/src/main/java/com/draco/buoy/utils/BatterySaverManager.kt b/app/src/main/java/com/draco/buoy/utils/BatterySaverManager.kt index d9ba9be..a72aa70 100644 --- a/app/src/main/java/com/draco/buoy/utils/BatterySaverManager.kt +++ b/app/src/main/java/com/draco/buoy/utils/BatterySaverManager.kt @@ -6,17 +6,6 @@ import com.draco.buoy.models.BatterySaverConstantsConfig import com.draco.buoy.repositories.constants.BatterySaverSecureSettings class BatterySaverManager(private val contentResolver: ContentResolver) { - /** - * Reset constants to default values - */ - fun resetConstants() { - Settings.Global.putString( - contentResolver, - BatterySaverSecureSettings.BATTERY_SAVER_CONSTANTS, - null - ) - } - /** * Enable or disable low power mode */ @@ -56,7 +45,7 @@ class BatterySaverManager(private val contentResolver: ContentResolver) { /** * Set the raw battery saver constants secure setting */ - fun setConstantsString(constants: String) { + fun setConstantsString(constants: String?) { Settings.Global.putString( contentResolver, BatterySaverSecureSettings.BATTERY_SAVER_CONSTANTS, @@ -84,9 +73,9 @@ class BatterySaverManager(private val contentResolver: ContentResolver) { /** * Quick way to apply either type of config */ - fun apply(config: Any) { + fun apply(config: Any?) { when (config) { - is String -> setConstantsString(config) + is String? -> setConstantsString(config) is BatterySaverConstantsConfig -> setConstantsConfig(config) } @@ -96,12 +85,9 @@ class BatterySaverManager(private val contentResolver: ContentResolver) { } /** - * Quick way to reset everything + * Reset constants to default values */ - fun reset() { - resetConstants() - setLowPower(false) - setLowPowerSticky(false) - setLowPowerStickyAutoDisableEnabled(true) + fun resetToDefault() { + apply(null) } } \ No newline at end of file -- cgit v1.2.3