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

WordPress Trac noreply at wordpress.org
Thu Aug 27 20:48:46 UTC 2015


#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:  minor                          |  Resolution:
 Keywords:  reporter-feedback needs-patch  |     Focuses:  administration
-------------------------------------------+------------------------------
Changes (by DrewAPicture):

 * keywords:  reporter-feedback has-patch needs-testing 2nd-opinion =>
     reporter-feedback needs-patch


Comment:

 Hi @Cybr,

 I'm sorry, I'm still not understanding what bug you're pointing out. The
 patch you uploaded is showing as a PHP file, not a diff, and is for
 `WP_Terms_List_Table`, yet your comment on the patch says it's for the
 `manage_{$post_type}_custom_column` hook, which isn't even in that file.

 Also, you should be patching against
 [https://develop.svn.wordpress.org/trunk/ WordPress trunk]. Assuming
 you're talking about the `manage_{$this->screen->taxonomy}_custom_column`
 hook, the version in trunk has three arguments:
 https://core.trac.wordpress.org/browser/trunk/src/wp-admin/includes/class-
 wp-terms-list-table.php#L509

 Can you please clarify what you're trying to accomplish and where you
 think the bug is?

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


More information about the wp-trac mailing list