[HttpFoundation] Request::getPathInfo() inconsistencies

[carlescliment]

I'm having some problems trying to modify a request in order to dinamically accept aliases without having to explicitly define the routes. Probably I'm not doing it right, but the problem leads me to think that there is something inconsistent in the Request class.

if I do this:

$request = Request::createFromGlobals();
$request->server->set('REQUEST_URI', '/my-path');
$path = $request->getRequestUri();

Then $path is /my-path.

But if I do:

$request = Request::createFromGlobals();
$request->getRequestUri();
$request->server->set('REQUEST_URI', '/my-path');
$path = $request->getRequestUri();

Then $path is the original path and not /my-path.

I think it is a strange behaviour, it is error-prone and makes you take wrong assumptions.

[Tobion]

It's because the getter caches the result, so modiying the ServerBag afterwards has no effect. I added this observation to #6406

[carlescliment]

Thanks for the link, @Tobion. Sad to read that it won't be fixed until 3.*.

I finally managed to bypass the problem by extending the Request class and adding a method to clear the cached attributes.

[carlescliment]

Closing the issue since it was already noticed.

[Tobion]

The reason it probably has to wait for version 3 is that it is a highly used class we don't want to break BC and we need to fix some things from groud up.