Spatie\SlashCommand\SlackCommandResponder

Complexity

Total Complexity

7

Size/Duplication

Total Lines	59
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	getResponse()	0	10	1
A	guardAgainstInvalidRequest()	0	10	3
A	determineResponseHandler()	0	15	2

<?php

namespace Spatie\SlashCommand;


use Illuminate\Http\Request;
use Spatie\SlashCommand\SlashCommandHandler\BaseHandler;

class SlackCommandResponder
{
    /**
     * @var array
     */
    private $commandConfig;
    /**
     * @var Request
     */
    private $request;

    public function __construct(array $commandConfig, Request $request)

You have injected the Request via parameter $request. This is generally not recommended as there might be multiple instances during a request cycle (f.e. when using sub-requests). Instead, it is recommended to inject the RequestStack and retrieve the current request each time you need it via getCurrentRequest().

    {
        $this->commandConfig = $commandConfig;

        $this->request = $request;
    }

    public function getResponse() {

        $this->guardAgainstInvalidRequest();

        $handler = $this->determineResponseHandler();

        $response = $handler->handleCurrentRequest();

        return $response->finalize();
    }

    protected function guardAgainstInvalidRequest()
    {
        if (!request()->has('token')) {
            throw InvalidSlashCommandRequest::tokenNotFound();
        }

        if (request()->get('token') != $this->commandConfig['verification_token']) {
            throw InvalidSlashCommandRequest::invalidToken(request()->get('token'));
        }
    }

    /**
     * @return mixed
     */
    protected function determineResponseHandler()
    {
        $handler = collect($this->commandConfig['handlers'])
            ->map(function (string $handlerClassName) {
                return new $handlerClassName(request());
            })
            ->filter(function (BaseHandler $handler) {
                return $handler->canHandleCurrentRequest();
            })->first();

        if (!$handler) {
            throw RequestCouldNotBeProcessed::noHandlerFound(request());
        }
        return $handler;
    }
}