Regression: columns query parameter is never urldecoded in PHP so it cannot be encoded in JS. #8905
+8
−2
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
When the
&columns=
query parameter is supplied in the URL fragment thehistory
angular service will encode the parameter. In PHP, however, Piwik will never decode the parameter, so values like:nb_visits%2Cnb_actions
will be interpreted incorrectly.
We can't change
Common::getRequestVar
to url decode for 2.15 in order to maintain BC, so this change makes sure columns is not url encoded when being detected in the history service.