Fixes Goal row evolution with double quotes in the goal name resulting in an error

[utrautmann]

If a goal has double quotes in its name, then Matomo cannot plot the conversion lines in the Row evolution graph. A javascript error occurs on the console.

Current Behavior to reproduce

1. Choose a goal with conversions
2. Rename the goal and insert double quote in the name. F.e. Newsletter "Sport"
3. Select the goal and go to "Goals by Referrers"
4. Select any row with a conversion
5. Open row evolution (see screenshot 1)
6. Select line with conversion (see screenshot 2)

In step 6 a javascript error will occur and no line will be drawn.

screenshot 1

screenshot 2

• Matomo Version: 4.10.1
• Browser: Firefox

[bx80]

@utrautmann Thanks for reporting this, the clear steps and screenshots are much appreciated 👍

I've managed to recreate the issue. It looks like a string escaping bug when passing axis names to jqPlot, when fixing this we should add UI tests for goals containing double and single quotes in their names.

[sgiehl]

That one is another issue with automatically sanitizing input values (See #11786 or #4231). The goals name is stored sanitized. E.g. a " is converted to ". The javascript so holds values that are sanitized. When clicking on a metric in the row evolution, the metric is read using .text() on the clicked element. But as it's a html element, reading text decodes " back to ", causing the values not to match anymore. I'll prepare a PR that workarounds that issue for that special case, but I'm pretty sure something similar might happen again til we get rid of the input sanitizing.