RegistrationGraphControl - Code Metrics - literat/srazvs - Measure and Improve Code Quality continuously with Scrutinizer

RegistrationGraphControl A
last analyzed 2017-12-30 08:03 UTC

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	60
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	60
rs	10
wmc	4
lcom	1
cbo	3

4 Methods

Rating	Name	Size	Complexity
A	__construct()	4	1
A	render()	14	1
A	getExportModel()	4	1
A	setExportModel()	6	1

<?php

namespace App\Components;

use App\Models\ExportModel;

class RegistrationGraphControl extends BaseControl
{

	const TEMPLATE_NAME = 'RegistrationGraph';
	const GRAPH_WIDTH = 94;
	const GRAPH_HEIGHT_INIT = 0;
	const GRAPH_HEIGHT_STEP = 21.5;
	const GRAPH_HEIGHT_MIN = 310;

	/**
	 * @var ExportModel
	 */
	private $exportModel;

	/**
	 * @param ExportModel $model
	 */
	public function __construct(ExportModel $model)
	{
		$this->setExportModel($model);
	}

	/**
	 * @return void
	 */
	public function render()
	{
		$exportModel = $this->getExportModel();

		$template = $this->getTemplate();
		$template->setFile($this->buildTemplatePath());
		$template->imgDir = IMG_DIR;
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

function someFunction(SomeInterface $object) {
    if ($object instanceof SomeClass) {
        $a = $object->a;
    }
}
		$template->graph = $exportModel->graph();
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

function someFunction(SomeInterface $object) {
    if ($object instanceof SomeClass) {
        $a = $object->a;
    }
}
		$template->graphMax = $exportModel->graphMax();
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

function someFunction(SomeInterface $object) {
    if ($object instanceof SomeClass) {
        $a = $object->a;
    }
}
		$template->graphWidth = static::GRAPH_WIDTH;
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

function someFunction(SomeInterface $object) {
    if ($object instanceof SomeClass) {
        $a = $object->a;
    }
}
		$template->graphHeight = static::GRAPH_HEIGHT_INIT;
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

function someFunction(SomeInterface $object) {
    if ($object instanceof SomeClass) {
        $a = $object->a;
    }
}
		$template->graphHeightStep = static::GRAPH_HEIGHT_STEP;
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

function someFunction(SomeInterface $object) {
    if ($object instanceof SomeClass) {
        $a = $object->a;
    }
}
		$template->render();
	}

	/**
	 * @return ExportModel
	 */
	protected function getExportModel()
	{
		return $this->exportModel->setMeetingId($this->getMeetingId());
	}

	/**
	 * @param  ExportModel $model
	 * @return $this
	 */
	protected function setExportModel(ExportModel $model)
	{
		$this->exportModel = $model;

		return $this;
	}

}


1			<?php
2
3			namespace App\Components;
4
5			use App\Models\ExportModel;
6
7			class RegistrationGraphControl extends BaseControl
8			{
9
10			const TEMPLATE_NAME = 'RegistrationGraph';
11			const GRAPH_WIDTH = 94;
12			const GRAPH_HEIGHT_INIT = 0;
13			const GRAPH_HEIGHT_STEP = 21.5;
14			const GRAPH_HEIGHT_MIN = 310;
15
16			/**
17			* @var ExportModel
18			*/
19			private $exportModel;
20
21			/**
22			* @param ExportModel $model
23			*/
24			public function __construct(ExportModel $model)
25			{
26			$this->setExportModel($model);
27			}
28
29			/**
30			* @return void
31			*/
32			public function render()
33			{
34			$exportModel = $this->getExportModel();
35
36			$template = $this->getTemplate();
37			$template->setFile($this->buildTemplatePath());
38			$template->imgDir = IMG_DIR;
			0 ignored issues – show Bug introduced 2017-09-10 21:00 UTC by Report Bug Copy Issue Report Accessing `imgDir` on the interface `Nette\Application\UI\ITemplate` suggest that you code against a concrete implementation. How about adding an `instanceof` check? If you access a property on an interface, you most likely code against a concrete implementation of the interface. Available Fixes Adding an additional type check: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeInterface $object) { if ($object instanceof SomeClass) { $a = $object->a; } } Changing the type hint: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeClass $object) { $a = $object->a; } Loading history...
39			$template->graph = $exportModel->graph();
			0 ignored issues – show Bug introduced 2017-09-10 21:00 UTC by Report Bug Copy Issue Report Accessing `graph` on the interface `Nette\Application\UI\ITemplate` suggest that you code against a concrete implementation. How about adding an `instanceof` check? If you access a property on an interface, you most likely code against a concrete implementation of the interface. Available Fixes Adding an additional type check: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeInterface $object) { if ($object instanceof SomeClass) { $a = $object->a; } } Changing the type hint: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeClass $object) { $a = $object->a; } Loading history...
40			$template->graphMax = $exportModel->graphMax();
			0 ignored issues – show Bug introduced 2017-09-10 21:00 UTC by Report Bug Copy Issue Report Accessing `graphMax` on the interface `Nette\Application\UI\ITemplate` suggest that you code against a concrete implementation. How about adding an `instanceof` check? If you access a property on an interface, you most likely code against a concrete implementation of the interface. Available Fixes Adding an additional type check: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeInterface $object) { if ($object instanceof SomeClass) { $a = $object->a; } } Changing the type hint: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeClass $object) { $a = $object->a; } Loading history...
41			$template->graphWidth = static::GRAPH_WIDTH;
			0 ignored issues – show Bug introduced 2017-09-10 21:00 UTC by Report Bug Copy Issue Report Accessing `graphWidth` on the interface `Nette\Application\UI\ITemplate` suggest that you code against a concrete implementation. How about adding an `instanceof` check? If you access a property on an interface, you most likely code against a concrete implementation of the interface. Available Fixes Adding an additional type check: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeInterface $object) { if ($object instanceof SomeClass) { $a = $object->a; } } Changing the type hint: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeClass $object) { $a = $object->a; } Loading history...
42			$template->graphHeight = static::GRAPH_HEIGHT_INIT;
			0 ignored issues – show Bug introduced 2017-09-10 21:00 UTC by Report Bug Copy Issue Report Accessing `graphHeight` on the interface `Nette\Application\UI\ITemplate` suggest that you code against a concrete implementation. How about adding an `instanceof` check? If you access a property on an interface, you most likely code against a concrete implementation of the interface. Available Fixes Adding an additional type check: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeInterface $object) { if ($object instanceof SomeClass) { $a = $object->a; } } Changing the type hint: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeClass $object) { $a = $object->a; } Loading history...
43			$template->graphHeightStep = static::GRAPH_HEIGHT_STEP;
			0 ignored issues – show Bug introduced 2017-09-10 21:00 UTC by Report Bug Copy Issue Report Accessing `graphHeightStep` on the interface `Nette\Application\UI\ITemplate` suggest that you code against a concrete implementation. How about adding an `instanceof` check? If you access a property on an interface, you most likely code against a concrete implementation of the interface. Available Fixes Adding an additional type check: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeInterface $object) { if ($object instanceof SomeClass) { $a = $object->a; } } Changing the type hint: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeClass $object) { $a = $object->a; } Loading history...
44			$template->render();
45			}
46
47			/**
48			* @return ExportModel
49			*/
50			protected function getExportModel()
51			{
52			return $this->exportModel->setMeetingId($this->getMeetingId());
53			}
54
55			/**
56			* @param ExportModel $model
57			* @return $this
58			*/
59			protected function setExportModel(ExportModel $model)
60			{
61			$this->exportModel = $model;
62
63			return $this;
64			}
65
66			}
67

literat / srazvs

RegistrationGraphControl A last analyzed 2017-12-30 08:03 UTC

Complexity

Size/Duplication

Coupling/Cohesion

Importance

4 Methods

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like

RegistrationGraphControl A
last analyzed 2017-12-30 08:03 UTC