Opened 11 years ago
Closed 11 years ago
#26209 closed defect (bug) (fixed)
Hook Docs (10): wp-admin/includes/class-wp-terms-list-table.php
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 3.9 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Taxonomy | Keywords: | has-patch commit |
Focuses: | docs | Cc: |
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
@
11 years ago
- Keywords needs-patch added; has-patch removed
- Owner set to kpdesign
- Status changed from new to reviewing
- Summary changed from Hooks Docs: wp-admin/includes/class-wp-terms-list-table.php to Hook Docs (10): wp-admin/includes/class-wp-terms-list-table.php
#2
@
11 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
@
11 years ago
- Owner changed from kpdesign to DrewAPicture
26209.1.diff contains improvements to descriptions, documents duplicate hooks, and some formatting changes.
This needs a secondary review and recommendation.
#5
@
11 years ago
- Keywords commit added
- Milestone changed from Awaiting Review to 3.9
- Status changed from reviewing to 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.