Started refactoring route patcher / makers

 - Patchers and makers are basically the same thing
 - The "default" patcher is the old route patcher class
 - All these things implement a single RouteMakerInterface
 - Will add a "maker" option to the builder unit to specify which maker
   to use
   - Defaults to "default" which will delegate depending on the context.

Author: dantleech

AutoRoute/RouteMaker.php

@@ -3,6 +3,7 @@
 namespace Symfony\Cmf\Bundle\RoutingAutoBundle\AutoRoute;
 
 use Doctrine\ODM\PHPCR\DocumentManager;
+use Symfony\Cmf\Bundle\RoutingAutoBundle\AutoRoute\RouteMakerInterface;
 use Symfony\Cmf\Bundle\RoutingAutoBundle\Document\AutoRoute;
 
 /**
@@ -11,7 +12,7 @@
  *
  * @author Daniel Leech <[email protected]>
  */
-class AutoRouteMaker
+class AutoRouteMaker implements RouteMakerInterface
 {
     protected $dm;
 
@@ -20,7 +21,7 @@ public function __construct(DocumentManager $dm)
         $this->dm = $dm;
     }
 
-    public function createOrUpdateAutoRoute(AutoRouteStack $autoRouteStack)
+    public function make(RouteStack $routeStack)
     {
         $context = $autoRouteStack->getContext();
         $content = $context->getContent();

AutoRoute/AutoRouteMaker.php renamed to AutoRoute/RouteMaker/AutoRouteMaker.php

@@ -0,0 +1,32 @@
+<?php
+
+namespace Symfony\Cmf\Bundle\RoutingAutoBundle\AutoRoute;
+
+use Symfony\Cmf\Bundle\RoutingAutoBundle\AutoRoute\RouteMakerInterface;
+
+/**
+ * Default route maker class - automatically delegates to an
+ * appropriate maker depending on the context.
+ *
+ * @author Daniel Leech <[email protected]>
+ */
+class RouteMaker implements RouteMakerInterface
+{
+    protected $autoRouteMaker;
+    protected $patcher;
+
+    public function __construct(RouteMakerInterface $autoRouteMaker, RouteMakerInterface $patcher)
+    {
+        $this->autoRouteMaker = $autoRouteMaker;
+        $this->patcher = $patcher;
+    }
+
+    public function make(RouteStack $routeStack)
+    {
+        if ($routeStack instanceOf AutoRouteStack) {
+            $this->autoRouteMaker->make($routeStack);
+        } else {
+            $this->patcher->make($routeStack);
+        }
+    }
+}

AutoRoute/RouteMaker/DefaultMaker.php

@@ -1,8 +1,8 @@
 <?php
 
-namespace Symfony\Cmf\Bundle\RoutingAutoBundle\AutoRoute\RoutePatcher;
+namespace Symfony\Cmf\Bundle\RoutingAutoBundle\AutoRoute\RouteMaker;
 
-use Symfony\Cmf\Bundle\RoutingAutoBundle\AutoRoute\RoutePatcherInterface;
+use Symfony\Cmf\Bundle\RoutingAutoBundle\AutoRoute\RouteMakerInterface;
 use Doctrine\ODM\PHPCR\DocumentManager;
 use Doctrine\ODM\PHPCR\Document\Generic;
 use Symfony\Cmf\Bundle\RoutingAutoBundle\AutoRoute\RouteStack;
@@ -13,14 +13,14 @@
  *
  * @author Daniel Leech <[email protected]>
  */
-class GenericPatcher implements RoutePatcherInterface
+class GenericMaker implements RouteMakerInterface
 {
     public function __construct(DocumentManager $dm)
     {
         $this->dm = $dm;
     }
 
-    public function patch(RouteStack $routeStack)
+    public function make(RouteStack $routeStack)
     {
         $paths = $routeStack->getFullPaths();
 

AutoRoute/RoutePatcher/GenericPatcher.php renamed to AutoRoute/RouteMaker/GenericMaker.php

@@ -0,0 +1,50 @@
+<?php
+
+namespace Symfony\Cmf\Bundle\RoutingAutoBundle\AutoRoute\RouteMaker;
+
+use Symfony\Cmf\Bundle\RoutingAutoBundle\AutoRoute\RouteMakerInterface;
+use Doctrine\ODM\PHPCR\DocumentManager;
+use Doctrine\ODM\PHPCR\Document\Generic;
+use Symfony\Cmf\Bundle\RoutingAutoBundle\AutoRoute\RouteStack;
+use Symfony\Cmf\Bundle\RoutingAutoBundle\AutoRoute\Exception\MissingOptionException;
+use Symfony\Cmf\Bundle\RoutingExtraBundle\Document\Route;
+
+/**
+ * This class will make the Route classes using
+ * Generic documents using.
+ *
+ * @author Daniel Leech <[email protected]>
+ */
+class RouteMaker implements RouteMakerInterface
+{
+    protected $defaults = array();
+
+    public function __construct(DocumentManager $dm)
+    {
+        $this->dm = $dm;
+    }
+
+    public function init(array $options)
+    {
+        $this->defaults = $defaults;
+    }
+
+    public function make(RouteStack $routeStack)
+    {
+        $paths = $routeStack->getFullPaths();
+
+        foreach ($paths as $path) {
+            $absPath = '/'.$path;
+            $doc = $this->dm->find(null, $absPath);
+
+            if (null === $doc) {
+                $doc = new Route;
+                $doc->setDefaults($this->defaults);
+                $doc->setId($absPath);
+            }
+
+            $routeStack->addRoute($doc);
+        }
+    }
+}
+

AutoRoute/RouteMaker/RouteMaker.php

@@ -0,0 +1,18 @@
+<?php
+
+namespace Symfony\Cmf\Bundle\RoutingAutoBundle\AutoRoute;
+
+use Symfony\Cmf\Bundle\RoutingAutoBundle\AutoRoute\RouteStack;
+
+/**
+ * Classes implementing this interface complete a route stack
+ * by adding one PHPCR-ODM document for each path element contained
+ * in the RouteStack.
+ *
+ * @author Daniel Leech <[email protected]>
+ * @date 13/03/24
+ */
+class RouteMakerInterface
+{
+    public function make(RouteStack $routeStack);
+}