Nip\Router\Router\Traits\HasMatcherTrait

Complexity

Total Complexity

2

Size/Duplication

Total Lines	14
Duplicated Lines	0 %

Test Coverage

Coverage

40%

Importance

Changes	1
Bugs	0	Features	1

1 Method

Rating	Name	Duplication	Size	Complexity
A	matchRequest()	0	8	2

<?php

namespace Nip\Router\Router\Traits;

use \Symfony\Component\HttpFoundation\Request;
use Nip\Router\Route\Route;
use Psr\Http\Message\ServerRequestInterface;

/**
 * Trait HasMatcherTrait
 * @package Nip\Router\Router\Traits
 */
trait HasMatcherTrait
{
    /**
     * @param Request|ServerRequestInterface $request
     * @return array
     */
    public function matchRequest(Request $request)
    {
        $return = parent::matchRequest($request);
        if (isset($return['_route'])) {
            $this->setCurrent($return['_route']);

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

//            $this->setCurrent($this->getRoute($return['_route']));
        }
        return $return;
    }
}