WordPress.org

Make WordPress Core

Opened 6 years ago

Closed 6 years ago

#31041 closed defect (bug) (fixed)

Make wpautop documentation more verbose

Reported by: ericlewis Owned by: DrewAPicture
Milestone: 4.2 Priority: normal
Severity: normal Version: 4.2
Component: Formatting Keywords: has-patch
Focuses: docs Cc:

Description

To help myself understand what's going on in wpautop, I added some more verbose documentation. Others for whom regex is a second language may also benefit.

Attachments (3)

31041.diff (5.9 KB) - added by ericlewis 6 years ago.
31041.patch (5.9 KB) - added by johneckman 6 years ago.
Refreshed patch
31041.2.patch (5.8 KB) - added by azaozz 6 years ago.

Download all attachments as: .zip

Change History (10)

@ericlewis
6 years ago

#1 @SergeyBiryukov
6 years ago

  • Milestone changed from Awaiting Review to 4.2

#2 @DrewAPicture
6 years ago

  • Keywords has-patch needs-refresh added

Looks like the patch needs a refresh.

@johneckman
6 years ago

Refreshed patch

#3 @johneckman
6 years ago

  • Keywords needs-refresh removed

This ticket was mentioned in Slack in #core by drew. View the logs.


6 years ago

#5 @DrewAPicture
6 years ago

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

31041.patch looks pretty good and still applies fine.

@azaozz: Can you pretty please take a look at the new docs for this just for a sanity check, then recommend commit or iteration? :-)

@azaozz
6 years ago

#6 @DrewAPicture
6 years ago

  • Owner changed from azaozz to DrewAPicture

Thanks @azaozz.

#7 @DrewAPicture
6 years ago

  • Resolution set to fixed
  • Status changed from reviewing to closed

In 31852:

Add more verbose inline documentation inside of wpautop() to more clearly explain what's going on.

Props ericlewis, johneckman, azaozz.
Fixes #31041.

Note: See TracTickets for help on using tickets.