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.

Sign up for GitHub

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

Jump to bottom

Getting an error when loading "Email eports" page when a segment no longer exists #16501

Closed
mattab opened this issue Oct 1, 2020 · 3 comments
Closed

Getting an error when loading "Email eports" page when a segment no longer exists #16501

mattab opened this issue Oct 1, 2020 · 3 comments
Assignees
@tsteur
Labels
Bug For errors / faults / flaws / inconsistencies etc.
Milestone
4.0.0-RC

Comments

@mattab
Copy link
Member

mattab commented Oct 1, 2020

Got this report by email:

Key "15" for array with keys ", 8, 9, 2, 3, 4, 13, 14, 23, 27, 1, 11, 24" does not exist. in /home/wp/disk/wordpress/matomo/plugins/ScheduledReports/templates/_listReports.twig 46 using PHP 7.4.8

Error when opening the Email Reports page.
@mattab mattab added the Bug For errors / faults / flaws / inconsistencies etc. label Oct 1, 2020
@mattab mattab added this to the 4.0.0-RC milestone Oct 1, 2020
@tsteur
Copy link
Member

tsteur commented Oct 1, 2020

You probably deleted a segment that was used in a report or so. As it's not a regression and bit more work to maybe cleanup reports that contain segments etc maybe can move this out of the RC?

@tsteur
Copy link
Member

tsteur commented Oct 1, 2020

Or as part of this issue we could simply avoid the twig error and then create a new issue to later find a solution for when segments get deleted and it affects other places (which is likely not straight forward)

@tsteur tsteur self-assigned this Oct 1, 2020
@sgiehl sgiehl mentioned this issue Oct 1, 2020
Merged
@tsteur tsteur closed this as completed Oct 1, 2020
@tsteur
Copy link
Member

tsteur commented Oct 1, 2020

won't create a separate issue for now for cleaning up reports that are referenced to segments under circumstances

@mattab mattab changed the title Getting an error when loading "Email eports" page Getting an error when loading "Email eports" page when a segment no longer exists Nov 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tsteur tsteur

Labels
Bug For errors / faults / flaws / inconsistencies etc.
Projects
None yet
Milestone
4.0.0-RC
Development

No branches or pull requests
2 participants
@tsteur @mattab