RowWrapper::start() - Code Metrics - Inspection of "update to phpspreadsheet to 1.6.0" - MewesK/TwigSpreadsheetBundle - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Pull Request — master (#23)

unknown

created 2019-04-25 10:45 UTC

RowWrapper::start() A

↳ Parent: RowWrapper

Complexity

Conditions	3
Paths	3

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	6
CRAP Score	3.0261

Importance

Changes

Metric	Value
dl	0
loc	12
ccs	6
cts	7
cp	0.8571
rs	9.8666
c	0
b	0
f	0
cc	3
nc	3
nop	1
crap	3.0261

<?php

namespace MyWheels\TwigSpreadsheetBundle\Wrapper;

/**
 * Class SheetWrapper.
 */
class RowWrapper extends BaseWrapper
{
    /**
     * @var SheetWrapper
     */
    protected $sheetWrapper;

    /**
     * RowWrapper constructor.
     *
     * @param array             $context
     * @param \Twig_Environment $environment
     * @param SheetWrapper      $sheetWrapper
     */
    public function __construct(array $context, \Twig_Environment $environment, SheetWrapper $sheetWrapper)
    {
        parent::__construct($context, $environment);

        $this->sheetWrapper = $sheetWrapper;
    }

    /**
     * @param null|int $index
     *
     * @throws \LogicException
     */
    public function start(int $index = null)
    {
        if ($this->sheetWrapper->getObject() === null) {
            throw new \LogicException();
        }

        if ($index === null) {
            $this->sheetWrapper->increaseRow();
        } else {
            $this->sheetWrapper->setRow($index);
        }
    }

    /**
     * @throws \LogicException
     */
    public function end()
    {
        if ($this->sheetWrapper->getObject() === null) {
            throw new \LogicException();
        }

        $this->sheetWrapper->setColumn(null);
    }
}


1		<?php
2
3		namespace MyWheels\TwigSpreadsheetBundle\Wrapper;
4
5		/**
6		* Class SheetWrapper.
7		*/
8		class RowWrapper extends BaseWrapper
9		{
10		/**
11		* @var SheetWrapper
12		*/
13		protected $sheetWrapper;
14
15		/**
16		* RowWrapper constructor.
17		*
18		* @param array $context
19		* @param \Twig_Environment $environment
20		* @param SheetWrapper $sheetWrapper
21		*/
22	108	public function __construct(array $context, \Twig_Environment $environment, SheetWrapper $sheetWrapper)
23		{
24	108	parent::__construct($context, $environment);
25
26	108	$this->sheetWrapper = $sheetWrapper;
27	108	}
28
29		/**
30		* @param null\|int $index
31		*
32		* @throws \LogicException
33		*/
34	76	public function start(int $index = null)
35		{
36	76	if ($this->sheetWrapper->getObject() === null) {
37		throw new \LogicException();
38		}
39
40	76	if ($index === null) {
41	76	$this->sheetWrapper->increaseRow();
42		} else {
43	3	$this->sheetWrapper->setRow($index);
44		}
45	76	}
46
47		/**
48		* @throws \LogicException
49		*/
50	72	public function end()
51		{
52	72	if ($this->sheetWrapper->getObject() === null) {
53		throw new \LogicException();
54		}
55
56	72	$this->sheetWrapper->setColumn(null);
57	72	}
58		}
59

MewesK / TwigSpreadsheetBundle

GitHub Access Token became invalid

Pull Request — master (#23)

RowWrapper::start() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like