App\Repositories\ProgramRepository

Complexity

Total Complexity

20

Size/Duplication

Total Lines	185
Duplicated Lines	8.11 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

16 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	1
A	setMeetingId()	0	7	1
A	all()	0	4	1
A	find()	0	4	1
A	findByVisitorId()	0	4	1
A	findTutor()	0	4	1
A	findVisitors()	0	4	1
A	create()	0	6	1
A	update()	0	6	1
A	delete()	0	4	1
A	assembleFormPrograms()	15	15	3
A	transformDisplayInRegValue()	0	10	3
A	getProgramModel()	0	4	1
A	setProgramModel()	0	6	1
A	getVisitorModel()	0	4	1
A	setVisitorModel()	0	6	1

<?php

namespace App\Repositories;

use App\Models\ProgramModel;
use App\Models\VisitorModel;
use Nette\Database\Table\ActiveRow;
use Nette\Utils\ArrayHash;

class ProgramRepository
{

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

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

	/**
	 * @param VisitorModel $visitorModel
	 * @param ProgramModel $programModel
	 */
	public function __construct(
		VisitorModel $visitorModel,
		ProgramModel $programModel
	) {
		$this->setVisitorModel($visitorModel);
		$this->setProgramModel($programModel);
	}

	/**
	 * @param int $meetingId
	 */
	public function setMeetingId(int $meetingId): self
	{
		$this->getProgramModel()->setMeetingId($meetingId);
		$this->getVisitorModel()->setMeetingId($meetingId);

		return $this;
	}

	/**
	 * @return array
	 */
	public function all(): array
	{
		return $this->getProgramModel()->all();
	}

	/**
	 * @param  int  $id
	 * @return Nette\Database\Table\ActiveRow
	 */
	public function find(int $id): ActiveRow
	{
		return $this->getProgramModel()->find($id);
	}

	/**
	 * @param  int  $VisitorId
	 * @return array
	 */
	public function findByVisitorId(int $VisitorId): array

The parameter $VisitorId is not used and could be removed.

	{
		return $this->getProgramModels()->findByVisitorId($visitorId);

The variable $visitorId does not exist. Did you mean $VisitorId?

The method getProgramModels() does not exist on App\Repositories\ProgramRepository. Did you maybe mean getProgramModel()?

	}

	/**
	 * @param  int    $programId
	 * @return array
	 */
	public function findTutor(int $programId): ActiveRow
	{
		return $this->getProgramModel()->getTutor($programId);
	}

	/**
	 * @param  int   $programId
	 * @return array
	 */
	public function findVisitors(int $programId): array
	{
		return $this->getProgramModel()->findProgramVisitors($programId);
	}

	/**
	 * @param  Nette\Utils\ArrayHash $program
	 * @return boolean
	 */
	public function create(ArrayHash $program)
	{
		$program = $this->transformDisplayInRegValue($program);

		return $this->getProgramModel()->create((array) $program);
	}

	/**
	 * @param  Nette\Utils\ArrayHash $program
	 * @return boolean
	 */
	public function update(int $id, ArrayHash $program)
	{
		$program = $this->transformDisplayInRegValue($program);

		return $this->getProgramModel()->update($id, (array) $program);
	}

	/**
	 * @param  int    $id
	 * @return boolean
	 */
	public function delete(int $id)
	{
		return $this->getProgramModel()->delete($id);
	}

	/**
	 * @param  int    $visitorId
	 * @return array
	 */
	public function assembleFormPrograms(int $visitorId): array

This method seems to be duplicated in your project.

	{
		$visitorPrograms = $this->getVisitorModel()->findVisitorPrograms($visitorId);

		$formPrograms = [];

		foreach ($visitorPrograms as $visitorProgram) {
			if($visitorProgram->program !== 0) {

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

				$program = $this->getProgramModel()->findByProgramId($visitorProgram->program);

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

				$formPrograms['blck_' . $program->block] = $visitorProgram->program;

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

			}
		}

		return $formPrograms;
	}

	/**
	 * @param  Nette\Utils\ArrayHash $program
	 * @return Nette\Utils\ArrayHash
	 */
	protected function transformDisplayInRegValue(ArrayHash $program): ArrayHash
	{
		if(array_key_exists('display_in_reg', $program) && empty($program['display_in_reg'])) {
			$program['display_in_reg'] = '1';
		} else {
			$program['display_in_reg'] = '0';
		}

		return $program;
	}

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

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

		return $this;
	}

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

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

		return $this;
	}

}