fix: Better locale detection and avoid launching null intents (#1382)

aniyomiorg · Feb 5, 2024 · 0bd1bb3 · 0bd1bb3
1 parent 8979594
commit 0bd1bb3
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 7 deletions.
diff --git a/app/src/main/java/eu/kanade/tachiyomi/ui/main/MainActivity.kt b/app/src/main/java/eu/kanade/tachiyomi/ui/main/MainActivity.kt
@@ -580,8 +580,8 @@ class MainActivity : BaseActivity() {
                 } catch (e: Exception) {
                     logcat(LogPriority.ERROR, e)
                     withUIContext { Injekt.get<Application>().toast(e.message) }
-                    return
-                }
+                    null
+                } ?: return
                 externalPlayerResult?.launch(intent) ?: return
             } else {
                 context.startActivity(

diff --git a/app/src/main/java/eu/kanade/tachiyomi/ui/player/ExternalIntents.kt b/app/src/main/java/eu/kanade/tachiyomi/ui/player/ExternalIntents.kt
@@ -188,7 +188,7 @@ class ExternalIntents {
 
             val localLangName = LocaleHelper.getSimpleLocaleDisplayName()
             video.subtitleTracks.firstOrNull {
-                it.lang.contains(localLangName)
+                it.lang.contains(localLangName, true)
             }?.let {
                 putExtra("subtitle", it.url)
             } ?: video.subtitleTracks.firstOrNull()?.let {
@@ -225,7 +225,7 @@ class ExternalIntents {
 
             val localLangName = LocaleHelper.getSimpleLocaleDisplayName()
             val langIndex = video.subtitleTracks.indexOfFirst {
-                it.lang.contains(localLangName)
+                it.lang.contains(localLangName, true)
             }
             val requestedLanguage = if (langIndex == -1) 0 else langIndex
             val requestedUrl = video.subtitleTracks.getOrNull(requestedLanguage)?.url

diff --git a/app/src/main/java/eu/kanade/tachiyomi/ui/player/PlayerActivity.kt b/app/src/main/java/eu/kanade/tachiyomi/ui/player/PlayerActivity.kt
@@ -1660,7 +1660,7 @@ class PlayerActivity : BaseActivity() {
             currentVideoList?.getOrNull(streams.quality.index)
                 ?.subtitleTracks?.let { tracks ->
                     val langIndex = tracks.indexOfFirst {
-                        it.lang.contains(localLangName)
+                        it.lang.contains(localLangName, true)
                     }
                     val requestedLanguage = if (langIndex == -1) 0 else langIndex
                     tracks.getOrNull(requestedLanguage)?.let { sub ->

diff --git a/app/src/main/java/eu/kanade/tachiyomi/util/system/LocaleHelper.kt b/app/src/main/java/eu/kanade/tachiyomi/util/system/LocaleHelper.kt
@@ -77,8 +77,7 @@ object LocaleHelper {
      * Return English display string from string language code
      */
     fun getSimpleLocaleDisplayName(): String {
-        val sp = Locale.getDefault().language.split("_", "-")
-        return Locale(sp[0]).getDisplayLanguage(LocaleListCompat.getDefault()[0]!!)
+        return LocaleListCompat.getDefault()[0]!!.displayLanguage
     }
 }