From 140b6f5234a9b49dd1ec431e7adcf10754daeab1 Mon Sep 17 00:00:00 2001
From: Amith Yamasani <yamasani@google.com>
Date: Tue, 12 Jun 2012 20:06:55 -0700
Subject: Show only the language if there's only one locale for the language.

If the system has only one instance of that language, then only show the language
without the country name.

Bug: 6522572
Change-Id: I1e99182b1c669b200090f6c360bb902376b63ab0
---
 .../InputMethodAndLanguageSettings.java            | 36 +++++++++++++++++++---
 1 file changed, 32 insertions(+), 4 deletions(-)

(limited to 'src/com/android/settings/inputmethod')

diff --git a/src/com/android/settings/inputmethod/InputMethodAndLanguageSettings.java b/src/com/android/settings/inputmethod/InputMethodAndLanguageSettings.java
index bc8b458..182c1eb 100644
--- a/src/com/android/settings/inputmethod/InputMethodAndLanguageSettings.java
+++ b/src/com/android/settings/inputmethod/InputMethodAndLanguageSettings.java
@@ -29,6 +29,7 @@ import android.content.Context;
 import android.content.Intent;
 import android.content.pm.PackageManager;
 import android.content.res.Configuration;
+import android.content.res.Resources;
 import android.database.ContentObserver;
 import android.hardware.input.InputManager;
 import android.hardware.input.KeyboardLayout;
@@ -229,10 +230,23 @@ public class InputMethodAndLanguageSettings extends SettingsPreferenceFragment
         if (!mIsOnlyImeSettings) {
             if (mLanguagePref != null) {
                 Configuration conf = getResources().getConfiguration();
-                String locale = conf.locale.getDisplayName(conf.locale);
-                if (locale != null && locale.length() > 1) {
-                    locale = Character.toUpperCase(locale.charAt(0)) + locale.substring(1);
-                    mLanguagePref.setSummary(locale);
+                String language = conf.locale.getLanguage();
+                String localeString;
+                // TODO: This is not an accurate way to display the locale, as it is
+                // just working around the fact that we support limited dialects
+                // and want to pretend that the language is valid for all locales.
+                // We need a way to support languages that aren't tied to a particular
+                // locale instead of hiding the locale qualifier.
+                if (hasOnlyOneLanguageInstance(language,
+                        Resources.getSystem().getAssets().getLocales())) {
+                    localeString = conf.locale.getDisplayLanguage(conf.locale);
+                } else {
+                    localeString = conf.locale.getDisplayName(conf.locale);
+                }
+                if (localeString.length() > 1) {
+                    localeString = Character.toUpperCase(localeString.charAt(0))
+                            + localeString.substring(1);
+                    mLanguagePref.setSummary(localeString);
                 }
             }
 
@@ -322,6 +336,20 @@ public class InputMethodAndLanguageSettings extends SettingsPreferenceFragment
         return super.onPreferenceTreeClick(preferenceScreen, preference);
     }
 
+    private boolean hasOnlyOneLanguageInstance(String languageCode, String[] locales) {
+        int count = 0;
+        for (String localeCode : locales) {
+            if (localeCode.length() > 2
+                    && localeCode.startsWith(languageCode)) {
+                count++;
+                if (count > 1) {
+                    return false;
+                }
+            }
+        }
+        return count == 1;
+    }
+
     private void saveInputMethodSelectorVisibility(String value) {
         try {
             int intValue = Integer.valueOf(value);
-- 
cgit v1.1