https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40141
--- Comment #7 from Pedro Amorim <[email protected]> --- (In reply to Paul Derscheid from comment #6) > I have some things to note: > 1. Maybe camel casing the functions would be more in line with the rest of > the module. > 2. I personally don't love the code duplication here but could maybe be > consolidated in a follow up. > > From a functionality standpoint this looks fine. I agree, but the eUsage part of ERM still needs to be moved into the bug 38201 framework. Until then I think these types of patches are to be expected. -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list [email protected] https://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/
