Make WordPress Core

Opened 15 years ago

Closed 15 years ago

Last modified 15 years ago

#17217 closed defect (bug) (fixed)

Walker_PageDropdown doesn't filter titles correctly

Reported by: Otto42 Owned by:
Priority: normal Milestone: 3.1.2
Component: Administration Version: 3.1
Severity: normal Keywords: has-patch
Cc: Focuses:

Description (last modified by Otto42)

The Walker_PageDropdown has this code:

$title = esc_html($page->post_title);
$title = apply_filters( 'list_pages', $page->post_title );

Meaning that the esc_html is not applied properly, since $title just gets replaces with the $post_title again.

Fix is this:

$title = esc_html($page->post_title);
$title = apply_filters( 'list_pages', $title );

Patch attached.

Attachments (1)

walkerpatch.patch (469 bytes ) - added by Otto42 15 years ago.

Download all attachments as: .zip

Change History (7)

@Otto42
15 years ago

#1 @Otto42
15 years ago

  • Description modified (diff)

#2 @Otto42
15 years ago

  • Component SecurityAdministration
  • Priority highnormal
  • Severity majornormal

#3 @nacin
15 years ago

  • Milestone Awaiting Review3.2

Going to tweak this a bit. Instead moving the esc_html() to after the filter, consistent with escaping as late as possible and also still passing post_title unescaped to the filter, as before.

#4 @nacin
15 years ago

  • Resolutionfixed
  • Status newclosed

(In [17683]) Apply esc_html properly in Walker_PageDropdown. fixes #17217.

#5 @nacin
15 years ago

  • Milestone 3.23.1.2

Broken in [16446].

#6 @nacin
15 years ago

(In [17685]) Apply esc_html properly in Walker_PageDropdown. fixes #17217 for the 3.1 branch.

Note: See TracTickets for help on using tickets.