Passed
Push — master ( cbdbb3...bd81e8 )
by Jhao
01:58
created

NormalizePhoneRequest::addPhone()   A

Complexity

Conditions 1
Paths 1

Size

Total Lines 8
Code Lines 4

Duplication

Lines 0
Ratio 0 %

Code Coverage

Tests 5
CRAP Score 1

Importance

Changes 1
Bugs 0 Features 0
Metric Value
cc 1
eloc 4
c 1
b 0
f 0
nc 1
nop 4
dl 0
loc 8
ccs 5
cts 5
cp 1
crap 1
rs 10
1
<?php
2
3
declare(strict_types=1);
4
5
namespace Appwilio\RussianPostSDK\Dispatching\Endpoints\Services\Requests;
6
7
use Appwilio\RussianPostSDK\Dispatching\DataAware;
8
use Appwilio\RussianPostSDK\Dispatching\Contracts\Arrayable;
9
10
final class NormalizePhoneRequest implements Arrayable
11
{
12
    use DataAware;
13
14 1
    public static function one(string $phone, ?string $area = null, ?string $place = null, ?string $region = null): self
15
    {
16 1
        $request = new self();
17
18 1
        $request->addPhone(...\func_get_args());
0 ignored issues
show
Bug introduced by
func_get_args() is expanded, but the parameter $phone of Appwilio\RussianPostSDK\...honeRequest::addPhone() does not expect variable arguments. ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-type  annotation

18
        $request->addPhone(/** @scrutinizer ignore-type */ ...\func_get_args());
Loading history...
19
20 1
        return $request;
21
    }
22
23 1
    public function addPhone(string $phone, ?string $area = null, ?string $place = null, ?string $region = null): void
24
    {
25 1
        $this->data[] = \array_merge(
26
            [
27 1
                'id'             => \sha1($phone),
28 1
                'original-phone' => $phone,
29
            ],
30 1
            \array_filter(\compact('area', 'place', 'region'))
31
        );
32 1
    }
33
34 1
    public function toArray(): array
35
    {
36 1
        return $this->data;
37
    }
38
}
39