Thanks to stvn and ciderpunx for the corections. And tsee I've added readmore around the code itself but left the POD as is.

stvn suggests adding a params method. I agree this is a good idea and will do so in some form before uploading to CPAN. Using Class::Params itself would be overkill but perhaps I can get some ideas from it.