Changeset 48387 for trunk/src/wp-includes/script-loader.php

Timestamp:

07/07/2020 04:59:22 PM (5 years ago)

Author:

ocean90

Message:

I18N: Use wp.i18n for translatable strings in wp-admin/js/widgets.js.

This removes the usage of $scripts->add_inline_script() for passing translations to the script and instead adds the translatable strings in the script directly through the use of wp.i18n and its utilities.

Fixes #50600.

File:

• trunk/src/wp-includes/script-loader.php (modified) (1 diff)

trunk/src/wp-includes/script-loader.php

@@ -1240 +1240 @@
 
         $scripts->add( 'admin-widgets', "/wp-admin/js/widgets$suffix.js", array( 'jquery-ui-sortable', 'jquery-ui-draggable', 'jquery-ui-droppable', 'wp-a11y' ), false, 1 );
-        did_action( 'init' ) && $scripts->add_inline_script(
-            'admin-widgets',
-            sprintf(
-                'wpWidgets.l10n = %s;',
-                wp_json_encode(
-                    array(
-                        'save'        => __( 'Save' ),
-                        'saved'       => __( 'Saved' ),
-                        'saveAlert'   => __( 'The changes you made will be lost if you navigate away from this page.' ),
-                        'widgetAdded' => __( 'Widget has been added to the selected sidebar' ),
-                    )
-                )
-            )
-        );
+        $scripts->set_translations( 'admin-widgets' );
 
         $scripts->add( 'media-widgets', "/wp-admin/js/widgets/media-widgets$suffix.js", array( 'jquery', 'media-models', 'media-views', 'wp-api-request' ) );