[Koha-bugs] [Bug 32476] Add caching for relatively expensive patron methods

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Tue Dec 20 23:33:16 CET 2022


https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32476

--- Comment #9 from David Cook <dcook at prosentient.com.au> ---
I still think the caching is a good idea, and I'm not saying my way is the only
way. 

Using Koha::Cache::Memory::Lite was just a suggestion/thought to explore. I
didn't mean that you had to change to using it. I think it has its pros and
cons. As you say, it's only cleared in a few places. This might not be the
right place to use it. I haven't thought it 100% through.

But I don't think I'd be alone in thinking there is too much copy/pasted code
in these patches. I don't like this terminology but it "smells wrong". 

Very possible that other people will disagree with me there though!

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


More information about the Koha-bugs mailing list