Option to disable only the comments feed about wordpress-seo

[jrfnl]

Hi Jeremy,

FYI: I just had a quick look at this - thinking it could be a quick fix to add -, but it's not actually that easy to implement - if at all possible -.

Notes for developers (in case it would be decided to implement this):

The main feed and the comments feed are echo'd out in one go by the feed_links() function in the wp-includes/general_template.php file. This function does not offer a way to filter the output, so it's either have both links or none at all (by removing the function from the wp_head actions queue - which is what the plugin currently does if the option to do so is turned on -).

The only way I can currently think of to do this, would be to copy the WP-core function into the plugin and add a conditional display option for the comments link there, remove the WP feed_links action and add an action to the copied function.

This however would create kind of a maintenance nightmare - quite apart from it being bad practice - as you'd have to keep an eye on any changes made to the WP-core function and be sure to emulate these into the function copied into the plugin.

A better route might be to submit a patch to WP core which adds an output filter to the feed_links() function. However getting these kind of patches accepted & merged is often a slow process, so no telling when the filter would be available.

Ok, that's my two pennies... ;-)

Smile,
Juliette

[GermanKiwi]

From what I've since read and learned, it's not possible to disable the comments feed without also disabling the main feed as well. It's either both or none.

So I've solved this for myself by disabling both feeds (using the appropriate 'remove_action' in my functions.php), and then manually adding back just the main feed via another function in my functions.php file. This does the trick for me and I think it's the only solution currently available.

I'll close this ticket as this request is obviously beyond the scope of this plugin.

[jdevalk]

Well, it's not possible right now, but I do want to make it possible, so I'm reopening.

[GermanKiwi]

Well, okay - thanks! :)

[fastmover]

This would be a great feature.

[jdevalk]

Just submitted a patch to core to make this possible. Find it here: https://core.trac.wordpress.org/ticket/23692

[GermanKiwi]

Nice! Thanks for doing so!

[tacoverdo]

Current status of core-patch that would make this possible:
has-patch

[Rarst]

Core patch merged for 4.4. I wonder though how to implement this specific to a version of core. Do we just silently hide setting when running on earlier versions?

[Rarst]

The removal of header links feature has been dropped from the plugin.

See #3599