DefaultZipCode::getService() - Code Metrics - Inspection of "0d8fc3b581a23a0bbafd560f15fc9370bd567e8e" - inspirum/balikobot-php - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Branch — master (0d8fc3)

by Tomáš

created 2022-08-07 19:19 UTC

DefaultZipCode::getService() A

↳ Parent: DefaultZipCode

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	2
CRAP Score	1

Importance

Changes	1
Bugs	0	Features	1

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

<?php

declare(strict_types=1);

namespace Inspirum\Balikobot\Model\ZipCode;

use Inspirum\Arrayable\BaseModel;

/**
 * @extends \Inspirum\Arrayable\BaseModel<string,mixed>
 */
final class DefaultZipCode extends BaseModel implements ZipCode
{
    public function __construct(
        private string $carrier,
        private ?string $service,
        private ?string $zipCode,
        private ?string $zipCodeEnd,
        private ?string $city,
        private ?string $country,
        private bool $morningDelivery
    ) {
    }

    public function getCarrier(): string
    {
        return $this->carrier;
    }

    public function getService(): ?string
    {
        return $this->service;
    }

    public function getZipCode(): ?string
    {
        return $this->zipCode;
    }

    public function getZipCodeEnd(): ?string
    {
        return $this->zipCodeEnd;
    }

    public function getCity(): ?string
    {
        return $this->city;
    }

    public function getCountry(): ?string
    {
        return $this->country;
    }

    public function isMorningDelivery(): bool
    {
        return $this->morningDelivery;
    }

    /** @inheritDoc */
    public function __toArray(): array
    {
        return [
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.
    }
}
            'carrier'         => $this->carrier,
            'service'         => $this->service,
            'zipCode'         => $this->zipCode,
            'zipCodeEnt'      => $this->zipCodeEnd,
            'city'            => $this->city,
            'country'         => $this->country,
            'morningDelivery' => $this->morningDelivery,
        ];
    }
}


1		<?php
2
3		declare(strict_types=1);
4
5		namespace Inspirum\Balikobot\Model\ZipCode;
6
7		use Inspirum\Arrayable\BaseModel;
8
9		/**
10		* @extends \Inspirum\Arrayable\BaseModel<string,mixed>
11		*/
12		final class DefaultZipCode extends BaseModel implements ZipCode
13		{
14	7	public function __construct(
15		private string $carrier,
16		private ?string $service,
17		private ?string $zipCode,
18		private ?string $zipCodeEnd,
19		private ?string $city,
20		private ?string $country,
21		private bool $morningDelivery
22		) {
23		}
24
25	1	public function getCarrier(): string
26		{
27	1	return $this->carrier;
28		}
29
30	1	public function getService(): ?string
31		{
32	1	return $this->service;
33		}
34
35	1	public function getZipCode(): ?string
36		{
37	1	return $this->zipCode;
38		}
39
40	1	public function getZipCodeEnd(): ?string
41		{
42	1	return $this->zipCodeEnd;
43		}
44
45	1	public function getCity(): ?string
46		{
47	1	return $this->city;
48		}
49
50	1	public function getCountry(): ?string
51		{
52	1	return $this->country;
53		}
54
55	1	public function isMorningDelivery(): bool
56		{
57	1	return $this->morningDelivery;
58		}
59
60		/** @inheritDoc */
61	1	public function __toArray(): array
62		{
63		return [
		0 ignored issues – show Bug Best Practice introduced 2022-08-07 19:31 UTC by Report Bug Copy Issue Report The expression `return array('carrier' =...$this->morningDelivery)` returns the type `array<string,boolean\|null\|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...
64	1	'carrier' => $this->carrier,
65	1	'service' => $this->service,
66	1	'zipCode' => $this->zipCode,
67	1	'zipCodeEnt' => $this->zipCodeEnd,
68	1	'city' => $this->city,
69	1	'country' => $this->country,
70	1	'morningDelivery' => $this->morningDelivery,
71		];
72		}
73		}
74

inspirum / balikobot-php

Branch — master (0d8fc3)

DefaultZipCode::getService() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like