Merge pull request #140 from SanojPunchihewa/bug-fixes

Fix app crash when settings write permission not given
SanojPunchihewa · Mar 20, 2020 · 7c33c70 · 7c33c70
2 parents 9ce80cf + 5b7db44
commit 7c33c70
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/app/src/main/java/com/mobilegenomics/f5n/activity/ConfirmationActivity.java b/app/src/main/java/com/mobilegenomics/f5n/activity/ConfirmationActivity.java
@@ -102,8 +102,6 @@ protected void onCreate(@Nullable final Bundle savedInstanceState) {
         window = getWindow();
 
         try {
-            Settings.System.putInt(cResolver,
-                    Settings.System.SCREEN_BRIGHTNESS_MODE, Settings.System.SCREEN_BRIGHTNESS_MODE_MANUAL);
             //Get the current system brightness
             int brightness = Settings.System.getInt(cResolver, Settings.System.SCREEN_BRIGHTNESS);
             PreferenceUtil.setSharedPreferenceInt(R.string.id_screen_brightness, brightness);

diff --git a/app/src/main/java/com/mobilegenomics/f5n/support/ScreenDimUtil.java b/app/src/main/java/com/mobilegenomics/f5n/support/ScreenDimUtil.java
@@ -17,6 +17,8 @@ public static void changeBrightness(ContentResolver cResolver, Window window, in
         }
 
         if (settingsCanWrite) {
+            Settings.System.putInt(cResolver,
+                    Settings.System.SCREEN_BRIGHTNESS_MODE, Settings.System.SCREEN_BRIGHTNESS_MODE_MANUAL);
             //Set the system brightness using the brightness variable value
             Settings.System.putInt(cResolver, Settings.System.SCREEN_BRIGHTNESS, value);
             //Get the current window attributes