Nip\Router\Legacy\Router\Traits\HasMatcherTrait

Complexity

Total Complexity

7

Size/Duplication

Total Lines	36
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	route()	0	7	2
A	populateRequest()	0	16	5

<?php

namespace Nip\Router\Legacy\Router\Traits;

use Nip\Request;
use Nip\Router\Route\Route;
use Psr\Http\Message\ServerRequestInterface;

/**
 * Trait HasMatcherTrait
 * @package Nip\Router\Legacy\Router\Traits
 */
trait HasMatcherTrait
{
    /**
     * @param Request|ServerRequestInterface $request
     * @return array
     * @deprecated Use matchRequest($request)
     */
    public function route($request)
    {
        $return = $this->matchRequest($request);

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

        if ($return['_route']) {
            $this->populateRequest($request, $return);
        }
        return $return;
    }

    /**
     * @param Request $request
     * @param $params
     */
    protected function populateRequest($request, $params)
    {
        foreach ($params as $param => $value) {
            switch ($param) {
                case 'module':
                    $request->setModuleName($value);
                    break;
                case 'controller':
                    $request->setControllerName($value);
                    break;
                case 'action':
                    $request->setActionName($value);
                    break;
                default:
                    $request->attributes->set($param, $value);
                    break;
            }
        }
    }
}