RSS feeds broken in 2.01

[incursio]

As reported in several forum threads.

Sometimes, we get just comments, and no posts in the RSS thread. I vaguely remember seeing an option in the admin panel to choose posts vs comments but with 2.01 it seems to have disappeared.

Plus, RSS feeds do not package into XML properly when encountering an ampersand. They are breaking - also reported in forums.

[incursio]

Followup/summary:

This works fine:
​http://www.scottburkett.com/?feed=rss2

These show only comments, no posts:
​http://www.scottburkett.com/index.php/feed
​http://www.scottburkett.com/index.php/feed/rss
​http://www.scottburkett.com/index.php/feed/atom

Unfortunately, Technorati automatically tries to pull the last 3 examples, not the first, so new posts are not getting indexed there (which sucks).

Cheers.

[jdybka]

Hello

Same problem here.

Shows only comments (no posts):
​http://www.poetryhut.com/wordpress/index.php/feed/

Shows only comments (no posts):
​http://www.poetryhut.com/wordpress/index.php/feed/rss

Shows only comments (no posts):
​http://www.poetryhut.com/wordpress/index.php/feed/atom

Shows only comments (works correctly):
​http://www.poetryhut.com/wordpress/index.php/comments/feed/

This feed also works correctly:
​http://www.poetryhut.com/wordpress/index.php?feed=rss2

My permalinks structure is set to "Date and name based."

Thanks
Jilly

[joshkidd]

fix for comments bug

[joshkidd]

I had the same problem reported in the comments on this bug. The attached file is a change that I made to the code. After making the change, I had to create a new static page to flush the rewrite rules. Now, I get the behavior I expect:

Shows posts: ​http://larameekidd.com/index.php/feed
Shows comments: ​http://larameekidd.com/index.php/comments/feed

[jdybka]

Thank you. That fixed my problem. To flush the rewrite rules I changed my permalinks structure from "Date and name based" to another structure and then back again.

[jbwhong]

Joshkidd, to which file do I apply your modification?

[joshkidd]

As it says in the file, the change is to be applied to wp-includes/classes.php

[ryan]

(In [3527]) Don't walk rewrite structure when generating comment feed rewrite rules. This fixes a bug uncovered when we switched from unions to merges. Props joshkidd. fixes #2379

[jbwhong]

It's not my software project, but here are my two cents:

A bug this big (that breaks people's subscriptions) shouldn't wait for a distant 2.1 release.

Now that this ticket is closed you are getting others submitting duplicate tickets.

Perhaps a 2.0.2 release is in order.

[jbwhong]

Joshkidd, sorry I couldn't figure out what file to patch based on reading the file.

How can I use your patch?

[davidhouse]

jbwhong:

The file you need to patch is wp-includes/classes.php, as it says at the top of the patch. Simply search that file for the line with the '-' in front of it in the patch (it won't actually have a - in the actual classes.php file), then replace it with the line with the '+' in front of it in the patch (again, don't actually put the + in the file, but the rest of the line).

[ryan]

(In [3551]) Don't walk rewrite structure when generating comment feed rewrite rules. This fixes a bug uncovered when we switched from unions to merges. Props joshkidd. fixes #2379

[(none)]

Milestone 2.0.2 deleted