Make WordPress Core

Opened 3 years ago

Last modified 3 months ago

#16790 reviewing enhancement

Functional spec for wpautop

Reported by: mrclay Owned by:
Milestone: Awaiting Review Priority: normal
Severity: minor Version:
Component: Formatting Keywords: wpautop
Focuses: Cc:


I've put together an algorithm-based functional spec that I hope could be useful in maintaining wpautop. Also might be handy is this combination of wpautop failure cases.

The algorithm is based on work I've started on a DOMDocument-based implementation (a project I contribute to needs an MIT-licensed version), and although it's incomplete and slow, it does a bang-up job so far.

DOMDocument may just be too slow or strict in its parsing to allow the variety of markup that wpautop needs to accept, but it's ace at the level of surgical precision needed to do this right.

Change History (4)

comment:1 scribu3 years ago

  • Owner set to scribu
  • Status changed from new to reviewing

Although not directly relevant, it's good to remember that the HTML runs through kses afterwards, which can alter it significantly.

comment:2 scribu3 years ago

  • Owner scribu deleted

comment:3 nacin19 months ago

  • Component changed from Unit Tests to Formatting

This is cool. For anything not currently covered by tests, it'd be great if we added unit tests for those.

Maybe such a document could find its way into http://make.wordpress.org/core/handbook/ as an appendix of some sort.

comment:4 nacin3 months ago

  • Keywords wpautop added
Note: See TracTickets for help on using tickets.