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:title>Something 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
--
Earle Martin
http://downlode.org/http://purl.oclc.org/net/earlemartin/
