From d26fb595814d87cb63a286f076c4e805bdc1caee Mon Sep 17 00:00:00 2001 From: Arild Matsson Date: Mon, 15 Apr 2024 14:45:42 +0200 Subject: [PATCH] Fix trend diagram with multiple series, fix #358 --- CHANGELOG.md | 1 + app/config/statistics_config.js | 2 +- app/scripts/trend_diagram/trend_util.js | 2 +- 3 files changed, 3 insertions(+), 2 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 0500dc4c3..d7aa43e3f 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -54,6 +54,7 @@ - Correct order of numbers in trend diagram table (first relative, then absolute) - Update number formatting in statistics table when switching language - Disable Trend diagram and Map buttons while waiting for statistics search to finish [#346](https://github.com/spraakbanken/korp-frontend/issues/346) +- Error when clicking trend diagram with multiple series [#358](https://github.com/spraakbanken/korp-frontend/issues/358) - Strip HTML from total hits in annotated KWIC dowload - Fix dynamic translation for tabs etc - Modes in "More" menu sorted locale-awarely diff --git a/app/config/statistics_config.js b/app/config/statistics_config.js index e1e8485e1..f010db661 100644 --- a/app/config/statistics_config.js +++ b/app/config/statistics_config.js @@ -24,7 +24,7 @@ export function getCqp(hitValues, ignoreCase) { } tokens.push("[" + andExpr.join(" & ") + "]") } - return ` ${tokens.join(" ")} []* ` + return ` ${tokens.join(" ")} []{0,} ` } function reduceCqp(type, tokens, ignoreCase) { diff --git a/app/scripts/trend_diagram/trend_util.js b/app/scripts/trend_diagram/trend_util.js index b2b8bffd7..b542af7f5 100644 --- a/app/scripts/trend_diagram/trend_util.js +++ b/app/scripts/trend_diagram/trend_util.js @@ -27,7 +27,7 @@ export function getTimeCQP(time, zoom, coarseGranularity) { timecqp = `[(${startsSameDate} & ${timeInside}) | (${startsBefore} & ${endsAfter})]` } - timecqp = ` ${timecqp} []* ` + timecqp = ` ${timecqp} []{0,} ` return timecqp }