Changeset 61542

Timestamp:

01/28/2026 01:06:35 AM (7 months ago)

Author:

wildworks

Message:

Script Loader: Use localized list separators in dependency warning messages.

Improve dependency warning messages so that list separators are localized according to the current locale when multiple dependencies are listed.

Follow-up to [61323], [60999], [61357].

Props mukeshpanchal27, jorbin, westonruter, wildworks.
See #64229.

Location:

trunk/src/wp-includes

Files:

• class-wp-dependencies.php (modified) (1 diff)
• class-wp-script-modules.php (modified) (1 diff)
• class-wp-scripts.php (modified) (1 diff)
• class-wp-styles.php (modified) (1 diff)

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

@@ -570 +570 @@
         protected function get_dependency_warning_message( $handle, $missing_dependency_handles ) {
                 return sprintf(
-                        /* translators: 1: Handle, 2: Comma-separated list of missing dependency handles. */
+                        /* translators: 1: Handle, 2: List of missing dependency handles. */
                         __( 'The handle "%1$s" was enqueued with dependencies that are not registered: %2$s.' ),
                         $handle,
-                        implode( ', ', $missing_dependency_handles )
+                        implode( wp_get_list_item_separator(), $missing_dependency_handles )
                 );
         }

trunk/src/wp-includes/class-wp-script-modules.php

@@ -740 +740 @@
                                         get_class( $this ) . '::register',
                                         sprintf(
-                                                /* translators: 1: Script module ID, 2: Comma-separated list of missing dependency IDs. */
+                                                /* translators: 1: Script module ID, 2: List of missing dependency IDs. */
                                                 __( 'The script module with the ID "%1$s" was enqueued with dependencies that are not registered: %2$s.' ),
                                                 $id,
-                                                implode( ', ', $missing_dependencies )
+                                                implode( wp_get_list_item_separator(), $missing_dependencies )
                                         ),
                                         '6.9.1'

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

@@ -1192 +1192 @@
         protected function get_dependency_warning_message( $handle, $missing_dependency_handles ) {
                 return sprintf(
-                        /* translators: 1: Script handle, 2: Comma-separated list of missing dependency handles. */
+                        /* translators: 1: Script handle, 2: List of missing dependency handles. */
                         __( 'The script with the handle "%1$s" was enqueued with dependencies that are not registered: %2$s.' ),
                         $handle,
-                        implode( ', ', $missing_dependency_handles )
+                        implode( wp_get_list_item_separator(), $missing_dependency_handles )
                 );
         }

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

@@ -494 +494 @@
         protected function get_dependency_warning_message( $handle, $missing_dependency_handles ) {
                 return sprintf(
-                        /* translators: 1: Style handle, 2: Comma-separated list of missing dependency handles. */
+                        /* translators: 1: Style handle, 2: List of missing dependency handles. */
                         __( 'The style with the handle "%1$s" was enqueued with dependencies that are not registered: %2$s.' ),
                         $handle,
-                        implode( ', ', $missing_dependency_handles )
+                        implode( wp_get_list_item_separator(), $missing_dependency_handles )
                 );
         }