sonrac\WAMP\Routers\RouterTrait

Complexity

Total Complexity

24

Size/Duplication

Total Lines	227
Duplicated Lines	0 %

Test Coverage

Coverage

85.48%

Importance

Changes

0

9 Methods

Rating	Name	Duplication	Size	Complexity
A	group()	0	10	4
B	parseCallback()	0	25	6
A	getRouter()	0	3	1
A	getClientSession()	0	3	1
A	setRouter()	0	3	1
A	getClient()	0	3	1
A	getGroups()	0	3	1
B	prepareCallback()	0	13	5
B	parseGroups()	0	25	4

<?php
/**
 * Created by PhpStorm.
 *
 * @author Donii Sergii <doniysa@gmail.com>
 * Date: 10/25/17
 * Time: 11:57 AM
 */

namespace sonrac\WAMP\Routers;

use Thruway\ClientSession;
use Thruway\Peer\RouterInterface;

/**
 * Trait RouterTrait.
 * Base router trait.
 */
trait RouterTrait
{
    /**
     * Controllers list
     *
     * @var \sonrac\WAMP\Abstracts\WAMPControllerInterface[]
     *
     * @author Donii Sergii <doniysa@gmail.com>
     */
    protected $controllers;

    /**
     * Router groups.
     *
     * @var null|array
     */
    protected $groups = null;

    /**
     * Main router
     *
     * @var null|\Thruway\Peer\RouterInterface|\sonrac\WAMP\Routers\Router
     *
     * @author Donii Sergii <doniysa@gmail.com>
     */
    protected $router = null;

    /**
     * Route path prefix.
     *
     * @var string|null
     *
     * @author Donii Sergii <doniysa@gmail.com>
     */
    private $prefix = null;

    /**
     * Controller namespace.
     *
     * @var string|null
     *
     * @author Donii Sergii <doniysa@gmail.com>
     */
    private $groupControllerNamespace = null;

    /**
     * Middleware list.
     *
     * @var null|array
     *
     * @author Donii Sergii <doniysa@gmail.com>
     */
    private $middleware = null;

    /**
     * Group routes.
     *
     * @param array    $config Group config
     * @param \Closure $runner Closure runner group
     *
     * @author Donii Sergii <doniysa@gmail.com>
     */
    public function group(array $config, \Closure $runner)
    {
        $middleware = isset($config['middleware']) ? explode('|', $config['middleware']) : [];
        $namespace = isset($config['namespace']) ? $config['namespace'] : 'App\Controllers\WAMP';

        $this->groups[] = [
            'middleware' => $middleware,
            'namespace'  => $namespace,
            'prefix'     => isset($config['prefix']) ? $config['prefix'] : '',
            'callback'   => $runner,
        ];
    }

    /**
     * Parse groups
     *
     * @return \sonrac\WAMP\GroupsConfigInterface[]|\stdClass[]
     *
     * @author Donii Sergii <doniysa@gmail.com>
     */
    public function parseGroups()
    {
        if (!is_array($this->groups) || !count($this->groups)) {
            return;
        }
        gc_enable();
        $callbacks = [];
        foreach ($this->groups as $group) {
            $this->prefix = $group['prefix'];
            $this->groupControllerNamespace = $group['namespace'];
            $this->middleware = $group['middleware'];
            $callbacks[] = $group['callback']($this->getClientSession(), $this->getClient());
        }

        $this->groups = null;
        unset($this->groups);
        $this->groups = [];

        $this->prefix = null;
        $this->groupControllerNamespace = null;

        gc_collect_cycles();
        gc_disable();

        return $callbacks;
    }

    /**
     * Get route groups
     *
     * @return null|\sonrac\WAMP\GroupsConfigInterface[]
     *
     * @author Donii Sergii <doniysa@gmail.com>
     */
    public function getGroups()
    {
        return $this->groups;
    }

    /**
     * Get router
     *
     * @return mixed|null|\sonrac\WAMP\Contracts\WAMPRouterInterface|\sonrac\WAMP\Routers\Router|\Thruway\Peer\RouterInterface
     *
     * @author Donii Sergii <doniysa@gmail.com>
     */
    public function getRouter()
    {
        return $this->router ?? $this->router = app()->wampRouter;

The function app was not found. Maybe you did not declare it correctly or list all dependencies?

    }

    /**
     * Set router.
     *
     * @author Donii Sergii <doniysa@gmail.com>
     */
    public function setRouter(RouterInterface $router)
    {
        $this->router = $router;
    }

    /**
     * Get client session.
     *
     * @return \Thruway\ClientSession
     *
     * @author Donii Sergii <doniysa@gmail.com>
     */
    public function getClientSession()
    {
        return $this->getRouter()->getClient()->getSession();

The method getClient() does not exist on Thruway\Peer\RouterInterface. It seems like you code against a sub-type of Thruway\Peer\RouterInterface such as sonrac\WAMP\Routers\Router.

    }

    /**
     * Get client.
     *
     * @return \sonrac\WAMP\Client|\Thruway\Peer\ClientInterface
     *
     * @author Donii Sergii <doniysa@gmail.com>
     */
    public function getClient()
    {
        return $this->getRouter()->getClient();
    }

    /**
     * Parse callback function
     *
     * @param string|\Closure $callback  Callback
     * @param null            $namespace Controller namespace

Are you sure the doc-type for parameter $namespace is correct as it would always require null to be passed?

     *
     * @return \Closure
     *
     * @author Donii Sergii <doniysa@gmail.com>
     */
    public function parseCallback($callback, $namespace = null)
    {
        if ($callback instanceof \Closure) {
            return $callback;
        }

        $namespace = $namespace ? $namespace.'\\' : '';

        $callback = explode('&', $callback);
        $self = $this;

        return function (ClientSession $clientSession) use ($callback, $namespace, $self) {
            if (count($callback) === 1) {
                return $this->{$callback[0]}($clientSession, $self->getClient());
            }

            if (!isset($this->controllers[$callback[0]])) {
                return $this->controllers[$callback[0]]->{$callback[1]}($clientSession, $this->getClient());
            }

            $className = class_exists($callback[0]) ? $callback[0] : $namespace.$callback[0];

            $this->controllers[$callback[0]] = app()->make($className);

The function app was not found. Maybe you did not declare it correctly or list all dependencies?

            return $this->controllers[$callback[0]]->{$callback[1]}($clientSession, $self->getClient());
        };
    }

    /**
     * Prepare path.
     *
     * @param \Closure|string $callback Callback
     *
     * @return array
     *
     * @author Donii Sergii <doniysa@gmail.com>
     */
    protected function prepareCallback($callback)
    {
        $namespace = $this->groupControllerNamespace ?? $this->getRouter()->getControllerNamespace();

The method getControllerNamespace() does not exist on Thruway\Peer\RouterInterface. It seems like you code against a sub-type of Thruway\Peer\RouterInterface such as sonrac\WAMP\Routers\Router.

        if ($this->groupControllerNamespace && $this->groupControllerNamespace

The expression $this->groupControllerNamespace of type null|string is loosely compared to true; this is ambiguous if the string can be empty. You might want to explicitly use !== null instead.

            && is_string($callback) && count(explode('&', $callback)) === 2) {
            $callback = rtrim($namespace, '\\').$callback;
        }

        return [
            'prefix'     => $this->prefix,
            'namespace'  => $namespace,
            'callback'   => $this->parseCallback($callback, $namespace),

It seems like $namespace can also be of type string; however, parameter $namespace of sonrac\WAMP\Routers\RouterTrait::parseCallback() does only seem to accept null, maybe add an additional type check?

            'middleware' => $this->middleware,
        ];
    }
}