diff --git a/core/modules/language/language.module b/core/modules/language/language.module index a94876e..ec87033 100644 --- a/core/modules/language/language.module +++ b/core/modules/language/language.module @@ -367,7 +367,7 @@ function language_language_negotiation_info() { 'file' => $file, 'weight' => -8, 'name' => t('URL'), - 'description' => t('Determine the language from the URL (Path prefix or domain).'), + 'description' => t('Language from the URL (Path prefix or domain).'), 'config' => 'admin/config/regional/language/detection/url', ); @@ -380,7 +380,7 @@ function language_language_negotiation_info() { 'file' => $file, 'weight' => -6, 'name' => t('Session'), - 'description' => t('Determine the language from a request/session parameter.'), + 'description' => t('Language from a request/session parameter.'), 'config' => 'admin/config/regional/language/detection/session', ); @@ -388,8 +388,8 @@ function language_language_negotiation_info() { 'callbacks' => array('negotiation' => 'language_from_user'), 'file' => $file, 'weight' => -4, - 'name' => t('User'), - 'description' => t("Follow the user's language preference."), + 'name' => t('Account'), + 'description' => t("Account site language setting."), ); $negotiation_info[LANGUAGE_NEGOTIATION_BROWSER] = array( @@ -398,7 +398,7 @@ function language_language_negotiation_info() { 'weight' => -2, 'cache' => 0, 'name' => t('Browser'), - 'description' => t("Determine the language from the browser's language settings."), + 'description' => t("Language from the browser's language settings."), ); $negotiation_info[LANGUAGE_NEGOTIATION_INTERFACE] = array( @@ -419,6 +419,15 @@ function language_language_negotiation_info() { 'description' => t('Use an already detected language for URLs if none is found.'), ); + $negotiation_info[LANGUAGE_NEGOTIATION_USER_ADMIN] = array( + 'types' => array(LANGUAGE_TYPE_INTERFACE), + 'callbacks' => array('negotiation' => 'language_from_user_admin'), + 'file' => $file, + 'weight' => 10, + 'name' => t('Account administration pages'), + 'description' => t("Account administration pages language setting."), + ); + return $negotiation_info; } diff --git a/core/modules/language/language.negotiation.inc b/core/modules/language/language.negotiation.inc index 48a8083..0091590 100644 --- a/core/modules/language/language.negotiation.inc +++ b/core/modules/language/language.negotiation.inc @@ -31,6 +31,11 @@ const LANGUAGE_NEGOTIATION_URL_FALLBACK = 'language-url-fallback'; const LANGUAGE_NEGOTIATION_USER = 'language-user'; /** + * The language is set based on the user admin language settings. + */ +const LANGUAGE_NEGOTIATION_USER_ADMIN = 'language-user-admin'; + +/** * The language is set based on the request/session parameters. */ const LANGUAGE_NEGOTIATION_SESSION = 'language-session'; @@ -153,7 +158,7 @@ function language_from_user($languages) { // User preference (only for logged users). global $user; - if ($user->uid && !empty($user->preferred_langcode)) { + if ($user->uid && !empty($user->preferred_langcode) && array_key_exists($user->preferred_langcode, $languages)) { return $user->preferred_langcode; } @@ -162,6 +167,27 @@ function language_from_user($languages) { } /** + * Identifies admin language from the user preferences. + * + * @param $languages + * An array of valid language objects. + * + * @return + * A valid language code on success, FALSE otherwise. + */ +function language_from_user_admin($languages) { + // User preference (only for logged users). + global $user; + + if ($user->uid && !empty($user->preferred_admin_langcode) && array_key_exists($user->preferred_admin_langcode, $languages) && path_is_admin(current_path())) { + return $user->preferred_admin_langcode; + } + + // No language preference from the user or not on an admin path. + return FALSE; +} + +/** * Identify language from a request/session parameter. * * @param $languages diff --git a/core/modules/language/lib/Drupal/language/Tests/LanguageUILanguageNegotiationTest.php b/core/modules/language/lib/Drupal/language/Tests/LanguageUILanguageNegotiationTest.php index d12bdfa..ff6ffc4 100644 --- a/core/modules/language/lib/Drupal/language/Tests/LanguageUILanguageNegotiationTest.php +++ b/core/modules/language/lib/Drupal/language/Tests/LanguageUILanguageNegotiationTest.php @@ -196,6 +196,107 @@ class LanguageUILanguageNegotiationTest extends WebTestBase { $this->drupalGet("$langcode_unknown/admin/config", array(), $http_header_browser_fallback); $this->assertResponse(404, "Unknown language path prefix should return 404"); + // Set preferred langcode for user to NULL. + $account = $this->loggedInUser; + $account->preferred_langcode = NULL; + $account->save(); + + $tests = array( + array( + 'language_negotiation' => array(LANGUAGE_NEGOTIATION_USER, LANGUAGE_NEGOTIATION_DEFAULT), + 'path' => 'admin/config', + 'expect' => $default_string, + 'expected_method_id' => LANGUAGE_NEGOTIATION_DEFAULT, + 'http_header' => array(), + 'message' => 'USER > DEFAULT: no preferred user language setting, the UI language is default', + ), + ); + + // Set preferred langcode for user to unknown language. + $account = $this->loggedInUser; + $account->preferred_langcode = $langcode_unknown; + $account->save(); + + $tests = array( + array( + 'language_negotiation' => array(LANGUAGE_NEGOTIATION_USER, LANGUAGE_NEGOTIATION_DEFAULT), + 'path' => 'admin/config', + 'expect' => $default_string, + 'expected_method_id' => LANGUAGE_NEGOTIATION_DEFAULT, + 'http_header' => array(), + 'message' => 'USER > DEFAULT: invalid preferred user language setting, the UI language is default', + ), + ); + + + // Set preferred langcode for user to non default. + $account->preferred_langcode = $langcode; + $account->save(); + + $tests = array( + array( + 'language_negotiation' => array(LANGUAGE_NEGOTIATION_USER, LANGUAGE_NEGOTIATION_DEFAULT), + 'path' => 'admin/config', + 'expect' => $language_string, + 'expected_method_id' => LANGUAGE_NEGOTIATION_USER, + 'http_header' => array(), + 'message' => 'USER > DEFAULT: defined prefereed user language setting, the UI language is based on user setting', + ), + ); + + foreach ($tests as $test) { + $this->runTest($test); + } + + // Set preferred admin langcode for user to NULL. + $account->preferred_admin_langcode = NULL; + $account->save(); + + $tests = array( + array( + 'language_negotiation' => array(LANGUAGE_NEGOTIATION_USER_ADMIN, LANGUAGE_NEGOTIATION_DEFAULT), + 'path' => 'admin/config', + 'expect' => $default_string, + 'expected_method_id' => LANGUAGE_NEGOTIATION_DEFAULT, + 'http_header' => array(), + 'message' => 'USER ADMIN > DEFAULT: no preferred user admin language setting, the UI language is default', + ), + ); + + // Set preferred admin langcode for user to unknown language. + $account->preferred_admin_langcode = $langcode_unknown; + $account->save(); + + $tests = array( + array( + 'language_negotiation' => array(LANGUAGE_NEGOTIATION_USER_ADMIN, LANGUAGE_NEGOTIATION_DEFAULT), + 'path' => 'admin/config', + 'expect' => $default_string, + 'expected_method_id' => LANGUAGE_NEGOTIATION_DEFAULT, + 'http_header' => array(), + 'message' => 'USER ADMIN > DEFAULT: invalid preferred user admin language setting, the UI language is default', + ), + ); + + // Set preferred admin langcode for user to non default. + $account->preferred_admin_langcode = $langcode; + $account->save(); + + $tests = array( + array( + 'language_negotiation' => array(LANGUAGE_NEGOTIATION_USER_ADMIN, LANGUAGE_NEGOTIATION_DEFAULT), + 'path' => 'admin/config', + 'expect' => $language_string, + 'expected_method_id' => LANGUAGE_NEGOTIATION_USER_ADMIN, + 'http_header' => array(), + 'message' => 'USER ADMIN > DEFAULT: defined prefereed user admin language setting, the UI language is based on user setting', + ), + ); + + foreach ($tests as $test) { + $this->runTest($test); + } + // Setup for domain negotiation, first configure the language to have domain // URL. $edit = array("domain[$langcode]" => $language_domain); diff --git a/core/modules/user/lib/Drupal/user/AccountFormController.php b/core/modules/user/lib/Drupal/user/AccountFormController.php index 31ca843..33f3476 100644 --- a/core/modules/user/lib/Drupal/user/AccountFormController.php +++ b/core/modules/user/lib/Drupal/user/AccountFormController.php @@ -205,6 +205,8 @@ abstract class AccountFormController extends EntityFormController { $user_preferred_language = $register ? $language_interface : user_preferred_language($account); + $user_preferred_admin_language = $register ? $language_interface : user_preferred_language($account, 'admin'); + // Is default the interface language? include_once DRUPAL_ROOT . '/core/includes/language.inc'; $interface_language_is_default = language_negotiation_method_get_first(LANGUAGE_TYPE_INTERFACE) != LANGUAGE_NEGOTIATION_DEFAULT; @@ -218,12 +220,20 @@ abstract class AccountFormController extends EntityFormController { $form['language']['preferred_langcode'] = array( '#type' => 'language_select', - '#title' => t('Language'), + '#title' => t('Site Language'), '#languages' => LANGUAGE_CONFIGURABLE, '#default_value' => $user_preferred_language->langcode, '#description' => $interface_language_is_default ? t("This account's preferred language for e-mails and site presentation.") : t("This account's preferred language for e-mails."), ); + $form['language']['preferred_admin_langcode'] = array( + '#type' => 'language_select', + '#title' => t('Administration pages language'), + '#languages' => LANGUAGE_CONFIGURABLE, + '#default_value' => $user_preferred_admin_language->langcode, + '#access' => user_access('access administration pages', $account), + ); + // User entities contain both a langcode property (for identifying the // language of the entity data) and a preferred_langcode property (see // above). Rather than provide a UI forcing the user to choose both diff --git a/core/modules/user/lib/Drupal/user/User.php b/core/modules/user/lib/Drupal/user/User.php index 42f2d29..74e2bf3 100644 --- a/core/modules/user/lib/Drupal/user/User.php +++ b/core/modules/user/lib/Drupal/user/User.php @@ -122,6 +122,13 @@ class User extends Entity { public $preferred_langcode = LANGUAGE_NOT_SPECIFIED; /** + * The user's preferred langcode for viewing administration pages. + * + * @var string + */ + public $preferred_admin_langcode = LANGUAGE_NOT_SPECIFIED; + + /** * The file ID of the user's picture. * * @var integer diff --git a/core/modules/user/user.install b/core/modules/user/user.install index abb985b..c44be33 100644 --- a/core/modules/user/user.install +++ b/core/modules/user/user.install @@ -224,6 +224,13 @@ function user_schema() { 'default' => '', 'description' => 'The {language}.langcode that the user prefers for receiving emails and viewing the site.', ), + 'preferred_admin_langcode' => array( + 'type' => 'varchar', + 'length' => 12, + 'not null' => TRUE, + 'default' => '', + 'description' => 'The {language}.langcode that the user prefers for viewing administration pages.', + ), 'picture' => array( 'type' => 'int', 'not null' => TRUE, @@ -475,5 +482,18 @@ function user_update_8003() { } /** + * Creates a preferred_admin_langcode column. + */ +function user_update_8004() { + $spec = array( + 'description' => 'The {language}.langcode that the user prefers for viewing administration pages.', + 'type' => 'varchar', + 'length' => 12, + 'not null' => FALSE, + ); + db_add_field('users', 'preferred_admin_langcode', $spec); +} + +/** * @} End of "addtogroup updates-7.x-to-8.x". */ diff --git a/core/modules/user/user.module b/core/modules/user/user.module index 44cb266..88be6be 100644 --- a/core/modules/user/user.module +++ b/core/modules/user/user.module @@ -2962,14 +2962,24 @@ function theme_user_signature($variables) { * * @param $account * User account to look up language for. + * @param $type + * Optional string to define which preferred langcode should be used. + * Default to 'preferred_langcode' property. + * If set 'preferred_$type_langcode' is used. * @param $default * Optional default language object to return if the account * has no valid language. */ -function user_preferred_language($account, $default = NULL) { +function user_preferred_language($account, $type = NULL, $default = NULL) { $language_list = language_list(); - if (!empty($account->preferred_langcode) && isset($language_list[$account->preferred_langcode])) { - return $language_list[$account->preferred_langcode]; + if (empty($type)) { + $preferred_langcode = $account->preferred_langcode; + } + else { + $preferred_langcode = $account->{'preferred_' . $type . '_langcode'}; + } + if (!empty($preferred_langcode) && isset($language_list[$preferred_langcode])) { + return $language_list[$preferred_langcode]; } else { return $default ? $default : language_default();