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

Fix segmentation using 'deviceType' segment #3989

Closed
anonymous-matomo-user opened this issue Jun 6, 2013 · 5 comments
Closed

Fix segmentation using 'deviceType' segment #3989

anonymous-matomo-user opened this issue Jun 6, 2013 · 5 comments
Assignees
@mattab
Labels
bug P: major
Milestone
Version 2.2

Comments

@anonymous-matomo-user
Copy link

If I create a segment on deviceType is 'tablet' I don't get tablets, but desktop & iPod Touch. Please see screenshots for results and segment settings

ACCEPTED VALUES: 'desktop', 'smartphone', 'tablet', 'feature phone', 'console', 'tv', 'car browser'
Keywords: DeviceType, Segment
@anonymous-matomo-user
Copy link
Author

Attachment:
DeviceType Segment.png

@anonymous-matomo-user
Copy link
Author

Attachment:
DeviceType Segment Result.png

@mattab
Copy link
Member

mattab commented Oct 8, 2013

In 4737bb6ab39879c8decd5c10ff38e390394e226d: Fixes #3989 The bug was that the sql filter callback was not set (my bad)

@jasonbukowski
Copy link

This still appears to be open on my 2.0-b12 instance as well as on demo.piwik.org.

The following API call works without segmentation ->
http://demo.piwik.org/?module=API&format=xml&method=VisitsSummary.get&period=day&date=2013-12-01,2013-12-31&idSite=3&format=xml&token_auth=anonymous

But if you define a segment, you get an empty result set
http://demo.piwik.org/?module=API&format=xml&method=VisitsSummary.get&period=day&date=2013-12-01,2013-12-31&segment=deviceType==desktop&idSite=3&format=xml&token_auth=anonymous

@mattab
Copy link
Member

mattab commented Dec 16, 2013

In 1e3af2e61bd34f1fb997359d386167f850f3f4fb: Refs #3989 thanks for the report, putting the fix back in
Reverting part of matomo-org/matomo@641e9ce

@anonymous-matomo-user anonymous-matomo-user added this to the Version 2.2 milestone Jul 8, 2014
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mattab mattab

Labels
bug P: major
Projects
None yet
Milestone
Version 2.2
Development

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