TicketType::getDescription() - Code Metrics - Inspection of "Merge pull request #79 from conferencetools/ticket..." - conferencetools/tickets-module - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 7264d5...1976a0 )

by Chris

created 2017-12-09 12:43 UTC

TicketType::getDescription() A

↳ Parent: TicketType

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes

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

<?php

namespace ConferenceTools\Tickets\Domain\ValueObject;

use Doctrine\ORM\Mapping as ORM;
use JMS\Serializer\Annotation as Jms;

/**
 * Class TicketType
 * @package ConferenceTools\Tickets\Domain\ValueObject
 * @ORM\Embeddable
 */
final class TicketType
{
    /**
     * @var string
     * @Jms\Type("string")
     * @ORM\Column(type="string")
     */
    private $identifier;

    /**
     * @var Money
     * @ORM\Embedded(class="ConferenceTools\Tickets\Domain\ValueObject\Price")
     * @Jms\Type("ConferenceTools\Tickets\Domain\ValueObject\Price")
     */
    private $price;

    /**
     * @var string
     * @Jms\Type("string")
     * @ORM\Column(type="string")
     */
    private $displayName;

    /**
     * @var string
     * @Jms\Type("string")
     * @ORM\Column(type="string")
     */
    private $description;

    public function __construct(string $identifier, Price $price, string $displayName, string $description = '')
    {
        $this->identifier = $identifier;
        $this->price = $price;

        $this->displayName = $displayName;
        $this->description = $description;
    }

    public function getIdentifier(): string
    {
        return $this->identifier;
    }

    public function getPrice(): Price
    {
        return $this->price;

    }

    public function getDisplayName(): string
    {
        return $this->displayName;
    }

    public function getDescription(): string
    {
        return $this->description;
    }
}

1		<?php
2
3		namespace ConferenceTools\Tickets\Domain\ValueObject;
4
5		use Doctrine\ORM\Mapping as ORM;
6		use JMS\Serializer\Annotation as Jms;
7
8		/**
9		* Class TicketType
10		* @package ConferenceTools\Tickets\Domain\ValueObject
11		* @ORM\Embeddable
12		*/
13		final class TicketType
14		{
15		/**
16		* @var string
17		* @Jms\Type("string")
18		* @ORM\Column(type="string")
19		*/
20		private $identifier;
21
22		/**
23		* @var Money
24		* @ORM\Embedded(class="ConferenceTools\Tickets\Domain\ValueObject\Price")
25		* @Jms\Type("ConferenceTools\Tickets\Domain\ValueObject\Price")
26		*/
27		private $price;
28
29		/**
30		* @var string
31		* @Jms\Type("string")
32		* @ORM\Column(type="string")
33		*/
34		private $displayName;
35
36		/**
37		* @var string
38		* @Jms\Type("string")
39		* @ORM\Column(type="string")
40		*/
41		private $description;
42
43	11	public function __construct(string $identifier, Price $price, string $displayName, string $description = '')
44		{
45	11	$this->identifier = $identifier;
46	11	$this->price = $price;
		0 ignored issues – show Documentation Bug introduced 2017-11-05 16:12 UTC by Report Bug Copy Issue Report It seems like `$price` of type `ConferenceTools\Tickets\Domain\ValueObject\Price` is incompatible with the declared type `ConferenceTools\Tickets\Domain\ValueObject\Money` of property `$price`. Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property. Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property.. Loading history...
47	11	$this->displayName = $displayName;
48	11	$this->description = $description;
49	11	}
50
51	5	public function getIdentifier(): string
52		{
53	5	return $this->identifier;
54		}
55
56	13	public function getPrice(): Price
57		{
58	13	return $this->price;
		0 ignored issues – show Bug Best Practice introduced 2017-12-08 18:18 UTC by Report Bug Copy Issue Report The expression `return $this->price` returns the type `ConferenceTools\Tickets\Domain\ValueObject\Money` which is incompatible with the type-hinted return `ConferenceTools\Tickets\Domain\ValueObject\Price`. Loading history...
59		}
60
61		public function getDisplayName(): string
62		{
63		return $this->displayName;
64		}
65
66		public function getDescription(): string
67		{
68		return $this->description;
69		}
70		}

conferencetools / tickets-module

Push — master ( 7264d5...1976a0 )

TicketType::getDescription() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like