WordPress.org
Get WordPress

Make WordPress Core

Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#25839 closed defect (bug) (fixed)

Hook Docs (5): wp-admin/includes/class-wp-media-list-table.php

Reported by: theorboman Owned by: kpdesign
Priority: normal Milestone: 3.9
Component: Media Version:
Severity: normal Keywords: has-patch commit
Cc: Focuses: docs

Description

Adding inline doc to all the hooks in wp-admin/includes/class-wp-media-list-table.php

Attachments (3)

25839.patch​ (2.6 KB ) - added by theorboman 13 years ago.
Patch for inline docs
25839.2.patch​ (2.9 KB ) - added by DrewAPicture 13 years ago.
Second pass.
25839.3.patch​ (2.9 KB ) - added by kpdesign 13 years ago.
Third pass

Download all attachments as: .zip

Change History (12)

#1 @kpdesign
13 years ago

  • Keywords needs-patch added

@theorboman
13 years ago

Patch for inline docs

#2 @theorboman
13 years ago

  • Keywords has-patch added; needs-patch removed

#3 @kpdesign
13 years ago

  • Owner set to kpdesign
  • Status new → reviewing

#4 @DrewAPicture
13 years ago

  • Keywords needs-patch added; has-patch removed
  • Summary Hook Docs: wp-admin/includes/class-wp-media-list-table.php → Hook Docs (5): wp-admin/includes/class-wp-media-list-table.php

25839.patch​ is a great first patch. Looks like there are some spacing issues, and a few docs-specific parameter variables need to be used as well.

@DrewAPicture
13 years ago

Second pass.

#5 @DrewAPicture
13 years ago

  • Keywords has-patch docs-feedback added; needs-patch removed
  • Milestone Awaiting Review → 3.9

25839.2.patch​ applies the suggested changes from comment:4 and clarifies some language. Needs a secondary review.

@kpdesign
13 years ago

Third pass

#6 @kpdesign
13 years ago

  • Keywords commit added; docs-feedback removed

25839.3.patch​ provides changes for consistency in language used.

I think this one's a good commit candidate.

#7 @DrewAPicture
13 years ago

  • Resolution → fixed
  • Status reviewing → closed

In 26916:

Inline documentation for hooks in wp-admin/includes/class-wp-media-list-table.php.

Props theorboman for the initial patch. Props kpdesign.
Fixes #25839.

#8 @DrewAPicture
13 years ago

  • Focuses docs added

#9 @DrewAPicture
13 years ago

  • Component Inline Docs → Media
Note: See TracTickets for help on using tickets.