DefaultMethod   A
last analyzed

Complexity

Total Complexity 3

Size/Duplication

Total Lines 17
Duplicated Lines 0 %

Test Coverage

Coverage 100%

Importance

Changes 1
Bugs 0 Features 1
Metric Value
eloc 4
c 1
b 0
f 1
dl 0
loc 17
ccs 7
cts 7
cp 1
rs 10
wmc 3

3 Methods

Rating   Name   Duplication   Size   Complexity  
A __construct() 0 3 1
A getCode() 0 3 1
A __toArray() 0 4 1
1
<?php
2
3
declare(strict_types=1);
4
5
namespace Inspirum\Balikobot\Model\Method;
6
7
use Inspirum\Arrayable\BaseModel;
8
9
/**
10
 * @extends \Inspirum\Arrayable\BaseModel<string,mixed>
11
 */
12
final class DefaultMethod extends BaseModel implements Method
13
{
14 7
    public function __construct(
15
        private readonly string $code,
16
    ) {
17 7
    }
18
19 2
    public function getCode(): string
20
    {
21 2
        return $this->code;
22
    }
23
24
    /** @inheritDoc */
25 2
    public function __toArray(): array
26
    {
27 2
        return [
0 ignored issues
show
Bug Best Practice introduced by
The expression return array('code' => $this->code) returns the type array<string,string> which is incompatible with the return type mandated by Inspirum\Arrayable\BaseModel::__toArray() of Inspirum\Arrayable\TValue[].

In the issue above, the returned value is violating the contract defined by the mentioned interface.

Let's take a look at an example:

interface HasName {
    /** @return string */
    public function getName();
}

class Name {
    public $name;
}

class User implements HasName {
    /** @return string|Name */
    public function getName() {
        return new Name('foo'); // This is a violation of the ``HasName`` interface
                                // which only allows a string value to be returned.
    }
}
Loading history...
28 2
            'code' => $this->code,
29 2
        ];
30
    }
31
}
32