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 Owned by: SergeyBiryukov
Priority: normal Milestone: 5.6
Component: Comments Version: 2.5
Severity: normal Keywords: good-first-bug has-patch
Cc: Focuses: administration

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 Administration → Comments
  • Focuses administration added
  • Milestone Awaiting Review → 5.6

#2 @johnbillion
6 years ago

  • Keywords needs-patch good-first-bug added
  • Version 5.4.2 → 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 → fixed
  • Status new → 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.