![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png){kind=logo}
Opened 13 years ago
Closed 13 years ago
#26209 closed defect (bug) (fixed)
Hook Docs (10): wp-admin/includes/class-wp-terms-list-table.php
| Reported by: | Ajay | Owned by: | DrewAPicture |
|---|---|---|---|
| Priority: | normal | Milestone: | 3.9 |
| Component: | Taxonomy | Version: | |
| Severity: | normal | Keywords: | has-patch commit |
| Cc: | Focuses: | docs |
Description
Attached patch has docs for edit_tags_per_page, tagsperpage, edit_categories_per_page, term_name, tag_row_actions, {$taxonomy}_row_actions, editable_slug, manage_{$this->screen->taxonomy}_custom_column filters and quick_edit_custom_box action
Attachments (4)
Change History (10)
#1
@
13 years ago
- Keywords needs-patch added; has-patch removed
- Owner set to
- Status new → reviewing
- Summary Hooks Docs: wp-admin/includes/class-wp-terms-list-table.php → Hook Docs (10): wp-admin/includes/class-wp-terms-list-table.php
#2
@
13 years ago
- Keywords has-patch added; needs-patch removed
I've attached an updated diff file 26209.diff which changes the tabs to spaces and also fixed a missing parameter. Please let me know if this is fine.
#4
@
13 years ago
- Owner changed from to
26209.1.diff contains improvements to descriptions, documents duplicate hooks, and some formatting changes.
This needs a secondary review and recommendation.
#5
@
13 years ago
- Keywords commit added
- Milestone Awaiting Review → 3.9
- Status reviewing → accepted
Double-checked all of the @since versions and 26209.2.diff fixes one. Also adds some spacing and s/tag/term in a couple of places. Ready to go in.
Note:
See TracTickets
for help on using tickets.
Hi, thanks for the patch.
Looking at class-wp-terms-list-table.patch, seems like spacing was done with tabs instead of spaces. Also, some parameter descriptions are missing. Could use iteration and tidying up.