League\Tactician\Doctrine\DBAL\TransactionMiddleware

Complexity

Total Complexity

4

Size/Duplication

Total Lines	45
Duplicated Lines	44.44 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	execute()	20	20	3

<?php

namespace League\Tactician\Doctrine\DBAL;

use Doctrine\DBAL\Driver\Connection;
use League\Tactician\Middleware;
use Exception;
use Throwable;

/**
 * Wraps command execution inside a Doctrine DBAL transaction
 */
class TransactionMiddleware implements Middleware
{
    /**
     * @var Connection
     */
    protected $connection;

    /**
     * @param Connection $connection
     */
    public function __construct(Connection $connection)
    {
        $this->connection = $connection;
    }

    /**
     * Executes the given command and optionally returns a value
     *
     * @param object $command
     * @param callable $next
     * @return mixed
     * @throws Exception
     * @throws Throwable
     */
    public function execute($command, callable $next)

This method seems to be duplicated in your project.

    {
        $this->connection->beginTransaction();

        try {
            $returnValue = $next($command);

            $this->connection->commit();
        } catch (Exception $exception) {
            $this->connection->rollBack();

            throw $exception;
        } catch (Throwable $exception) {

The class Throwable does not exist. Is this class maybe located in a folder that is not analyzed, or in a newer version of your dependencies than listed in your composer.lock/composer.json?

            $this->connection->rollBack();

            throw $exception;
        }

        return $returnValue;
    }
}