(fixed #4439, BP from #4438) Fix sfPatternRouting->getRoutes() sometimes returning serialized routes

FriendsOfSymfony1/symfony1#169
参照

Author: nishizoe

lib/vendor/symfony/lib/routing/sfPatternRouting.class.php

@@ -144,6 +144,14 @@ public function getCurrentRouteName()
    */
   public function getRoutes()
   {
+    foreach ($this->routes as $name => $route) {
+      if (is_string($route)) {
+        $route = unserialize($route);
+        $route->setDefaultParameters($this->defaultParameters);
+        $this->routes[$name] = $route;
+      }
+    }
+
     return $this->routes;
   }
 

lib/vendor/symfony/test/unit/routing/sfPatternRoutingTest.php

@@ -639,8 +639,8 @@ function configureRouting($event)
 // see fixtures/config_routing.yml.php
 $r = new sfPatternRoutingTest(new sfEventDispatcher(), new sfNoCache(), array('load_configuration' => true));
 $t->ok($r->hasRouteName('test1'), '->loadConfiguration() Config file is loaded');
-$routes = $r->getRoutes();
-$t->ok(is_string($routes['test1']), '->loadConfiguration() Route objects are not serialized in cache');
 $route = $r->getRoute('test1');
 $t->ok(is_object($route), '->loadConfiguration() Route objects are unserialized on demand');
+$routes = $r->getRoutes();
+$t->ok(is_object($routes['test1']), '->loadConfiguration() Route objects are not serialized in cache');
 $t->is_deeply($r->parse('/'), array('module' => 'default', 'action' => 'index'), '->parse() Default parameters are applied to serialized routes');