From 52441b7b1bdae7a20174d0feb00c500bb0fa99ab Mon Sep 17 00:00:00 2001 From: RD WebDesign Date: Mon, 15 Sep 2025 22:11:44 -0300 Subject: [PATCH] Remove unnecessary code The outer `if` (line 92) already guarantees only 2 possible values. If `isQueryTypeChart` is false, `isForwardDestinationChart` must be true. Signed-off-by: RD WebDesign --- scripts/js/charts.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/js/charts.js b/scripts/js/charts.js index ecb99b61..224abda7 100644 --- a/scripts/js/charts.js +++ b/scripts/js/charts.js @@ -96,7 +96,7 @@ globalThis.htmlLegendPlugin = { if (isQueryTypeChart) { link.href = `queries?type=${item.text}`; - } else if (isForwardDestinationChart) { + } else { // Encode the forward destination as it may contain an "#" character link.href = `queries?upstream=${encodeURIComponent(upstreams[item.text])}`; }