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.

Sign up for GitHub

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

Different results with get_browser and browscap-php #900

Closed
asgrim opened this issue Feb 15, 2016 · 0 comments
Closed

Different results with get_browser and browscap-php #900

asgrim opened this issue Feb 15, 2016 · 0 comments
Assignees
@mimmi20
Labels
Update Agent
Milestone
6015

Comments

@asgrim
Copy link
Member

asgrim commented Feb 15, 2016

Originally reported by @wggdeveloper in browscap/browscap-php#136

While we have been working on our browscap implementation in LUA I run all the UA strings from UserAgentString.com through browscap-php master and the built-in get_browser in PHP5.6. The test gave us different results for 109 UAs out of 10770.

The browscap.ini was the full version of the 6011 release.

Please check the attached file for the test results. The lines marked with ! are the ones in question.
@asgrim asgrim added this to the 6013 milestone Feb 15, 2016
@mimmi20 mimmi20 self-assigned this Feb 16, 2016
@asgrim asgrim modified the milestones: 6013, 6014 Mar 15, 2016
@mimmi20 mimmi20 mentioned this issue Apr 6, 2016
Closed
@mimmi20 mimmi20 mentioned this issue Apr 14, 2016
Merged
@asgrim asgrim modified the milestones: 6014, 6015 Apr 18, 2016
@asgrim asgrim closed this as completed in 68dc100 Apr 19, 2016
@mimmi20 mimmi20 mentioned this issue Nov 20, 2016
Closed
@jaydiablo jaydiablo mentioned this issue Apr 17, 2017
Closed
@jaydiablo jaydiablo mentioned this issue Jan 26, 2018
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mimmi20 mimmi20

Labels
Update Agent
Projects
None yet
Milestone
6015
Development

No branches or pull requests
2 participants
@asgrim @mimmi20