Re: Pyrss2 plugin version 0.1 released

Gordon,

You are way ahead of me. I appreciate your comments,
but I've simply not come far enough in the process to
address the issues you are mentioning. Also, I suspect your
coding skills are greatly superior to mine :) But thanks
for taking the time to reply.

I wonder if we should try to coordinate our efforts and
figure out how best to support feeds in pyblosxom. I find
the current non-plugin native rss0.91 + rss3 solution a
bit clunky  (no offense to wari and the other developers btw).

I can't say that my plugin is any better, though ...

On 10/30/03 03:23:06, Wari Wahab wrote:

[snip]
> > I chose not to implement a new renderer. If this syndication
> > business should be done properly, I'd say one would be best off
> > implementing a general xml/syndication renderer and then
> > subclassing that further to implement rss0.9x, rss1.0,
> > rss2.0 and atom feeds. Hope the devs are listening to me
> > on this ...
> 
> Unfortunately, looking at XML rendering using XML modules looks quite
> unrefactorable. And I have no idea if making this allows the users to
> plainly change their templates without the need to know python, or
> (gasp!) XML processing.
> 
> > But then again, that might just be rubbish - what do I know?
> 
> I'm all for inputs on this one, but I believe that it could be a
> futile experiment. I'm too young in python to know XML generation the  
> 'easy' way.

Boy, that sucks. Well, I guess the template scheme is general enough
to do some nifty stuff anyway.

[snip]
> > If you want to check the output it produces by default, check:
> > http://u7.dk:8088/index.rss2
> 
> That don't work, I guess the link should really be
>
> http://u7.dk:8088/blog/index.rss2

DUH! - you are right, of course. Silly me.

> One problem though, a quick glance at it shows:
> 
> <title>Roadmap og Milestones for u7.dk</title>
> <description> ![CDATA[]] </description>
> <pubDate>Tue, 21 Oct 2003 20:37 GMT</pubDate>
> 
> A few problems, ![CDATA[]] is invalid, and shows it as is. Also the
> date
> format for pubdate is wrong I believe, it should be in the ISO8601
> format., that is - 2003-10-21T03:57:37Z instead of 'Tue, 21 Oct 2003
> 20:37 GMT'.. You can use $w3cdate in your templates to get the dates.

If I remove the ![CDATA[]] stuff, my mozilla browser will complain
when using the content-type text/xml. This happens because the   
truncated entrydata included in the description field for each entry is  
marked up with xhtml in my setup. That part should definitely be made  
smarter, either by stripping any and all html or only including the  
CDATA escaping if the description field is non-empty.

Wrt the date stuff, I can only support my choice of format by referring  
you to the feed validator I use for my rss2 feed. It is maintained by  
Mark Pilgrim and Sam Ruby. It is located at:

http://feedvalidator.org/

It states that the format of the feed should be RFC 822 compliant
(this I need to fix - mine isn't compliant at all), but it doesn't  
mention the ISO 8601 format. I'm curious to know where you've happened  
upon that requirement...

> > Please drop me a note if you use it - I'd like to hear about bugs,
> > suggestions, and anything related you can think of.
> 
> I hope I pointed you to the bugs I encountered, at least if the link
> is correct.

Thanks a bunch!

/Rune


-------------------------------------------------------
This SF.net email is sponsored by: SF.net Giveback Program.
Does SourceForge.net help you be more productive?  Does it
help you create better code?   SHARE THE LOVE, and help us help
YOU!  Click Here: http://sourceforge.net/donate/