Bug 18509

Summary: Minor performance improvement for overdue_notices.pl
Product: Koha Reporter: Martin Renvoize (ashimema) <martin.renvoize>
Component: Command-line UtilitiesAssignee: Galen Charlton <gmcharlt>
Status: RESOLVED DUPLICATE QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: barton, robin
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:

Description Martin Renvoize (ashimema) 2017-04-28 14:56:18 UTC
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.
Comment 1 Barton Chittenden 2017-05-01 16:39:58 UTC
This is fixed in bug 17952
Comment 2 Martin Renvoize (ashimema) 2017-05-01 18:31:25 UTC
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 ***