Merge pull request #2177 from akrabat/revert-1946

Revert #1946 - BugFix URL Encoding

Author: geggleto

Slim/Http/Uri.php

@@ -203,7 +203,6 @@ public static function createFromEnvironment(Environment $env)
         // parse_url() requires a full URL. As we don't extract the domain name or scheme,
         // we use a stand-in.
         $requestUri = parse_url('http://example.com' . $env->get('REQUEST_URI'), PHP_URL_PATH);
-        $requestUri = rawurldecode($requestUri);
 
         $basePath = '';
         $virtualPath = $requestUri;

tests/Http/UriTest.php

@@ -562,20 +562,6 @@ public function testCreateEnvironmentWithBasePath()
         $this->assertEquals('http://localhost/foo/bar', (string) $uri);
     }
 
-    public function testCreateEnvironmentWithBasePathContainingSpace()
-    {
-        $environment = Environment::mock([
-            'SCRIPT_NAME' => "/f'oo bar/index.php",
-            'REQUEST_URI' => "/f'oo%20bar/baz",
-        ]);
-        $uri = Uri::createFromEnvironment($environment);
-
-        $this->assertEquals("/f%27oo%20bar", $uri->getBasePath());
-        $this->assertEquals('baz', $uri->getPath());
-
-        $this->assertEquals('http://localhost/f%27oo%20bar/baz', (string) $uri);
-    }
-
     public function testGetBaseUrl()
     {
         $environment = Environment::mock([