Use the idGoal from Sparkline URL, when updating the DataTable

[fealXX]

Description:

Use the idGoal from Sparkline URL to update the datatable, instead of always using page default / first idGoal, so that the Ecommerce Overview is able to load the abandoned cart metrics, instead of always falling back to idGoal=ecommerceOrder

adresses #22713 (edit: also #21957 / #22014 )

Review

• Functional review done
• Potential edge cases thought about (behavior of the code with strange input, with strange internal state or possible interactions with other Matomo subsystems)
• Usability review done (is anything maybe unclear or think about anything that would cause people to reach out to support)
• Security review done
• Wording review done
• Code review done
• Tests were added if useful/possible
• Reviewed for breaking changes
• Developer changelog updated if needed
• Documentation added if needed
• Existing documentation updated if needed

[github-actions]

If you don't want this PR to be closed automatically in 28 days then you need to assign the label 'Do not close'.

[fealXX]

Anything I can do to help facilitate merging this? :)

[sgiehl]

Hey @fealXX. I've just had a look at the fix. It works in terms of loading the overview chart with the correct metric.
However when switching to "Abandoned Carts" metric, the metric selector within the evolution chart then only allows selecting such metrics and in addition also some metrics that shouldn't be displayed at all.
The optimal way for sure would be to allow the chart to show all metrics together. But that might be a bit tricky due to the differing goal id.

[fealXX]

@sgiehl while I do agree on the optimal way, it seems (to me) a rework of this logic is out of scope for the issue that is being adressed. I will have a look at the metric selector and why it behaves that way, thank you for spotting this. :)