[Koha-bugs] [Bug 16441] C4::Letters::getletter is not plack safe
bugzilla-daemon at bugs.koha-community.org
bugzilla-daemon at bugs.koha-community.org
Fri Jun 10 11:14:30 CEST 2016
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16441
--- Comment #11 from Marcel de Rooy <m.de.rooy at rijksmuseum.nl> ---
Created attachment 52217
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=52217&action=edit
Bug 16441: Do not use a package variable to cache C4::Letters::getletter
C4::Letters::getletter use a package variable (%letter) to cache letter
returned by the subroutine.
I have not found any direct issues caused by that but it is safer to
remove it.
It won't be a big deal to hit the DBMS to get a valid letter when
needed.
No test plan here, just confirm that the changes make sense.
Signed-off-by: Srdjan <srdjan at catalyst.net.nz>
Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Confirm that performance loss is just a millisecond or so per
subsequent call of getletter.
--
You are receiving this mail because:
You are watching all bug changes.
More information about the Koha-bugs
mailing list