WordPress.org

Make WordPress Core

Opened 7 years ago

Closed 3 years ago

Last modified 3 years ago

#5829 closed enhancement (invalid)

Make _wp_comment_list_item filter the post's title

Reported by: henrikmelin Owned by:
Milestone: Priority: low
Severity: normal Version: 2.3.2
Component: Administration Keywords:
Focuses: Cc:

Description

In the _wp_comment_list_item() function the post title is currently formatted using wp_specialchars. Since this has no hook it is not possible to change or format the title prior to it being displayed.

I propose that the standard formatting filter 'the_title' is applied instead, using get_the_title($post->ID), making it accessible to plugins (and formatted in the same way it is formatted everywhere else, for consistency).

This is dependent on ticket #5746. This change makes the ID available to the 'the_title' filter.

Attachments (1)

5829.diff (621 bytes) - added by henrikmelin 7 years ago.
Patch for 5829

Download all attachments as: .zip

Change History (9)

@henrikmelin7 years ago

Patch for 5829

comment:1 @lloydbudd7 years ago

  • Milestone set to 2.5

comment:2 @mattyrob7 years ago

  • Priority changed from normal to low

-1 on this props.

See this ticket:
http://trac.wordpress.org/ticket/4267

A new filter is required rather than mis-applying an existing one IMHO.

comment:3 @ffemtcj7 years ago

  • Milestone changed from 2.5 to 2.6

comment:4 @Denis-de-Bernardy6 years ago

  • Keywords 2nd-opinion removed

comment:5 @Denis-de-Bernardy6 years ago

  • Keywords needs-patch added; has-patch removed

comment:6 @Denis-de-Bernardy6 years ago

  • Milestone changed from 2.9 to Future Release
  • Summary changed from Make _wp_comment_list_item use 'the_title' filter to format the title to Make _wp_comment_list_item filter the post's title

-1 for using the_title, but a filter could be added.

comment:7 @johnbillion3 years ago

  • Keywords needs-patch removed
  • Resolution set to invalid
  • Status changed from new to closed

_wp_comment_list_item() disappeared in 2.5.

comment:8 @SergeyBiryukov3 years ago

  • Milestone Future Release deleted
Note: See TracTickets for help on using tickets.