WordPress.org

Make WordPress Core

Opened 12 days ago

Closed 4 days ago

#50736 closed enhancement (fixed)

`manage_comments_nav` hook miss parameter

Reported by: Tkama Owned by: 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 11 days ago.
manage-comments-nav.2.diff (816 bytes) - added by wpgurudev 10 days ago.

Download all attachments as: .zip

Change History (6)

#1 @SergeyBiryukov
12 days ago

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

#2 @johnbillion
12 days ago

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

#3 @SergeyBiryukov
4 days ago

  • Keywords has-patch added; needs-patch removed

#4 @SergeyBiryukov
4 days 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.