Opened 7 years ago
Closed 7 years ago
#48857 closed enhancement (fixed)
I found wrong variables in documentation of handle_bulk_actions-{$screen} filter.
| Reported by: | dilipbheda | Owned by: | SergeyBiryukov |
|---|---|---|---|
| Priority: | normal | Milestone: | 5.4 |
| Component: | Comments | Version: | |
| Severity: | normal | Keywords: | has-patch 2nd-opinion |
| Cc: | Focuses: | docs |
Description
I updated as per WordPress. guidelines/standard
I have added a patch for the same.
Attachments (1)
Change History (4)
Note:
See TracTickets
for help on using tickets.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Thanks for the patch @dilipbheda.
I believe this filter is a special case because this it's applied in several places in core with differing parameter names, hence the generic
$itemsvariable name rather than the more specific$comment_idswhich only applies to the bulk management of comments.That said, the inconsistency here isn't ideal. Some instances of the filter still use the non-interpolated version of the filter name.