Add Config.loadSkins() and fix crashes for Android versions < Oreo

osudroid · Jan 8, 2022 · dafbcd0 · dafbcd0
1 parent 954bc7f
commit dafbcd0
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 14 deletions.
diff --git a/src/ru/nsu/ccfit/zuev/osu/Config.java b/src/ru/nsu/ccfit/zuev/osu/Config.java
@@ -138,13 +138,13 @@ public static void loadConfig(final Context context) {
             soundVolume = prefs.getInt("soundvolume", 100) / 100f;
             bgmVolume = prefs.getInt("bgmvolume", 100) / 100f;
             cursorSize = prefs.getInt("cursorSize", 50) / 100f;
-        }catch(RuntimeException e) { // migrate to integers to prevent crash
+        }catch(RuntimeException e) { // use valid integer since this makes the game crash on android m
             prefs.edit()
-                .putInt("offset", Integer.parseInt(prefs.getString("offset", "0")))
-                .putInt("bgbrightness", Integer.parseInt(prefs.getString("bgbrightness", "25")))
-                .putInt("soundvolume", Integer.parseInt(prefs.getString("soundvolume", "100")))
-                .putInt("bgmvolume", Integer.parseInt(prefs.getString("bgmvolume", "100")))
-                .putInt("cursorSize", Integer.parseInt(prefs.getString("cursorSize", "50")))
+                .putInt("offset", 0)
+                .putInt("bgbrightness", 25)
+                .putInt("soundvolume", 100)
+                .putInt("bgmvolume", 100)
+                .putInt("cursorSize", 50)
                 .commit();
             Config.loadConfig(context);
         }
@@ -196,14 +196,6 @@ public static void loadConfig(final Context context) {
             comboColors[i - 1] = RGBColor.hex2Rgb(ColorPickerPreference.convertToRGB(prefs.getInt("combo" + i, 0xff000000)));
         }
 
-        // skins
-        File[] folders = FileUtils.listFiles(new File(skinTopPath), file -> file.isDirectory() && !file.getName().startsWith("."));
-        skins = new HashMap<String, String>();
-        for(File folder : folders) {
-            skins.put(folder.getName(), folder.getPath());
-            Debug.i("skins: " + folder.getName() + " - " + folder.getPath());
-        }
-
         // beatmaps
         DELETE_OSZ = prefs.getBoolean("deleteosz", true);
         SCAN_DOWNLOAD = prefs.getBoolean("scandownload", false);
@@ -767,6 +759,15 @@ public static String getDefaultCorePath() {
         return defaultCorePath;
     }
 
+    public static void loadSkins() {
+        File[] folders = FileUtils.listFiles(new File(skinTopPath), file -> file.isDirectory() && !file.getName().startsWith("."));
+        skins = new HashMap<String, String>();
+        for(File folder : folders) {
+            skins.put(folder.getName(), folder.getPath());
+            Debug.i("skins: " + folder.getName() + " - " + folder.getPath());
+        }
+    }
+
     public static Map<String, String> getSkins(){
         return skins;
     }

diff --git a/src/ru/nsu/ccfit/zuev/osu/MainActivity.java b/src/ru/nsu/ccfit/zuev/osu/MainActivity.java
@@ -309,6 +309,7 @@ public void run() {
                 GlobalManager.getInstance().init();
                 analytics.logEvent(FirebaseAnalytics.Event.APP_OPEN, null);
                 GlobalManager.getInstance().setLoadingProgress(50);
+                Config.loadSkins();
                 checkNewSkins();
                 checkNewBeatmaps();
                 if (!LibraryManager.getInstance().loadLibraryCache(MainActivity.this, true)) {