[wp-trac] [WordPress Trac] #33521: manage_${post_type}_posts_columns parameters shifted

[WordPress Trac]

#33521: manage_${post_type}_posts_columns parameters shifted
-------------------------------+------------------------------
 Reporter:  Cybr               |       Owner:
     Type:  defect (bug)       |      Status:  new
 Priority:  normal             |   Milestone:  Awaiting Review
Component:  Taxonomy           |     Version:  4.3
 Severity:  normal             |  Resolution:
 Keywords:  reporter-feedback  |     Focuses:  administration
-------------------------------+------------------------------
Changes (by DrewAPicture):

 * keywords:   => reporter-feedback


Comment:

 Hi @Cybr, welcome to Trac!

 Not sure I'm following. There appears to be some discrepancy in the hooks
 you're referencing in your report.

 First you reference the `manage_${post_type}_posts_columns` hook, then
 you've linked to the `manage_{$post->post_type}_posts_custom_column` hook
 call in the source, and finally, in your example, referenced
 `manage_{$slug}_custom_column` where `$slug` would need to be `posts` or
 `pages` for post types and a screen ID for taxonomies.

 So, which hook exactly are you reporting an issue with? And where are you
 seeing an indication that that hook "requires" 3 parameters?

 Side note:
 In your example, you're using `manage_{$slug}_custom_column` for both post
 types and taxonomies, in which case you'd want to pass the `$type` value
 instead of `$slug` because the hook in the Terms list table is expecting a
 screen ID.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/33521#comment:1>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform