WordPress.org
Get WordPress

Make WordPress Core

Opened 12 years ago

Closed 11 years ago

#31041 closed defect (bug) (fixed)

Make wpautop documentation more verbose

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

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 12 years ago.
31041.patch​ (5.9 KB ) - added by johneckman 12 years ago.
Refreshed patch
31041.2.patch​ (5.8 KB ) - added by azaozz 11 years ago.

Download all attachments as: .zip

Change History (10)

@ericlewis
12 years ago

#1 @SergeyBiryukov
12 years ago

  • Milestone Awaiting Review → 4.2

#2 @DrewAPicture
12 years ago

  • Keywords has-patch needs-refresh added

Looks like the patch needs a refresh.

@johneckman
12 years ago

Refreshed patch

#3 @johneckman
12 years ago

  • Keywords needs-refresh removed

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

#5 @DrewAPicture
11 years ago

  • Owner set to azaozz
  • Status new → 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
11 years ago

#6 @DrewAPicture
11 years ago

  • Owner changed from azaozz to DrewAPicture

Thanks @azaozz.

#7 @DrewAPicture
11 years ago

  • Resolution → fixed
  • Status reviewing → 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.