Bug 41440 - Add caching to language_get_description and get_rfc4646_from_iso639
Summary: Add caching to language_get_description and get_rfc4646_from_iso639
Status: Signed Off
Alias: None
Product: Koha
Classification: Unclassified
Component: Architecture, internals, and plumbing (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement
Assignee: Nick Clemens (kidclamp)
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2025-12-12 14:29 UTC by Nick Clemens (kidclamp)
Modified: 2025-12-17 15:47 UTC (History)
2 users (show)

See Also:
GIT URL:
Initiative type: ---
Sponsorship status: ---
Comma delimited list of Sponsors:
Crowdfunding goal: 0
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:


Attachments
Bug 41440: Add caching to language_get_description and get_rfc4646_from_iso639 (2.02 KB, patch)
2025-12-12 14:31 UTC, Nick Clemens (kidclamp)
Details | Diff | Splinter Review
Bug 41440: Add caching to language_get_description and get_rfc4646_from_iso639 (2.06 KB, patch)
2025-12-13 07:23 UTC, David Nind
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Nick Clemens (kidclamp) 2025-12-12 14:29:27 UTC
These are called a number of times in search results - caching saves us some processing
Comment 1 Nick Clemens (kidclamp) 2025-12-12 14:31:40 UTC
Created attachment 190466 [details] [review]
Bug 41440: Add caching to language_get_description and get_rfc4646_from_iso639

This should have no visible effect, just internal caching

To test:
1 - Do a search, check the language facets
2 - Apply patch, restart all
3 - Do a search, check the language facets
4 - No change
5 - Sign off
Comment 2 David Nind 2025-12-13 07:23:26 UTC
Created attachment 190477 [details] [review]
Bug 41440: Add caching to language_get_description and get_rfc4646_from_iso639

This should have no visible effect, just internal caching

To test:
1 - Do a search, check the language facets
2 - Apply patch, restart all
3 - Do a search, check the language facets
4 - No change
5 - Sign off

Signed-off-by: David Nind <david@davidnind.com>
Comment 3 David Nind 2025-12-13 07:25:22 UTC
Testing notes (using KTD):

1. Language facet only shows up in the staff interface and OPAC when using Elasticsearch.