Author: jsuchome Date: Thu Jan 7 15:09:21 2010 New Revision: 60268 URL: http://svn.opensuse.org/viewcvs/yast?rev=60268&view=rev Log: - added Language::SwitchToEnglishIfNeeded to be called when it might be necessary to use English in UI, due to possible displaying problems with another language (bnc#479529) - 2.17.40 Modified: branches/SuSE-Code-11-SP1-Branch/country/VERSION branches/SuSE-Code-11-SP1-Branch/country/language/src/modules/Language.ycp branches/SuSE-Code-11-SP1-Branch/country/language/src/select_language.ycp branches/SuSE-Code-11-SP1-Branch/country/package/yast2-country.changes Modified: branches/SuSE-Code-11-SP1-Branch/country/VERSION URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-SP1-Branch/country/VERSION?rev=60268&r1=60267&r2=60268&view=diff ============================================================================== --- branches/SuSE-Code-11-SP1-Branch/country/VERSION (original) +++ branches/SuSE-Code-11-SP1-Branch/country/VERSION Thu Jan 7 15:09:21 2010 @@ -1 +1 @@ -2.17.39 +2.17.40 Modified: branches/SuSE-Code-11-SP1-Branch/country/language/src/modules/Language.ycp URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-SP1-Branch/country/language/src/modules/Language.ycp?rev=60268&r1=60267&r2=60268&view=diff ============================================================================== --- branches/SuSE-Code-11-SP1-Branch/country/language/src/modules/Language.ycp (original) +++ branches/SuSE-Code-11-SP1-Branch/country/language/src/modules/Language.ycp Thu Jan 7 15:09:21 2010 @@ -134,7 +134,6 @@ return contains (cjk_languages, l); } - /** * return the value of text_mode (true for ncurses) */ @@ -1239,5 +1238,40 @@ } } +/** + * Set current YaST language to English if method for showing text in + * current language is not supported (usually for CJK languages) + * See http://bugzilla.novell.com/show_bug.cgi?id=479529 for discussion + * @boolean show_popup if information popup about the change should be shown + * @return true if UI language was changed + */ +global boolean SwitchToEnglishIfNeeded (boolean show_popup) { + + if (Stage::normal ()) + { + y2milestone ("no language switching in normal stage"); + return false; + } + if (GetTextMode () && + // current language is CJK + CJKLanguage (language) && + // fbiterm is not running + getenv ("TERM") != "iterm") + { + if (show_popup) + { + // popup message (user selected CJK language in text mode) + Popup::Message (_("The selected language cannot be used in text mode. English is used for +installation, but the selected language will be used for the new system.")); + } + WfmSetGivenLanguage ("en_US"); + return true; + } + return false; +} + + + + /* EOF */ } Modified: branches/SuSE-Code-11-SP1-Branch/country/language/src/select_language.ycp URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-SP1-Branch/country/language/src/select_language.ycp?rev=60268&r1=60267&r2=60268&view=diff ============================================================================== --- branches/SuSE-Code-11-SP1-Branch/country/language/src/select_language.ycp (original) +++ branches/SuSE-Code-11-SP1-Branch/country/language/src/select_language.ycp Thu Jan 7 15:09:21 2010 @@ -528,17 +528,10 @@ if (Stage::initial () || Stage::firstboot ()) { - if ((set_default && ret == `language || - (!set_default && ret == `next)) && - Language::CJKLanguage (language) && - !Language::CJKLanguage (preselected) && - Language::GetTextMode ()) - { - // popup message (user selected CJK language in text mode) - Popup::Message (_("The selected language cannot be used in text mode. English is used for -installation, but the selected language will be used for the new system.")); - - Language::WfmSetGivenLanguage ("en_US"); + if ((set_default && ret == `language || (!set_default && ret == `next)) && + Language::SwitchToEnglishIfNeeded (true)) + { + y2debug ("UI switched to en_US"); } else if (ret == `next || (set_default && ret == `language)) { Modified: branches/SuSE-Code-11-SP1-Branch/country/package/yast2-country.changes URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-SP1-Branch/country/package/yast2-country.changes?rev=60268&r1=60267&r2=60268&view=diff ============================================================================== --- branches/SuSE-Code-11-SP1-Branch/country/package/yast2-country.changes (original) +++ branches/SuSE-Code-11-SP1-Branch/country/package/yast2-country.changes Thu Jan 7 15:09:21 2010 @@ -1,4 +1,12 @@ ------------------------------------------------------------------- +Thu Jan 7 14:45:07 CET 2010 - jsuchome@suse.cz + +- added Language::SwitchToEnglishIfNeeded to be called when it might + be necessary to use English in UI, due to possible displaying + problems with another language (bnc#479529) +- 2.17.40 + +------------------------------------------------------------------- Fri Dec 4 14:12:20 CET 2009 - jsuchome@suse.cz - install ntp together with yast2-ntp-client (bnc#560732) -- To unsubscribe, e-mail: yast-commit+unsubscribe@opensuse.org For additional commands, e-mail: yast-commit+help@opensuse.org