LineChart::getType() - Code Metrics - Inspection of "Improve QA" - cmen/CMENGoogleChartsBundle - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#38)

by Christophe

created 2019-12-06 15:35 UTC

LineChart::getType() A

↳ Parent: LineChart

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

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

<?php

namespace CMEN\GoogleChartsBundle\GoogleCharts\Charts;

use CMEN\GoogleChartsBundle\GoogleCharts\Chart;
use CMEN\GoogleChartsBundle\GoogleCharts\EventType;
use CMEN\GoogleChartsBundle\GoogleCharts\Options\LineChart\LineChartOptions;

/**
 * @author Christophe Meneses
 */
class LineChart extends Chart
{
    /**
     * @var LineChartOptions
     */
    protected $options;

    public function __construct()
    {
        parent::__construct();

        $this->options = new LineChartOptions();
    }

    public function getType()
    {
        return 'LineChart';
    }

    public function getPackage()
    {
        return 'corechart';
    }

    public function getAvailableEventTypes()
    {
        return [
            EventType::ANIMATION_FINISH,
            EventType::CLICK,
            EventType::ERROR,
            EventType::ON_MOUSE_OUT,
            EventType::ON_MOUSE_OVER,
            EventType::READY,
            EventType::SELECT,
        ];
    }

    /**
     * @return LineChartOptions
     */
    public function getOptions()
    {
        return $this->options;
    }

    /**
     * @param LineChartOptions $options
     *
     * @return LineChart
     */
    public function setOptions($options)
    {
        $this->options = $options;

        return $this;
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.
    }
}
    }
}


1		<?php
2
3		namespace CMEN\GoogleChartsBundle\GoogleCharts\Charts;
4
5		use CMEN\GoogleChartsBundle\GoogleCharts\Chart;
6		use CMEN\GoogleChartsBundle\GoogleCharts\EventType;
7		use CMEN\GoogleChartsBundle\GoogleCharts\Options\LineChart\LineChartOptions;
8
9		/**
10		* @author Christophe Meneses
11		*/
12		class LineChart extends Chart
13		{
14		/**
15		* @var LineChartOptions
16		*/
17		protected $options;
18
19	1	public function __construct()
20		{
21	1	parent::__construct();
22
23	1	$this->options = new LineChartOptions();
24	1	}
25
26	1	public function getType()
27		{
28	1	return 'LineChart';
29		}
30
31	1	public function getPackage()
32		{
33	1	return 'corechart';
34		}
35
36		public function getAvailableEventTypes()
37		{
38		return [
39		EventType::ANIMATION_FINISH,
40		EventType::CLICK,
41		EventType::ERROR,
42		EventType::ON_MOUSE_OUT,
43		EventType::ON_MOUSE_OVER,
44		EventType::READY,
45		EventType::SELECT,
46		];
47		}
48
49		/**
50		* @return LineChartOptions
51		*/
52	1	public function getOptions()
53		{
54	1	return $this->options;
55		}
56
57		/**
58		* @param LineChartOptions $options
59		*
60		* @return LineChart
61		*/
62		public function setOptions($options)
63		{
64		$this->options = $options;
65
66		return $this;
		0 ignored issues – show Bug Best Practice introduced 2019-12-06 15:40 UTC by Report Bug Copy Issue Report The expression `return $this` returns the type `CMEN\GoogleChartsBundle\...Charts\Charts\LineChart` which is incompatible with the return type mandated by `CMEN\GoogleChartsBundle\...rts\Chart::setOptions()` of `CMEN\GoogleChartsBundle\...s\ChartOptionsInterface`. 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...
67		}
68		}
69

cmen / CMENGoogleChartsBundle

Pull Request — master (#38)

LineChart::getType() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like