#55904 closed enhancement (fixed)
Add a set of fine-grained filters to disable the different types of feed links separately
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | 6.1 | Priority: | normal |
| Severity: | normal | Version: | |
| Component: | Feeds | Keywords: | has-patch has-unit-tests needs-testing needs-dev-note |
| Focuses: | Cc: |
Description (last modified by )
The current set of filters to enable/disable display feed links is quite limited.
We have just:
feed_links_show_posts_feedto control the main feedfeed_links_show_comments_feedto control both the global comments feed, and the comment feed for singular posts.
If you want do disable the other feeds (post type archive, category, tag, custom taxonomy, author archive, search results) you have to unhook feed_links_extra from wp_head, but this removes completely all those feeds + the single post comments feed.
We should introduce a full set of filters to control each one of the feeds independently, included a way to enable/disable the single post comments feed when the global comments feed is disabled/enabled:
feed_links_extra_show_post_comments_feedfeed_links_extra_show_post_type_archive_feedfeed_links_extra_show_category_feedfeed_links_extra_show_tag_feedfeed_links_extra_show_tax_feedfeed_links_extra_show_author_feedfeed_links_extra_show_search_feed
All of them should get true as default input, except for feed_links_extra_show_post_comments_feed which should get the output of feed_links_show_comments_feed to ensure backwards compatibility.
Change History (11)
This ticket was mentioned in PR #2773 on WordPress/wordpress-develop by enricobattocchi.
4 years ago
#1
- Keywords has-patch added
#4
follow-up:
↓ 5
@
4 years ago
Hi there!
Thanks for the ticket and PR.
I have added one small piece of feedback on PR. Please check and address it.
#5
in reply to:
↑ 4
@
4 years ago
Replying to mukesh27:
Hi there!
Thanks for the ticket and PR.
I have added one small piece of feedback on PR. Please check and address it.
Thanks! I just fixed it.
#6
@
3 years ago
- Keywords commit added
- Owner set to audrasjb
- Status changed from assigned to accepted
- Version trunk deleted
The patch looks good to me.
Self assigning for commit.
#7
@
3 years ago
- Keywords needs-testing added; commit removed
After further review, removing commit keyword to allow further testing.
SergeyBiryukov commented on PR #2773:
3 years ago
#9
Thanks for the PR! Merged in r54161.
Add a set of fine-grained filters to disable the different types of feed links separately.
Trac ticket: https://core.trac.wordpress.org/ticket/55904