App\Components\AProgramOverviewControl

Complexity

Total Complexity

3

Size/Duplication

Total Lines	45
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	render()	0	11	1
A	createComponentBlocksByDay()	0	4	1
A	setBlocksBayControl()	0	6	1

<?php

namespace App\Components;

use App\Components\ProgramsControl;

abstract class AProgramOverviewControl extends BaseControl implements IProgramOverviewControl
{

	const TEMPLATE_NAME = '';

	/**
	 * @var BlocksByDayControl
	 */
	private $blocksByDay;

	/**
	 * @return void
	 */
	public function render()
	{
		$template = $this->getTemplate();
		$template->setFile($this->buildTemplatePath());
		$template->weekendDays = [

Accessing weekendDays on the interface Nette\Application\UI\ITemplate suggest that you code against a concrete implementation. How about adding an instanceof check?

			'pátek',
			'sobota',
			'neděle'
		];
		$template->render();
	}

	/**
	 * @return BlocksByDayControl
	 */
	protected function createComponentBlocksByDay(): IBlocksByDayControl
	{
		return $this->blocksByDay->setMeetingId($this->getMeetingId());
	}

	/**
	 * @param  BlocksByDayControl $control
	 * @return self
	 */
	protected function setBlocksBayControl(IBlocksByDayControl $control): self
	{
		$this->blocksByDay = $control;

$control is of type object<App\Components\IBlocksByDayControl>, but the property $blocksByDay was declared to be of type object<App\Components\BlocksByDayControl>. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

		return $this;
	}

}