Make WordPress Core

Opened 13 years ago

Closed 13 years ago

#24293 closed enhancement (fixed)

$allowedposttags to allow value for <li>

Reported by: kraftbj Owned by: nacin
Priority: normal Milestone: 3.7
Component: Formatting Version:
Severity: normal Keywords: has-patch commit 3.7-early
Cc: Focuses:

Description

HTML 5 allowed the value attribute in <li> elements for use with ols (e.g. setting <ol><li value="31"> would result in that item of the ol rendering as 31.

$allowedposttags strips that out currently, as that's not allowed.

Submitting a patch to allow the value attribute.

Attachments (1)

24293.patch (314 bytes ) - added by kraftbj 13 years ago.
added value to li in $allowedposttags in /wp-includes/kses.php

Download all attachments as: .zip

Change History (7)

@kraftbj
13 years ago

added value to li in $allowedposttags in /wp-includes/kses.php

#1 @kraftbj
13 years ago

  • Cc bk@… added
  • Keywords has-patch added

#2 @SergeyBiryukov
13 years ago

  • Keywords commit added
  • Milestone Awaiting Review3.6

#3 @kraftbj
13 years ago

From the last IRC chat, nacin mentioned this may not be so straightforward of a patch, needing to ensure proper escaping, etc. I'm not sure the best way to ensure of this since we're just needing to add an extra attribute.

Happy to do the work to learn the process; can someone help me with what direction to go in to ensure that adding value attribute to the li tag works as it should?

A very similar "start" attribute to the ol tag is already in $allowedposttags.

#4 @ocean90
13 years ago

  • Component SecurityFormatting
  • Keywords 3.7-early added
  • Milestone 3.6Future Release
  • Type defect (bug)enhancement

#5 @wonderboymusic
13 years ago

  • Milestone Future Release3.7

these are all marked 3.7-early

#6 @nacin
13 years ago

  • Owner set to nacin
  • Resolutionfixed
  • Status newclosed

In 25460:

Allow the value attribute for list items processed by kses.

props kraftbj.
fixes #24293.

Note: See TracTickets for help on using tickets.