App\Presenters\RegistrationPresenter

Complexity

Total Complexity

38

Size/Duplication

Total Lines	458
Duplicated Lines	7.64 %

Coupling/Cohesion

Components	1
Dependencies	13

Importance

Changes	1
Bugs	1	Features	0

27 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	23	1
A	getRegistrationFormFactory()	0	4	1
A	injectRegistrationFormFactory()	0	4	1
A	startup()	0	22	3
A	actionCreate()	18	18	2
A	actionUpdate()	17	17	2
A	renderDefault()	0	13	3
A	renderCheck()	0	14	1
A	renderEdit()	0	15	2
B	createComponentRegistrationForm()	0	34	3
B	useLoggedVisitor()	0	29	3
A	getMealModel()	0	4	1
A	setMealModel()	0	6	1
A	getMeetingModel()	0	4	1
A	setMeetingModel()	0	6	1
A	getVisitorModel()	0	4	1
A	setVisitorModel()	0	6	1
A	getProgramModel()	0	4	1
A	setProgramModel()	0	6	1
A	getUserService()	0	4	1
A	setUserService()	0	6	1
A	getProgramService()	0	4	1
A	setProgramService()	0	6	1
A	getEventService()	0	4	1
A	setEventService()	0	6	1
A	getSettingsModel()	0	4	1
A	setSettingsModel()	0	6	1

<?php

namespace App\Presenters;

use DateTime;
use App\Entities\VisitorEntity;
use App\Models\MeetingModel;
use App\Models\VisitorModel;
use App\Models\ProgramModel;
use App\Models\MealModel;
use App\Services\SkautIS\UserService;
use App\Services\Emailer;
use App\Services\VisitorService;
use App\Services\ProgramService;
use Tracy\Debugger;
use App\Components\Forms\RegistrationForm;
use App\Components\Forms\Factories\IRegistrationFormFactory;
use App\Services\SkautIS\EventService;
use Skautis\Wsdl\WsdlException;
use App\Models\SettingsModel;

/**
 * Registration controller
 *
 * This file handles the registration of visitors
 *
 * @author Tomas Litera
 * @copyright 2013-06-12 <tomaslitera@hotmail.com>
 * @package srazvs
 */
class RegistrationPresenter extends VisitorPresenter
{

	/**
	 * @var VisitorModel
	 */
	private $visitorModel;

	/**
	 * @var ProgramModel
	 */
	private $programModel;

	/**
	 * @var UserService
	 */
	private $userService;

	/**
	 * @var ProgramService
	 */
	private $programService;

	/**
	 * @var SettingsModel
	 */
	protected $settingsModel;

	/**
	 * @var boolean
	 */
	private $disabled = false;

	/**
	 * @var IRegistrationFormFactory
	 */
	private $registrationFormFactory;

	/**
	 * @var EventService
	 */
	protected $skautisEventService;

	/**
	 * @param MeetingModel   $meetingModel
	 * @param UserService    $userService
	 * @param VisitorModel   $visitorModel
	 * @param MealModel      $mealModel
	 * @param ProgramModel   $programModel
	 * @param VisitorService $visitorService
	 * @param SettingsModel  $settingsModel
	 */
	public function __construct(
		MeetingModel $meetingModel,
		UserService $userService,
		VisitorModel $visitorModel,
		MealModel $mealModel,
		ProgramModel $programModel,
		Emailer $emailer,
		VisitorService $visitorService,
		ProgramService $programService,
		EventService $skautisEvent,
		SettingsModel $settingsModel
	) {
		$this->setMeetingModel($meetingModel);
		$this->setUserService($userService);
		$this->setVisitorModel($visitorModel);
		$this->setMealModel($mealModel);
		$this->setProgramModel($programModel);
		$this->setEmailer($emailer);
		$this->setVisitorService($visitorService);
		$this->setProgramService($programService);
		$this->setEventService($skautisEvent);
		$this->setSettingsModel($settingsModel);
	}

	/**
	 * @return IRegistrationFormFactory
	 */
	public function getRegistrationFormFactory(): IRegistrationFormFactory
	{
		return $this->registrationFormFactory;
	}

	/**
	 * Injector
	 *
	 * @param  IRegistrationFormFactory $factory
	 */

	protected $error = FALSE;

	protected $hash = NULL;
	private $item;
	private $mealData;
	private $user;

Consider using a different property name as you override a private property of the parent class.

	private $event;

	public function injectRegistrationFormFactory(IRegistrationFormFactory $factory)
	{
		$this->registrationFormFactory = $factory;
	}

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

		$this->getMeetingModel()->setMeetingId($this->getMeetingId());

		if($this->getDebugMode() || $this->getSettingsModel()->findDebugRegime()) {
			$this->getMeetingModel()->setRegistrationHandlers(1);
			$this->setMeetingId(1);
		} else {
			$this->getMeetingModel()->setRegistrationHandlers($this->getMeetingId());
		}

		//$this->user = $this->container->getService('userService');
		//$this->event = $this->container->getService('eventService');

		$template = $this->getTemplate();

		$template->page_title = "Registrace srazu VS";

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->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?

		$template->isRegistrationOpen = $this->getMeetingModel()->isRegOpen($this->getDebugMode());

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

	}

	/**
	 * Process data from form
	 *
	 * @return void
	 */
	public function actionCreate()

This method seems to be duplicated in your project.

	{
		try {
			$postData = $this->getHttpRequest()->getPost();
			$postData['meeting'] = $this->getMeetingId();

			$guid = $this->getVisitorService()->create($postData);
			$result = $this->sendRegistrationSummary($postData, $guid);

			Debugger::log('Creation of registration('. $guid .') successfull, result: ' . json_encode($result), Debugger::INFO);
			$this->flashMessage('Registrace(' . $guid . ') byla úspěšně založena.', self::FLASH_TYPE_OK);
		} catch(Exception $e) {

The class App\Presenters\Exception does not exist. Did you forget a USE statement, or did you not list all dependencies?

			Debugger::log('Creation of registration('. $guid .') failed, result: ' .  $e->getMessage(), Debugger::ERROR);

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

			$this->flashMessage('Creation of registration failed, result: ' . $e->getMessage(), self::FLASH_TYPE_ERROR);
		}

		$this->redirect('Registration:check', $guid);
	}

	/**
	 * @param  string  $guid
	 * @return void
	 */
	public function actionUpdate($guid)

This method seems to be duplicated in your project.

	{
		try {
			$postData = $this->getHttpRequest()->getPost();

			$result = $this->getVisitorService()->update($guid, $postData);
			$result = $this->sendRegistrationSummary($postData, $guid);

			Debugger::log('Modification of registration('. $guid .') successfull, result: ' . json_encode($result), Debugger::INFO);
			$this->flashMessage('Registrace(' . $guid . ') byla úspěšně upravena.', self::FLASH_TYPE_OK);
		} catch(Exception $e) {

The class App\Presenters\Exception does not exist. Did you forget a USE statement, or did you not list all dependencies?

			Debugger::log('Modification of registration('. $guid .') failed, result: ' .  $e->getMessage(), Debugger::ERROR);
			$this->flashMessage('Modification of registration(' . $guid . ') failed, result: ' . $e->getMessage(), self::FLASH_TYPE_ERROR);
		}

		$this->redirect('Registration:check', $guid);
	}

	/**
	 * @return void
	 */
	public function renderDefault()
	{
		$template = $this->getTemplate();

		////otevirani a uzavirani prihlasovani
		$disabled = $this->getMeetingModel()->isRegOpen($this->getDebugMode()) ? "" : "disabled";
		$template->disabled = $disabled;

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

		$template->loggedIn = $this->getUserService()->isLoggedIn();

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

		if($this->getUserService()->isLoggedIn()) {
			$this['registrationForm']->setDefaults(($this->useLoggedVisitor())->toArray());
		}
	}

	/**
	 * @param  string  $guid
	 * @return void
	 */
	public function renderCheck($guid)
	{
		$visitor = $this->getVisitorModel()->findByGuid($guid);

		$this->getMeetingModel()->setRegistrationHandlers($visitor->meeting);

		$template = $this->getTemplate();
		$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->visitor = $visitor;

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

		$template->meetingId = $visitor->meeting;

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

		$template->meals = $this->getMealModel()->findByVisitorId($visitor->id);

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

		$template->province = $this->getMeetingModel()->getProvinceNameById($visitor->province);

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

		$template->programs = $this->getProgramModel()->findByVisitorId($visitor->id);

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

	}

	/**
	 * @param  string $guid
	 * @return void
	 */
	public function renderEdit($guid)
	{
		$visitor = $this->getVisitorService()->findByGuid($guid);
		$meetingId = $visitor['meeting'];

		$this->getMeetingModel()->setRegistrationHandlers($meetingId);

		$template = $this->getTemplate();
		$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->meetingId = $meetingId;

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

		$template->loggedIn = $this->getUserService()->isLoggedIn();

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

		$template->disabled = $this->getMeetingModel()->isRegOpen($this->getDebugMode()) ? "" : "disabled";

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

		$this['registrationForm']->setDefaults($visitor);
	}

	/**
	 * @return RegistrationFormControl
	 */
	protected function createComponentRegistrationForm(): RegistrationForm
	{
		$control = $this->registrationFormFactory->create();
		$control->setMeetingId($this->getMeetingId());
		$control->onRegistrationSave[] = function(RegistrationForm $control, $newVisitor) {
			try {
				$guid = $this->getVisitorService()->create((array) $newVisitor);
/*
				if($this->getUserService()->isLoggedIn() && $this->getMeetingModel()->findCourseId()) {
					$this->getEventService()->insertEnroll(
						$this->getUserService()->getSkautis()->getUser()->getLoginId(),
						$this->getMeetingModel()->findCourseId(),
						// TODO: get real phone number
						'123456789'
					);
				}
*/
				$result = $this->sendRegistrationSummary((array) $newVisitor, $guid);

				Debugger::log('Storage of visitor('. $guid .') successfull, result: ' . json_encode($result), Debugger::INFO);
				$this->flashMessage('Účastník(' . $guid . ') byl úspěšně uložen.', self::FLASH_TYPE_OK);
			} catch(WsdlException $e) {
				Debugger::log('Storage of visitor('. $guid .') failed, result: ' . $e->getMessage(), Debugger::WARNING);
				$this->flashMessage('Uložení účastníka (' . $guid . ') selhalo. Účastník je již zaregistrován.', self::FLASH_TYPE_ERROR);
			} catch(Exception $e) {

The class App\Presenters\Exception does not exist. Did you forget a USE statement, or did you not list all dependencies?

				Debugger::log('Storage of visitor('. $guid .') failed, result: ' .  $e->getMessage(), Debugger::ERROR);
				$this->flashMessage('Uložení účastníka selhalo, chyba: ' . $e->getMessage(), self::FLASH_TYPE_ERROR);
			}

			$this->redirect('Registration:check', $guid);
		};

		return $control;
	}

	/**
	 * @return VisitorEntity
	 */
	protected function useLoggedVisitor(): VisitorEntity
	{
		$userDetail = $this->getUserService()->getUserDetail();
		$skautisUser = $this->getUserService()->getPersonalDetail($userDetail->ID_Person);
		$membership = $this->getUserService()->getPersonUnitDetail($userDetail->ID_Person);

		if(!preg_match('/^[1-9]{1}[0-9a-zA-Z]{2}\.[0-9a-zA-Z]{1}[0-9a-zA-Z]{1}$/', $membership->RegistrationNumber)) {
			$skautisUserUnit = $this->getUserService()->getParentUnitDetail($membership->ID_Unit)[0];
		} else {
			$skautisUserUnit = $this->getUserService()->getUnitDetail($membership->ID_Unit);
		}

		$visitor = new VisitorEntity;
		$visitor->name = $skautisUser->FirstName;
		$visitor->surname = $skautisUser->LastName;
		$visitor->nick = $skautisUser->NickName;
		$visitor->email = $skautisUser->Email;
		$visitor->street = $skautisUser->Street;
		$visitor->city = $skautisUser->City;
		$visitor->postal_code = preg_replace('/\s+/', '', $skautisUser->Postcode);
		$visitor->birthday = (new DateTime($skautisUser->Birthday))->format('d. m. Y');
		$visitor->group_name = $skautisUserUnit->DisplayName;
		$visitor->group_num = $skautisUserUnit->RegistrationNumber;
		if(isset($membership->Unit)) {
			$visitor->troop_name = $membership->Unit;
		}

		return $visitor;
	}

	/**
	 * @return MealModel
	 */
	protected function getMealModel()
	{
		return $this->mealModel;
	}

	/**
	 * @param  MealModel $model
	 * @return $this
	 */
	protected function setMealModel(MealModel $model)
	{
		$this->mealModel = $model;

		return $this;
	}

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

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

		return $this;
	}

	/**
	 * @return VisitorModel
	 */
	protected function getVisitorModel()
	{
		return $this->visitorModel;
	}

	/**
	 * @param  VisitorModel $model
	 * @return $this
	 */
	protected function setVisitorModel(VisitorModel $model)
	{
		$this->visitorModel = $model;

		return $this;
	}

	/**
	 * @return ProgramModel
	 */
	protected function getProgramModel()
	{
		return $this->programModel;
	}

	/**
	 * @param  ProgramModel $model
	 * @return $this
	 */
	protected function setProgramModel(ProgramModel $model)
	{
		$this->programModel = $model;

		return $this;
	}

	/**
	 * @return UserService
	 */
	protected function getUserService()
	{
		return $this->userService;
	}

	/**
	 * @param  UserService $service
	 * @return $this
	 */
	protected function setUserService(UserService $service)
	{
		$this->userService = $service;

		return $this;
	}

	/**
	 * @return ProgramService
	 */
	protected function getProgramService()
	{
		return $this->programService;
	}

	/**
	 * @param  ProgramService $service
	 * @return $this
	 */
	protected function setProgramService(ProgramService $service)
	{
		$this->programService = $service;

		return $this;
	}

	/**
	 * @return EventService
	 */
	protected function getEventService(): EventService
	{
		return $this->skautisEventService;
	}

	/**
	 * @param EventService $skautisEvent

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

	 *
	 * @return self
	 */
	protected function setEventService(EventService $service): self
	{
		$this->skautisEventService = $service;

		return $this;
	}


	/**
	 * @return SettingsModel
	 */
	public function getSettingsModel()
	{
		return $this->settingsModel;
	}

	/**
	 * @param SettingsModel $model
	 *
	 * @return self
	 */
	public function setSettingsModel(SettingsModel $model): self
	{
		$this->settingsModel = $model;

		return $this;
	}

}