[Bug 18509] New: Minor performance improvement for overdue_notices.pl
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18509 Bug ID: 18509 Summary: Minor performance improvement for overdue_notices.pl Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Command-line Utilities Assignee: gmcharlt@gmail.com Reporter: martin.renvoize@ptfs-europe.com QA Contact: testopia@bugs.koha-community.org CC: robin@catalyst.net.nz Don't re-instantiate Koha::Calendar objects unnecessarily We were calling Koha::Calender->new on the same branchcode within a set of nested loops. This is just silly, only one object is required. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18509 Barton Chittenden <barton@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |barton@bywatersolutions.com --- Comment #1 from Barton Chittenden <barton@bywatersolutions.com> --- This is fixed in bug 17952 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18509 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |DUPLICATE Status|NEW |RESOLVED --- Comment #2 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Oh, nice spot, thanks for pointing that out.. means I won't have to extract it from our local branch :) Will mark resolved duplicate and take a look at the other bug. *** This bug has been marked as a duplicate of bug 17952 *** -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org