DateTimeWizard::wrapLiteral() - 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

DateTimeWizard::wrapLiteral() A

↳ Parent: DateTimeWizard

Complexity

Conditions	2
Paths	2

Size

Total Lines	8
Code Lines	3

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	4
CRAP Score	2

Importance

Changes	1
Bugs	0	Features	0

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

<?php

namespace PhpOffice\PhpSpreadsheet\Style\NumberFormat\Wizard;

abstract class DateTimeWizard implements Wizard
{
    protected const NO_ESCAPING_NEEDED = "$+-/():!^&'~{}<>= ";

    protected function padSeparatorArray(array $separators, int $count): array
    {
        $lastSeparator = array_pop($separators);

        return $separators + array_fill(0, $count, $lastSeparator);
    }

    protected function escapeSingleCharacter(string $value): string
    {
        if (strpos(self::NO_ESCAPING_NEEDED, $value) !== false) {
            return $value;
        }

        return "\\{$value}";
    }

    protected function wrapLiteral(string $value): string
    {
        if (mb_strlen($value, 'UTF-8') === 1) {
            return $this->escapeSingleCharacter($value);
        }

        // Wrap any other string literals in quotes, so that they're clearly defined as string literals
        return '"' . str_replace('"', '""', $value) . '"';
    }

    protected function intersperse(string $formatBlock, ?string $separator): string
    {
        return "{$formatBlock}{$separator}";
    }

    public function __toString(): string
    {
        return $this->format();
    }
}


1		<?php
2
3		namespace PhpOffice\PhpSpreadsheet\Style\NumberFormat\Wizard;
4
5		abstract class DateTimeWizard implements Wizard
6		{
7		protected const NO_ESCAPING_NEEDED = "$+-/():!^&'~{}<>= ";
8
9	14	protected function padSeparatorArray(array $separators, int $count): array
10		{
11	14	$lastSeparator = array_pop($separators);
12
13	14	return $separators + array_fill(0, $count, $lastSeparator);
14		}
15
16	2	protected function escapeSingleCharacter(string $value): string
17		{
18	2	if (strpos(self::NO_ESCAPING_NEEDED, $value) !== false) {
19	1	return $value;
20		}
21
22	1	return "\\{$value}";
23		}
24
25	5	protected function wrapLiteral(string $value): string
26		{
27	5	if (mb_strlen($value, 'UTF-8') === 1) {
28	2	return $this->escapeSingleCharacter($value);
29		}
30
31		// Wrap any other string literals in quotes, so that they're clearly defined as string literals
32	4	return '"' . str_replace('"', '""', $value) . '"';
33		}
34
35	14	protected function intersperse(string $formatBlock, ?string $separator): string
36		{
37	14	return "{$formatBlock}{$separator}";
38		}
39
40	14	public function __toString(): string
41		{
42	14	return $this->format();
43		}
44		}
45

PHPOffice / PhpSpreadsheet

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

DateTimeWizard::wrapLiteral() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like