WordPress.org
Get WordPress

Make WordPress Core

Opened 6 years ago

Closed 6 years ago

#50736 closed enhancement (fixed)

`manage_comments_nav` hook miss parameter

Reported by: tkama's profile Tkama Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 5.6 Priority: normal
Severity: normal Version: 2.5
Component: Comments Keywords: good-first-bug has-patch
Focuses: administration Cc:

Description

It would be great if manage_comments_nav hook pass $which as the second parameter.

do_action( 'manage_comments_nav', $comment_status, $which );

See: ​https://developer.wordpress.org/reference/classes/wp_comments_list_table/extra_tablenav/

Attachments (2)

manage-comments-nav.diff​ (663 bytes) - added by wpgurudev 6 years ago.
manage-comments-nav.2.diff​ (816 bytes) - added by wpgurudev 6 years ago.

Download all attachments as: .zip

Change History (6)

#1 @SergeyBiryukov
6 years ago

  • Component changed from Administration to Comments
  • Focuses administration added
  • Milestone changed from Awaiting Review to 5.6

#2 @johnbillion
6 years ago

  • Keywords needs-patch good-first-bug added
  • Version changed from 5.4.2 to 2.5

@wpgurudev
6 years ago

@wpgurudev
6 years ago

#3 @SergeyBiryukov
6 years ago

  • Keywords has-patch added; needs-patch removed

#4 @SergeyBiryukov
6 years ago

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

In 48693:

Comments: Pass the $which parameter to the manage_comments_nav action.

This brings some consistency with the manage_users_extra_tablenav and manage_users_extra_tablenav actions.

Props Tkama, wpgurudev.
Fixes #50736.

Note: See TracTickets for help on using tickets.