[wp-trac] [WordPress Trac] #12398: Reinstate manage_pages_columns filter missing in wp_manage_posts_columns()

WordPress Trac wp-trac at lists.automattic.com
Sat Feb 27 11:45:21 UTC 2010


#12398: Reinstate manage_pages_columns filter missing in wp_manage_posts_columns()
-------------------------------+--------------------------------------------
 Reporter:  studiograsshopper  |        Owner:  nacin 
     Type:  defect (bug)       |       Status:  closed
 Priority:  high               |    Milestone:  3.0   
Component:  Editor             |      Version:  3.0   
 Severity:  normal             |   Resolution:  fixed 
 Keywords:  needs-patch        |  
-------------------------------+--------------------------------------------

Comment(by studiograsshopper):

 Replying to [comment:3 nacin]:
 > manage_{$post_type}_columns might not work as it might clash with other
 existing filters, such as one in get_column_headers() and
 'manage_media_columns'.
 >
 > This is a good start though. Just a style note, we generally put the
 variable on the right in comparisons, that way you don't accidentally do
 an assignment (see your first patch).
 Thanks for pointing that out. :-)

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/12398#comment:5>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list