[Koha-bugs] [Bug 14769] Authorities merge: Set correct indicators in biblio field

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Wed Apr 11 13:26:53 CEST 2018


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

--- Comment #73 from Marcel de Rooy <m.de.rooy at 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.


More information about the Koha-bugs mailing list