Reviews by Ton Voon

cpanratings
 

RSS

XML-Atom-Syndication (0.901) **

Advice on RSS feeds is confusing me! Some say RSS 2.0 is not good and that Atom is the future. Makes sense, so I'd like to be able to generate some simple Atom feeds. Discovered XML::Atom::Syndication from www.intertwingly.net/wiki/pie/Rss20An....

Since XML::Atom is not updated to Atom 1.0, XML::Atom::Syndication appears to be the only choice. However, it lacks some straightforward documentation on how to start - a simple example on a basic feed would be be very productive.

A basic requirement is also to keep the list of entries to a fixed number, but had to delve into the code to work out how to do this ( @entries = @{$feed->elem->contents; @entries = splice @entries, 0, $max; $feed->elem->contents(\@entries) if you were interested ).

Also got a problem where $entry->author("person") is not correct as this returns "<author>person</author>" when the spec says it should be "<author><name>person</name></author>".

I think this module just needs a bit more cooking, but my knowledge is not strong enough to help move it forward. Sigh. Back to XML::RSS for me...

WWW-Mechanize-Timed (0.42) ***

Nice, simple plugin to get the individual timings for each state from LWP. I was a bit confused because the synopsis suggests that client_response_receive_time was a cumulative time for the all 4 states, but in fact is only the value returned for that single state.

Check the patch I submitted at rt.cpan.org/Ticket/Display.html?id=19099 which provides a convenience function to return the summation of the 4 states & a new method to return the time take to call WWW::Mechanize->get().

Net-Dev-MIBLoadOrder (1.1.0) ****

Great module! Has saved me a lot of time and stress in parsing MIB files myself.

However, the interface to call the loading is not particularly nice (3 parameter reply? $error appears to be set even if there are no errors), and some of the documentation doesn't seem to match (using -StandardMIBs as the key to mib_load does not seem to work, but dropping the hyphen does).

There are no tests to the module either.

I've added a patch to rt which includes the start of some tests, which I hope the author takes on. I've given this module 4 stars, even though all the individual ratings are 3 or below, just because it has saved me a lot of time and effort.

Unix-ConfigFile (0.06) ****

Good implementation of passwd/group changes. Saved me a lot of work in amending these files, so I could concentrate on our specific implementation.

Have added some patches in the bug tracker which are worth reviewing it helps you.

Did get a bit confused with the documentation for commit(). For instance, if you want to commit with backups, should have commit ( backup => ".bak" ). An example would clarify. Used to CGI module which specifies options as ( { backup => ".bak" } ) which uses anonymous hash refs instead.