From 60a16dc08cae3d4dee77e2aae5e25b2e108c5cfb Mon Sep 17 00:00:00 2001 From: Nick Clemens Date: Fri, 3 Mar 2023 16:43:54 +0000 Subject: [PATCH] Bug 32613: (follow-up) Cache the tables This shouldn't be expected to change, except maybe after an upgrade, seems worth caching. To test: 1 - Apply patch 2 - Confirm the feature still works --- reports/guided_reports.pl | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/reports/guided_reports.pl b/reports/guided_reports.pl index fcd3cf2298..facd2a1d01 100755 --- a/reports/guided_reports.pl +++ b/reports/guided_reports.pl @@ -1036,6 +1036,9 @@ sub header_cell_loop { #get a list of available tables for auto-complete sub get_tables { my $result = {}; + my $cache = Koha::Caches->get_instance(); + my $tables = $cache->get_from_cache("Reports-SQL_tables-for-autocomplete"); + return $tables if $tables; my $tables = C4::Reports::Guided->get_all_tables(); for my $table (@{$tables}) { my $sql = "SHOW COLUMNS FROM $table"; @@ -1044,6 +1047,7 @@ sub get_tables { push @{$result->{$table}}, $row->{Field}; } } + $cache->set_in_cache("Reports-SQL_tables-for-autocomplete",$result); return $result; } -- 2.30.2