8000 merged branch Crell/routecollection-count (PR #4642) · symfony/symfony@db9a8c1 · GitHub
[go: up one dir, main page]
Skip to content

Commit db9a8c1

Browse files
fabpotfabpot
committed
merged branch Crell/routecollection-count (PR #4642)
Commits ------- c350944 Add the Countable interface to RouteCollection. Discussion ---------- Add the Countable interface to RouteCollection. Since RouteCollection is a fancy-pants array of Routes, and it already is iterable, it would be nice if it were also countable. There may be optimizations to be had here, but I figure this is a decent start. There should be no BC breaks here, just some DX convenience. --------------------------------------------------------------------------- by Crell at 2012-06-23T17:01:43Z You Symfony people and your gratuitous whitespace... :-P Both fixed and rebased. --------------------------------------------------------------------------- by travisbot at 2012-06-23T17:25:31Z This pull request [fails](http://travis-ci.org/symfony/symfony/builds/1689898) (merged 6b588eb3 into 0d4b02e). --------------------------------------------------------------------------- by travisbot at 2012-06-23T17:26:06Z This pull request [fails](http://travis-ci.org/symfony/symfony/builds/1689902) (merged abd74cf5 into 0d4b02e). --------------------------------------------------------------------------- by travisbot at 2012-06-23T18:10:57Z This pull request [fails](http://travis-ci.org/symfony/symfony/builds/1689975) (merged c350944 into 0d4b02e). --------------------------------------------------------------------------- by mvrhov at 2012-06-24T07:11:12Z @Crell running [php-cs-fixer](http://cs.sensiolabs.org/) on your changes will prevent you from being stoffed :P about CS --------------------------------------------------------------------------- by stof at 2012-06-24T09:34:24Z @mvrhov depends. The CS fixer will replace tabs with 4 spaces, but it will not fix it if using 2 spaces instead of 4. it cannot fix everything --------------------------------------------------------------------------- by vicb at 2012-06-24T17:14:14Z @Crell what is your use case ? --------------------------------------------------------------------------- by fabpot at 2012-06-25T06:59:00Z I'm not opposed to the change but what's the use case? --------------------------------------------------------------------------- by lsmith77 at 2012-06-25T11:22:01Z This is semi related to ideas we have for the dynamic router in the CMF. The 2 main reasons for the existence of the dynamic router are: 1) the fact that end users should be able to define new routes 2) the fact that there might just be too many routes to dump those efficiently to PHP Now in some cases despite 1) the users might still want to dump key routes or even all routes as part of some deployment process to mod_rewrite or PHP when moving content from a staging database to a production database for better performance and reduced load on the database server. Also even with the dynamic router it would be good to have tools available like ``app/console router:debug``. But for both the use case of dumping some/all routes or when listing routes it might be necessary to implement some stop gap solutions to first check if this wouldn't lead to a too big collection. So it could be useful to be able to use whatever efficient solution the dynamic data store has for determining the count before starting to actually read the data from the data source.
2 parents 03c8d4d + c350944 commit db9a8c1

File tree

2 files changed

+23
-1
lines changed

2 files changed

+23
-1
lines changed

‎src/Symfony/Component/Routing/RouteCollection.php

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@
2424
*
2525
* @api
2626
*/
27-
class RouteCollection implements \IteratorAggregate
27+
class RouteCollection implements \IteratorAggregate, \Countable
2828
{
2929
private $routes;
3030
private $resources;
@@ -88,6 +88,16 @@ public function getIterator()
8888
return new \ArrayIterator($this->routes);
8989
}
9090

91+
/**
92+
* Gets the number of Routes in this collection.
93+
*
94+
* @return int The number of routes in this collection, including nested collections
95+
*/
96+
public function count()
97+
{
98+
return count($this->routes->all());
99+
}
100+
91101
/**
92102
* Adds a route.
93103
*

‎src/Symfony/Component/Routing/Tests/RouteCollectionTest.php

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -76,6 +76,18 @@ public function testIteratorWithOverridenRoutes()
7676
$this->assertEquals('/foo1', $this->getFirstNamedRoute($collection, 'foo')->getPattern());
7777
}
7878

79+
public function testCount()
80+
{
81+
$collection = new RouteCollection();
82+
$collection->add('foo', new Route('/foo'));
83+
84+
$collection1 = new RouteCollection();
85+
$collection->addCollection($collection1);
86+
$collection1->add('foo', new Route('/foo1'));
87+
88+
$this->assertCount(2, $collection);
89+
}
90+
7991
protected function getFirstNamedRoute(RouteCollection $routeCollection, $name)
8092
{
8193
foreach ($routeCollection as $key => $route) {

0 commit comments

Comments
 (0)
0