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

Bing search engine keyword parameter change #1695

Closed
robocoder opened this issue Sep 11, 2010 · 5 comments
Closed

Bing search engine keyword parameter change #1695

robocoder opened this issue Sep 11, 2010 · 5 comments
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.
Milestone
1.1

Comments

@robocoder
Copy link
Contributor

Bing changed the param from 'q' to 'Q'.
@sgiehl
Copy link
Member

sgiehl commented Sep 11, 2010

Well, it's still working with 'q' when I use it. But 'Q' seems to work aswell.

@robocoder
Copy link
Contributor Author

m.bing.com uses Q. I'm pretty sure piwik treats url parameters as case sensitive. I'll add a unit test.

@sgiehl
Copy link
Member

sgiehl commented Sep 11, 2010

Ah. Ok. But m.bing.com is not in list at all. There is only www.

@sgiehl
Copy link
Member

sgiehl commented Sep 11, 2010

Oh. I guess you missunderstood me. I meant Bing is able to handle 'q' and 'Q'. www. uses 'q' and m. uses 'Q' by default. Piwik is treating url parameters case sensitive afaik.

@robocoder
Copy link
Contributor Author

(In [3125]) fixes #1695 - add missing m.bing.com

@robocoder robocoder added this to the Piwik 1.1 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
Assignees
No one assigned
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
Milestone
1.1
Development

No branches or pull requests
2 participants
@robocoder @sgiehl