diogo/symfony
Archived
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix bug when imported routes are prefixed

Browse Source
This commit is contained in:
smaine 2018-05-15 00:21:15 +02:00
parent f59ce97eff
commit cb5ce8f32e
2 changed files with 16 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/Symfony/Component/Routing/RouteCollection.php
View File

@ -162,6 +162,9 @@ class RouteCollection implements \IteratorAggregate, \Countable
foreach ($this->routes as $name => $route) {
$prefixedRoutes[$prefix.$name] = $route;
if (null !== $name = $route->getDefault('_canonical_route')) {
$route->setDefault('_canonical_route', $prefix.$name);
}
}
$this->routes = $prefixedRoutes;

13
src/Symfony/Component/Routing/Tests/RouteCollectionTest.php
View File

@ -317,4 +317,17 @@ class RouteCollectionTest extends TestCase
$this->assertNull($collection->get('foo'));
$this->assertNull($collection->get('bar'));
}
public function testAddNamePrefixCanonicalRouteName()
{
$collection = new RouteCollection();
$collection->add('foo', new Route('/foo', array('_canonical_route' => 'foo')));
$collection->add('bar', new Route('/bar', array('_canonical_route' => 'bar')));
$collection->add('api_foo', new Route('/api/foo', array('_canonical_route' => 'api_foo')));
$collection->addNamePrefix('api_');
$this->assertEquals('api_foo', $collection->get('api_foo')->getDefault('_canonical_route'));
$this->assertEquals('api_bar', $collection->get('api_bar')->getDefault('_canonical_route'));
$this->assertEquals('api_api_foo', $collection->get('api_api_foo')->getDefault('_canonical_route'));
}
}