Changeset 52929 for trunk/src/wp-content/themes/twentyeleven/content.php

Timestamp:

03/13/2022 08:01:05 PM (3 years ago)

Author:

audrasjb

Message:

i18n: Define List item separator as a WP_Locale property.

The list item separator is a locale property, and it doesn't make much sense to translate it separately in multiple projects. This changeset implements the following modifications:

• Define list item separator as a new WP_Locale property
• Add wp_get_list_item_separator() as a wrapper for WP_Locale::get_list_item_separator
• Replace $wp_locale->get_list_item_separator() calls with wp_get_list_item_separator()
• Added a compatibility layer for bundled themes

Props SergeyBiryukov, swissspidy, rsiddharth, johnbillion, audrasjb.
Fixes #39733.

File:

• trunk/src/wp-content/themes/twentyeleven/content.php (modified) (2 diffs)

trunk/src/wp-content/themes/twentyeleven/content.php

@@ -56 +56 @@
             <?php if ( is_object_in_taxonomy( get_post_type(), 'category' ) ) : // Hide category text when not supported. ?>
                 <?php
-                /* translators: Used between list items, there is a space after the comma. */
-                $categories_list = get_the_category_list( __( ', ', 'twentyeleven' ) );
+                $categories_list = get_the_category_list( wp_get_list_item_separator() );
 
                 if ( $categories_list ) :
@@ -73 +72 @@
             <?php if ( is_object_in_taxonomy( get_post_type(), 'post_tag' ) ) : // Hide tag text when not supported. ?>
                 <?php
-                /* translators: Used between list items, there is a space after the comma. */
-                $tags_list = get_the_tag_list( '', __( ', ', 'twentyeleven' ) );
+                $tags_list = get_the_tag_list( '', wp_get_list_item_separator() );
 
                 if ( $tags_list && ! is_wp_error( $tags_list ) ) :