Make WordPress Core

Opened 10 years ago

Closed 10 years ago

#26209 closed defect (bug) (fixed)

Hook Docs (10): wp-admin/includes/class-wp-terms-list-table.php

Reported by: ajay's profile Ajay Owned by: drewapicture's profile DrewAPicture
Milestone: 3.9 Priority: normal
Severity: normal Version:
Component: Taxonomy Keywords: has-patch commit
Focuses: docs Cc:


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)

class-wp-terms-list-table.patch (4.4 KB) - added by Ajay 10 years ago.
26209.diff (4.4 KB) - added by Ajay 10 years ago.
26209.1.diff (4.6 KB) - added by kpdesign 10 years ago.
Third pass
26209.2.diff (4.8 KB) - added by DrewAPicture 10 years ago.
Final pass.

Download all attachments as: .zip

Change History (10)

#1 @DrewAPicture
10 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

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.

10 years ago

#2 @Ajay
10 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.

#3 @jeremyfelt
10 years ago

  • Component changed from Inline Docs to Taxonomy
  • Focuses docs added

10 years ago

Third pass

#4 @kpdesign
10 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.

10 years ago

Final pass.

#5 @DrewAPicture
10 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.

#6 @DrewAPicture
10 years ago

  • Resolution set to fixed
  • Status changed from accepted to closed

In 27089:

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

Props Ajay, kpdesign.
Fixes #26209.

Note: See TracTickets for help on using tickets.