ReaderOptions::setShouldPreserveEmptyRows() - Code Metrics - Inspection of "Refactor readers for better di" - box/spout - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — develop_3.0 (#457)

by Adrien

created 2017-08-26 21:44 UTC

ReaderOptions::setShouldPreserveEmptyRows() A

↳ Parent: ReaderOptions

Complexity

Conditions	1
Paths	1

Size

Total Lines	5
Code Lines	3

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes	1
Bugs	0	Features	0

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

<?php

namespace Box\Spout\Reader\Common;

/**
 * Class ReaderOptions
 * Readers' common options
 *
 * @package Box\Spout\Reader\Common
 */
class ReaderOptions
{
    /** @var bool Whether date/time values should be returned as PHP objects or be formatted as strings */
    protected $shouldFormatDates = false;

    /** @var bool Whether empty rows should be returned or skipped */
    protected $shouldPreserveEmptyRows = false;

    /**
     * @return bool Whether date/time values should be returned as PHP objects or be formatted as strings.
     */
    public function shouldFormatDates()
    {
        return $this->shouldFormatDates;
    }

    /**
     * Sets whether date/time values should be returned as PHP objects or be formatted as strings.
     *
     * @param bool $shouldFormatDates
     * @return ReaderOptions
     */
    public function setShouldFormatDates($shouldFormatDates)
    {
        $this->shouldFormatDates = $shouldFormatDates;
        return $this;
    }

    /**
     * @return bool Whether empty rows should be returned or skipped.
     */
    public function shouldPreserveEmptyRows()
    {
        return $this->shouldPreserveEmptyRows;
    }

    /**
     * Sets whether empty rows should be returned or skipped.
     *
     * @param bool $shouldPreserveEmptyRows
     * @return ReaderOptions
     */
    public function setShouldPreserveEmptyRows($shouldPreserveEmptyRows)
    {
        $this->shouldPreserveEmptyRows = $shouldPreserveEmptyRows;
        return $this;
    }
}


1			<?php
2
3			namespace Box\Spout\Reader\Common;
4
5			/**
6			* Class ReaderOptions
7			* Readers' common options
8			*
9			* @package Box\Spout\Reader\Common
10			*/
11			class ReaderOptions
12			{
13			/** @var bool Whether date/time values should be returned as PHP objects or be formatted as strings */
14			protected $shouldFormatDates = false;
15
16			/** @var bool Whether empty rows should be returned or skipped */
17			protected $shouldPreserveEmptyRows = false;
18
19			/**
20			* @return bool Whether date/time values should be returned as PHP objects or be formatted as strings.
21			*/
22			public function shouldFormatDates()
23			{
24			return $this->shouldFormatDates;
25			}
26
27			/**
28			* Sets whether date/time values should be returned as PHP objects or be formatted as strings.
29			*
30			* @param bool $shouldFormatDates
31			* @return ReaderOptions
32			*/
33			public function setShouldFormatDates($shouldFormatDates)
34			{
35			$this->shouldFormatDates = $shouldFormatDates;
36			return $this;
37			}
38
39			/**
40			* @return bool Whether empty rows should be returned or skipped.
41			*/
42			public function shouldPreserveEmptyRows()
43			{
44			return $this->shouldPreserveEmptyRows;
45			}
46
47			/**
48			* Sets whether empty rows should be returned or skipped.
49			*
50			* @param bool $shouldPreserveEmptyRows
51			* @return ReaderOptions
52			*/
53			public function setShouldPreserveEmptyRows($shouldPreserveEmptyRows)
54			{
55			$this->shouldPreserveEmptyRows = $shouldPreserveEmptyRows;
56			return $this;
57			}
58			}
59

box / spout

Pull Request — develop_3.0 (#457)

ReaderOptions::setShouldPreserveEmptyRows() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like