Since we merged #20090 when comparing dates we no longer need to force day periods when comparing dates since we're ensuring the correct period is being used in the Sparklines visualisation.

Deployed that PR on production and noticed the URL looks like:

Even though both periods given are week it falsely chose day for the initial period. Which not only made it compare wrong data but also got very slow (since it had to load 365 days which is slower than 52 weeks).

This PR fixes it and now looks like

Description:

Please include a description of this change and which issue it fixes. If no issue exists yet please include context and what problem it solves.

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