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.

Sign up for GitHub

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

Fix for #3318 #1377

Merged
merged 1 commit into from
Jul 2, 2013
Merged

Fix for #3318 #1377

merged 1 commit into from
Jul 2, 2013

Commits on Jun 30, 2013

  1. Fix for cakephp#3318 

    Fixes https://cakephp.lighthouseapp.com/projects/42648-cakephp/tickets/3318

It seems fixing this in the htaccess file(s) isn't going to work even though a url rewriting based solution was more clean. On the plus side this works for any web server.
If a url is called with "index.php" in it then the CakeRequest swallows this part and fixes the path. Any linked url from the requested page will have a clean url. Thus after following one of these urls this problem is gone anyway.

Some code docblock improvements to CakeRequestTest.php
Added test case for fix
Also now you can call just index.php even if you have url rewriting enabled
    @ravage84
    ravage84 committed Jun 30, 2013
    Configuration menu
    Copy the full SHA
    f930a50 View commit details
    Browse the repository at this point in the history