Description:

As part of the recent Visual indication of incomplete periods on charts change (#18086), a check is performed on the currently selected period and date to determine if the period contains the current date. (eg. Period = 'week', Date = '2022-01-20'). This check is called by jqplot.js and executed by the periods service.

The A/B testing plugin for some internal reason needs to set the period to 'day' but the date to a range (eg. '2021-01-20,2021-01-27). This causes the containsToday check to fail as a valid date cannot be parsed from a date range.

This PR adds a simple check before calling attempting to parse the date period, if the date is a range then the period type will be forced to type 'range' for the purposes of the containsToday check only.

Ref L3-205

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