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

window_look_back_for_visitor break js visit length #3957

Closed
anonymous-matomo-user opened this issue May 28, 2013 · 4 comments
Closed

window_look_back_for_visitor break js visit length #3957

anonymous-matomo-user opened this issue May 28, 2013 · 4 comments
Assignees
Labels
Bug For errors / faults / flaws / inconsistencies etc. Major Indicates the severity or impact or benefit of an issue is much higher than normal but not critical.

Comments

@anonymous-matomo-user
Copy link

hi there,

I have combined instance of piwik with both js tracked sites and log imported sites.

I have set window_look_back_for_visitor = 2678400 so I can track log imported returning visitors, no problem with this it works fine and doesn't cause any performance problem even for such long period.

However it breaks js tracked sites see attached screenshot

  • first record on the screenshot is with entirely disabled window_look_back_for_visitor and is perfectly correct
  • other lines are with window_look_back_for_visitor enabled and as you can see wrong

using 1.11.1 - is this something to be fixed in 1.12? I cannot run this on production right now.

@anonymous-matomo-user
Copy link
Author

Attachment:
visitor_log.png

@mattab
Copy link
Member

mattab commented May 29, 2013

In fa36f40: Adding failing test case showing the bug (visit over several days, creates too many visits) refs #3957

@mattab
Copy link
Member

mattab commented May 29, 2013

In 9e16d35: This fixture says "14 visits" which is the bug: it should say only "10" visits Refs #3957

@mattab
Copy link
Member

mattab commented May 29, 2013

In 4f2628f: Fixes #3957 thanks for the report and nice find! The bug was the missing ORDER clause.

Also fixing the "look ahead code" to lookahead at most the visit standard length

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. Major Indicates the severity or impact or benefit of an issue is much higher than normal but not critical.
Projects
None yet
Development

No branches or pull requests

2 participants