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

regression trunk: URL append character = which breaks it all #2119

Closed
mattab opened this issue Mar 2, 2011 · 2 comments
Closed

regression trunk: URL append character = which breaks it all #2119

mattab opened this issue Mar 2, 2011 · 2 comments
Labels
Bug For errors / faults / flaws / inconsistencies etc. Critical Indicates the severity of an issue is very critical and the issue has a very high priority.
Milestone

Comments

@mattab
Copy link
Member

mattab commented Mar 2, 2011

See on demo: http://demo.piwik.org/index.php?module=CoreHome&action=index&idSite=7&period=day&date=yesterday#module=Actions&action=getEntryPageUrls&idSite=7&period=day&date=yesterday

very critical, not sure how this is possible...

@mattab
Copy link
Member Author

mattab commented Mar 2, 2011

It is because on the forum, the URL look like:
http://forum.piwik.org/read.php?2,71865

Node there is no = character.

Piwik trying to fix the string appends a = character with empty value, but this wrong (adding a test case in tests should show this easily)

@robocoder
Copy link
Contributor

(In [4008]) fixes #2119

@mattab mattab added this to the Piwik 1.2.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
Labels
Bug For errors / faults / flaws / inconsistencies etc. Critical Indicates the severity of an issue is very critical and the issue has a very high priority.
Projects
None yet
Development

No branches or pull requests

2 participants