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

API getLastVisitsDetails - sorting a range of data #4992

Closed
jasonbukowski opened this issue Apr 15, 2014 · 2 comments
Closed

API getLastVisitsDetails - sorting a range of data #4992

jasonbukowski opened this issue Apr 15, 2014 · 2 comments
Assignees
Labels
Bug For errors / faults / flaws / inconsistencies etc.
Milestone

Comments

@jasonbukowski
Copy link

filter_sort_column and filter_sort_order do not seem to produce the proper results.

Consider the following example:
http://demo.piwik.org/?module=API&method=Live.getLastVisitsDetails&idSite=7&period=range&date=2014-04-01,2014-04-11&filter_limit=10&filter_sort_column=idVisit&filter_sort_order=desc&format=xml&token_auth=anonymous

Ordering DESC (which appears to be the default) The first result in the result set is a visit late on April 11th. This looks correct.

But, when you change to ASC:
http://demo.piwik.org/?module=API&method=Live.getLastVisitsDetails&idSite=7&period=range&date=2014-04-01,2014-04-11&filter_limit=10&filter_sort_column=idVisit&filter_sort_order=asc&format=xml&token_auth=anonymous

The results to not change order. The first result is from late on April 11th. My expectation would be that upon sorting ASC, the first result would be the first visit on April 1st.

@tsteur
Copy link
Member

tsteur commented May 6, 2014

In 95d93ac: fixes #4992 added support for filter_sort_order in Live.getLastVisitsDetails API

@mattab
Copy link
Member

mattab commented May 7, 2014

In c0cfdf8: Refs #4992 The visitor log should not be sorted by time ASC, but by time DESC

@jasonbukowski jasonbukowski added this to the 2.2.1 - Piwik 2.2.1 milestone Jul 8, 2014
sabl0r pushed a commit to sabl0r/piwik that referenced this issue Sep 23, 2014
sabl0r pushed a commit to sabl0r/piwik that referenced this issue Sep 23, 2014
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug For errors / faults / flaws / inconsistencies etc.
Projects
None yet
Development

No branches or pull requests

3 participants