NestedTransactionDbAdapterAbstract::rollbackTransaction() - Code Metrics - Inspection of "added composer scripts" - bartko-s/stefano-tree - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — develop ( 4ea5a3...a8c617 )

by Bartko

created 2016-05-19 16:39 UTC

rollbackTransaction() A

↳ Parent: NestedTransactionDbAdapterAbstract

Complexity

Conditions	2
Paths	2

Size

Total Lines	6
Code Lines	3

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	6

Importance

Changes	2
Bugs	1	Features	0

Metric	Value
c	2
b	1
f	0
dl	0
loc	6
ccs	0
cts	5
cp	0
rs	9.4285
cc	2
eloc	3
nc	2
nop	0
crap	6

<?php
namespace StefanoTree\NestedSet\Adapter;

abstract class NestedTransactionDbAdapterAbstract
    implements AdapterInterface

{
    private $outerTransactionWasStarted = false;

    /**
     * Start DB transaction
     */
    abstract protected function _beginTransaction();

    /**
     * Commit DB transaction
     */
    abstract protected function _commitTransaction();

    /**
     * Rollback DB transaction
     */
    abstract protected function _rollbackTransaction();

    /**
     * Check if DB transaction has been started
     * @return boolean
     */
    abstract protected function _isInTransaction();

    public function beginTransaction()
    {
        $this->outerTransactionWasStarted = $this->_isInTransaction();

        if (!$this->outerTransactionWasStarted) {
            $this->_beginTransaction();
        }
    }

    public function commitTransaction()
    {
        if (!$this->outerTransactionWasStarted) {
            $this->_commitTransaction();
        }
    }

    public function rollbackTransaction()
    {
        if (!$this->outerTransactionWasStarted) {
            $this->_rollbackTransaction();
        }
    }
}


1		<?php
2		namespace StefanoTree\NestedSet\Adapter;
3
4		abstract class NestedTransactionDbAdapterAbstract
5		implements AdapterInterface
		0 ignored issues – show Coding Style introduced 2016-01-09 17:22 UTC by Report Bug Copy Issue Report The implements keyword must be on the same line as the class name Loading history...
6		{
7		private $outerTransactionWasStarted = false;
8
9		/**
10		* Start DB transaction
11		*/
12		abstract protected function _beginTransaction();
13
14		/**
15		* Commit DB transaction
16		*/
17		abstract protected function _commitTransaction();
18
19		/**
20		* Rollback DB transaction
21		*/
22		abstract protected function _rollbackTransaction();
23
24		/**
25		* Check if DB transaction has been started
26		* @return boolean
27		*/
28		abstract protected function _isInTransaction();
29
30	12	public function beginTransaction()
31		{
32	12	$this->outerTransactionWasStarted = $this->_isInTransaction();
33
34	12	if (!$this->outerTransactionWasStarted) {
35	12	$this->_beginTransaction();
36	12	}
37	12	}
38
39	12	public function commitTransaction()
40		{
41	12	if (!$this->outerTransactionWasStarted) {
42	12	$this->_commitTransaction();
43	12	}
44	12	}
45
46		public function rollbackTransaction()
47		{
48		if (!$this->outerTransactionWasStarted) {
49		$this->_rollbackTransaction();
50		}
51		}
52		}
53

bartko-s / stefano-tree

Push — develop ( 4ea5a3...a8c617 )

rollbackTransaction() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like