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

--- Comment #73 from Marcel de Rooy <m.de.r...@rijksmuseum.nl> ---
(In reply to Jonathan Druart from comment #72)
> I do not have time to test but I am pretty sure the way you cache
> $cached_indicators in Koha::Authority::ControlledIndicators is buggy.
> You should use Koha::Cache::Memory::Lite instead.

Hmm. Must say that I have been thinking about using Koha::Cache for it, but it
would not be a real benefit in time. Problem with Koha::Cache::Memory::Lite is
the same as handling it in the module; the L1 cache and also the
cached_indicators are kept per Plack worker for some time. So would still be
buggy.

But note that this is strictly hypothetical. The pref is not expected to change
on a regular basis. I decided not to add a clear statement at pref change event
but to add it in the new method. This effectively makes it a minimal issue.

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to