Bug #23649
closed
felogin::getPreserveGetVars() decode encoded values
100%
Description
if a urlencoded var is provided the method just forward the encode value. By doing this it's "adding" more parameter to the url with the result of a malfunction.
Check
&test=urlencode(7&test1=2&test2=3)
will create &test=7&test1=2&test2=3
which is wrong.
Patch attached.
(issue imported from #M15861)
Files
Updated by Gerrit Code Review over 12 years ago
- Status changed from New to Under Review
Patch set 3 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7638
Updated by Gerrit Code Review over 12 years ago
Patch set 4 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7638
Updated by Gerrit Code Review over 12 years ago
Patch set 5 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7638
Updated by Gerrit Code Review over 12 years ago
Patch set 6 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7638
Updated by Gerrit Code Review over 12 years ago
Patch set 7 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7638
Updated by Gerrit Code Review about 12 years ago
Patch set 8 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7638
Updated by Gerrit Code Review about 12 years ago
Patch set 9 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7638
Updated by Gerrit Code Review about 12 years ago
Patch set 10 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7638
Updated by Gerrit Code Review about 12 years ago
Patch set 11 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7638
Updated by Gerrit Code Review about 12 years ago
Patch set 12 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7638
Updated by Gerrit Code Review about 12 years ago
Patch set 13 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7638
Updated by Gerrit Code Review about 12 years ago
Patch set 14 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7638
Updated by Gerrit Code Review about 12 years ago
Patch set 15 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7638
Updated by Gerrit Code Review about 12 years ago
Patch set 16 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7638
Updated by Gerrit Code Review about 12 years ago
Patch set 17 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7638
Updated by Jigal van Hemert about 12 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset a0f4d13c9f53431a62845bbad31ff11bc5d5c2ef.
Updated by Gerrit Code Review about 12 years ago
- Status changed from Resolved to Under Review
Patch set 1 for branch TYPO3_4-7 has been pushed to the review server.
It is available at http://review.typo3.org/16452
Updated by Gerrit Code Review almost 12 years ago
Patch set 2 for branch TYPO3_4-7 has been pushed to the review server.
It is available at https://review.typo3.org/16452
Updated by Jigal van Hemert almost 12 years ago
- Status changed from Under Review to Resolved
Applied in changeset 9eca09b08b257fbdcc4053d88d1df3feeb9afd66.
Updated by