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

--- Comment #7 from Matt Blenkinsop <[email protected]> ---
Created attachment 204009
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=204009&action=edit
Bug 43126: Improve caching of circulation rules to reduce requests when
checking hold availability

get_effective_rule_value only returned a cached value when the value was true,
so a rule that is unset, zero or empty missed the cache and queried again on
every call. Those are common, and a caller checking one rule per item on a
record paid a query for each. The value is now wrapped so a false one caches
too.

That exposed a second gap: the cache was cleared in set_rule alone, so a write
by any other route left a value behind that no longer existed. It was harmless
only because false values never cached. Koha::CirculationRule->store and
->delete now clear it themselves, and set_rule calls the same method.

Assisted-by: Sonnet 5 (Anthropic)

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list -- [email protected]
To unsubscribe send an email to [email protected]
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to