App\Components\PublicBlockDetailControl

Complexity

Total Complexity

4

Size/Duplication

Total Lines	49
Duplicated Lines	100 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	4	4	1
A	render()	7	7	1
A	getBlockModel()	4	4	1
A	setBlockModel()	6	6	1

<?php

namespace App\Components;

use App\Models\BlockModel;

class PublicBlockDetailControl extends BaseControl

This class seems to be duplicated in your project.

{

	const TEMPLATE_NAME = 'PublicBlockDetail';

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

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

	/**
	 * @return void
	 */
	public function render($blockId)
	{
		$template = $this->getTemplate();
		$template->setFile($this->buildTemplatePath());
		$template->block = $this->getBlockModel()->find($blockId);

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

		$template->render();
	}

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

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

		return $this;
	}

}