Nip\Router\Generator\Traits\HasDefaultRouteTrait

Complexity

Total Complexity

8

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	generate()	0	5	1
B	initDefaultRoute()	0	21	7

<?php

namespace Nip\Router\Generator\Traits;

use Nip\Router\Route\Route;

/**
 * Trait HasDefaultRouteTrait
 * @package Nip\Router\Generator\Traits
 */
trait HasDefaultRouteTrait
{

    /**
     * {@inheritdoc}
     */
    public function generate($name, $parameters = [], $referenceType = self::ABSOLUTE_PATH)
    {
        $name = $this->initDefaultRoute($name, $parameters);
        return parent::generate($name, $parameters, $referenceType);
    }

    /**
     * @param $name
     * @param array $params
     * @return null|Route
     */
    protected function initDefaultRoute($name, &$params = [])
    {
        $route = $this->hasRoute($name);

It seems like hasRoute() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        if ($route) {
            return $name;
        }
        $parts = explode(".", $name);
        $count = count($parts);
        if ($count <= 3) {
            if (in_array(reset($parts), app('mvc.modules')->getNames())) {
                $module = array_shift($parts);
                $params['controller'] = isset($parts[0]) ? $parts[0] : null;
                $params['action'] = isset($parts[1]) ? $parts[1] : null;
                $defaultRoute = $module . '.default';
                if ($this->hasRoute($defaultRoute)) {

It seems like hasRoute() must be provided by classes using this trait. How about adding it as abstract method to this trait?

                    return $defaultRoute;
                }
            }
        }
        return $name;
    }
}