Merge pull request #127 from symfony-cmf/fix_handling_of_root_parents

Fix the case when the uri is in fact the root

Author: lsmith77

Adapter/PhpcrOdmAdapter.php

@@ -117,7 +117,7 @@ public function removeAutoRoute(AutoRouteInterface $autoRoute)
     public function createAutoRoute($uri, $contentDocument, $autoRouteTag)
     {
         $path = $this->baseRoutePath;
-        $parentDocument = $this->dm->find(null, $path);
+        $document = $parentDocument = $this->dm->find(null, $path);
         $segments = preg_split('#/#', $uri, null, PREG_SPLIT_NO_EMPTY);
         $headName = array_pop($segments);
         foreach ($segments as $segment) {

Tests/Functional/EventListener/AutoRouteListenerTest.php

@@ -22,6 +22,7 @@
 use Symfony\Cmf\Bundle\RoutingAutoBundle\Tests\Resources\Document\SeoArticleMultilang;
 use Symfony\Cmf\Bundle\RoutingAutoBundle\Tests\Resources\Document\SeoArticle;
 use Symfony\Cmf\Component\RoutingAuto\Model\AutoRouteInterface;
+use Symfony\Cmf\Bundle\RoutingAutoBundle\Tests\Resources\Document\Page;
 
 class AutoRouteListenerTest extends BaseTestCase
 {
@@ -467,6 +468,20 @@ public function testConflictResolverAutoIncrement()
         }
     }
 
+    public function testCreationOfChildOnRoot()
+    {
+        $page = new Page;
+        $page->title = 'Home';
+        $page->path = '/test/home';
+        $this->getDm()->persist($page);
+        $this->getDm()->flush();
+
+        $expectedRoute = '/test/auto-route/home';
+        $route = $this->getDm()->find('Symfony\Cmf\Bundle\RoutingAutoBundle\Model\AutoRoute', $expectedRoute);
+
+        $this->assertNotNull($route);
+    }
+
     /**
      * @expectedException Symfony\Cmf\Component\RoutingAuto\ConflictResolver\Exception\ExistingUriException
      */

Tests/Resources/Document/Page.php

@@ -0,0 +1,25 @@
+<?php
+
+/*
+ * This file is part of the Symfony CMF package.
+ *
+ * (c) 2011-2014 Symfony CMF
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+
+namespace Symfony\Cmf\Bundle\RoutingAutoBundle\Tests\Resources\Document;
+
+use Doctrine\ODM\PHPCR\Mapping\Annotations as PHPCR;
+
+/**
+ * Document mapped to /{title} schema
+ *
+ * @PHPCR\Document(referenceable=true)
+ */
+class Page extends SeoArticle
+{
+}
+

Tests/Resources/app/config/routing_auto.yml

@@ -35,3 +35,8 @@ Symfony\Cmf\Bundle\RoutingAutoBundle\Tests\Resources\Document\AbstractContent:
     uri_schema: /articles/{article_title}
     token_providers:
         article_title: [content_method, { method: getTitle } ]
+
+Symfony\Cmf\Bundle\RoutingAutoBundle\Tests\Resources\Document\Page:
+    uri_schema: /{title}
+    token_providers:
+        title: [content_method, { method: getTitle } ]