Changeset 44403 for trunk/src/wp-includes/class.wp-scripts.php

Timestamp:

01/06/2019 04:22:46 PM (3 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.

Props swissspidy.
See #45441.

File:

• trunk/src/wp-includes/class.wp-scripts.php (modified) (1 diff)

trunk/src/wp-includes/class.wp-scripts.php

@@ -547 +547 @@
         }
 
-        $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 ) {