Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix date range handling to prevent evolution graph display being confused when changing the period #19393

Closed
Chardonneaur opened this issue Jun 23, 2022 · 1 comment · Fixed by #19395
Assignees
Labels
Bug For errors / faults / flaws / inconsistencies etc.
Milestone

Comments

@Chardonneaur
Copy link

When we change the period of a graph with the "green icon calendar" for example changing from Day to Week, and then when we are coming back to a Day period, the graph is getting confused and not coming back to its original state.
For example, if you choose June the 22th, then you click on the green icon calendar and select week and then select back the day period then instead of coming back to June the 22th it will be June the 26th... rather than June the 22th.
I checked this out on https://demo.matomo.cloud.

@Chardonneaur Chardonneaur added the Potential Bug Something that might be a bug, but needs validation and confirmation it can be reproduced. label Jun 23, 2022
@sgiehl
Copy link
Member

sgiehl commented Jun 23, 2022

Hi @Chardonneaur
Thanks for the report. I was able to reproduce that. It seems for some reason the date parameter in the request seems to be changed to the last day of the period in that case.
I'll try to set up a quick fix for that.

@sgiehl sgiehl added Bug For errors / faults / flaws / inconsistencies etc. and removed Potential Bug Something that might be a bug, but needs validation and confirmation it can be reproduced. labels Jun 23, 2022
@sgiehl sgiehl self-assigned this Jun 23, 2022
@sgiehl sgiehl added this to the 4.12.0 milestone Jun 23, 2022
@justinvelluppillai justinvelluppillai changed the title Evolution graph display is being confused when changing the period Fix date range handling to prevent evolution graph display being confused when changing the period Sep 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug For errors / faults / flaws / inconsistencies etc.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants