BrainExe\Core\DependencyInjection\CompilerPass\ControllerCompilerPass

Complexity

Total Complexity

8

Size/Duplication

Total Lines	80
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

92.86%

Importance

Changes	15
Bugs	1	Features	3

3 Methods

Rating	Name	Duplication	Size	Complexity
A	dumpMatcher()	0	15	1
B	process()	0	26	5
A	createRoute()	0	17	2

<?php

namespace BrainExe\Core\DependencyInjection\CompilerPass;

use BrainExe\Core\Annotations\CompilerPass;
use BrainExe\Core\Annotations\Route as RouteAnnotation;
use BrainExe\Core\Application\SerializedRouteCollection;
use BrainExe\Core\Traits\FileCacheTrait;
use Exception;
use Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface;
use Symfony\Component\DependencyInjection\ContainerBuilder;
use Symfony\Component\Routing\Matcher\Dumper\PhpMatcherDumper;
use Symfony\Component\Routing\Route;

/**
 * @CompilerPass(priority=5)
 */
class ControllerCompilerPass implements CompilerPassInterface
{
    use FileCacheTrait;

    const CONTROLLER_TAG = 'controller';
    const ROUTE_TAG      = 'route';

    /**
     * {@inheritdoc}
     */
    public function process(ContainerBuilder $container)
    {
        $controllers = $container->findTaggedServiceIds(self::ROUTE_TAG);

        $serialized = [];
        foreach ($controllers as $controllerId => $tag) {
            foreach ($tag as $routeRaw) {
                /** @var RouteAnnotation $route */
                $route = $routeRaw[0];

                $name = $route->getName();
                if (empty($name)) {
                    throw new Exception(sprintf('"name" is missing for @Route(%s)', $controllerId));
                } elseif (isset($serialized[$name])) {
                    throw new Exception(sprintf('Route name %s does already exits in %s', $name, $controllerId));
                }

                $serialized[$name] = serialize($this->createRoute($route));
            }

            $controller = $container->getDefinition($controllerId);
            $controller->clearTag(self::ROUTE_TAG);
        }

        $this->dumpMatcher($container, $serialized);
    }

    /**
     * @param RouteAnnotation $route
     * @return Route
     */
    private function createRoute(RouteAnnotation $route)
    {
        if ($route->isCsrf()) {
            $route->setOptions(['csrf' => true]);
        }

        return new Route(
            $route->getPath(),
            $route->getDefaults(),
            $route->getRequirements(),
            $route->getOptions(),
            $route->getHost(),
            $route->getSchemes(),
            $route->getMethods(),
            $route->getCondition()
        );
    }

    /**
     * @param ContainerBuilder $container
     * @param array $routes
     * @codeCoverageIgnore
     */
    protected function dumpMatcher(ContainerBuilder $container, array $routes)
    {
        ksort($routes);

        $this->dumpVariableToCache(SerializedRouteCollection::CACHE_FILE, $routes);

        /** @var SerializedRouteCollection $routerCollection */
        $routerCollection = $container->get('Core.RouteCollection');

        $routerFile  = sprintf('%scache/router_matcher.php', ROOT);
        $routeDumper = new PhpMatcherDumper($routerCollection);
        $content     = $routeDumper->dump();
        file_put_contents($routerFile, $content);
        @chmod($routerFile, 0777);

It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended.

    }
}