WordPress.org

Make WordPress Core

Opened 10 years ago

Closed 8 years ago

Last modified 6 years ago

#2254 closed defect (bug) (wontfix)

single_post_title filtering

Reported by: fredcz Owned by:
Milestone: Priority: normal
Severity: normal Version: 2.0
Component: General Keywords: has-patch
Focuses: Cc:

Description

could change this code:

if (is_single() || is_page()) {
$title = strip_tags( $posts[0]->post_title );
$title = apply_filters('single_post_title', $title);
}

in template-functions-general.php to:

if (is_single() || is_page()) {
$title = $posts[0]->post_title;
$title = apply_filters('single_post_title', $title);
$title = strip_tags($title);
}

Some plugins (such as Polyglot) need tags in the title for filtering.
More info: http://fredfred.net/skriker/index.php/polyglot/

Attachments (1)

template-functions-general.php.diff (525 bytes) - added by fredcz 10 years ago.
suggested fix

Download all attachments as: .zip

Change History (5)

@fredcz10 years ago

suggested fix

comment:1 @davidhouse10 years ago

  • Keywords bg|has-patch added

Would this break any plugins? It shouldn't, as we don't allow tags in titles by default, so plugins shouldn't rely on it.

comment:2 @fredcz9 years ago

I just think that you should pass raw title for filtering, that's all. New version of Polyglot doesn't need tags anymore and is using [lang_xx]lang_xx instead.

comment:3 @fredcz9 years ago

I just think that you should pass raw title for filtering, that's all. New version of Polyglot doesn't need tags anymore and is using [lang_xx]lang_xx instead.

comment:4 @Nazgul8 years ago

  • Keywords has-patch added; bg|has-patch removed
  • Resolution set to wontfix
  • Status changed from new to closed

No real traction for almost a year, so closing it as wontfix.

If somebody has suggestions/patches/... to add, feel free to reopen.

Note: See TracTickets for help on using tickets.