From 4125cc80cafa68855d4ed3dc74faf79d0f32e162 Mon Sep 17 00:00:00 2001 From: Owen Leonard Date: Thu, 7 Oct 2021 14:19:56 +0000 Subject: [PATCH] Bug 28470: Typo: Are you sure you with to chart this report? This patch corrects the text of a warning on the report results page, from this: _("Fetching full chart data for reports with many rows can cause performance issues. Are you sure you with to chart this report?") ...to this: _("Fetching full chart data for reports with many rows can cause performance issues. Are you sure you want to chart this report?") To test, apply the patch and go to Reports -> Use saved. - Run a report which, unlimited, will return more than 1000 rows. - Click the "Fetch all data for chart" button. - You should be asked for confirmation. Check that the text is correct. Signed-off-by: David Nind Signed-off-by: Katrin Fischer Signed-off-by: Jonathan Druart Signed-off-by: Kyle M Hall (cherry picked from commit f58085e2629b4cb6c93f438b4c102212094ad98b) Signed-off-by: Fridolin Somers --- .../prog/en/modules/reports/guided_reports_start.tt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/reports/guided_reports_start.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/reports/guided_reports_start.tt index e24fa91311..4bdbba3040 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/reports/guided_reports_start.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/reports/guided_reports_start.tt @@ -1396,7 +1396,7 @@ $("body").on('click',".fetch_chart_data",function(){ if( [% unlimited_total || 0 | $raw %] > 1000 ){ - if( confirm("Fetching full chart data for reports with many rows can cause performance issues. Are you sure you with to chart this report?") ){ + if( confirm("Fetching full chart data for reports with many rows can cause performance issues. Are you sure you want to chart this report?") ){ return true; } else { return false; -- 2.39.5