It matches the route "http://localhost/hello" without the extra "/" at the end
please use symfony's generate url function to get the correct url

AFAIK this case is handled in one of the routing component PRs

@mahmouds You are correct, it does match without the trailing slash, but that doesn't match the pattern as it's defined. The pattern should be /hello/{optionalParameter}, not /hello{/optionalParameter}. If I use the generateUrl() function it does return /hello as the relative url, but that's still wrong.

@mvrhov Can you please clarify PR?

@chrismcmacken Here is the PR #5410

@mvrhov @mahmouds That PR doesn't fix this issue as far as I can tell.

You are correct, it does match without the trailing slash, but that doesn't match the pattern as it's defined. The pattern should be /hello/{optionalParameter}, not /hello{optionalParameter}. If I use the generateUrl() function it does return /hello as the relative url, but that's still wrong.

@chrismcmacken /hello/{optionalParameter} matches /hello and /hello/.... Not /hello/ because the / is considered a seperator that is left out without the optional parameter. Your comparison with /hello{optionalParameter} is not valid here. The generated path /hello is correct and it's not a "relative url" either. Its an abolute path.

Furthermore, #5410 has nothing to do with this use case here. Your use case can be achieved by with #5424.
You can close this issue as this is not a bug but a feature request that is already covered by the linked issue.

@Tobion Yeah, this was a misunderstanding on my part as to the way the routing configuration works. I'm closing it as invalid, thanks for the help. It might be worth noting that in the documentation though.

Then please open an issue on symfony-docs that mentions that handling of optional parameters should be clarified.

Will do, I'll write it up tonight. Thanks again @Tobion

No problem.

I have a issue in this code:

return $this->redirect($this->generateUrl('acme_hello_homepage',array('name'=>$nombre)));

i have this error:
FatalErrorException: Error: Cannot use object of type Acme\TaskBundle\Entity\Task as array in /var/www/Symfony/src/Acme/TaskBundle/Controller/DefaultController.php line 31

@jesmarquez Is $nombre an object? Has to be some scalar value.

Why are you posting this in a closed issue that was opened on documentation?

$nombre i get using the getData() method, this is the code: 
 
// data is an array with "name", "email", and "message" keys
$data = $form->getData();

$nombre = $data['task'];

El Jueves 9 de enero de 2014 13:56, hacfi [email protected] escribió:

@jesmarquez Is $nombre an object? Has to be some scalar value.
—
Reply to this email directly or view it on GitHub.

@jesmarquez Looks like your task field is an entity field so you get a Task object back which you can’t use for generating the url.

Try

return $this->redirect($this->generateUrl('acme_hello_homepage',array('name'=>$nombre->getId())));

Thank hacfi, $nombre is a instance!

Excuse me by write a post here when is close!

El Viernes 10 de enero de 2014 18:24, hacfi [email protected] escribió:

@jesmarquez Looks like your task field is an entity field so you get a Task object back which you can’t use for generating the url.
Try
return $this->redirect($this->generateUrl('acme_hello_homepage',array('name'=>$nombre->getId())));
—
Reply to this email directly or view it on GitHub.