RowWrapper::end() - Code Metrics - Inspection of "modify namespace, fix some deprecations, fix some..." - erelke/TwigSpreadsheetBundle - Measure and Improve Code Quality continuously with Scrutinizer

Test Setup Failed

Retried Inspection

Push — master ( 78f8b7...0f754d )

by Elemér

created 2019-07-02 14:10 UTC

RowWrapper::end() A

↳ Parent: RowWrapper

Complexity

Conditions	2
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	8
rs	10
c	0
b	0
f	0
cc	2
nc	2
nop	0

<?php

namespace Erelke\TwigSpreadsheetBundle\Wrapper;

use LogicException;
use Twig\Environment as Twig_Environment;

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

erelke / TwigSpreadsheetBundle

GitHub Access Token became invalid

Push — master ( 78f8b7...0f754d )

RowWrapper::end() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like