Make WordPress Core

Opened 7 years ago

Closed 7 years ago

#47959 closed enhancement (fixed)

'disable_formats_dropdown' filter does not pass no post_type.

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

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 7 years ago.
Post type is now passed into the filter.
47959-2.diff​ (739 bytes ) - added by birgire 7 years ago.

Download all attachments as: .zip

Change History (12)

#1 @mukesh27
7 years ago

  • Component Posts, Post Types → Post Formats
  • Version 5.2.2 → 5.2

#2 @SergeyBiryukov
7 years ago

  • Keywords good-first-bug added
  • Milestone Awaiting Review → Future Release

@MatheusFD
7 years ago

Post type is now passed into the filter.

#3 @MatheusFD
7 years ago

  • Keywords has-patch added; needs-patch removed

#4 @SergeyBiryukov
7 years ago

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

#5 @davidbaumwald
7 years 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
7 years ago

#6 @birgire
7 years 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
7 years ago

  • Milestone 5.3 → 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!

#8 @MatheusFD
7 years ago

Hey @SergeyBiryukov and @davidbaumwald,

Maybe we can deliver this patch in the next release?

#9 @SergeyBiryukov
7 years ago

  • Milestone Future Release → 5.5

#10 @SergeyBiryukov
7 years ago

  • Resolution → fixed
  • Status reviewing → closed

In 47472:

Post Formats: Pass post type to the disable_formats_dropdown filter in WP_Posts_List_Table::formats_dropdown().

Props MatheusFD, birgire.
Fixes #47959.

Note: See TracTickets for help on using tickets.