WordPress.org
Get WordPress

Make WordPress Core

Opened 10 years ago

Closed 10 years ago

#36338 closed defect (bug) (fixed)

Update docblock for export_wp()

Reported by: theMikeD Owned by: DrewAPicture
Priority: normal Milestone: 4.7
Component: Export Version:
Severity: normal Keywords: has-patch
Cc: Focuses: docs

Description

The docblock needs to have the array of values it accepts documented, especially now that devhub is pulling from code.

Attachments (5)

36338.diff​ (1.8 KB ) - added by theMikeD 10 years ago.
36338.1.diff​ (1.8 KB ) - added by theMikeD 10 years ago.
Updated docblock to coding standards
36338.2.diff​ (2.3 KB ) - added by theMikeD 10 years ago.
Third time's a charm
36338.3.diff​ (2.6 KB ) - added by theMikeD 10 years ago.
Last one. I think I have captured the current behaviour adequately
36338.4.diff​ (2.7 KB ) - added by theMikeD 10 years ago.
Tricky one to nail down. I'm done now.

Download all attachments as: .zip

Change History (11)

@theMikeD
10 years ago

#1 @theMikeD
10 years ago

  • Keywords has-patch added

This ticket was mentioned in ​Slack in #meta-devhub by themiked. ​View the logs.
10 years ago

@theMikeD
10 years ago

Updated docblock to coding standards

#3 @DrewAPicture
10 years ago

  • Milestone Awaiting Review → Future Release
  • Owner set to DrewAPicture
  • Status new → reviewing

36338.1.diff​ looks great, thanks!

@theMikeD
10 years ago

Third time's a charm

@theMikeD
10 years ago

Last one. I think I have captured the current behaviour adequately

@theMikeD
10 years ago

Tricky one to nail down. I'm done now.

#4 @DrewAPicture
10 years ago

  • Version trunk

#5 @DrewAPicture
10 years ago

  • Milestone Future Release → 4.7
  • Type enhancement → defect (bug)

#6 @DrewAPicture
10 years ago

  • Resolution → fixed
  • Status reviewing → closed

In 38253:

Docs: Add documentation for all arguments accepted by export_wp().

Props theMikeD for the initial patch.
Fixes #36338.

Note: See TracTickets for help on using tickets.