Add Ability to Fetch Post Types by Supports

[wpsmith]

I would be great to see a WordPress function or functions to fetch a list of registered post types that support a specific feature or a set of features.

​https://gist.github.com/wpsmith/6d6679e05c1ab62752ea

[wpsmith]

get_post_types_by_support()

[wpsmith]

get_post_types_by_supports()

[swissspidy]

It's not uncommon to loop through to all post types to see if a post type supports a given feature, so this is certainly of value in my opinion. I don't think it's necessary to query by multiple features though.

34010.diff​ proposes a simple function get_post_types_by_support( $feature ) function (get_post_types_supporting sounds good too). Basically ​like in this gist, but without needing to access the global variable.

[barryceelen]

Allow getting post types by multiple features

[barryceelen]

@swissspidy Why are you looping through $post_types in stead of using the $_wp_post_type_features global? I see all other post_type_support related functions are using it.

[swissspidy]

@barryceelen I don't know anymore, probably just to not rely on the global :)

I don't think it's necessary to query by multiple features though.

Since it's so simple and there are situations where this can indeed be necessary, I changed my mind here. I just uploaded 34010.2.diff​ now which includes unit tests.

[barryceelen]

:)
While we're at it might as well also add the $operator parameter like get_post_types() does (One of the patches by @wpsmith had it but the server is unreachable at the time).

[barryceelen]

Include and/or operator

[swissspidy]

Replying to barryceelen:

:)
While we're at it might as well also add the $operator parameter like get_post_types() does (One of the patches by @wpsmith had it but the server is unreachable at the time).

Why don't we just allow passing a 'supports' filter to get_post_types()? Makes more sense IMHO.

[barryceelen]

In fact, the get_post_types() function is where I started looking for this functionality, kinda makes sense to me. Something held me back from modifying that function but I can't recall why :)

[swissspidy]

Adding this to get_post_types() seems to make more sense, but it would also make the function more complicated.

Let's stick with 34010.2.diff​ with the operator argument suggested in 34010.with-operator.diff​.

[barryceelen]

Thanks for getting back to this. Adapting the get_post_types() function indeed got unwieldy when trying to combine/diff the two different $_wp_post_type_features and $wp_post_types arrays. Also, the $_wp_post_type_features array contains 'meta' post types like attachment.video and attachment.audio, unsure how those would be handled.

[swissspidy]

34010.3.diff​ is an updated patch based on my last comment.

[ocean90]

Patch looks good.

[swissspidy]

34010.4.diff​ uses assertContains instead of assertEqualSets because some other tests register some random post types, causing these tests to fail.

[swissspidy]

In 36652:

Posts: Introduce get_post_types_by_support().

Similar to get_post_types(), this new function returns a list of post type names that support a specific feature.

Props wpsmith, barryceelen, swissspidy.
Fixes #34010.

[DrewAPicture]

In 36704:

Posts: Rename the $args parameter in get_post_types_by_support() to $feature for better self-documentation.

See #34010. See #32246.