App\Components\BlocksByDayControl

Complexity

Total Complexity

6

Size/Duplication

Total Lines	78
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	render()	0	10	1
A	createComponentPrograms()	0	4	1
A	setProgramsControl()	0	6	1
A	getBlockModel()	0	4	1
A	setBlockModel()	0	6	1

<?php

namespace App\Components;

use App\Models\BlockModel;

class BlocksByDayControl extends BaseControl
{

	const TEMPLATE_NAME = 'BlocksByDay';

	/**
	 * @var ProgramsControl
	 */
	private $programs;

	/**
	 * @var BlockModel
	 */
	private $blockModel;

	/**
	 * @param BlockModel $model
	 */
	public function __construct(BlockModel $model, ProgramsControl $control)
	{
		$this->setBlockModel($model);
		$this->setProgramsControl($control);
	}

	/**
	 * @param  string $day
	 * @return void
	 */
	public function render($day)
	{
		$this->getBlockModel()->setMeetingId($this->getMeetingId());
		$blocks = $this->getBlockModel()->findByDay($day);

		$template = $this->getTemplate();
		$template->setFile($this->buildTemplatePath());
		$template->blocks = $blocks;

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

		$template->render();
	}

	/**
	 * @return ProgramsControl
	 */
	protected function createComponentPrograms()
	{
		return $this->programs;
	}

	/**
	 * @param  ProgramsControl $control
	 * @return $this
	 */
	protected function setProgramsControl(ProgramsControl $control)
	{
		$this->programs = $control;

		return $this;
	}

	/**
	 * @return BlockModel
	 */
	protected function getBlockModel()
	{
		return  $this->blockModel;
	}

	/**
	 * @param  BlockModel $model
	 * @return $this
	 */
	protected function setBlockModel(BlockModel $model)
	{
		$this->blockModel = $model;

		return $this;
	}

}