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

Timestamp:

07/07/2020 06:28:41 PM (5 years ago)

Author:

ocean90

Message:

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

This removes the usage of wp_localize_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.

Props swissspidy, ocean90.
See #20491.
Fixes #50603.

File:

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

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

@@ -1313 +1313 @@
          */
         $scripts->add( 'nav-menu', "/wp-admin/js/nav-menu$suffix.js", array( 'jquery-ui-sortable', 'jquery-ui-draggable', 'jquery-ui-droppable', 'wp-lists', 'postbox', 'json2', 'underscore' ) );
-        did_action( 'init' ) && $scripts->localize(
-            'nav-menu',
-            'navMenuL10n',
-            array(
-                'noResultsFound' => __( 'No results found.' ),
-                'warnDeleteMenu' => __( "You are about to permanently delete this menu. \n 'Cancel' to stop, 'OK' to delete." ),
-                'saveAlert'      => __( 'The changes you made will be lost if you navigate away from this page.' ),
-                'untitled'       => _x( '(no label)', 'missing menu item navigation label' ),
-            )
-        );
+        $scripts->set_translations( 'nav-menu' );
 
         $scripts->add( 'custom-header', '/wp-admin/js/custom-header.js', array( 'jquery-masonry' ), false, 1 );