App\Presenters\ProgramPresenter

Complexity

Total Complexity

35

Size/Duplication

Total Lines	434
Duplicated Lines	3.23 %

Coupling/Cohesion

Components	1
Dependencies	8

Importance

Changes

0

26 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	17	1
A	injectProgramFormFactory()	0	4	1
A	startup()	0	9	1
B	actionCreate()	0	24	2
B	actionUpdate()	0	25	2
A	actionDelete()	0	20	2
B	actionMail()	0	24	2
A	renderPublic()	0	18	3
A	renderListing()	0	7	1
A	renderNew()	0	5	1
A	renderEdit()	0	12	1
B	createComponentProgramForm()	14	27	4
A	createComponentProgramOverview()	0	4	1
A	createComponentCategoryStyles()	0	4	1
A	createComponentProgramVisitors()	0	4	1
A	setProgramOverviewControl()	0	6	1
A	getEmailer()	0	4	1
A	setEmailer()	0	6	1
A	getBlockModel()	0	4	1
A	setBlockModel()	0	6	1
A	getMeetingModel()	0	4	1
A	setMeetingModel()	0	6	1
A	getProgramRepository()	0	4	1
A	setProgramRepository()	0	6	1
A	setCategoryStylesControl()	0	6	1
A	setProgramVisitorsControl()	0	6	1

<?php

namespace App\Presenters;

use App\Components\CategoryStylesControl;
use App\Components\Forms\Factories\IProgramFormFactory;
use App\Components\Forms\ProgramForm;
use App\Components\IProgramOverviewControl;
use App\Components\ProgramOverviewControl;
use App\Components\PublicProgramOverviewControl;
use App\Components\ProgramVisitorsControl;
use App\Models\BlockModel;
use App\Models\MeetingModel;
use App\Repositories\ProgramRepository;
use App\Services\Emailer;
use Nette\Utils\ArrayHash;
use Tracy\Debugger;
use Exception;

/**
 * Program controller
 *
 * This file handles the retrieval and serving of blocks
 *
 * @created 2013-06-05
 * @author Tomas Litera <tomaslitera@hotmail.com>
 */
class ProgramPresenter extends BasePresenter
{

	/**
	 * @var integer
	 */
	private $programId = null;

	/**
	 * @var Emailer
	 */
	private $emailer;

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

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

	/**
	 * @var ProgramRepository
	 */
	private $programRepository;

	/**
	 * @var IProgramFormFactory
	 */
	private $programFormFactory;

	/**
	 * @var ProgramOverviewControl
	 */
	private $programOverview;

	/**
	 * @var CategoryStylesControl
	 */
	private $categoryStyles;

	/**
	 * @var ProgramVisitorsControl
	 */
	private $programVisitors;

	/**
	 * Prepare model classes and get meeting id
	 */
	public function __construct(
		Emailer $emailer,
		BlockModel $blockModel,
		MeetingModel $meetingModel,
		ProgramRepository $programRepository,
		PublicProgramOverviewControl $publicProgramOverview,
		CategoryStylesControl $categoryStyles,
		ProgramVisitorsControl $programVisitors
	) {
		$this->setEmailer($emailer);
		$this->setBlockModel($blockModel);
		$this->setMeetingModel($meetingModel);
		$this->setProgramRepository($programRepository);
		$this->setProgramOverviewControl($publicProgramOverview);
		$this->setCategoryStylesControl($categoryStyles);
		$this->setProgramVisitorsControl($programVisitors);
	}

	/**
	 * @param  IProgramFormFactory $factory
	 */
	public function injectProgramFormFactory(IProgramFormFactory $factory)
	{
		$this->programFormFactory = $factory;
	}

	/**
	 * @return void
	 */
	public function startup()
	{
		parent::startup();

		$meetingId = $this->getMeetingId();
		$this->getProgramRepository()->setMeetingId($meetingId);
		$this->getMeetingModel()->setMeetingId($meetingId);
		$this->getBlockModel()->setMeetingId($meetingId);
	}

	/**
	 * Stores program into storage
	 *
	 * @param  Nette\Utils\ArrayHash  $program
	 * @return boolean
	 */
	public function actionCreate(ArrayHash $program)
	{
		try {
			$this->logInfo('Storing new program.');

			$result = $this->getProgramRepository()->create($program);

			$this->logInfo('Storing of new program with data %s successfull, result: %s', [
				json_encode($program),
				json_encode($result),
			]);

			$this->flashSuccess('Položka byla úspěšně vytvořena');
		} catch(Exception $e) {
			$this->logError('Creation of program with data %s failed, result: %s', [
				json_encode($data),

The variable $data does not exist. Did you forget to declare it?

				$e->getMessage()
			]);

			$this->flashError('Položku se nepodařilo vytvořit.');
		}

		return $result;

The variable $result does not seem to be defined for all execution paths leading up to this point.

	}

	/**
	 * Updates program in storage
	 *
	 * @param  int                    $id
	 * @param  Nette\Utils\ArrayHash  $program
	 * @return boolean
	 */
	public function actionUpdate(int $id, ArrayHash $program)
	{
		try {
			$this->logInfo('Updating program(%s).', [$id]);

			$result = $this->getProgramRepository()->update($id, $program);

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

			$this->flashSuccess('Položka byla úspěšně upravena');
		} catch(Exception $e) {
			$this->logError('Updating of program(%s) failed, result: %s', [
				$program->guid,
				$e->getMessage(),
			]);

			$this->flashError('Položku se nepodařilo upravit.');
		}

		return $result;

The variable $result does not seem to be defined for all execution paths leading up to this point.

	}

	/**
	 * @param  integer  $id
	 * @return void
	 */
	public function actionDelete($id)
	{
		try {
			$result = $this->getProgramRepository()->delete($id);

			$this->logInfo('Destroying of program successfull, result: %s', [
				json_encode($result)
			]);
			$this->flashSuccess('Položka byla úspěšně smazána.');
		} catch(Exception $e) {
			$this->logError('Destroying of program failed, result: %s', [
				$e->getMessage()
			]);
			$this->flashError('Smazání programu se nezdařilo, result: %s', [

The call to ProgramPresenter::flashError() has too many arguments starting with array($e->getMessage()).

				$e->getMessage()
			]);
		}

		$this->redirect('Program:listing');
	}

	/**
	 * @return void
	 */
	public function actionMail($id)
	{
		try {
			$tutors = $this->getProgramRepository()->findTutor($id);
			$recipients = $this->parseTutorEmail($tutors);

			$this->getEmailer()->tutor($recipients, $tutors->guid, 'program');

			$this->logInfo('Sending email to program tutor successfull, result: %s, %s', [
				json_encode($recipients),
				$tutors->guid
			]);
			$this->flashSuccess('Email lektorovi byl odeslán.');
		} catch(Exception $e) {
			$this->logError('Sending email to program tutor failed, result: %s', [
				$e->getMessage()
			]);
			$this->flashError('Email lektorovi nebyl odeslán, result: %s', [

The call to ProgramPresenter::flashError() has too many arguments starting with array($e->getMessage()).

				$e->getMessage()
			]);
		}

		$this->redirect('Program:edit', $id);
	}

	/**
	 * View public program
	 *
	 * @return void
	 */
	public function renderPublic()
	{
		$this->getMeetingModel()->setRegistrationHandlers($this->getMeetingId());

		$template = $this->getTemplate();
		$template->meeting_heading = $this->getMeetingModel()->getRegHeading();

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

			////otevirani a uzavirani prihlasovani
		if(($this->getMeetingModel()->getRegOpening() < time()) || $this->getDebugMode()) {
			$template->display_program = true;

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

		} else {
			$template->display_program = false;

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

		}
		$template->page_title = 'Srazy VS - veřejný program';

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

		$template->style = 'table { border-collapse:separate; width:100%; }

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

				td { .width:100%; text-align:center; padding:0px; }
				td.day { border:1px solid black; background-color:#777777; width:80px; }
				td.time { background-color:#cccccc; width:80px; }';
	}

	/**
	 * @return void
	 */
	public function renderListing()
	{
		$template = $this->getTemplate();
		$template->programs = $this->getProgramRepository()->all();

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

		$template->mid = $this->meetingId;

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

		$template->heading = $this->heading;

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

	}

	/**
	 * @return void
	 */
	public function renderNew()
	{
		$template = $this->getTemplate();
		$template->heading = 'nový program';

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

	}

	/**
	 * @param  integer $id
	 * @return void
	 */
	public function renderEdit($id)
	{
		$this->programId = $id;
		$program = $this->getProgramRepository()->find($id);

		$template = $this->getTemplate();
		$template->heading = 'úprava programu';

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

		$template->program = $program;

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

		$template->id = $id;

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

		$this['programForm']->setDefaults($program);
	}

	/**
	 * @return ProgramForm
	 */
	protected function createComponentProgramForm(): ProgramForm
	{
		$control = $this->programFormFactory->create();
		$control->setMeetingId($this->getMeetingId());
		$control->onProgramSave[] = function(ProgramForm $control, $program) {

This code seems to be duplicated across your project.

			//$guid = $this->getParameter('guid');
			$id = $this->getParameter('id');

			$this->setBacklinkFromArray($program);

			if($id) {
				$this->actionUpdate($id, $program);
			} else {
				$this->actionCreate($program);
			}

			$this->redirect($this->getBacklink() ?: 'Program:listing');
		};

		$control->onProgramReset[] = function(ProgramForm $control, $program) {
			$this->setBacklinkFromArray($program);

			$this->redirect($this->getBacklink() ?: 'Program:listing');
		};

		return $control;
	}

	/**
	 * @return AProgramOverviewControl
	 */
	protected function createComponentProgramOverview(): IProgramOverviewControl
	{
		return $this->programOverview->setMeetingId($this->getMeetingId());
	}

	/**
	 * @return CategoryStylesControl
	 */
	protected function createComponentCategoryStyles(): CategoryStylesControl
	{
		return $this->categoryStyles;
	}

	/**
	 * @return ProgramVisitorsControl
	 */
	protected function createComponentProgramVisitors(): ProgramVisitorsControl
	{
		return $this->programVisitors;
	}

	/**
	 * @param  ProgramOverviewControl $control
	 * @return $this
	 */
	protected function setProgramOverviewControl(IProgramOverviewControl $control)
	{
		$this->programOverview = $control;

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

		return $this;
	}

	/**
	 * @return Emailer
	 */
	protected function getEmailer(): Emailer
	{
		return $this->emailer;
	}

	/**
	 * @param  Emailer $emailer
	 * @return self
	 */
	protected function setEmailer(Emailer $emailer): self
	{
		$this->emailer = $emailer;

		return $this;
	}

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

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

		return $this;
	}

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

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

		return $this;
	}

	/**
	 * @return ProgramRepository
	 */
	protected function getProgramRepository(): ProgramRepository
	{
		return $this->programRepository;
	}

	/**
	 * @param  ProgramRepository $model

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

	 * @return $this
	 */
	protected function setProgramRepository(ProgramRepository $repository):self
	{
		$this->programRepository = $repository;

		return $this;
	}

	/**
	 * @param CategoryStylesControl $control
	 *
	 * @return self
	 */
	public function setCategoryStylesControl(CategoryStylesControl $control): self
	{
		$this->categoryStyles = $control;

		return $this;
	}

	/**
	 * @param  ProgramVisitorsControl $control
	 * @return self
	 */
	public function setProgramVisitorsControl(ProgramVisitorsControl $control): self
	{
		$this->programVisitors = $control;

		return $this;
	}

}