Make WordPress Core

#25566 closed defect (bug) (fixed)

Popular tags edit links do not respect current post type

Reported by: mordauk Owned by: wonderboymusic
Milestone: 3.9 Priority: normal
Severity: normal Version: 3.8
Component: Taxonomy Keywords: has-patch wcorl needs-testing
Focuses: administration Cc:


When a post type registers a non-hierarchical taxonomy, there is a "Popular Tags" section displayed on that taxonomies overview. Each of the terms included in the popular tags Tag Cloud is linked to an edit page for that term, but when clicking on the edit link, the current post type isn't respected, meaning the admin menu goes from having the current post type highlighted to having the standard Posts menu highlighted.

For example, the URL for the edit link of a popular "Download Tag" from my Easy Digital Downloads plugin is:


Since &post_type=download is not included in the URL, the dashboard menu highlight changes to Posts.

To fix this, a couple of things are needed:

  1. post_type needs to be passed as an argument to wp_tag_cloud() in wp-admin/edit-tags.php if the current user can edit the term.

2, The wp_tag_cloud() function should call get_edit_term_link() instead of get_edit_tag_link() when the link parameter is set to edit. The post_type arg passed should also be passed as the third parameter to get_edit_term_link(), for the object type.

Attachments (5)

25566.patch (1.7 KB) - added by mordauk 18 months ago.
25566.2.patch (2.1 KB) - added by mordauk 13 months ago.
Refreshed patch with documentation for the post_type argument of wp_tag_cloud
25566-test.diff (1.3 KB) - added by fahmiadib 13 months ago.
25566.diff (3.5 KB) - added by wonderboymusic 12 months ago.
25566.2.diff (3.6 KB) - added by wonderboymusic 12 months ago.

Download all attachments as: .zip

Change History (16)

@mordauk18 months ago

comment:1 @mordauk18 months ago

  • Keywords has-patch added
  • Version set to trunk

comment:2 @mordauk17 months ago

  • Keywords wcorl added

comment:3 @mordauk13 months ago

  • Keywords needs-testing added

comment:4 @mordauk13 months ago

This is ready for further testing if anyone wants to tackle it.

comment:5 @DrewAPicture13 months ago

  • Focuses administration added

25566.patch works as advertised for me. I also registered the taxonomy for multiple post types and both types the links respected the current post type.

Probably worth adding a short bit of documentation to the description about the new post_type parameter that wp_tag_cloud() accepts. We'll probably convert that section of the PHPDoc to a hash notation in the future but a small addition now should suffice.

@mordauk13 months ago

Refreshed patch with documentation for the post_type argument of wp_tag_cloud

comment:6 @mordauk13 months ago

Updated patch attached with documentation for the post_type argument.

comment:7 @DrewAPicture13 months ago

  • Milestone changed from Awaiting Review to 3.9

25566.2.patch looks good to me.

@fahmiadib13 months ago

comment:8 @fahmiadib13 months ago

Added unit test. review?

@wonderboymusic12 months ago

comment:9 @wonderboymusic12 months ago

25566.diff combines the patches - the unit tests need some work.

@wonderboymusic12 months ago

comment:10 @wonderboymusic12 months ago

25566.2.diff contains safer unit tests

comment:11 @wonderboymusic12 months ago

  • Owner set to wonderboymusic
  • Resolution set to fixed
  • Status changed from new to closed

In 27720:

Popular tags' edit links should respect the current post type. Adds unit test.

Props mordauk, fahmiadib.
Fixes #25566.

Note: See TracTickets for help on using tickets.