Fix language spinner

Author: maskara

app/src/main/java/fr/free/nrw/commons/upload/Language.java

@@ -2,34 +2,7 @@
 
 import java.util.Locale;
 
-import fr.free.nrw.commons.R;
-
 class Language {
-
-    public static int[] languageGroups={R.string.desc_language_Worldwide,
-            R.string.desc_language_America,
-            R.string.desc_language_Europe,
-            R.string.desc_language_Middle_East,
-            R.string.desc_language_Africa,
-            R.string.desc_language_Asia,
-            R.string.desc_language_Pacific };
-    public static int[] languageNames={R.array.desc_languages_Worldwide,
-            R.array.desc_languages_America,
-            R.array.desc_languages_Europe,
-            R.array.desc_languages_Middle_East,
-            R.array.desc_languages_Africa,
-            R.array.desc_languages_Asia,
-            R.array.desc_languages_Pacific
-    };
-    public static int[] languageCodes={R.array.desc_language_codes_Worldwide,
-            R.array.desc_language_codes_America,
-            R.array.desc_language_codes_Europe,
-            R.array.desc_language_codes_Middle_East,
-            R.array.desc_language_codes_Africa,
-            R.array.desc_language_codes_Asia,
-            R.array.desc_language_codes_Pacific
-    };
-
     private Locale locale;
     private boolean isSet = false;
 

app/src/main/java/fr/free/nrw/commons/upload/SpinnerLanguagesAdapter.java

@@ -1,7 +1,6 @@
 package fr.free.nrw.commons.upload;
 
 import android.content.Context;
-import android.content.res.Resources;
 import android.graphics.Color;
 import android.os.Build;
 import android.support.annotation.NonNull;
@@ -15,7 +14,6 @@
 import android.widget.TextView;
 
 import java.util.ArrayList;
-import java.util.Arrays;
 import java.util.List;
 import java.util.Locale;
 
@@ -42,23 +40,18 @@ public SpinnerLanguagesAdapter(@NonNull Context context,
         this.layoutInflater = LayoutInflater.from(context);
         languageNamesList = new ArrayList<>();
         languageCodesList = new ArrayList<>();
-        prepareLanguages(context);
+        prepareLanguages();
         this.selectedLanguages = selectedLanguages;
     }
 
-    private void prepareLanguages(Context context) {
-        Resources resources = context.getResources();
+    private void prepareLanguages() {
         List<Language> languages = getLocaleSupportedByDevice();
 
         for(Language language: languages) {
-            languageNamesList.add(language.getLocale().getDisplayName());
-            languageCodesList.add(language.getLocale().getDisplayScript());
-        }
-        for (int i = 0; i < languages.size(); i++) {
-            languageNamesList.add(resources.getString(Language.languageGroups[i]));
-            languageCodesList.add("");
-            languageNamesList.addAll(Arrays.asList(resources.getStringArray(Language.languageNames[i])));
-            languageCodesList.addAll(Arrays.asList(resources.getStringArray(Language.languageCodes[i])));
+            if(!languageCodesList.contains(language.getLocale().getLanguage())) {
+                languageNamesList.add(language.getLocale().getDisplayName());
+                languageCodesList.add(language.getLocale().getLanguage());
+            }
         }
     }
 
@@ -146,11 +139,11 @@ public void init(int position, boolean isDropDownView) {
         }
     }
 
-    public String getLanguageCode(int position) {
+    String getLanguageCode(int position) {
         return languageCodesList.get(position);
     }
 
-    public int getIndexOfUserDefaultLocale(Context context) {
+    int getIndexOfUserDefaultLocale(Context context) {
         return languageCodesList.indexOf(context.getResources().getConfiguration().locale.getLanguage());
     }