Moving towards working functionality (i hope)

Author: dantleech

AutoRoute/AutoRouteMaker.php

@@ -0,0 +1,27 @@
+<?php
+
+namespace Symfony\Cmf\Bundle\RoutingAutoBundle\AutoRoute;
+
+use Symfony\Cmf\Bundle\RoutingAutoBundle\AutoRoute\RouteStack\Builder;
+use Symfony\Cmf\Bundle\RoutingAutoBundle\AutoRoute\RouteStack\BuilderUnit;
+
+/**
+ * @author Daniel Leech <[email protected]>
+ */
+class AutoRouteMaker
+{
+    protected $builder;
+    protected $builderUnit;
+
+    public function __construct(Builder $builder, BuilderUnit $builderUnit)
+    {
+        $this->builder = $builder;
+        $this->bulderUnit = $builderUnit;
+    }
+
+    public function createOrUpdateAutoRoute(BuilderContext $context)
+    {
+        $stack = new AutoRouteStack($context);
+        $this->builder->build($stack, $this->builderUnit);
+    }
+}

AutoRoute/AutoRouteStack.php

@@ -0,0 +1,31 @@
+<?php
+
+namespace Symfony\Cmf\Bundle\RoutingAutoBundle\AutoRoute;
+
+use Symfony\Cmf\Bundle\RoutingAutoBundle\Document\AutoRoute;
+
+/**
+ * Special sub class for AutoRoutes.
+ *
+ * This enables us to reuse the same strategies for the
+ * creation of auto routes as for the creation of the content
+ * path routes.
+ *
+ * @author Daniel Leech <[email protected]>
+ */
+class AutoRouteStack extends RouteStack
+{
+    public function close()
+    {
+        if (count($this->routes) > 1) {
+            throw new \RuntimeException('You can only add one route to the AutoRouteStack.');
+        }
+
+        parent::close();
+    }
+
+    public function addRoute(AutoRoute $route)
+    {
+        parent::addRoute($route);
+    }
+}

AutoRoute/Factory.php

@@ -57,10 +57,13 @@ public function getRouteStackBuilderUnitChain($classFqn)
         return $this->routeStackChains[$classFqn];
     }
 
-    public function getContentRouteBuilder($classFqn)
+    public function getAutoRouteMaker($classFqn)
     {
-        $mapping = $this->getMapping($classFqn);
-        return $this->generateBuilderUnit($mapping['content_name']);
+        if (!isset($this->autoRouteMakers[$classFqn])) {
+            $this->autoRouteMakers[$classFqn] = $this->generateAutoRouteMaker($classFqn);
+        }
+
+        return $this->autoRouteMakers[$classFqn];
     }
 
     public function hasMapping($classFqn)
@@ -83,6 +86,16 @@ protected function generateRouteStackChain($classFqn)
         return $routeStackChain;
     }
 
+    protected function generateAutoRouteMaker($classFqn)
+    {
+        $mapping = $this->getMapping($classFqn);
+        $unit = $this->generateBuilderUnit($mapping['content_name']);
+
+        $arm = new AutoRouteMaker($this->builder);
+
+        return $arm;
+    }
+
     protected function generateBuilderUnit($config)
     {
         $pathProvider = $this->getBuilderService($config, 'provider', 'name');
@@ -122,8 +135,21 @@ private function validateMapping($classFqn, $mapping)
             }
         };
 
-        $exists('content_path', function ($mapping) { return isset($mapping['content_path']); });
-        $exists('content_name', function ($mapping) { return isset($mapping['content_name']); });
+        $exists('content_path', function ($mapping) { 
+            return isset($mapping['content_path']); 
+        });
+        $exists('content_name', function ($mapping) { 
+            return isset($mapping['content_name']); 
+        });
+        $exists('content_name/provider', function ($mapping) { 
+            return isset($mapping['content_name']['provider']); 
+        });
+        $exists('content_name/exists', function ($mapping) { 
+            return isset($mapping['content_name']['exists']); 
+        });
+        $exists('content_name/not_exists', function ($mapping) { 
+            return isset($mapping['content_name']['not_exists']); 
+        });
     }
 
     private function getBuilderService($builderConfig, $type, $aliasKey)

AutoRoute/RouteMaker.php

@@ -3,21 +3,29 @@
 namespace Symfony\Cmf\Bundle\RoutingAutoBundle\AutoRoute;
 
 /**
+ * Class responsible for delegating the creation of
+ * routes to the appropriate class.
+ *
  * @author Daniel Leech <[email protected]>
+ * @date 13/03/17
  */
 class RouteMaker
 {
+    protected $arm;
     protected $patcher;
 
-    public function __construct(RoutePatcherInterface $patcher)
+    public function __construct(AutoRouteMaker $arm, RoutePatcherInterface $patcher)
     {
+        $this->arm = $arm;
         $this->patcher = $patcher;
     }
 
-    public function makeRoute(RouteStack $routeStack, BuilderContext $context)
+    public function makeRoutes(RouteStack $routeStack)
     {
-        foreach ($context->getRouteStacks() as $stack) {
-            $this->patcher->patchStack($stack, $context);
+        if ($routeStack instanceOf AutoRouteStack) {
+            $this->arm->makeAutoRoute($routeStack);
+        } else {
+            $this->patcher->patch($routeStack);
         }
     }
 }

AutoRoute/RoutePatcher/GenericPatcher.php

@@ -12,8 +12,6 @@
  * This class will make the Route classes using
  * Generic documents using.
  *
- * @todo: Make this use PHPCR\Util\NodeHelper:makePath
- *
  * @author Daniel Leech <[email protected]>
  */
 class GenericPatcher implements RoutePatcherInterface
@@ -23,32 +21,20 @@ public function __construct(DocumentManager $dm)
         $this->dm = $dm;
     }
 
-    public function makeRoutes(BuilderUnitContext $buc)
+    public function makeRoutes(RouteStack $routeStack)
     {
-        $components = $buc->getPathComponents();
-
-        foreach ($components as $i => $component) {
-
-            $path .= '/'.$component;
+        $paths = $routeStack->getFullPaths();
 
+        foreach ($paths as $path) {
             $doc = $this->dm->find(null, $path);
 
             if (null === $doc) {
-                $parent = $context->getLastRoute();
-
-                if (null === $parent) {
-                    $parent = $this->dm->find(null, '/');
-                }
-
-                // otherwise create a generic document
                 $doc = new Generic;
-                $doc->setNodename($component);
-                $doc->setParent($parent);
-                    var_dump($parent);
-                }
+                $meta = $this->dm->getClassMetadata(get_class($doc));
+                $meta->setIdentifierValue($doc, $path);
             }
 
-            $context->addRoute($doc);
+            $routeStack->addRoute($doc);
         }
     }
 }

AutoRoute/RouteStack.php

@@ -9,9 +9,15 @@ class RouteStack
 {
     protected $pathElements;
     protected $routes = array();
+    protected $context;
 
     protected $closed = false;
 
+    public function __construct(BuilderContext $context)
+    {
+        $this->context = $context;
+    }
+
     public function addPathElements(array $pathElements)
     {
         foreach ($pathElements as $pathElement) {
@@ -45,6 +51,19 @@ public function getPaths()
         return $paths;
     }
 
+    public function getFullPaths()
+    {
+        $parentPath = $this->context->getRouteStackPath($this);
+
+        $paths = $this->getPaths();
+
+        array_walk($paths, function (&$path) use ($parentPath) {
+            $path = $parentPath.'/'.$path;
+        });
+
+        return $paths;
+    }
+
     public function getPath()
     {
         return implode('/', $this->pathElements);