App\Zapheus\CompositeRouter

Complexity

Total Complexity

2

Size/Duplication

Total Lines	38
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	routes()	0	9	1
A	merge()	0	7	1

<?php

namespace App\Zapheus;

use Zapheus\Routing\Router;
use Zapheus\Routing\RouterInterface;

/**
 * Route Collection
 *
 * @package App
 * @author  Rougin Royce Gutib <rougingutib@gmail.com>
 */
class CompositeRouter extends Router
{
    /**
     * Namespace applied to all class-based routes.
     *
     * @var string
     */
    protected $namespace = 'App\Zapheus';

    /**
     * Merges the new route instances.
     *
     * @param  \Zapheus\Routing\RouterInterface $router
     * @return self
     */
    public function merge(RouterInterface $router)
    {
        $routes = $router->routes();

        $this->routes = array_merge($this->routes, $routes);

        return $this;
    }

    /**
     * Returns an array of route instances.
     *
     * @return \Zapheus\Routing\Route[]
     */
    public function routes()
    {
        $this->get('/scream', 'GreetController@scream');

        $this->get('/', 'GreetController@greet');

        // $this->get('/{name}', 'GreetController@greet');

73% of this comment could be valid code. Did you maybe forget this after debugging?

        return $this->routes;
    }
}