WordPress.org

Make WordPress Core

Opened 6 weeks ago

Last modified 3 weeks ago

#47959 reviewing enhancement

'disable_formats_dropdown' filter does not pass no post_type.

Reported by: chrstffr Owned by: SergeyBiryukov
Milestone: Future Release Priority: normal
Severity: minor Version: 5.2
Component: Post Formats Keywords: good-first-bug has-patch commit
Focuses: Cc:
PR Number:

Description

On line 477 in class-wp-posts-list-table.php, the post type is not passed into the filter.

Attachments (2)

47959.patch (537 bytes) - added by MatheusFD 5 weeks ago.
Post type is now passed into the filter.
47959-2.diff (739 bytes) - added by birgire 3 weeks ago.

Download all attachments as: .zip

Change History (9)

#1 @mukesh27
6 weeks ago

  • Component changed from Posts, Post Types to Post Formats
  • Version changed from 5.2.2 to 5.2

#2 @SergeyBiryukov
6 weeks ago

  • Keywords good-first-bug added
  • Milestone changed from Awaiting Review to Future Release

@MatheusFD
5 weeks ago

Post type is now passed into the filter.

#3 @MatheusFD
5 weeks ago

  • Keywords has-patch added; needs-patch removed

#4 @SergeyBiryukov
5 weeks ago

  • Milestone changed from Future Release to 5.3
  • Owner set to SergeyBiryukov
  • Status changed from new to reviewing

#5 @davidbaumwald
3 weeks ago

  • Keywords commit added

@SergeyBiryukov The patch here looks good and still applies cleanly to trunk. Is this something that can be included for 5.3 Beta 1 tomorrow? If not, can we move this to a future milestone?

@birgire
3 weeks ago

#6 @birgire
3 weeks ago

Thanks for the patch @MatheusFD

47959-2.diff adds:

* @since 5.3.0 The `$post_type` parameter was added.

for the filter's changelog.

#7 @davidbaumwald
3 weeks ago

  • Milestone changed from 5.3 to Future Release

With 5.3 Beta 1 shipping today, I’m punting this to Future Release so as not to clog up the 5.4 milestone. However, @SergeyBiryukov, as the ticket owner feel free to move to 5.4 if you feel comfortable committing to resolving this fully during that release cycle. Thanks!

Note: See TracTickets for help on using tickets.