[Koha-bugs] [Bug 17712] Move availability calculation to the Koha namespace

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Wed May 10 16:31:55 CEST 2017


https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17712

--- Comment #52 from Lari Taskula <lari.taskula at jns.fi> ---
(In reply to Benjamin Rokseth from comment #42)
> Sorry to see you're so alone on this, as it sounds like a very good idea! I
> will sign it off, though I have some concerns, particularly on performance.
> 
> Passing koha objects around this way seems very javaish, and I don't really
> see the point, if all you really need is foreign keys. 
> 
> But then again, its circumstantial, and could be remedied at a later stage
> if needed. And it's a bit unfair on such a large codechange, which is
> actually relatively easy to read and follow.

Thanks for the support, new ideas, and the sign-off, Benjamin, and glad to hear
from you! I found this a very challenging project and surely there is room for
further performance improvements as well as other changes - under heavy
pressure some of the choices made may have not been optimal. Follow-ups are
welcome :)

BTW, this is also under busy testing by our librarians and in the following
days I'll have a desk-full of test reports and possibly follow-ups coming in
for this Bug :)

-- 
You are receiving this mail because:
You are watching all bug changes.


More information about the Koha-bugs mailing list