RemiSan\TransactionManager\MultipleTransactionManager

Complexity

Total Complexity

8

Size/Duplication

Total Lines	55
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes	1
Bugs	0	Features	1

4 Methods

Rating	Name	Duplication	Size	Complexity
A	beginTransaction()	0	8	2
A	commit()	0	11	3
A	rollback()	0	9	2
A	reset()	0	4	1

<?php

namespace RemiSan\TransactionManager;

use RemiSan\TransactionManager\Exception\NoRunningTransactionException;

class MultipleTransactionManager extends SimpleTransactionManager
{
    /**
     * @var int number of transactions currently running
     */
    private $transactionCpt = 0;

    /**
     * {@inheritdoc}
     */
    public function beginTransaction()
    {
        $this->transactionCpt++;

        if ($this->transactionCpt === 1) {
            parent::beginTransaction();
        }
    }

    /**
     * {@inheritdoc}
     */
    public function commit()
    {
        $this->transactionCpt--;

        if ($this->transactionCpt < 0) {
            throw new NoRunningTransactionException('Cannot commit before a transaction has begun');
        } elseif ($this->transactionCpt === 0) {
            parent::commit();
            $this->reset();
        }
    }

    /**
     * {@inheritdoc}
     */
    public function rollback()
    {
        if ($this->transactionCpt === 0) {
            return;
        }

        parent::rollback();
        $this->reset();
    }

    /**
     * Reset the transaction.
     */
    private function reset()

Consider using a different method name as you override a private method of the parent class.

    {
        $this->transactionCpt = 0;
    }
}