Make WordPress Core

Opened 11 years ago

Closed 8 years ago

#16864 closed defect (bug) (fixed)

WP_Terms_List_Table doesn't call sanitize_term() on display

Reported by: scribu Owned by: wonderboymusic
Milestone: 4.0 Priority: normal
Severity: minor Version: 2.9
Component: Taxonomy Keywords: has-patch easy
Focuses: administration Cc:

Description (last modified by scribu)

Steps to reproduce:

Add this filter:

add_filter( 'category_slug', function() {
  return 'filtered';
} );

and go to edit-tags.php?taxonomy=category

Attachments (3)

16864.diff (494 bytes) - added by scribu 11 years ago.
16864.2.diff (493 bytes) - added by c3mdigital 8 years ago.
Refresh of original patch
16864.3.diff (540 bytes) - added by oso96_2000 8 years ago.
Refreshed patch

Download all attachments as: .zip

Change History (12)

11 years ago

#1 @scribu
11 years ago

  • Description modified (diff)

#2 @scribu
11 years ago

For the slug in particular, there's 'editable_slug', which doesn't pass the taxonomy.

Also, that still leaves out all the other fields, like 'name', 'description' etc.

#3 @nacin
11 years ago

  • Version set to 2.9

8 years ago

Refresh of original patch

#4 @c3mdigital
8 years ago

  • Keywords needs-refresh added

#5 @c3mdigital
8 years ago

  • Keywords needs-refresh removed

#6 @nacin
8 years ago

  • Component changed from Administration to Taxonomy
  • Focuses administration added

8 years ago

Refreshed patch

#7 @oso96_2000
8 years ago

  • Keywords dev-feedback added

#8 @wonderboymusic
8 years ago

  • Keywords easy added; dev-feedback removed
  • Milestone changed from Awaiting Review to 4.0

#9 @wonderboymusic
8 years ago

  • Owner set to wonderboymusic
  • Resolution set to fixed
  • Status changed from new to closed

In 28360:

In WP_Terms_List_Table::single_row(), call sanitize_term() on the passed term ($tag).

Props oso96_2000, c3mdigital, scribu.
It takes a village.
Fixes #16864.

Note: See TracTickets for help on using tickets.