On Fri, Jan 20, 2012 at 9:55 AM, Galen Charlton <gmc@esilibrary.com> wrote:
Hi,
On Jan 20, 2012, at 9:02 AM, Chris Nighswonger wrote:
> That's pretty slick. I'd have no objection to adding it in. I don't think we should slash Data::Dumper simply because there are many places it is currently used. We could work toward eliminating it, though.
Data::Dumper isn't just a debug tool, though -- more broadly, it's a way of serializing Perl data structures. Not the only way, of course -- we've already got YAML and JSON modules as dependencies that can serve the same purpose, but since Data::Dumper is also a core module, I don't think making a point of eliminating its use would gain us much. Of course, it doesn't hurt to continually evaluate usage of Koha's many Perl modules.
And there is a nagging half-memory which tells me we use it for things other than debugging already...
Kind Regards,
Chris