Ticket #36171: 36171.diff

src/wp-includes/category-template.php

@@ -168 +168 @@
  * @param int $post_id Optional. Post ID to retrieve categories.
  * @return string
  */
-function get_the_category_list( $separator = '', $parents='', $post_id = false ) {
+function get_the_category_list( $separator = '', $parents = '', $post_id = false ) {
         global $wp_rewrite;
+
         if ( ! is_object_in_taxonomy( get_post_type( $post_id ), 'category' ) ) {
                 /** This filter is documented in wp-includes/category-template.php */
                 return apply_filters( 'the_category', '', $separator, $parents );
@@ -193 +194 @@
 
         $rel = ( is_object( $wp_rewrite ) && $wp_rewrite->using_permalinks() ) ? 'rel="category tag"' : 'rel="category"';
 
+        $links = array();
+        foreach ( $categories as $category ) {
+                /**
+                 * Break the link for better link building
+                 */
+                $start_link = '<a href="' . esc_url( get_category_link( $category->term_id ) ) . '" ' . $rel . '>';
+                $end_link   = $category->name . '</a>';
+
+                /**
+                 * Build the category links
+                 */
+                $the_link_list = '';
+                switch ( strtolower( $parents ) ) {
+                        case 'multiple':
+                                $cat_parents   = $category->parent ? get_category_parents( $category->parent, true, $separator ) : '';
+                                $the_link_list = $cat_parents . ' ' . $start_link . $end_link;
+                                break;
+                        case 'single':
+                                $cat_parents   = $category->parent ? get_category_parents( $category->parent, false, $separator ) : '';
+                                $the_link_list = $start_link . $cat_parents . $end_link;
+                                break;
+                        case '':
+                        default:
+                                $the_link_list = $start_link . $end_link;
+                }
+
+                /**
+                 * Filter the category links on category level.
+                 *
+                 * @since X.X.X
+                 *
+                 * @param string $the_link_list Post category link.
+                 * @param object $category      The current category object
+                 * @param $cat_parents          Link list of parents of the current category
+                 */
+                $links[] = apply_filters( 'the_category_list_links', $the_link_list, $category, $cat_parents );
+        }
+
         $thelist = '';
-        if ( '' == $separator ) {
+        if ( '' === $separator ) {
                 $thelist .= '<ul class="post-categories">';
-                foreach ( $categories as $category ) {
-                        $thelist .= "\n\t<li>";
-                        switch ( strtolower( $parents ) ) {
-                                case 'multiple':
-                                        if ( $category->parent )
-                                                $thelist .= get_category_parents( $category->parent, true, $separator );
-                                        $thelist .= '<a href="' . esc_url( get_category_link( $category->term_id ) ) . '" ' . $rel . '>' . $category->name.'</a></li>';
-                                        break;
-                                case 'single':
-                                        $thelist .= '<a href="' . esc_url( get_category_link( $category->term_id ) ) . '"  ' . $rel . '>';
-                                        if ( $category->parent )
-                                                $thelist .= get_category_parents( $category->parent, false, $separator );
-                                        $thelist .= $category->name.'</a></li>';
-                                        break;
-                                case '':
-                                default:
-                                        $thelist .= '<a href="' . esc_url( get_category_link( $category->term_id ) ) . '" ' . $rel . '>' . $category->name.'</a></li>';
-                        }
-                }
+                $thelist .= "\n\t<li>";
+                $thelist .= implode( "</li>\n\t<li>", $links );
+                $thelist .= '</li>';
                 $thelist .= '</ul>';
         } else {
-                $i = 0;
-                foreach ( $categories as $category ) {
-                        if ( 0 < $i )
-                                $thelist .= $separator;
-                        switch ( strtolower( $parents ) ) {
-                                case 'multiple':
-                                        if ( $category->parent )
-                                                $thelist .= get_category_parents( $category->parent, true, $separator );
-                                        $thelist .= '<a href="' . esc_url( get_category_link( $category->term_id ) ) . '" ' . $rel . '>' . $category->name.'</a>';
-                                        break;
-                                case 'single':
-                                        $thelist .= '<a href="' . esc_url( get_category_link( $category->term_id ) ) . '" ' . $rel . '>';
-                                        if ( $category->parent )
-                                                $thelist .= get_category_parents( $category->parent, false, $separator );
-                                        $thelist .= "$category->name</a>";
-                                        break;
-                                case '':
-                                default:
-                                        $thelist .= '<a href="' . esc_url( get_category_link( $category->term_id ) ) . '" ' . $rel . '>' . $category->name.'</a>';
-                        }
-                        ++$i;
-                }
+                $thelist .= implode( $separator, $links );
         }
 
         /**