Need XHTML Alternative

shelleyp - November 8, 2008 - 16:47
Project:Atom
Version:6.x-1.0
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:closed
Description

Atom feeds support HTML and XHTML content types, but the Drupal Atom module only supports HTML content type. I created a patch that adds an XHTML option to the Atom maintenance page, and that applies a new variable supporting content type when generating the feed.

Attached and for review.

AttachmentSize
atomxhtml.patch3.03 KB

#1

deekayen - December 27, 2008 - 07:14
Status:needs review» needs work

Please just make the patch upgrade HTML to XHTML (drop the old HTML) and run coder review on your change. I see some coding standards deviations in your patch.

#2

deekayen - December 27, 2008 - 07:52
Status:needs work» fixed

nevermind, I just copied and pasted the new tags in by hand to 5, 6, and head (currently 7)

#3

System Message - January 10, 2009 - 08:00
Status:fixed» closed

Automatically closed -- issue fixed for two weeks with no activity.

 
 

Drupal is a registered trademark of Dries Buytaert.