DateTime::__construct() - Code Metrics - Inspection of "Merge pull request #3458 from PHPOffice/NumberForm..." - PHPOffice/PhpSpreadsheet - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 66b95b...1192d3 )

by Mark

created 2023-03-17 12:30 UTC

DateTime::__construct() A

↳ Parent: DateTime

Complexity

Conditions	2
Paths	1

Size

Total Lines	7
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	6
CRAP Score	2

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
eloc	4
c	1
b	0
f	0
dl	0
loc	7
rs	10
ccs	6
cts	6
cp	1
cc	2
nc	1
nop	2
crap	2

<?php

namespace PhpOffice\PhpSpreadsheet\Style\NumberFormat\Wizard;

class DateTime extends DateTimeWizard
{
    /**
     * @var string[]
     */
    protected array $separators;

    /**
     * @var array<DateTimeWizard|string>
     */
    protected array $formatBlocks;

    /**
     * @param null|string|string[] $separators
     *          If you want to use only a single format block, then pass a null as the separator argument
     * @param DateTimeWizard|string ...$formatBlocks
     */
    public function __construct($separators, ...$formatBlocks)
    {
        $this->separators = $this->padSeparatorArray(
            is_array($separators) ? $separators : [$separators],
            count($formatBlocks) - 1
        );
        $this->formatBlocks = array_map([$this, 'mapFormatBlocks'], $formatBlocks);
    }

    /**
     * @param DateTimeWizard|string $value
     */
    private function mapFormatBlocks($value): string
    {
        // Any date masking codes are returned as lower case values
        if (is_object($value)) {
            // We can't explicitly test for Stringable until PHP >= 8.0
            return $value;
        }

        // Wrap any string literals in quotes, so that they're clearly defined as string literals
        return $this->wrapLiteral($value);
    }

    public function format(): string
    {
        return implode('', array_map([$this, 'intersperse'], $this->formatBlocks, $this->separators));
    }
}


1		<?php
2
3		namespace PhpOffice\PhpSpreadsheet\Style\NumberFormat\Wizard;
4
5		class DateTime extends DateTimeWizard
6		{
7		/**
8		* @var string[]
9		*/
10		protected array $separators;
11
12		/**
13		* @var array<DateTimeWizard\|string>
14		*/
15		protected array $formatBlocks;
16
17		/**
18		* @param null\|string\|string[] $separators
19		* If you want to use only a single format block, then pass a null as the separator argument
20		* @param DateTimeWizard\|string ...$formatBlocks
21		*/
22	2	public function __construct($separators, ...$formatBlocks)
23		{
24	2	$this->separators = $this->padSeparatorArray(
25	2	is_array($separators) ? $separators : [$separators],
26	2	count($formatBlocks) - 1
27	2	);
28	2	$this->formatBlocks = array_map([$this, 'mapFormatBlocks'], $formatBlocks);
29		}
30
31		/**
32		* @param DateTimeWizard\|string $value
33		*/
34	2	private function mapFormatBlocks($value): string
35		{
36		// Any date masking codes are returned as lower case values
37	2	if (is_object($value)) {
38		// We can't explicitly test for Stringable until PHP >= 8.0
39	2	return $value;
40		}
41
42		// Wrap any string literals in quotes, so that they're clearly defined as string literals
43	2	return $this->wrapLiteral($value);
44		}
45
46	2	public function format(): string
47		{
48	2	return implode('', array_map([$this, 'intersperse'], $this->formatBlocks, $this->separators));
49		}
50		}
51

PHPOffice / PhpSpreadsheet

Push — master ( 66b95b...1192d3 )

DateTime::__construct() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like