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

Make sure to group the label in Referrer Websites report #13884

Open
wants to merge 2 commits into
base: 3.x-dev
from

Conversation

Projects
None yet
1 participant
@tsteur
Copy link
Member

tsteur commented Dec 19, 2018

I just debugged and can confirm behaviour described in #13882
The label includes a URL like http://demo.matomo.org/foo/bar and the label is then removed in the ColumnCallbackReplace filter meaning there could be several entries with the same label and a group by is needed.

Make sure to group the label in Referrer Websites report
I just debugged and can confirm behaviour described in #13882 
The label includes a URL like `http://demo.matomo.org/foo/bar` and the label is then removed in the `ColumnCallbackReplace` filter meaning there could be several entries with the same label and a group by is needed.
@tsteur

This comment has been minimized.

Copy link
Member

tsteur commented Dec 19, 2018

User confirmed it works, might cause the report to load a little less slow since the filters needed to be executed directly and the GroupBy is not the fastest either.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment