http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7417
--- Comment #31 from Jared Camins-Esakov <[email protected]> --- Jonathan, I think the thing to do here is move forward with the Class::Accessor-based code. While I am willing to rewrite this to use Moo, I think that it might be a bit premature to do that at this point. Better, I think, to make sure that there's consensus on using Moo as a compromise framework first, per comment 19. In the meantime, if you don't mind I would like to incorporate any further QA comments you might have so that this patch can move forward. Regarding what exactly goes in Koha::DataObject::Authority, I think we all agree that we can't just leave C4::AuthoritiesMarc as it is. I suggest that taking a minimalist approach to Koha::DataObject::Authority is the best we can do. My patch does that We can't possibly know what metadata we might have about authorities in the future, so any precise list of what can go in Koha::DataObject::Authority is guaranteed to be obsolete almost as soon as we have written it. -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list [email protected] http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
