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

fields config_os_version and config_device_type are not large enough #4203

Closed
mattab opened this issue Oct 8, 2013 · 2 comments
Closed

fields config_os_version and config_device_type are not large enough #4203

mattab opened this issue Oct 8, 2013 · 2 comments
Assignees
@mattab
Labels
Bug For errors / faults / flaws / inconsistencies etc.
Milestone
2.0

Comments

@mattab
Copy link
Member

mattab commented Oct 8, 2013

These two fields can be longer than 10 characters, which may fail to record properly when some special settings are set in mysql such as strict mode.

Reported in forums


Meanwhile, we have identified the cause: Sometime after the version 1.10.1 four new fields for the table log_visit have been introduced, one is called "config_os_version". In our intranet many user sessions are running under Citrix. The corresponding calculated value for this field is "Server 2003". With this value, the write operation goes wrong. The complete record is not written.
The call Piwik_Tracker getDatabase :: () -> query ($ sql, $ bind) in Visit.php is not coming back, the action is not counted!.
So 3,500 visitors per day with all actions are lost in our statistics.
@mattab
Copy link
Member Author

mattab commented Oct 8, 2013

In 16f8dd6: Fixes #4203 DevicesDetection 1.14 version schema update to account for longer strings ( fields config_os_version and config_device_type are not large enough )

Refs #4204 Plugin schema example with DevicesDetection

@mattab
Copy link
Member Author

mattab commented Oct 8, 2013

In ded987a: Fixes #4203 DevicesDetection 1.14 version schema update to account for longer strings ( fields config_os_version and config_device_type are not large enough )
Refs #4204 Plugin schema example with DevicesDetection

@mattab mattab added this to the 2.0 - Piwik 2.0 milestone Jul 8, 2014
@mattab mattab self-assigned this Jul 8, 2014
@mattab mattab mentioned this issue Oct 8, 2013
Closed
sabl0r pushed a commit to sabl0r/piwik that referenced this issue Sep 23, 2014
@mattab
Fixes matomo-org#4203 DevicesDetection 1.14 version schema update to …
16f8dd6 
…account for longer strings ( fields config_os_version and config_device_type are not large enough )

Refs matomo-org#4204 Plugin schema example with DevicesDetection
sabl0r pushed a commit to sabl0r/piwik that referenced this issue Sep 23, 2014
@mattab
Fixes matomo-org#4203 DevicesDetection 1.14 version schema update to …
ded987a 
…account for longer strings ( fields config_os_version and config_device_type are not large enough )

Refs matomo-org#4204 Plugin schema example with DevicesDetection
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 For errors / faults / flaws / inconsistencies etc.
Projects
None yet
Milestone
2.0
Development

No branches or pull requests
1 participant
@mattab