App\Components\MaterialsControl

Complexity

Total Complexity

4

Size/Duplication

Total Lines	49
Duplicated Lines	100 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes	1
Bugs	0	Features	0

4 Methods

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

<?php

namespace App\Components;

use App\Models\ExportModel;

class MaterialsControl extends BaseControl

This class seems to be duplicated in your project.

{

	const TEMPLATE_NAME = 'Materials';

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

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

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

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

		$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;
	}

}