[wp-trac] [WordPress Trac] #38772: restrict_manage_posts in wp-media-list-table.php missing 2nd param

WordPress Trac noreply at wordpress.org
Mon Nov 21 20:52:35 UTC 2016


#38772: restrict_manage_posts in wp-media-list-table.php missing 2nd param
----------------------------+------------------
 Reporter:  wpsmith         |       Owner:
     Type:  defect (bug)    |      Status:  new
 Priority:  normal          |   Milestone:  4.8
Component:  Administration  |     Version:  4.6
 Severity:  normal          |  Resolution:
 Keywords:  has-patch       |     Focuses:
----------------------------+------------------
Changes (by flixos90):

 * keywords:   => has-patch
 * version:  3.1 => 4.6
 * type:  enhancement => defect (bug)
 * milestone:  Awaiting Review => 4.8


Comment:

 The `$which` parameter was added to the `restrict_manage_posts` action in
 [changeset:37422], so I assume the filter occurrence in the media list
 table was overlooked. The existing patch looks good to fix that.

 Regarding the ticket type, I agree with @wpsmith that this is a bug, since
 it should be expected that a hook has parameter parity wherever it is
 executed. Since it's not a bug against trunk, let's move it to 4.8.

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


More information about the wp-trac mailing list