http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7417 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |chris@bigballofwax.co.nz --- Comment #14 from Chris Cormack <chris@bigballofwax.co.nz> --- (In reply to comment #13)
Hi Jared,
For discussion:
This patch is interesting. It introduces some new notions into Koha. For example, objects appear into Koha with an (not one of the least) Authority module \o/ But ... :)
Why do you choose an old method to do OOP in Perl (bless, SUPER) ? I think you may have heard about Moose or Moo, why don't use one of them directly from the beginning of the Koha::* rewriting?
This is using Class::Accessor http://search.cpan.org/~rjbs/perl-5.16.0/pod/perlootut.pod#Class::Accessor Which introduces no new dependencies, and importantly it's fast Class::Accessor does not employ an autoloader, thus it is much faster than you'd think. Its generated methods incur no special penalty over ones you'd write yourself. And for those who want it, it has a moose like interface http://search.cpan.org/~kasei/Class-Accessor-0.34/lib/Class/Accessor.pm#Moos... -- You are receiving this mail because: You are watching all bug changes.