App\Presenters\AnnotationPresenter

Complexity

Total Complexity

9

Size/Duplication

Total Lines	124
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	injectAnnotationFormFactory()	0	4	1
A	renderEdit()	0	13	1
B	createComponentAnnotationForm()	0	25	2
A	getAnnotationService()	0	4	1
A	setAnnotationService()	0	6	1
A	getMeetingModel()	0	4	1
A	setMeetingModel()	0	6	1

<?php

namespace App\Presenters;

use App\Components\Forms\AnnotationForm;
use App\Components\Forms\Factories\IAnnotationFormFactory;
use App\Models\MeetingModel;
use App\Services\AnnotationService;
use \Exception;

class AnnotationPresenter extends BasePresenter
{

	/**
	 * @var IAnnotationFormFactory
	 */
	private $annotationFormFactory;

	/**
	 * @var AnnotationService
	 */
	private $annotationService;

	/**
	 * @var MeetingModel
	 */
	private $meetingModel;

	/**
	 * @param AnnotationService $service
	 * @param MeetingModel $model
	 */
	public function __construct(AnnotationService $service, MeetingModel $model)
	{
		$this->setAnnotationService($service);
		$this->setMeetingModel($model);
	}

	/**
	 * @param  IAnnotationFormFactory $factory
	 */
	public function injectAnnotationFormFactory(IAnnotationFormFactory $factory)
	{
		$this->annotationFormFactory = $factory;
	}

	/**
	 * @param  string $id

There is no parameter named $id. Was it maybe removed?

	 * @param  string $type
	 */
	public function renderEdit(string $guid, string $type)
	{
		$template = $this->getTemplate();

		$annotation = $this->getAnnotationService()->findByType($guid, $type);

		$template->guid = $guid;

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

		$template->annotation = $annotation;

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

		$template->meeting = $this->getMeetingModel()->find($this->getMeetingId());

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

		$template->program = $this->getAnnotationService()->findParentProgram($annotation);

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

$annotation is of type object<Nette\Database\Table\IRow>|false, but the function expects a object<Nette\Database\Table\ActiveRow>.

		$this['annotationForm']->setDefaults($annotation->toArray());
	}

	/**
	 * @return AnnotationFormControl
	 */
	protected function createComponentAnnotationForm(): AnnotationForm
	{
		$control = $this->annotationFormFactory->create();
		$control->setMeetingId($this->getMeetingId());
		$type = $this->getParameter('type');
		$control->onAnnotationSave[] = function(AnnotationForm $control, $annotation) use ($type) {
			try {
				$result = $this->getAnnotationService()->updateByType($type, $annotation);

				$this->logInfo('Modification of annotation id %s with data %s successfull, result: %s',	[
					$annotation->guid,
					json_encode($annotation),
					json_encode($result),
				]);

				$this->flashSuccess('Položka byla úspěšně upravena');
			} catch(Exception $e) {
				$this->logError("Modification of annotation id {$annotation->guid} failed, result: {$e->getMessage()}");

				$this->flashError("Modification of annotation id {$annotation->guid} failed, result: {$e->getMessage()}");
			}
		};

		return $control;
	}

	/**
	 * @return AnnotationService
	 */
	public function getAnnotationService(): AnnotationService
	{
		return $this->annotationService;
	}

	/**
	 * @param AnnotationService $annotationService
	 *
	 * @return self
	 */
	public function setAnnotationService(AnnotationService $annotationService): self
	{
		$this->annotationService = $annotationService;

		return $this;
	}

	/**
	 * @return MeetingModel
	 */
	public function getMeetingModel(): MeetingModel
	{
		return $this->meetingModel;
	}

	/**
	 * @param MeetingModel $meetingModel
	 *
	 * @return self
	 */
	public function setMeetingModel(MeetingModel $meetingModel): self
	{
		$this->meetingModel = $meetingModel;

		return $this;
	}

}