WordPress.org

Make WordPress Core

Ticket #29783: 29783.2.diff

File 29783.2.diff, 16.2 KB (added by swissspidy, 21 months ago)
  • src/wp-admin/admin-header.php

    diff --git src/wp-admin/admin-header.php src/wp-admin/admin-header.php
    index 9ec3999..b5bbec0 100644
    if ( $current_screen->taxonomy ) 
    156156$admin_body_class .= ' branch-' . str_replace( array( '.', ',' ), '-', floatval( get_bloginfo( 'version' ) ) );
    157157$admin_body_class .= ' version-' . str_replace( '.', '-', preg_replace( '/^([.0-9]+).*/', '$1', get_bloginfo( 'version' ) ) );
    158158$admin_body_class .= ' admin-color-' . sanitize_html_class( get_user_option( 'admin_color' ), 'fresh' );
    159 $admin_body_class .= ' locale-' . sanitize_html_class( strtolower( str_replace( '_', '-', get_locale() ) ) );
     159$admin_body_class .= ' locale-' . sanitize_html_class( strtolower( str_replace( '_', '-', get_user_locale() ) ) );
    160160
    161161if ( wp_is_mobile() )
    162162        $admin_body_class .= ' mobile';
  • src/wp-admin/includes/class-wp-plugin-install-list-table.php

    diff --git src/wp-admin/includes/class-wp-plugin-install-list-table.php src/wp-admin/includes/class-wp-plugin-install-list-table.php
    index 73df40f..db6c86d 100644
    class WP_Plugin_Install_List_Table extends WP_List_Table { 
    134134                                'active_installs' => true
    135135                        ),
    136136                        // Send the locale and installed plugin slugs to the API so it can provide context-sensitive results.
    137                         'locale' => get_locale(),
     137                        'locale' => get_user_locale(),
    138138                        'installed_plugins' => $this->get_installed_plugin_slugs(),
    139139                );
    140140
  • src/wp-admin/includes/class-wp-press-this.php

    diff --git src/wp-admin/includes/class-wp-press-this.php src/wp-admin/includes/class-wp-press-this.php
    index 3ff242b..e275b16 100644
    class WP_Press_This { 
    13121312        $admin_body_class .= ' branch-' . str_replace( array( '.', ',' ), '-', floatval( $wp_version ) );
    13131313        $admin_body_class .= ' version-' . str_replace( '.', '-', preg_replace( '/^([.0-9]+).*/', '$1', $wp_version ) );
    13141314        $admin_body_class .= ' admin-color-' . sanitize_html_class( get_user_option( 'admin_color' ), 'fresh' );
    1315         $admin_body_class .= ' locale-' . sanitize_html_class( strtolower( str_replace( '_', '-', get_locale() ) ) );
     1315        $admin_body_class .= ' locale-' . sanitize_html_class( strtolower( str_replace( '_', '-', get_user_locale() ) ) );
    13161316
    13171317        /** This filter is documented in wp-admin/admin-header.php */
    13181318        $admin_body_classes = apply_filters( 'admin_body_class', '' );
  • src/wp-admin/includes/credits.php

    diff --git src/wp-admin/includes/credits.php src/wp-admin/includes/credits.php
    index 6634724..005858c 100644
     
    1616 */
    1717function wp_credits() {
    1818        $wp_version = get_bloginfo( 'version' );
    19         $locale = get_locale();
     19        $locale = get_user_locale();
    2020
    2121        $results = get_site_transient( 'wordpress_credits_' . $locale );
    2222
  • src/wp-admin/includes/dashboard.php

    diff --git src/wp-admin/includes/dashboard.php src/wp-admin/includes/dashboard.php
    index f3d2f79..98dfc85 100644
    function wp_dashboard_browser_nag() { 
    13531353                $notice .= "<p class='browser-update-nag{$browser_nag_class}'>{$msg}</p>";
    13541354
    13551355                $browsehappy = 'http://browsehappy.com/';
    1356                 $locale = get_locale();
     1356                $locale = get_user_locale();
    13571357                if ( 'en_US' !== $locale )
    13581358                        $browsehappy = add_query_arg( 'locale', $locale, $browsehappy );
    13591359
  • src/wp-admin/includes/import.php

    diff --git src/wp-admin/includes/import.php src/wp-admin/includes/import.php
    index 4f0b785..0d53442 100644
    function wp_import_handle_upload() { 
    125125function wp_get_popular_importers() {
    126126        include( ABSPATH . WPINC . '/version.php' ); // include an unmodified $wp_version
    127127
    128         $locale = get_locale();
     128        $locale = get_user_locale();
    129129        $cache_key = 'popular_importers_' . md5( $locale . $wp_version );
    130130        $popular_importers = get_site_transient( $cache_key );
    131131
    132132        if ( ! $popular_importers ) {
    133133                $url = add_query_arg( array(
    134                         'locale'  => get_locale(),
     134                        'locale'  => get_user_locale(),
    135135                        'version' => $wp_version,
    136136                ), 'http://api.wordpress.org/core/importers/1.1/' );
    137137                $options = array( 'user-agent' => 'WordPress/' . $wp_version . '; ' . home_url() );
  • src/wp-admin/includes/plugin-install.php

    diff --git src/wp-admin/includes/plugin-install.php src/wp-admin/includes/plugin-install.php
    index a33797e..b0c2e39 100644
    function plugins_api( $action, $args = array() ) { 
    109109        }
    110110
    111111        if ( ! isset( $args->locale ) ) {
    112                 $args->locale = get_locale();
     112                $args->locale = get_user_locale();
    113113        }
    114114
    115115        /**
  • src/wp-admin/includes/template.php

    diff --git src/wp-admin/includes/template.php src/wp-admin/includes/template.php
    index 4eac951..01cb080 100644
    do_action( "admin_head-$hook_suffix" ); 
    16221622/** This action is documented in wp-admin/admin-header.php */
    16231623do_action( 'admin_head' );
    16241624
    1625 $admin_body_class .= ' locale-' . sanitize_html_class( strtolower( str_replace( '_', '-', get_locale() ) ) );
     1625$admin_body_class .= ' locale-' . sanitize_html_class( strtolower( str_replace( '_', '-', get_user_locale() ) ) );
    16261626
    16271627if ( is_rtl() )
    16281628        $admin_body_class .= ' rtl';
  • src/wp-admin/includes/theme.php

    diff --git src/wp-admin/includes/theme.php src/wp-admin/includes/theme.php
    index 9fbd712..edd6cb1 100644
    function themes_api( $action, $args = array() ) { 
    412412        }
    413413
    414414        if ( ! isset( $args->locale ) ) {
    415                 $args->locale = get_locale();
     415                $args->locale = get_user_locale();
    416416        }
    417417
    418418        /**
  • src/wp-admin/includes/user.php

    diff --git src/wp-admin/includes/user.php src/wp-admin/includes/user.php
    index fc46dd3..6bf826b 100644
    function edit_user( $user_id = 0 ) { 
    9494                $user->rich_editing = isset( $_POST['rich_editing'] ) && 'false' == $_POST['rich_editing'] ? 'false' : 'true';
    9595                $user->admin_color = isset( $_POST['admin_color'] ) ? sanitize_text_field( $_POST['admin_color'] ) : 'fresh';
    9696                $user->show_admin_bar_front = isset( $_POST['admin_bar_front'] ) ? 'true' : 'false';
     97                $user->locale = '';
     98
     99                if ( isset( $_POST['locale'] ) ) {
     100                        $locale = sanitize_text_field( $_POST['locale'] );
     101                        if ( ! in_array( $locale, get_available_languages(), true ) ) {
     102                                $locale = '';
     103                        }
     104
     105                        $user->locale = ( '' === $locale ) ? 'en_US' : $locale;
     106                }
    97107        }
    98108
    99109        $user->comment_shortcuts = isset( $_POST['comment_shortcuts'] ) && 'true' == $_POST['comment_shortcuts'] ? 'true' : '';
  • src/wp-admin/plugin-editor.php

    diff --git src/wp-admin/plugin-editor.php src/wp-admin/plugin-editor.php
    index 9919005..6225c72 100644
    foreach ( $plugin_files as $plugin_file ) : 
    260260                <input type="hidden" name="scrollto" id="scrollto" value="<?php echo $scrollto; ?>" />
    261261                </div>
    262262                <?php if ( !empty( $docs_select ) ) : ?>
    263                 <div id="documentation" class="hide-if-no-js"><label for="docs-list"><?php _e('Documentation:') ?></label> <?php echo $docs_select ?> <input type="button" class="button" value="<?php esc_attr_e( 'Look Up' ) ?> " onclick="if ( '' != jQuery('#docs-list').val() ) { window.open( 'https://api.wordpress.org/core/handbook/1.0/?function=' + escape( jQuery( '#docs-list' ).val() ) + '&amp;locale=<?php echo urlencode( get_locale() ) ?>&amp;version=<?php echo urlencode( get_bloginfo( 'version' ) ) ?>&amp;redirect=true'); }" /></div>
     263                <div id="documentation" class="hide-if-no-js"><label for="docs-list"><?php _e('Documentation:') ?></label> <?php echo $docs_select ?> <input type="button" class="button" value="<?php esc_attr_e( 'Look Up' ) ?> " onclick="if ( '' != jQuery('#docs-list').val() ) { window.open( 'https://api.wordpress.org/core/handbook/1.0/?function=' + escape( jQuery( '#docs-list' ).val() ) + '&amp;locale=<?php echo urlencode( get_user_locale() ) ?>&amp;version=<?php echo urlencode( get_bloginfo( 'version' ) ) ?>&amp;redirect=true'); }" /></div>
    264264                <?php endif; ?>
    265265<?php if ( is_writeable($real_file) ) : ?>
    266266        <?php if ( in_array( $file, (array) get_option( 'active_plugins', array() ) ) ) { ?>
  • src/wp-admin/theme-editor.php

    diff --git src/wp-admin/theme-editor.php src/wp-admin/theme-editor.php
    index 0b100cf..c4a6207 100644
    else : ?> 
    263263                <div id="documentation" class="hide-if-no-js">
    264264                <label for="docs-list"><?php _e('Documentation:') ?></label>
    265265                <?php echo $docs_select; ?>
    266                 <input type="button" class="button" value="<?php esc_attr_e( 'Look Up' ); ?>" onclick="if ( '' != jQuery('#docs-list').val() ) { window.open( 'https://api.wordpress.org/core/handbook/1.0/?function=' + escape( jQuery( '#docs-list' ).val() ) + '&amp;locale=<?php echo urlencode( get_locale() ) ?>&amp;version=<?php echo urlencode( get_bloginfo( 'version' ) ) ?>&amp;redirect=true'); }" />
     266                <input type="button" class="button" value="<?php esc_attr_e( 'Look Up' ); ?>" onclick="if ( '' != jQuery('#docs-list').val() ) { window.open( 'https://api.wordpress.org/core/handbook/1.0/?function=' + escape( jQuery( '#docs-list' ).val() ) + '&amp;locale=<?php echo urlencode( get_user_locale() ) ?>&amp;version=<?php echo urlencode( get_bloginfo( 'version' ) ) ?>&amp;redirect=true'); }" />
    267267                </div>
    268268        <?php endif; ?>
    269269
  • src/wp-admin/user-edit.php

    diff --git src/wp-admin/user-edit.php src/wp-admin/user-edit.php
    index bb3c62e..e2ade0d 100644
    if ( !( IS_PROFILE_PAGE && !$user_can_edit ) ) : ?> 
    269269</fieldset>
    270270</td>
    271271</tr>
     272
     273<?php
     274$languages = get_available_languages();
     275if ( $languages ) : ?>
     276<tr class="user-language-wrap">
     277        <th scope="row">
     278                <label for="site_language"><?php _e( 'Site Language' ); ?></label>
     279        </th>
     280        <td>
     281                <?php
     282                $locale = get_user_option( 'locale', $profileuser->ID );
     283                if ( 'en_US' === $locale ) { // en_US
     284                        $locale = false;
     285                } elseif ( ! in_array( $locale, $languages ) ) {
     286                        $locale = get_locale();
     287                }
     288
     289                wp_dropdown_languages( array(
     290                        'name'                        => 'locale',
     291                        'id'                          => 'locale',
     292                        'selected'                    => $locale,
     293                        'languages'                   => $languages,
     294                        'show_available_translations' => false
     295                ) );
     296                ?>
     297        </td>
     298</tr>
     299<?php
     300endif;
     301?>
     302
    272303<?php
    273304/**
    274305 * Fires at the end of the 'Personal Options' settings table on the user editing screen.
  • src/wp-includes/class-wp-editor.php

    diff --git src/wp-includes/class-wp-editor.php src/wp-includes/class-wp-editor.php
    index 6e6fecc..11075ef 100644
    final class _WP_Editors { 
    351351                        if ( empty( self::$first_init ) ) {
    352352                                self::$baseurl = includes_url( 'js/tinymce' );
    353353
    354                                 $mce_locale = get_locale();
     354                                $mce_locale = get_user_locale();
    355355                                self::$mce_locale = $mce_locale = empty( $mce_locale ) ? 'en' : strtolower( substr( $mce_locale, 0, 2 ) ); // ISO 639-1
    356356
    357357                                /** This filter is documented in wp-admin/includes/media.php */
    final class _WP_Editors { 
    672672                                }
    673673                        }
    674674
    675                         $body_class .= ' locale-' . sanitize_html_class( strtolower( str_replace( '_', '-', get_locale() ) ) );
     675                        $body_class .= ' locale-' . sanitize_html_class( strtolower( str_replace( '_', '-', get_user_locale() ) ) );
    676676
    677677                        if ( !empty($set['tinymce']['body_class']) ) {
    678678                                $body_class .= ' ' . $set['tinymce']['body_class'];
  • src/wp-includes/class-wp-theme.php

    diff --git src/wp-includes/class-wp-theme.php src/wp-includes/class-wp-theme.php
    index 40af39d..8136956 100644
    final class WP_Theme implements ArrayAccess { 
    13951395         * @param array $themes Array of themes to sort, passed by reference.
    13961396         */
    13971397        public static function sort_by_name( &$themes ) {
    1398                 if ( 0 === strpos( get_locale(), 'en_' ) ) {
     1398                if ( 0 === strpos( get_user_locale(), 'en_' ) ) {
    13991399                        uasort( $themes, array( 'WP_Theme', '_name_sort' ) );
    14001400                } else {
    14011401                        uasort( $themes, array( 'WP_Theme', '_name_sort_i18n' ) );
  • src/wp-includes/class-wp-user.php

    diff --git src/wp-includes/class-wp-user.php src/wp-includes/class-wp-user.php
    index bcb82eb..e3c2608 100644
     
    3131 * @property string $display_name
    3232 * @property string $spam
    3333 * @property string $deleted
     34 * @property string $locale
    3435 */
    3536class WP_User {
    3637        /**
  • src/wp-includes/l10n.php

    diff --git src/wp-includes/l10n.php src/wp-includes/l10n.php
    index aaa7611..f442cd9 100644
    function get_locale() { 
    7676}
    7777
    7878/**
     79 * Get the locale of the current user.
     80 *
     81 * Returns {@see get_locale()} for non-admin requests.
     82 *
     83 * If the user has a locale set to a non-empty string then it will be
     84 * returned. Otherwise it returns the locale of {@see get_locale()}.
     85 *
     86 * @since 4.7.0
     87 *
     88 * @return string The locale of the current user.
     89 */
     90function get_user_locale() {
     91        if ( ! is_admin() ) {
     92                return get_locale();
     93        }
     94
     95        $user = wp_get_current_user();
     96
     97        $locale = $user->locale;
     98        return ( '' === $locale ) ? get_locale() : $locale;
     99}
     100
     101/**
    79102 * Retrieve the translation of $text.
    80103 *
    81104 * If there is no translation, or the text domain isn't loaded, the original text is returned.
    function unload_textdomain( $domain ) { 
    633656 */
    634657function load_default_textdomain( $locale = null ) {
    635658        if ( null === $locale ) {
    636                 $locale = get_locale();
     659                $locale = get_user_locale();
    637660        }
    638661
    639662        // Unload previously loaded strings so we can switch translations.
    function is_rtl() { 
    11481171                return false;
    11491172        }
    11501173        return $wp_locale->is_rtl();
    1151 }
    1152  No newline at end of file
     1174}
  • src/wp-includes/user.php

    diff --git src/wp-includes/user.php src/wp-includes/user.php
    index 9ff6455..5641695 100644
    function wp_insert_user( $userdata ) { 
    15921592
    15931593        $meta['show_admin_bar_front'] = empty( $userdata['show_admin_bar_front'] ) ? 'true' : $userdata['show_admin_bar_front'];
    15941594
     1595        $meta['locale'] = isset( $userdata['locale'] ) ? $userdata['locale'] : '';
     1596
    15951597        $user_nicename_check = $wpdb->get_var( $wpdb->prepare("SELECT ID FROM $wpdb->users WHERE user_nicename = %s AND user_login != %s LIMIT 1" , $user_nicename, $user_login));
    15961598
    15971599        if ( $user_nicename_check ) {
    function wp_create_user($username, $password, $email = '') { 
    19511953 * @return array List of user keys to be populated in wp_update_user().
    19521954 */
    19531955function _get_additional_user_keys( $user ) {
    1954         $keys = array( 'first_name', 'last_name', 'nickname', 'description', 'rich_editing', 'comment_shortcuts', 'admin_color', 'use_ssl', 'show_admin_bar_front' );
     1956        $keys = array( 'first_name', 'last_name', 'nickname', 'description', 'rich_editing', 'comment_shortcuts', 'admin_color', 'use_ssl', 'show_admin_bar_front', 'locale' );
    19551957        return array_merge( $keys, array_keys( wp_get_user_contact_methods( $user ) ) );
    19561958}
    19571959
  • src/wp-settings.php

    diff --git src/wp-settings.php src/wp-settings.php
    index 80f556c..a67e5c6 100644
    wp_templating_constants( ); 
    371371// Load the default text localization domain.
    372372load_default_textdomain();
    373373
    374 $locale = get_locale();
     374$locale = get_user_locale();
    375375$locale_file = WP_LANG_DIR . "/$locale.php";
    376376if ( ( 0 === validate_file( $locale ) ) && is_readable( $locale_file ) )
    377377        require( $locale_file );
  • new file tests/phpunit/tests/l10n/getUserLocale.php

    diff --git tests/phpunit/tests/l10n/getUserLocale.php tests/phpunit/tests/l10n/getUserLocale.php
    new file mode 100644
    index 0000000..36ac496
    - +  
     1<?php
     2
     3/**
     4 * @group l10n
     5 * @group i18n
     6 */
     7class Tests_Get_User_Locale extends WP_UnitTestCase {
     8        protected $user_id;
     9
     10        public function setUp() {
     11                parent::setUp();
     12
     13                $this->user_id = $this->factory()->user->create( array(
     14                        'role'   => 'administrator',
     15                        'locale' => 'de_DE',
     16                ) );
     17
     18                wp_set_current_user( $this->user_id );
     19        }
     20
     21        public function tearDown() {
     22                delete_user_option( $this->user_id, 'locale' );
     23                set_current_screen( 'front' );
     24
     25                parent::tearDown();
     26        }
     27
     28        public function test_returns_site_locale_if_not_in_admin() {
     29                $this->assertSame( get_locale(), get_user_locale() );
     30        }
     31
     32        public function test_returns_site_locale_if_empty() {
     33                set_current_screen( 'dashboard' );
     34                update_user_meta( $this->user_id, 'locale', '' );
     35                $this->assertSame( get_locale(), get_user_locale() );
     36        }
     37
     38        public function test_returns_correct_user_locale() {
     39                set_current_screen( 'dashboard' );
     40                $this->assertSame( 'de_DE', get_user_locale() );
     41        }
     42}