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

When visitors are recorded in a random chronological order, Piwik should assign the actions correctly #3927

Closed
mattab opened this issue May 7, 2013 · 1 comment
Assignees
Labels
Bug For errors / faults / flaws / inconsistencies etc.
Milestone

Comments

@mattab
Copy link
Member

mattab commented May 7, 2013

This is a feature request from Takeshi, piwikjapan.org webmaster. It had been requested by other users before: allow Piwik to import recent visitors, then old visits from the same visit and record them in the past.

When importing log files in random order, the visits are recorded for the last visitor instead of being recorded each day as we would expect. This happens because the "get visit" query does not restrict the "max date". So when recording an old visitor, if "new visits" from this same visitor have already been recorded, the algorithm would select this "new" visit row instead of creating a visit for the past date.

I will fix this by adding a test case and putting in simple solution.

This is follow up of #2584

@mattab
Copy link
Member Author

mattab commented May 7, 2013

In 56554b5: Fixes #3927
Adding future look ahead window to the SQL trying to detect the current visitor

  • fixing small bug discovered (in the tests) as a result of this fix

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

1 participant