Mcustiel\PowerRoute\Actions\Psr7MiddlewareAction

Complexity

Total Complexity

1

Size/Duplication

Total Lines	21
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes	3
Bugs	0	Features	1

1 Method

Rating	Name	Duplication	Size	Complexity
A	execute()	0	12	1

<?php
namespace Mcustiel\PowerRoute\Actions;

use Mcustiel\PowerRoute\Common\TransactionData;

class Psr7MiddlewareAction implements ActionInterface
{
    /**
     * Executes a PSR-7 middleware.
     *
     * @param \Mcustiel\PowerRoute\Common\TransactionData                $transactionData
     * @param \Mcustiel\PowerRoute\Common\Conditions\ClassArgumentObject $argument
     */
    public function execute(TransactionData $transactionData, $argument = null)
    {
        $middleware = $argument->getInstance();

It seems like $argument is not always an object, but can also be of type null. Maybe add an additional type check?

        $transactionData->setResponse(
            $middleware(
                $transactionData->getRequest(),
                $transactionData->getResponse(),
                $argument->getArgument()
            )
        );
    }
}