Changeset 44404 for branches/5.0/src/wp-includes/class.wp-scripts.php

Timestamp:

01/06/2019 05:05:24 PM (6 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:

• . (modified) (1 prop)
• src/wp-includes/class.wp-scripts.php (modified) (2 diffs)

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

@@ -533 +533 @@
         }
 
-        $output = '(function( translations ){' .
-                      'translations.locale_data.messages[""].domain = "' . $domain . '";' .
-                      'wp.i18n.setLocaleData( translations.locale_data.messages, "' . $domain . '" );' .
-                  '})(' . $json_translations . ');';
+        $output = <<<JS
+( function( domain, translations ) {
+    var localeData = translations.locale_data[ domain ] || translations.locale_data.messages;
+    localeData[""].domain = domain;
+    wp.i18n.setLocaleData( localeData, domain );
+} )( "{$domain}", {$json_translations} );
+JS;
 
         if ( $echo ) {
@@ -547 +550 @@
     /**
      * Determines script dependencies.
-     *
+     *
      * @since 2.1.0
      *