http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13719

--- Comment #13 from Martin Renvoize <martin.renvo...@ptfs-europe.com> ---
OK, So, you're always calling ->all on he dbic resultset, even when you may
only actually want the ->first result :(. Bad mojo, this is a big memory leak
in my opinion.

Yes calling ->all is a good idea when you know you want the full set (or you
know the full set will always be sufficiently small that your not worried about
the memory overhead, but calling it for all requests is just asking for trouble
in my opinion.

We should be encouraging thought behind these calls and good practice. Not
coding around other coders deficiencies.. that's what QA is for, spotting such
performance mistakes during QA time.

All in all, this gets a definitive thumbs down from me

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
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/

Reply via email to