Ticket #4704 (closed defect (bug): fixed)
Invalid RSS2 Comments Feed
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Priority: | high | Milestone: | 2.2.3 |
| Component: | Administration | Version: | 2.2.1 |
| Severity: | normal | Keywords: | has-patch |
| Cc: |
Description
Version 2.2.2. My comments RSS2 feed was invalid because the author tags did not enclose a valid email address.
I modified the feed-rss2-comments.php file to include xlmns:dc="http://purl.org/dc/elements/1.1/" in the head space and changed the author tags to dc:creator tags. This validated the comments feed.
I apologize for not submitting the patch as a diff but I'm quite a noob when it comes to this stuff.
Attachments
Change History
- Keywords has-patch added
- Priority changed from normal to high
- Version set to 2.2.1
- Milestone set to 2.3 (trunk)
Ahh.. It just occurred to me that it might not be desirable to publish email addresses of comment authors, considering it says that you won't on the default theme page. So... forget the above patch, perhaps the dc:creator would indeed be a better way to go.
comment:3
Viper007Bond — 5 years ago
Yeah, publishing e-mails = major eww and should be avoided at all costs.
I'll attach my patched feed-rss2-comments.php so if someone wants to make a diff out of it, go for it.
-
attachment
rss2comments-patch.zip
added
Patched feed-rss2-comments.php file for feed validation
Added dc:creator based patches for 2.2.2 and trunk. These change the author to dc:creator, which eliminates the validation issue.
- Status changed from new to closed
- Resolution set to fixed

Agreed, this causes all comment feeds to be invalid. The author element is supposed to conform to RFC822.
http://cyber.law.harvard.edu/rss/rss.html#ltauthorgtSubelementOfLtitemgt http://feedvalidator.org/docs/error/InvalidContact.html
I'm attaching a patch to fix this for 2.2.2. Trunk patch coming soon.