[Koha-bugs] [Bug 7567] News by Library: refactor, enhance, and fix

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Mon Dec 16 04:38:08 CET 2013


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

--- Comment #38 from M. Tompsett <mtompset at hotmail.com> ---
Created attachment 23559
  -->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=23559&action=edit
Bug 7567 - Removed finish calls

"When all the data has been fetched from a SELECT statement,
the driver will automatically call finish for you. So you should
not call it explicitly except when you know that you've not
fetched all the data from a statement handle and the handle
won't be destroyed soon."
(http://search.cpan.org/~timb/DBI-1.627/DBI.pm#finish)

All the $sth variables were scoped within the functions,
and would be destroyed immediately. Additionally, there was
one after a SELECT, for only a single idnew, and so it was
not necessary.

TEST PLAN
---------
1) prove -v t/db_dependent/NewsChannels.t
2) apply patch
3) prove -v t/db_dependent/NewsChannels.t

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


More information about the Koha-bugs mailing list