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

Opera 15+ detection isn't working #4816

Closed
rluik opened this issue Mar 6, 2014 · 5 comments
Closed

Opera 15+ detection isn't working #4816

rluik opened this issue Mar 6, 2014 · 5 comments

Comments

@rluik
Copy link

rluik commented Mar 6, 2014

Even after #3964 (Piwik 2.1.0) Opera 20 detection isn't working.

UA-String example:
Mozilla/5.0 (Windows NT 6.3; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/33.0.1750.117 Safari/537.36 OPR/20.0.1387.64

Piwik says it's Chrome 33 instead of Opera.

The same is on mobile (Android), Opera 19 is detected as Chrome 32. An UA-String example:

Mozilla/5.0 (Linux; Android 4.4.2; Evo 3D GSM Build/KOT49H) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/32.0.1700.72 Mobile Safari/537.36 OPR/19.0.1340.69721

@sgiehl
Copy link
Member

sgiehl commented Mar 6, 2014

Those UAs are detected as Opera Next and Opera Mobile.

Have you enabled the DevicesDetection Plugin? Otherwise the old useragentparser is used which might have a wrong detection.

DevicesDetection plugin is not enabled by default. We will change that behaviour in Piwik 2.2. As the old useragentparser will be removed then.

@rluik
Copy link
Author

rluik commented Mar 6, 2014

Oh ok thanks, it detects Opera now, but there are two problems...

First is this is not Opera Next (beta stream), this is the regular Opera UA-String.
The other is the icon for Opera Mobile is blank.

Should I open other reports?

@sgiehl
Copy link
Member

sgiehl commented Mar 6, 2014

In 23a76f08465487395db3f9d5a485bea3b2122234: refs #4816 fixed detections of Opera and Opera Next

@sgiehl
Copy link
Member

sgiehl commented Mar 6, 2014

Detection for Opera/Opera Next should be fixed now. Icon for Opera Mobile should be the same as for Opera.

@rluik
Copy link
Author

rluik commented Mar 7, 2014

Thank you!

@rluik rluik added this to the Version 2.2 milestone Jul 8, 2014
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants