configmgr/source/access.cxx              |    2 +-
 sd/source/core/CustomAnimationPreset.cxx |    4 ++--
 sd/source/core/TransitionPreset.cxx      |    4 ++--
 3 files changed, 5 insertions(+), 5 deletions(-)

New commits:
commit b516089abd4ec2ec615cdf15ea570ed1e4996cfe
Author:     Andras Timar <andras.ti...@collabora.com>
AuthorDate: Mon Jun 15 09:20:24 2020 +0200
Commit:     Andras Timar <andras.ti...@collabora.com>
CommitDate: Mon Jun 15 21:41:50 2020 +0200

    Use  getBcp47() instead of getLanguage()
    
    getLanguage() only returns the language code which can be
    ambiguous in case of UI localization, when there are more
    than one language variants. Fallback to the first variant
    caused mixed language UI, e.g. en-US vs. en-GB, pt vs. pt-BR,
    zh-CN vs. zh-TW.
    
    Change-Id: I888423cd4b3c65dbdb900ff3b6bc308fa119be4d
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/96284
    Tested-by: Andras Timar <andras.ti...@collabora.com>
    Reviewed-by: Andras Timar <andras.ti...@collabora.com>

diff --git a/configmgr/source/access.cxx b/configmgr/source/access.cxx
index 9fe7f31cc00f..e07fc8446f51 100644
--- a/configmgr/source/access.cxx
+++ b/configmgr/source/access.cxx
@@ -1382,7 +1382,7 @@ rtl::Reference< ChildAccess > Access::getChild(OUString 
const & name) {
         // here we override it with the actual current-user's language to
         // support per-view localization in LOK.
         if (comphelper::LibreOfficeKit::isActive())
-            locale = 
comphelper::LibreOfficeKit::getLanguageTag().getLanguage();
+            locale = comphelper::LibreOfficeKit::getLanguageTag().getBcp47();
 
         if (!locale.isEmpty()) {
             // Find best match using an adaptation of RFC 4647 lookup matching
diff --git a/sd/source/core/CustomAnimationPreset.cxx 
b/sd/source/core/CustomAnimationPreset.cxx
index e54c67d95b5c..426d8cc05924 100644
--- a/sd/source/core/CustomAnimationPreset.cxx
+++ b/sd/source/core/CustomAnimationPreset.cxx
@@ -507,8 +507,8 @@ const CustomAnimationPresets& 
CustomAnimationPresets::getCustomAnimationPresets(
     // Support localization per-view. Currently not useful for Desktop
     // but very much critical for LOK. The cache now is per-language.
     const OUString aLang = comphelper::LibreOfficeKit::isActive()
-                               ? 
comphelper::LibreOfficeKit::getLanguageTag().getLanguage()
-                               : 
SvtSysLocaleOptions().GetLanguageTag().getLanguage();
+                               ? 
comphelper::LibreOfficeKit::getLanguageTag().getBcp47()
+                               : 
SvtSysLocaleOptions().GetLanguageTag().getBcp47();
 
     SolarMutexGuard aGuard;
     const auto it = mPresetsMap.find(aLang);
diff --git a/sd/source/core/TransitionPreset.cxx 
b/sd/source/core/TransitionPreset.cxx
index 951959cdb69a..1a8f55b90aed 100644
--- a/sd/source/core/TransitionPreset.cxx
+++ b/sd/source/core/TransitionPreset.cxx
@@ -367,8 +367,8 @@ const TransitionPresetList& 
TransitionPreset::getTransitionPresetList()
     // Support localization per-view. Currently not useful for Desktop
     // but very much critical for LOK. The cache now is per-language.
     const OUString aLang = comphelper::LibreOfficeKit::isActive()
-                               ? 
comphelper::LibreOfficeKit::getLanguageTag().getLanguage()
-                               : 
SvtSysLocaleOptions().GetLanguageTag().getLanguage();
+                               ? 
comphelper::LibreOfficeKit::getLanguageTag().getBcp47()
+                               : 
SvtSysLocaleOptions().GetLanguageTag().getBcp47();
 
     SolarMutexGuard aGuard;
     const auto it = mPresetsMap.find(aLang);
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to