fix: always sort languages alphabetically, no matter the engine used

you-apps · Feb 4, 2025 · 735c27e · 735c27e
1 parent 3335fd7
commit 735c27e
Show file tree

Hide file tree

Showing 6 changed files with 5 additions and 6 deletions.
diff --git a/app/src/main/java/com/bnyro/translate/api/ap/ApEngine.kt b/app/src/main/java/com/bnyro/translate/api/ap/ApEngine.kt
@@ -148,7 +148,7 @@ class ApEngine: TranslationEngine(
             "zh" to "Chinese",
             "zlm" to "Malay",
             "zu" to "Zulu"
-        ).map { Language(it.first, it.second) }.sortedBy { it.name }
+        ).map { Language(it.first, it.second) }
     }
 
     override suspend fun translate(query: String, source: String, target: String): Translation {

diff --git a/app/src/main/java/com/bnyro/translate/api/mm/MMEngine.kt b/app/src/main/java/com/bnyro/translate/api/mm/MMEngine.kt
@@ -139,7 +139,7 @@ class MMEngine : TranslationEngine(
             "zu-ZA" to "Zulu"
         )
             .map { Language(it.first, it.second) }
-            .sortedBy { it.name }
+
 
         /* DEPRECATED endpoint apparently
         val request = Request.Builder()

diff --git a/app/src/main/java/com/bnyro/translate/api/or/OneRingEngine.kt b/app/src/main/java/com/bnyro/translate/api/or/OneRingEngine.kt
@@ -61,7 +61,6 @@ class OneRingEngine: TranslationEngine(
             Language(it.isO3Language, it.getDisplayName(Locale.getDefault()))
         }
             .distinctBy { it.code }
-            .sortedBy { it.name }
     }
 
     override suspend fun translate(query: String, source: String, target: String): Translation {

diff --git a/app/src/main/java/com/bnyro/translate/api/po/PonsEngine.kt b/app/src/main/java/com/bnyro/translate/api/po/PonsEngine.kt
@@ -40,7 +40,7 @@ class PonsEngine : TranslationEngine(
     override suspend fun getLanguages(): List<Language> {
         return api.getLanguages().languages.map { (code, langInfo) ->
             Language(code = code, name = langInfo.display)
-        }.sortedBy { it.name }
+        }
     }
 
     override suspend fun translate(query: String, source: String, target: String): Translation {

diff --git a/app/src/main/java/com/bnyro/translate/api/st/STEngine.kt b/app/src/main/java/com/bnyro/translate/api/st/STEngine.kt
@@ -44,7 +44,7 @@ class STEngine : TranslationEngine(
     override suspend fun getLanguages(): List<Language> {
         return api.getLanguages(getSelectedEngine()).map { (code, name) ->
             Language(code = code, name = name)
-        }.sortedBy { it.code }
+        }
     }
 
     override suspend fun translate(query: String, source: String, target: String): Translation {

diff --git a/app/src/main/java/com/bnyro/translate/ui/models/TranslationModel.kt b/app/src/main/java/com/bnyro/translate/ui/models/TranslationModel.kt
@@ -208,7 +208,7 @@ class TranslationModel : ViewModel() {
         viewModelScope.launch {
             val languages = try {
                 Log.e("engine", engine.name)
-                engine.getLanguages()
+                engine.getLanguages().sortedBy { it.name }
             } catch (e: Exception) {
                 Log.e("Fetching languages", e.toString())
                 onError.invoke(e)