WordPress.org
Get WordPress

Make WordPress Core

Changeset 44404 for branches/5.0


Ignore:
Timestamp:
01/06/2019 05:05:24 PM (8 years ago)
Author:
ocean90
Message:

I18N/Script Loader: Support text domains other than "messages".

The inline JavaScript added by WP_Scripts::print_translations() should check whether locale_data.$text_domain exists and fall back to locale_data.messages otherwise.

Merge of [44403] to the 5.0 branch.

Props swissspidy.
Fixes #45441.

Location:
branches/5.0
Files:
3 edited

Legend:

Unmodified
Added
Removed

  • branches/5.0

  • branches/5.0/src/wp-includes/class.wp-scripts.php

    r43898 r44404  
    533533                }
    534534
    535                 $output = '(function( translations ){' .
    536                               'translations.locale_data.messages[""].domain = "' . $domain . '";' .
    537                               'wp.i18n.setLocaleData( translations.locale_data.messages, "' . $domain . '" );' .
    538                           '})(' . $json_translations . ');';
     535                $output = <<<JS
     536( function( domain, translations ) {
     537        var localeData = translations.locale_data[ domain ] || translations.locale_data.messages;
     538        localeData[""].domain = domain;
     539        wp.i18n.setLocaleData( localeData, domain );
     540} )( "{$domain}", {$json_translations} );
     541JS;
    539542
    540543                if ( $echo ) {
    … …  
    547550        /**
    548551         * Determines script dependencies.
    549     *
     552        *
    550553         * @since 2.1.0
    551554         *

  • branches/5.0/tests/phpunit/tests/dependencies/scripts.php

    r43898 r44404  
    55 */
    66class Tests_Dependencies_Scripts extends WP_UnitTestCase {
    7         var $old_wp_scripts;
     7        protected $old_wp_scripts;
     8
     9        protected $wp_scripts_print_translations_output;
    810
    911        function setUp() {
    … …  
    1416                $GLOBALS['wp_scripts'] = new WP_Scripts();
    1517                $GLOBALS['wp_scripts']->default_version = get_bloginfo( 'version' );
     18
     19                $this->wp_scripts_print_translations_output  = <<<JS
     20<script type='text/javascript'>
     21( function( domain, translations ) {
     22        var localeData = translations.locale_data[ domain ] || translations.locale_data.messages;
     23        localeData[""].domain = domain;
     24        wp.i18n.setLocaleData( localeData, domain );
     25} )( "__DOMAIN__", __JSON_TRANSLATIONS__ );
     26</script>
     27JS;
     28                $this->wp_scripts_print_translations_output .= "\n";
    1629        }
    1730
    … …  
    776789                wp_set_script_translations( 'test-example', 'default',  DIR_TESTDATA . '/languages' );
    777790
    778                 $expected  = "<script type='text/javascript' src='/wp-includes/js/dist/wp-i18n.js'></script>";
    779                 $expected .= "\n<script type='text/javascript'>\n(function( translations ){" .
    780                              "translations.locale_data.messages[\"\"].domain = \"default\";" .
    781                              "wp.i18n.setLocaleData( translations.locale_data.messages, \"default\" );" .
    782                              "})(" . file_get_contents( DIR_TESTDATA . '/languages/en_US-813e104eb47e13dd4cc5af844c618754.json' ) . ");\n</script>\n";
     791                $expected  = "<script type='text/javascript' src='/wp-includes/js/dist/wp-i18n.js'></script>\n";
     792                $expected .= str_replace(
     793                        array(
     794                                '__DOMAIN__',
     795                                '__JSON_TRANSLATIONS__',
     796                        ),
     797                        array(
     798                                'default',
     799                                file_get_contents( DIR_TESTDATA . '/languages/en_US-813e104eb47e13dd4cc5af844c618754.json' ),
     800                        ),
     801                        $this->wp_scripts_print_translations_output
     802                );
    783803                $expected .= "<script type='text/javascript' src='/wp-includes/js/script.js'></script>\n";
    784804
    … …  
    794814                wp_set_script_translations( 'plugin-example', 'internationalized-plugin',  DIR_TESTDATA . '/languages/plugins' );
    795815
    796                 $expected  = "<script type='text/javascript' src='/wp-includes/js/dist/wp-i18n.js'></script>";
    797                 $expected .= "\n<script type='text/javascript'>\n(function( translations ){" .
    798                              "translations.locale_data.messages[\"\"].domain = \"internationalized-plugin\";" .
    799                              "wp.i18n.setLocaleData( translations.locale_data.messages, \"internationalized-plugin\" );" .
    800                              "})(" . file_get_contents( DIR_TESTDATA . '/languages/plugins/internationalized-plugin-en_US-2f86cb96a0233e7cb3b6f03ad573be0b.json' ) . ");\n</script>\n";
     816                $expected  = "<script type='text/javascript' src='/wp-includes/js/dist/wp-i18n.js'></script>\n";
     817                $expected .= str_replace(
     818                        array(
     819                                '__DOMAIN__',
     820                                '__JSON_TRANSLATIONS__',
     821                        ),
     822                        array(
     823                                'internationalized-plugin',
     824                                file_get_contents( DIR_TESTDATA . '/languages/plugins/internationalized-plugin-en_US-2f86cb96a0233e7cb3b6f03ad573be0b.json' ),
     825                        ),
     826                        $this->wp_scripts_print_translations_output
     827                );
    801828                $expected .= "<script type='text/javascript' src='/wp-content/plugins/my-plugin/js/script.js'></script>\n";
    802829
    … …  
    812839                wp_set_script_translations( 'theme-example', 'internationalized-theme',  DIR_TESTDATA . '/languages/themes' );
    813840
    814                 $expected  = "<script type='text/javascript' src='/wp-includes/js/dist/wp-i18n.js'></script>";
    815                 $expected .= "\n<script type='text/javascript'>\n(function( translations ){" .
    816                              "translations.locale_data.messages[\"\"].domain = \"internationalized-theme\";" .
    817                              "wp.i18n.setLocaleData( translations.locale_data.messages, \"internationalized-theme\" );" .
    818                              "})(" . file_get_contents( DIR_TESTDATA . '/languages/themes/internationalized-theme-en_US-2f86cb96a0233e7cb3b6f03ad573be0b.json' ) . ");\n</script>\n";
     841                $expected  = "<script type='text/javascript' src='/wp-includes/js/dist/wp-i18n.js'></script>\n";
     842                $expected .= str_replace(
     843                        array(
     844                                '__DOMAIN__',
     845                                '__JSON_TRANSLATIONS__',
     846                        ),
     847                        array(
     848                                'internationalized-theme',
     849                                file_get_contents( DIR_TESTDATA . '/languages/themes/internationalized-theme-en_US-2f86cb96a0233e7cb3b6f03ad573be0b.json' ),
     850                        ),
     851                        $this->wp_scripts_print_translations_output
     852                );
    819853                $expected .= "<script type='text/javascript' src='/wp-content/themes/my-theme/js/script.js'></script>\n";
    820854
    … …  
    830864                wp_set_script_translations( 'script-handle', 'admin',  DIR_TESTDATA . '/languages/' );
    831865
    832                 $expected  = "<script type='text/javascript' src='/wp-includes/js/dist/wp-i18n.js'></script>";
    833                 $expected .= "\n<script type='text/javascript'>\n(function( translations ){" .
    834                              "translations.locale_data.messages[\"\"].domain = \"admin\";" .
    835                              "wp.i18n.setLocaleData( translations.locale_data.messages, \"admin\" );" .
    836                              "})(" . file_get_contents( DIR_TESTDATA . '/languages/admin-en_US-script-handle.json' ) . ");\n</script>\n";
     866                $expected  = "<script type='text/javascript' src='/wp-includes/js/dist/wp-i18n.js'></script>\n";
     867                $expected .= str_replace(
     868                        array(
     869                                '__DOMAIN__',
     870                                '__JSON_TRANSLATIONS__',
     871                        ),
     872                        array(
     873                                'admin',
     874                                file_get_contents( DIR_TESTDATA . '/languages/admin-en_US-script-handle.json' ),
     875                        ),
     876                        $this->wp_scripts_print_translations_output
     877                );
    837878                $expected .= "<script type='text/javascript' src='/wp-admin/js/script.js'></script>\n";
    838879
    … …  
    863904                wp_set_script_translations( 'test-example', 'admin',  DIR_TESTDATA . '/languages/' );
    864905
    865                 $expected  = "<script type='text/javascript' src='/wp-includes/js/dist/wp-i18n.js'></script>";
    866                 $expected .= "\n<script type='text/javascript'>\n(function( translations ){" .
    867                              "translations.locale_data.messages[\"\"].domain = \"admin\";" .
    868                              "wp.i18n.setLocaleData( translations.locale_data.messages, \"admin\" );" .
    869                              "})({ \"locale_data\": { \"messages\": { \"\": {} } } });\n</script>\n";
     906                $expected  = "<script type='text/javascript' src='/wp-includes/js/dist/wp-i18n.js'></script>\n";
     907                $expected .= str_replace(
     908                        array(
     909                                '__DOMAIN__',
     910                                '__JSON_TRANSLATIONS__',
     911                        ),
     912                        array(
     913                                'admin',
     914                                '{ "locale_data": { "messages": { "": {} } } }',
     915                        ),
     916                        $this->wp_scripts_print_translations_output
     917                );
    870918                $expected .= "<script type='text/javascript' src='/wp-admin/js/script.js'></script>\n";
    871919
    … …  
    879927                wp_register_script( 'wp-i18n', '/wp-includes/js/dist/wp-i18n.js', array(), null );
    880928                wp_register_script( 'test-example', '/wp-includes/js/script.js', array(), null );
    881                 wp_set_script_translations( 'test-example', 'default',  DIR_TESTDATA . '/languages' );
     929                wp_set_script_translations( 'test-example', 'default', DIR_TESTDATA . '/languages' );
    882930
    883931                wp_enqueue_script( 'test-example' );
    884932
    885                 $expected  = "<script type='text/javascript' src='/wp-includes/js/dist/wp-i18n.js'></script>";
    886                 $expected .= "\n<script type='text/javascript'>\n(function( translations ){" .
    887                              "translations.locale_data.messages[\"\"].domain = \"default\";" .
    888                              "wp.i18n.setLocaleData( translations.locale_data.messages, \"default\" );" .
    889                              "})(" . file_get_contents( DIR_TESTDATA . '/languages/en_US-813e104eb47e13dd4cc5af844c618754.json' ) . ");\n</script>\n";
     933                $expected  = "<script type='text/javascript' src='/wp-includes/js/dist/wp-i18n.js'></script>\n";
     934                $expected .= str_replace(
     935                        array(
     936                                '__DOMAIN__',
     937                                '__JSON_TRANSLATIONS__',
     938                        ),
     939                        array(
     940                                'default',
     941                                file_get_contents( DIR_TESTDATA . '/languages/en_US-813e104eb47e13dd4cc5af844c618754.json' ),
     942                        ),
     943                        $this->wp_scripts_print_translations_output
     944                );
    890945                $expected .= "<script type='text/javascript' src='/wp-includes/js/script.js'></script>\n";
    891946
    … …  
    899954                wp_register_script( 'wp-i18n', '/wp-includes/js/dist/wp-i18n.js', array(), null );
    900955                wp_register_script( 'test-dependency', '/wp-includes/js/script.js', array(), null );
    901                 wp_set_script_translations( 'test-dependency', 'default',  DIR_TESTDATA . '/languages' );
     956                wp_set_script_translations( 'test-dependency', 'default', DIR_TESTDATA . '/languages' );
    902957
    903958                wp_enqueue_script( 'test-example', '/wp-includes/js/script2.js', array( 'test-dependency' ), null );
    904959
    905                 $expected  = "<script type='text/javascript' src='/wp-includes/js/dist/wp-i18n.js'></script>";
    906                 $expected .= "\n<script type='text/javascript'>\n(function( translations ){" .
    907                              "translations.locale_data.messages[\"\"].domain = \"default\";" .
    908                              "wp.i18n.setLocaleData( translations.locale_data.messages, \"default\" );" .
    909                              "})(" . file_get_contents( DIR_TESTDATA . '/languages/en_US-813e104eb47e13dd4cc5af844c618754.json' ) . ");\n</script>\n";
     960                $expected  = "<script type='text/javascript' src='/wp-includes/js/dist/wp-i18n.js'></script>\n";
     961                $expected .= str_replace(
     962                        array(
     963                                '__DOMAIN__',
     964                                '__JSON_TRANSLATIONS__',
     965                        ),
     966                        array(
     967                                'default',
     968                                file_get_contents( DIR_TESTDATA . '/languages/en_US-813e104eb47e13dd4cc5af844c618754.json' ),
     969                        ),
     970                        $this->wp_scripts_print_translations_output
     971                );
    910972                $expected .= "<script type='text/javascript' src='/wp-includes/js/script.js'></script>\n";
    911973                $expected .= "<script type='text/javascript' src='/wp-includes/js/script2.js'></script>\n";
Note: See TracChangeset for help on using the changeset viewer.