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

Jump to bottom

Commas in Segment values break Visitors Overview Reports #4024

Closed
anonymous-matomo-user opened this issue Jun 27, 2013 · 4 comments
Closed

Commas in Segment values break Visitors Overview Reports #4024

anonymous-matomo-user opened this issue Jun 27, 2013 · 4 comments
Labels
Bug For errors / faults / flaws / inconsistencies etc. worksforme The issue cannot be reproduced and things work as intended.
Milestone
Backlog (Help wan...

Comments

@anonymous-matomo-user
Copy link

Hi,

I'm using version 1.12, PHP 5.4.7.
Actually, I'm using XAMPP 1.8.1.

When defining a segment, if I add a comma to a Custom Variable value and then try to see a report from the "Visitors > Overview", an error shows up.

How to reproduce:

  1. Create a segment (e.g. "My Segment").
    1.a) Select the "Custom Variable value 1 (scope visit)"
    1.b) Filter operator: Is
    1.c) Value: "Doe, John" (in my case, cust var with ID 1 has names).
  2. Select the new segment.
  3. Go to "Visitors > Overview".
  4. Click on the first report (e.g. "X visits, X unique visitors").

You should see an error message:
The segment ' John' is not valid.

I found that the problem is the comma inside the value.
Don't know if there are other characters that cause this error.

Regards,

Marcos
@anonymous-matomo-user
Copy link
Author

Did you try to in value 1.c to use regex to filter?

It could just be that which will allow you to select John, Doe

Perhaps a "Doe,*John"

@anonymous-matomo-user
Copy link
Author

Hi lesjokolat,

Yes, I tried that, but it does not filter the required user name.
Besides the use of regexp in the custom variable value, I think nothing should break because a custom variable value has a comma inside. That is why I reported the issue as a bug.

Regards,

Marcos

@mattab
Copy link
Member

mattab commented Jun 29, 2013

Thanks for the report!

@anonymous-matomo-user anonymous-matomo-user added this to the 2.5.0 - Piwik 2.5.0 milestone Jul 8, 2014
@mattab
Copy link
Member

mattab commented Dec 1, 2014

works in modern Piwik

@mattab mattab closed this as completed Dec 1, 2014
@mattab mattab added the worksforme The issue cannot be reproduced and things work as intended. label Dec 1, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug For errors / faults / flaws / inconsistencies etc. worksforme The issue cannot be reproduced and things work as intended.
Projects
None yet
Milestone
Backlog (Help wanted)
Development

No branches or pull requests
2 participants
@mattab @anonymous-matomo-user