WordPress.org
Get WordPress

Make WordPress Core

Opened 10 years ago

Closed 10 years ago

#31041 closed defect (bug) (fixed)

Make wpautop documentation more verbose

Reported by: ericlewis's profile ericlewis Owned by: drewapicture's profile 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 10 years ago.
31041.patch​ (5.9 KB) - added by johneckman 10 years ago.
Refreshed patch
31041.2.patch​ (5.8 KB) - added by azaozz 10 years ago.

Download all attachments as: .zip

Change History (10)

@ericlewis
10 years ago

#1 @SergeyBiryukov
10 years ago

  • Milestone changed from Awaiting Review to 4.2

#2 @DrewAPicture
10 years ago

  • Keywords has-patch needs-refresh added

Looks like the patch needs a refresh.

@johneckman
10 years ago

Refreshed patch

#3 @johneckman
10 years ago

  • Keywords needs-refresh removed

This ticket was mentioned in ​Slack in #core by drew. ​View the logs.
10 years ago

#5 @DrewAPicture
10 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
10 years ago

#6 @DrewAPicture
10 years ago

  • Owner changed from azaozz to DrewAPicture

Thanks @azaozz.

#7 @DrewAPicture
10 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.