![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png){kind=logo}
#26075 closed defect (bug) (fixed)
Hook Docs (4): wp-admin/media-upload.php
| Reported by: | piontkowski | Owned by: | kpdesign |
|---|---|---|---|
| Priority: | normal | Milestone: | 3.9 |
| Component: | Media | Version: | |
| Severity: | normal | Keywords: | has-patch commit |
| Cc: | Focuses: | docs |
Description
Attached patch has inline docs for media_upload_default_type and media_upload_default_tab filters and media_upload_$type and media_upload_$tab actions.
Attachments (2)
Change History (9)
#2
@
13 years ago
- Keywords needs-patch added; has-patch removed
- Owner set to
- Status new → reviewing
- Summary Hook Docs: wp-admin/media-upload.php → Hook Docs (4): wp-admin/media-upload.php
#3
@
13 years ago
- Keywords has-patch docs-feedback added; needs-patch removed
- Milestone Awaiting Review → 3.9
26075.2.diff is a complete re-pass of these hooks. It attempts to navigate the complex conditions under which the media_upload_$type and media_upload_$tab hooks fire. The language needs a secondary review.
Note:
See TracTickets
for help on using tickets.
Thanks for the patch. Looks like you missed documenting the single parameters in the
media_upload_default_typeandmedia_upload_default_tabfilters. Also, some of the spacing and wrapping is a little bit off. Should be able to wrap this one up today during the sprint.