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
possibly avoid infinite looping if group_concat cuts off result #17244
Conversation
core/DataAccess/Model.php
Outdated
* @param array $idSites | ||
* @param bool $setMaxLen for tests only |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* @param array $idSites | |
* @param bool $setMaxLen for tests only | |
* @param bool $setGroupContentMaxLen for tests only |
@sgiehl applied pr feedback |
Looks good @diosmosis is the test https://travis-ci.com/github/matomo-org/matomo/jobs/485720146#L778 failing because of this PR? Feel free to merge otherwise |
@tsteur was looking at this yesterday. Something changed w/ the tracking, now only one visit gets tracked. Looking into why. |
build failure was random. I was able to reproduce it locally yesterday, but not today... |
Description:
As title, test forthcoming.
Review