Nip\Dispatcher\Resolver\Pipeline\Stages\MethodCallStage

Complexity

Total Complexity

5

Size/Duplication

Total Lines	34
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	3
Bugs	1	Features	2

2 Methods

Rating	Name	Duplication	Size	Complexity
A	processCommand()	0	11	2
A	callMethod()	0	11	3

<?php

namespace Nip\Dispatcher\Resolver\Pipeline\Stages;

use Nip\Controllers\Controller;
use Nip\Dispatcher\Exceptions\InvalidCommandException;
use Psr\Http\Message\RequestInterface;
use Psr\Http\Message\ResponseInterface;

/**
 * Class MethodCallStage
 * @package Nip\Dispatcher\Resolver\Pipeline\Stages
 */
class MethodCallStage extends AbstractStage
{
    /**
     * @return void
     * @throws \Exception
     */
    public function processCommand()
    {
        if (!$this->hasInstanceAction()) {
            throw new InvalidCommandException(
                "No valid instance for callback in dispatcher command " .
                "[" . print_r($this->getCommand(), true) . "]"

Are you sure print_r($this->getCommand(), true) of type string|true can be used in concatenation?

            );
        }

        $return = $this->callMethod();
        $this->getCommand()->setReturn($return);
    }

    /**
     * @return ResponseInterface
     * @throws \Exception
     */
    protected function callMethod()
    {
        $command = $this->getCommand();
        /** @var Controller $controllerInstance */
        $controllerInstance = $this->getCommand()->getActionParam('instance');
        if ($this->getCommand()->hasRequest()) {
            $controllerInstance->setRequest($this->getCommand()->getRequest());
        }
        $method = $command->getActionParam('action');
        $params = $command->hasActionParam('params') ? $command->getActionParam('params') : [];
        return $controllerInstance->{$method}(...$params);
    }
}