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

EZP-29246: Avoid double decoding of URL #1360

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Commits on May 19, 2018

  1. Avoid double decoding of URL

    eZSYS::requestURI is already decoding the URL. The eZURI class does it a 2nd time.
    
    Example of the problem. Given URL:
    http://<sitedomain>/(foo)/bar%2Bbar
    
    Double decoded version is is /(foo)/bar bar (that's a space in between)
    But the correct value is /(foo)/bar+bar
    
    You can test this by adding following into your pagelayout.tpl:
    {$view_parameters|dump()}
    
    Use following URL and look at the dump output:
    http://<sitedomain>/(foo)/bar%2Bbar
    
    The correct value is " /(foo)/bar+bar" - the pull request should produce the correct output.
    pkamps authored May 19, 2018
    Configuration menu
    Copy the full SHA
    6aaf17e View commit details
    Browse the repository at this point in the history

Commits on May 22, 2018

  1. Configuration menu
    Copy the full SHA
    d5aaad9 View commit details
    Browse the repository at this point in the history

Commits on May 25, 2018

  1. Configuration menu
    Copy the full SHA
    b367591 View commit details
    Browse the repository at this point in the history

Commits on May 28, 2018

  1. Tabs replaced with spaces

    pkamps authored May 28, 2018
    Configuration menu
    Copy the full SHA
    1e429e8 View commit details
    Browse the repository at this point in the history