literat / srazvs

app/presenters/VisitorPresenter.php

<?php

namespace App\Presenters;

use App\Components\Forms\Factories\IVisitorFormFactory;
use App\Components\Forms\VisitorForm;
use App\Models\MeetingModel;
use App\Services\Emailer;
use App\Repositories\VisitorRepository;
use Exception;

/**
 * Visitor controller
 *
 * This file handles the retrieval and serving of visitors
 *
 * @author Tomas Litera
 * @copyright 2013-06-12 <tomaslitera@hotmail.com>
 * @package srazvs
 */
class VisitorPresenter extends BasePresenter
{

	const TEMPLATE_DIR = __DIR__ . '/../templates/visitor/';
	const TEMPLATE_EXT = 'latte';
	const REDIRECT_DEFAULT = 'Visitor:listing';

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

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

	/**
	 * @var VisitorRepository
	 */
	protected $visitorRepository;

	/**
	 * @var IVisitorFormFactory
	 */
	private $visitorFormFactory;

	/**
	 * @param MeetingModel      $meetings
	 * @param Emailer           $emailer
	 * @param VisitorRepository $visitor
	 */
	public function __construct(
		MeetingModel $meetings,
		Emailer $emailer,
		VisitorRepository $visitor
	) {
		$this->setMeetingModel($meetings);
		$this->setEmailer($emailer);
		$this->setVisitorRepository($visitor);
	}

	/**
	 * @return IVisitorFormFactory
	 */
	public function getVisitorFormFactory(): IVisitorFormFactory
	{
		return $this->visitorFormFactory;
	}

	/**
	 * Injector
	 *
	 * @param  IVisitorFormFactory $factory
	 */
	public function injectVisitorFormFactory(IVisitorFormFactory $factory)
	{
		$this->visitorFormFactory = $factory;
	}

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

		$this->getMeetingModel()->setMeetingId($this->getMeetingId());
		$this->getVisitorRepository()->setMeeting($this->getMeetingId())
;	}

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

This method seems to be duplicated in your project.

	{
		try {
			$guid = $this->getVisitorRepository()->create($visitor);
			$result = $this->sendRegistrationSummary($visitor, $guid);

			$this->logInfo('Creation of visitor('. $guid .') successfull, result: ' . json_encode($result));
			$this->flashSuccess('Účastník(' . $guid . ') byl úspěšně vytvořen.');
		} catch(Exception $e) {
			$this->logError('Creation of visitor('. $guid .') failed, result: ' .  $e->getMessage());
			$this->flashError('Creation of visitor failed, result: ' . $e->getMessage());
		}

		return $result;

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

	}

	/**
	 * Process data from editing
	 *
	 * @param  integer 	$id
	 * @return void
	 */
	public function actionUpdate($id, $visitor)

This method seems to be duplicated in your project.

	{
		try {
			$result = $this->getVisitorRepository()->update($id, $visitor);
			$result = $this->sendRegistrationSummary($visitor, $id);

			$this->logInfo('Modification of visitor('. $id .') successfull, result: ' . json_encode($result));
			$this->flashSuccess('Účastník(' . $id . ') byl úspěšně upraven.');
		} catch(Exception $e) {
			$this->logError('Modification of visitor('. $id .') failed, result: ' .  $e->getMessage());
			$this->flashFailure('Modification of visitor failed, result: ' . $e->getMessage());
			$result = false;
		}

		return $result;
	}

	/**
	 * @param  int  $id
	 * @return void
	 */
	public function actionDelete($id)

This method seems to be duplicated in your project.

	{
		try {
			$result = $this->getVisitorRepository()->delete($id);

			$this->logInfo('Destroying of visitor('. $id .') successfull, result: ' . json_encode($result));
			$this->flashSuccess('Položka byla úspěšně smazána');
		} catch(Exception $e) {
			$this->logError('Destroying of visitor('. $id .') failed, result: ' .  $e->getMessage());
			$this->flashFailure('Destroying of visitor failed, result: ' . $e->getMessage());
		}

		$this->redirect(self::REDIRECT_DEFAULT);
	}

	/**
	 * Prepare mass mail form
	 *
	 * @return void
	 */
	public function actionSend()
	{
		try {
			$request = $this->getHttpRequest();
			$subject = $request->getPost('subject', '');
			$message = $request->getPost('message', '');
			$bcc = explode(',', preg_replace("/\s+/", "", $request->getPost('recipients', '')));
			// fill bcc name and address
			$bcc = array_combine($bcc, $bcc);

			$mailParameters = $this->getContainer()->parameters['mail'];
			$recipient = [
				$mailParameters['senderAddress'] => $mailParameters['senderName'],
			];

			$template = $this->createMailTemplate();
			$template->subject = $subject;

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

			$template->message = $message;

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

			$result = $this->getEmailer()->sendMail($recipient, $subject, (string) $template, $bcc);

			$this->logInfo('E-mail was send successfully, result: ' . json_encode($result));
			$this->flashSuccess('E-mail byl úspěšně odeslán');
		} catch(Exception $e) {
			$this->logError('Sending of e-mail failed, result: ' .  $e->getMessage());
			$this->flashFailure('Sending of e-mail failed, result: ' . $e->getMessage());
		}

		$this->redirect(self::REDIRECT_DEFAULT);
	}

	/**
	 * @param  integer|string $ids

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

	 * @return void
	 */
	public function actionPay($id)

This method seems to be duplicated in your project.

	{
		try {
			if(!$id) {
				$id = $this->getHttpRequest()->getPost('checker');
			}

			$visitor = $this->getVisitorRepository();
			$visitor->payCostCharge($id);
			$recipients = $visitor->findRecipients($id);
			$this->getEmailer()->sendPaymentInfo($recipients, 'cost');

			$this->logInfo('Visitor: Action pay cost for id %s executed successfully.', [json_encode($id)]);
			$this->flashSuccess('Platba byla zaplacena.');
		} catch(Exception $e) {
			$this->logError('Visitor: Action pay for id %s failed, result: %s', [
			    json_encode($id),
                $e->getMessage()
            ]);
			$this->flashFailure('Visitor: Action pay for id ' . $id . ' failed, result: ' . $e->getMessage());
		}

		$this->redirect(self::REDIRECT_DEFAULT);
	}

	/**
	 * @param  string|interger $ids

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

	 * @return void
	 */
	public function actionAdvance(int $id = null)

This method seems to be duplicated in your project.

	{
		try {
			if(!$id) {

The expression $id of type null|integer is loosely compared to false; this is ambiguous if the integer can be zero. You might want to explicitly use === null instead.

				$id = $this->getHttpRequest()->getPost('checker');
			}

			$visitor = $this->getVisitorRepository();
			$visitor->payAdvanceCharge($id);
			$recipients = $visitor->findRecipients($id);
			$this->getEmailer()->sendPaymentInfo($recipients, 'advance');

			$this->logInfo('Visitor: Action pay advance for id %s executed successfully.', [json_encode($id)]);
			$this->flashSuccess('Záloha byla zaplacena.');
		} catch(Exception $e) {
			$this->logError(
				'Visitor: Action pay advance for id %s failed, result: %s', [
					json_encode($id),
					$e->getMessage(),
				]);
			$this->flashFailure('Zaplacení zálohy pro účastníka s id '.json_encode($id).' neprošlo: '.$e->getMessage());
		}

		$this->redirect(self::REDIRECT_DEFAULT);
	}

	/**
	 * Set item as checked by id
	 *
	 * @param  integer $id
	 * @return void
	 */
	public function actionChecked($id)

This method seems to be duplicated in your project.

	{
		try {
			$result = $this->getVisitorRepository()->setChecked($id);
			$this->logInfo('Check of visitor('. $id .') successfull, result: ' . json_encode($result));
			$this->flashSuccess('Položka byla úspěšně zkontrolována');
		} catch(Exception $e) {
			$this->logError('Check of visitor('. $id .') failed, result: ' .  $e->getMessage());
			$this->flashFailure('Check of visitor failed, result: ' . $e->getMessage());
		}

		$this->redirect(self::REDIRECT_DEFAULT);
	}

	/**
	 * Set item as unchecked by id
	 *
	 * @param  integer $id
	 * @return void
	 */
	public function actionUnchecked($id)

This method seems to be duplicated in your project.

	{
		try {
			$result = $this->getVisitorRepository()->setUnchecked($id);
			$this->logInfo('Uncheck of visitor('. $id .') successfull, result: ' . json_encode($result));
			$this->flashSuccess('Položka byla nastavena jako nekontrolována');
		} catch(Exception $e) {
			$this->logError('Uncheck of visitor('. $id .') failed, result: ' .  $e->getMessage());
			$this->flashFailure('Uncheck of visitor failed, result: ' . $e->getMessage());
		}

		$this->redirect(self::REDIRECT_DEFAULT);
	}

	/**
	 * Prepare page for new item
	 *
	 * @return void
	 */
	public function renderNew()
	{
		$template = $this->getTemplate();
		$template->heading = 'nový účastník';

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)
	{
		$visitor = $this->getVisitorRepository()->findExpandedById($id);

		$template = $this->getTemplate();
		$template->heading = 'úprava účastníka';

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

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

	/**
	 * Prepare mass mail form
	 *
	 * @return void
	 */
	public function renderMail()
	{
		$ids = $this->getHttpRequest()->getPost('checker');

		$template = $this->getTemplate();
		$template->recipientMailAddresses = $this->getVisitorRepository()->getSerializedMailAddress($ids);

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

The method getSerializedMailAddress() does not seem to exist on object<App\Repositories\VisitorRepository>.

	}


	/**
	 * @return void
	 */
	public function renderListing()
	{
		$search = $this->getHttpRequest()->getQuery('search') ?: '';

		$visitorRepository = $this->getVisitorRepository();

		$template = $this->getTemplate();
		$template->render = $visitorRepository->findBySearch($search);

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

		$template->visitorCount = $visitorRepository->count();

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

		$template->meetingPrice	= $this->getMeetingModel()->getPrice('cost');

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

		$template->search = $search;

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

	}

	/**
	 * @return VisitorFormControl
	 */
	protected function createComponentVisitorForm(): VisitorForm
	{
		$control = $this->visitorFormFactory->create();
		$control->setMeetingId($this->getMeetingId());

		$control->onVisitorSave[] = function(VisitorForm $control, $visitor) {

This code seems to be duplicated across your project.

			$id = $this->getParameter('id');
			$this->setBacklinkFromArray($visitor);

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

			$this->redirect($this->getBacklink() ?: self::REDIRECT_DEFAULT);
		};

		$control->onVisitorReset[] = function(VisitorForm $control, $visitor) {
			$this->setBacklinkFromArray($visitor);

			$this->redirect($this->getBacklink() ?: self::REDIRECT_DEFAULT);
		};

		return $control;
	}

	/**
	 * @return Latte
	 */
	protected function createMailTemplate()
	{
		$template = $this->createTemplate();
		$template->setFile(
			sprintf(
				'%s%s.%s',
				self::TEMPLATE_DIR,
				'mail_body',
				self::TEMPLATE_EXT
			)
		);

		return $template;
	}

	/**
	 * @param  array   $visitor
	 * @return boolean
	 */
	protected function sendRegistrationSummary($visitor, $guid)
	{
		$recipient = [
			$visitor,
		];

		$code4bank = $this->getVisitorRepository()->calculateCode4Bank(
			$visitor['name'],
			$visitor['surname'],
			$visitor['birthday']->format('d. m. Y')
		);
		$result = $this->getEmailer()->sendRegistrationSummary($recipient, $guid, $code4bank);

		return $result;
	}

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

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

		return $this;
	}

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

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

		return $this;
	}

	/**
	 * @return VisitorRepository
	 */
	protected function getVisitorRepository(): VisitorRepository
	{
		return $this->visitorRepository;
	}

	/**
	 * @param  VisitorRepository $repository
	 * @return self
	 */
	protected function setVisitorRepository(VisitorRepository $repository): self
	{
		$this->visitorRepository = $repository;

		return $this;
	}

}