Hello,

(Quiet round these parts.)

In responding to a request from Jo Walsh yesterday to add some rdfs:seeAlso properties to the items in the london.openguides.org RSS feed, I took a look at the code of C::W::P::R::MW for the first time and realised it could do with some changing.

Namely, the code appears to have been based on part of http://www.usemod.com/cgi-bin/wiki.pl?WikiPatches/XmlRss which in itself is really not the best code it could be. In particular, it uses XML::RSS to do a lot of stuff that could be seamlessly replaced with simple string concatenation, and by relying on XML::RSS's pre-rolled methods to generate things like "dc:titleSomething Or Other</dc:title>" you lose the ability to write your own custom RDF.

So, I stripped out anything to do with XML::RSS and replaced it with a bunch of simple strings. I can't prove it, but my instincts tell me the resulting code actually runs a little faster. (Certainly no objects are generated and laden with attributes etcetera.) I've actually put it to run live, and the result can be seen at http://london.openguides.org/?action=rss .

A patch is attached to this message for my update. Kake, could you do the honours? I'm not sure if CGI::Wiki provides an RDF option, though - at least, I don't think it does - so you'll need to remove the four lines that match "$rdf_url". This is the complete patch just so you can see what I did. I should probably work out a way to get those lines into OpenGuides::RDF somehow - any suggestions, anyone?

I'm not sure how many CGI::Wiki modules (and consequently OpenGuides modules) use any of the XML::* modules, but hopefully this will cut down the dependencies a bit.

Cheers,

Earle.

hopefully this will cut down