[Routing][XML Loader] Add a possibility to set a default value to null

aitboudad · aitboudad · commit 94a9cdc473e7 · 2013-04-11T23:37:28.000Z
diff --git a/src/Symfony/Component/Routing/Loader/XmlFileLoader.php b/src/Symfony/Component/Routing/Loader/XmlFileLoader.php
@@ -141,7 +141,11 @@ protected function parseRoute(RouteCollection $collection, \DOMElement $definiti
 
             switch ($node->tagName) {
                 case 'default':
-                    $defaults[(string) $node->getAttribute('key')] = trim((string) $node->nodeValue);
+                    if ($node->hasAttribute('xsi:nil') && 'true' == $node->getAttribute('xsi:nil')) {
+                        $defaults[(string) $node->getAttribute('key')] = null;
+                    } else {
+                        $defaults[(string) $node->getAttribute('key')] = trim((string) $node->nodeValue);
+                    }
                     break;
                 case 'option':
                     $options[(string) $node->getAttribute('key')] = trim((string) $node->nodeValue);
diff --git a/src/Symfony/Component/Routing/Loader/schema/routing/routing-1.0.xsd b/src/Symfony/Component/Routing/Loader/schema/routing/routing-1.0.xsd
@@ -16,7 +16,7 @@
 
   <xsd:complexType name="route">
     <xsd:sequence>
-      <xsd:element name="default" type="element" minOccurs="0" maxOccurs="unbounded" />
+      <xsd:element name="default" nillable="true" type="element" minOccurs="0" maxOccurs="unbounded" />
       <xsd:element name="requirement" type="element" minOccurs="0" maxOccurs="unbounded" />
       <xsd:element name="option" type="element" minOccurs="0" maxOccurs="unbounded" />
     </xsd:sequence>
diff --git a/src/Symfony/Component/Routing/Tests/Fixtures/validpattern.xml b/src/Symfony/Component/Routing/Tests/Fixtures/validpattern.xml
@@ -6,6 +6,7 @@
 
     <route id="blog_show" pattern="/blog/{slug}">
         <default key="_controller">MyBundle:Blog:show</default>
+        <default key="slug" xsi:nil="true" />
         <requirement key="_method">GET</requirement>
         <option key="compiler_class">RouteCompiler</option>
     </route>
diff --git a/src/Symfony/Component/Routing/Tests/Loader/XmlFileLoaderTest.php b/src/Symfony/Component/Routing/Tests/Loader/XmlFileLoaderTest.php
@@ -47,7 +47,9 @@ public function testLoadWithRoute()
 
         $this->assertEquals(1, count($routes), 'One route is loaded');
         $this->assertContainsOnly('Symfony\Component\Routing\Route', $routes);
+        
         $route = $routes['blog_show'];
+        $this->assertSame(null, $route->getDefault('slug'));
         $this->assertEquals('RouteCompiler', $route->getOption('compiler_class'));
     }
 

Original file line number	Diff line number	Diff line change
`@@ -47,7 +47,9 @@ public function testLoadWithRoute()`
`47`	`47`
`48`	`48`	`$this->assertEquals(1, count($routes), 'One route is loaded');`
`49`	`49`	`$this->assertContainsOnly('Symfony\Component\Routing\Route', $routes);`
	`50`	`+`
`50`	`51`	`$route = $routes['blog_show'];`
	`52`	`+ $this->assertSame(null, $route->getDefault('slug'));`
`51`	`53`	`$this->assertEquals('RouteCompiler', $route->getOption('compiler_class'));`
`52`	`54`	`}`
`53`	`55`