From f373e41a91e997b81e859487f994da59eedaede2 Mon Sep 17 00:00:00 2001
From: Chris Caron <lead2gold@gmail.com>
Date: Tue, 4 Jun 2019 22:13:15 -0400
Subject: [PATCH] handles exception thrown when no localization is detected
 refs #118

---
 apprise/AppriseLocale.py | 28 ++++++++++++++++++----------
 test/test_locale.py      | 34 ++++++++++++++++++++++++++++++----
 2 files changed, 48 insertions(+), 14 deletions(-)

diff --git a/apprise/AppriseLocale.py b/apprise/AppriseLocale.py
index ea88acc6..714d2016 100644
--- a/apprise/AppriseLocale.py
+++ b/apprise/AppriseLocale.py
@@ -192,16 +192,24 @@ class AppriseLocale(object):
 
             if hasattr(ctypes, 'windll'):
                 windll = ctypes.windll.kernel32
-                lang = locale.windows_locale[
-                    windll.GetUserDefaultUILanguage()]
-            else:
                 try:
-                    # Detect language
-                    lang = locale.getdefaultlocale()[0]
+                    lang = locale.windows_locale[
+                        windll.GetUserDefaultUILanguage()]
 
-                except TypeError:
-                    # None is returned if the default can't be determined
-                    # we're done in this case
-                    return None
+                    # Our detected windows language
+                    return lang[0:2].lower()
 
-        return lang[0:2].lower()
+                except (TypeError, KeyError):
+                    # Fallback to posix detection
+                    pass
+
+            try:
+                # Detect language
+                lang = locale.getdefaultlocale()[0]
+
+            except TypeError:
+                # None is returned if the default can't be determined
+                # we're done in this case
+                return None
+
+        return None if not lang else lang[0:2].lower()
diff --git a/test/test_locale.py b/test/test_locale.py
index 15de3d1b..ce2fcc8d 100644
--- a/test/test_locale.py
+++ b/test/test_locale.py
@@ -27,6 +27,7 @@ import mock
 import ctypes
 
 from apprise import AppriseLocale
+from apprise.utils import environ
 
 try:
     # Python v3.4+
@@ -133,8 +134,7 @@ def test_gettext_installs(mock_gettext_trans):
         pass
 
 
-@mock.patch('locale.getdefaultlocale')
-def test_detect_language(mock_getlocale):
+def test_detect_language_windows_users():
     """
     API: Apprise() Detect language
 
@@ -147,14 +147,40 @@ def test_detect_language(mock_getlocale):
         setattr(ctypes, 'windll', windll)
 
     # The below accesses the windows fallback code
-    assert AppriseLocale.AppriseLocale.detect_language() == 'en'
+    with environ('LANG', 'LANGUAGE', 'LC_ALL', 'LC_CTYPE', LANG="en_CA"):
+        assert AppriseLocale.AppriseLocale.detect_language() == 'en'
 
     assert AppriseLocale.AppriseLocale\
         .detect_language(detect_fallback=False) is None
 
+    # 0 = IndexError
+    windll.kernel32.GetUserDefaultUILanguage.return_value = 0
+    setattr(ctypes, 'windll', windll)
+    with environ('LANG', 'LC_ALL', 'LC_CTYPE', LANGUAGE="en_CA"):
+        assert AppriseLocale.AppriseLocale.detect_language() == 'en'
+
+    # The below accesses the windows fallback code and fail
+    # then it will resort to the environment variables
+    with environ('LANG', 'LANGUAGE', 'LC_ALL', 'LC_CTYPE'):
+        # Language can't be detected
+        assert AppriseLocale.AppriseLocale.detect_language() is None
+
+    with environ('LANGUAGE', 'LC_ALL', 'LC_CTYPE', LANG="fr_CA"):
+        # Detect french language
+        assert AppriseLocale.AppriseLocale.detect_language() == 'fr'
+
+    # Tidy
+    delattr(ctypes, 'windll')
+
+
+@mock.patch('locale.getdefaultlocale')
+def test_detect_language_defaultlocale(mock_getlocale):
+    """
+    API: Apprise() Default locale detection
+
+    """
     # Handle case where getdefaultlocale() can't be detected
     mock_getlocale.return_value = None
-    delattr(ctypes, 'windll')
     assert AppriseLocale.AppriseLocale.detect_language() is None
 
     # if detect_language and windows env fail us, then we don't