literat / srazvs

app/presenters/ProgramPresenter.php

Unused Use Statements -1 removed lines

@@ -14,7 +14,6 @@
 use App\Repositories\ProgramRepository;
 use App\Services\Emailer;
 use Nette\Utils\ArrayHash;
-use Tracy\Debugger;
 use Exception;
 
 /**

Doc Comments +4 added lines, -5 removed lines

@@ -118,7 +118,7 @@ 
 	/**
 	 * Stores program into storage
 	 *
-	 * @param  Nette\Utils\ArrayHash  $program
+	 * @param  ArrayHash  $program
 	 * @return boolean
 	 */
 	public function actionCreate(ArrayHash $program)
@@ -150,8 +150,8 @@ 
 	 * Updates program in storage
 	 *
 	 * @param  int                    $id
-	 * @param  Nette\Utils\ArrayHash  $program
-	 * @return boolean
+	 * @param  ArrayHash  $program
+	 * @return integer|null
 	 */
 	public function actionUpdate(int $id, ArrayHash $program)
 	{
@@ -325,7 +325,7 @@ 
 	}
 
 	/**
-	 * @return AProgramOverviewControl
+	 * @return IProgramOverviewControl
 	 */
 	protected function createComponentProgramOverview(): IProgramOverviewControl
 	{
@@ -425,7 +425,6 @@ 
 	}
 
 	/**
-	 * @param  ProgramRepository $model
 	 * @return $this
 	 */
 	protected function setProgramRepository(ProgramRepository $repository):self

app/models/ProgramModel.php

Spacing +25 added lines, -25 removed lines

@@ -65,7 +65,7 @@ 
 			->limit(10)
 			->fetchAll();
 
-		if(!$blocks){
+		if(!$blocks) {
 			$html = "";
 		} else {
 			/*
@@ -80,7 +80,7 @@ 
 
 			$checked_flag = false;
 			$html_input = "";
-			foreach($blocks as $data){
+			foreach($blocks as $data) {
 				// full program capacity with visitors
 				$fullProgramData = $this->database
 					->query('SELECT COUNT(visitor) AS visitors FROM `kk_visitor-program` AS visprog
@@ -94,21 +94,21 @@ 
 					->where('program ? AND visitor ?', $data['id'], $vid)
 					->fetch();
 
-				if($program){
+				if($program) {
 					$checked = "checked='checked'";
 					$checked_flag = true;
 				} else {
 					$checked = "";
 				}
 				// if the capacity is full
-				if($fullProgramData['visitors'] >= $data['capacity']){
-					$html_input .= "<input id='".$data['id'].$blockId."' ".$checked." disabled type='radio' name='blck_".$blockId."' value='".$data['id']."' />\n";
+				if($fullProgramData['visitors'] >= $data['capacity']) {
+					$html_input .= "<input id='" . $data['id'] . $blockId . "' " . $checked . " disabled type='radio' name='blck_" . $blockId . "' value='" . $data['id'] . "' />\n";
 					$fullProgramInfo = " (NELZE ZAPSAT - kapacita programu je již naplněna!)";
 				} else {
-					$html_input .= "<input id='".$data['id'].$blockId."' ".$checked." type='radio' name='blck_".$blockId."' value='".$data['id']."' /> \n";
+					$html_input .= "<input id='" . $data['id'] . $blockId . "' " . $checked . " type='radio' name='blck_" . $blockId . "' value='" . $data['id'] . "' /> \n";
 					$fullProgramInfo = "";
 				}
-				$html_input .= '<label for="'.$data['id'].$blockId.'">'.$data['name'].'</label>';
+				$html_input .= '<label for="' . $data['id'] . $blockId . '">' . $data['name'] . '</label>';
 				$html_input .= $fullProgramInfo;
 				$html_input .= "<br />\n";
 			}
@@ -117,7 +117,7 @@ 
 			if(!$checked_flag) $checked = "checked='checked'";
 			else $checked = "";
 
-			$html .= "<input ".$checked." type='radio' name='blck_".$blockId."' value='0' /> Nebudu přítomen <br />\n";
+			$html .= "<input " . $checked . " type='radio' name='blck_" . $blockId . "' value='0' /> Nebudu přítomen <br />\n";
 			$html .= $html_input;
 
 			$html .= "</div>\n";
@@ -138,7 +138,7 @@ 
 			$html = "";
 		} else {
 			$html = "<table>\n";
-			foreach($exportPrograms as $data){
+			foreach($exportPrograms as $data) {
 				$html .= "<tr>";
 				//// resim kapacitu programu a jeho naplneni navstevniky
 				$fullProgramData = $this->database
@@ -149,16 +149,16 @@ 
 							AND vis.deleted = ?',
 							$data['id'], '0')->fetch();
 
-				if($fullProgramData['visitors'] >= $data['capacity']){
+				if($fullProgramData['visitors'] >= $data['capacity']) {
 					//$html .= "<input disabled type='radio' name='".$id."' value='".$data['id']."' />\n";
-					$fullProgramInfo = "<span style='font-size:12px; font-weight:bold;'>".$fullProgramData['visitors']."/".$data['capacity']."</span> (kapacita programu je naplněna!)";
+					$fullProgramInfo = "<span style='font-size:12px; font-weight:bold;'>" . $fullProgramData['visitors'] . "/" . $data['capacity'] . "</span> (kapacita programu je naplněna!)";
 				}
 				else {
 					//$html .= "<input type='radio' name='".$id."' value='".$data['id']."' /> \n";
-					$fullProgramInfo = "<span style='font-size:12px; font-weight:bold;'>".$fullProgramData['visitors']."/".$data['capacity']."</span>";
+					$fullProgramInfo = "<span style='font-size:12px; font-weight:bold;'>" . $fullProgramData['visitors'] . "/" . $data['capacity'] . "</span>";
 				}
 				$html .= "<td style='min-width:270px;'>";
-				$html .= "<a rel='programDetail' href='".PRJ_DIR."program/?id=".$data['id']."&cms=edit&page=export' title='".$data['name']."'>".$data['name']."</a>\n";
+				$html .= "<a rel='programDetail' href='" . PRJ_DIR . "program/?id=" . $data['id'] . "&cms=edit&page=export' title='" . $data['name'] . "'>" . $data['name'] . "</a>\n";
 				$html .= "</td>";
 				$html .= "<td>";
 				$html .= $fullProgramInfo;
@@ -188,13 +188,13 @@ 
 			->order('day ASC, from ASC')
 			->fetchAll();
 
-		if(!$progSql){
+		if(!$progSql) {
 			$programs .= "<div class='emptyTable' style='width:400px;'>Nejsou žádná aktuální data.</div>\n";
 		} else {
 			//// prasarnicka kvuli programu raftu - resim obsazenost dohromady u dvou polozek
 			//$raftCountSql = "SELECT COUNT(visitor) AS raft FROM `kk_visitor-program` WHERE program='56|57'";
 
-			foreach($progSql as $progData){
+			foreach($progSql as $progData) {
 				//nemoznost volit predsnemovni dikusi
 				if($progData['id'] == 63) $notDisplayed = "style='display:none;'";
 				//obsazenost raftu
@@ -205,8 +205,8 @@ 
 				}
 				*/
 				else $notDisplayed = "";
-				$programs .= "<div ".$notDisplayed.">".$progData['day'].", ".$progData['from']." - ".$progData['to']." : ".$progData['name']."</div>\n";
-				if($progData['program'] == 1) $programs .= "<div ".$notDisplayed.">".$this->getExportPrograms($progData['id'])."</div>";
+				$programs .= "<div " . $notDisplayed . ">" . $progData['day'] . ", " . $progData['from'] . " - " . $progData['to'] . " : " . $progData['name'] . "</div>\n";
+				if($progData['program'] == 1) $programs .= "<div " . $notDisplayed . ">" . $this->getExportPrograms($progData['id']) . "</div>";
 				$programs .= "<br />";
 			}
 		}
@@ -307,20 +307,20 @@ 
 			->fetchAll();
 	}
 
-	public static function getPdfPrograms($id, $vid, $database){
+	public static function getPdfPrograms($id, $vid, $database) {
 		$result = $database
 			->table('kk_programs')
 			->where('block ? AND deleted ?', $id, '0')
 			->limit(10)
 			->fetchAll();
 
-		if(!$result){
+		if(!$result) {
 			$html = "";
 		} else {
 
 			$html = "<div class='program'>\n";
 
-			foreach($result as $data){
+			foreach($result as $data) {
 				$rows = $database
 					->table('kk_visitor-program')
 					->where('program ? AND visitor ?', $data->id, $vid)
@@ -333,7 +333,7 @@ 
 		return $html;
 	}
 
-	public function getProgramsLarge($id){
+	public function getProgramsLarge($id) {
 		$result = $this->database
 			->query('SELECT progs.name AS name,
 						cat.style AS style
@@ -347,8 +347,8 @@ 
 		else {
 			$html = "<table>";
 			$html .= " <tr>";
-			foreach($result as $data){
-				$html .= "<td class='category cat-".$data['style']."' >".$data['name']."</td>\n";
+			foreach($result as $data) {
+				$html .= "<td class='category cat-" . $data['style'] . "' >" . $data['name'] . "</td>\n";
 			}
 			$html .= " </tr>\n";
 			$html .= "</table>\n";
@@ -369,8 +369,8 @@ 
 
 		if(!$result) $html = "";
 		else {
-			foreach($result as $data){
-				$html .= $data['name'].",\n";
+			foreach($result as $data) {
+				$html .= $data['name'] . ",\n";
 			}
 		}
 		return $html;

Braces +28 added lines, -14 removed lines

@@ -114,8 +114,11 @@ 
 			}
 
 			// pokud uz jednou bylo zaskrtnuto, nezaskrtavam znovu
-			if(!$checked_flag) $checked = "checked='checked'";
-			else $checked = "";
+			if(!$checked_flag) {
+				$checked = "checked='checked'";
+			} else {
+				$checked = "";
+			}
 
 			$html .= "<input ".$checked." type='radio' name='blck_".$blockId."' value='0' /> Nebudu přítomen <br />\n";
 			$html .= $html_input;
@@ -152,8 +155,7 @@ 
 				if($fullProgramData['visitors'] >= $data['capacity']){
 					//$html .= "<input disabled type='radio' name='".$id."' value='".$data['id']."' />\n";
 					$fullProgramInfo = "<span style='font-size:12px; font-weight:bold;'>".$fullProgramData['visitors']."/".$data['capacity']."</span> (kapacita programu je naplněna!)";
-				}
-				else {
+				} else {
 					//$html .= "<input type='radio' name='".$id."' value='".$data['id']."' /> \n";
 					$fullProgramInfo = "<span style='font-size:12px; font-weight:bold;'>".$fullProgramData['visitors']."/".$data['capacity']."</span>";
 				}
@@ -196,7 +198,9 @@ 
 
 			foreach($progSql as $progData){
 				//nemoznost volit predsnemovni dikusi
-				if($progData['id'] == 63) $notDisplayed = "style='display:none;'";
+				if($progData['id'] == 63) {
+					$notDisplayed = "style='display:none;'";
+				}
 				//obsazenost raftu
 				/*
 				elseif($raftCountData['raft'] >= 18){
@@ -204,9 +208,13 @@ 
 					else $notDisplayed = "";
 				}
 				*/
-				else $notDisplayed = "";
+				else {
+					$notDisplayed = "";
+				}
 				$programs .= "<div ".$notDisplayed.">".$progData['day'].", ".$progData['from']." - ".$progData['to']." : ".$progData['name']."</div>\n";
-				if($progData['program'] == 1) $programs .= "<div ".$notDisplayed.">".$this->getExportPrograms($progData['id'])."</div>";
+				if($progData['program'] == 1) {
+					$programs .= "<div ".$notDisplayed.">".$this->getExportPrograms($progData['id'])."</div>";
+				}
 				$programs .= "<br />";
 			}
 		}
@@ -307,7 +315,8 @@ 
 			->fetchAll();
 	}
 
-	public static function getPdfPrograms($id, $vid, $database){
+	public static function getPdfPrograms($id, $vid, $database)
+	{
 		$result = $database
 			->table('kk_programs')
 			->where('block ? AND deleted ?', $id, '0')
@@ -325,7 +334,9 @@ 
 					->table('kk_visitor-program')
 					->where('program ? AND visitor ?', $data->id, $vid)
 					->fetchAll();
-				if($rows) $html .= $data['name'];
+				if($rows) {
+					$html .= $data['name'];
+				}
 			}
 			$html .= "</div>\n";
 		}
@@ -333,7 +344,8 @@ 
 		return $html;
 	}
 
-	public function getProgramsLarge($id){
+	public function getProgramsLarge($id)
+	{
 		$result = $this->database
 			->query('SELECT progs.name AS name,
 						cat.style AS style
@@ -343,8 +355,9 @@ 
 				LIMIT 10',
 				$id, '0')->fetchAll();
 
-		if(!$result) $html = "";
-		else {
+		if(!$result) {
+			$html = "";
+		} else {
 			$html = "<table>";
 			$html .= " <tr>";
 			foreach($result as $data){
@@ -367,8 +380,9 @@ 
 
 		$html = '';
 
-		if(!$result) $html = "";
-		else {
+		if(!$result) {
+			$html = "";
+		} else {
 			foreach($result as $data){
 				$html .= $data['name'].",\n";
 			}

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -219,7 +219,7 @@
 	 *
 	 * @return	string	html of a table
 	 */
-	public function getData($program_id = NULL)
+	public function getData($program_id = null)
 	{
 		if(isset($program_id)) {
 			$data = $this->database

app/components/Forms/RegistrationForm.php

Doc Comments -1 removed lines

@@ -25,7 +25,6 @@
 	protected $userService;
 
 	/**
-	 * @param ProvinceModel $model
 	 */
 	public function __construct(
 		ProvinceModel $province,

app/components/Forms/VisitorForm.php

Doc Comments +1 added lines, -1 removed lines

@@ -459,7 +459,7 @@
 
 	/**
 	 * @param  array  $programs
-	 * @return array
+	 * @return boolean
 	 */
 	protected function filterFilledCapacity(array $programs = []): array
 	{

Indentation +52 added lines, -52 removed lines

@@ -25,10 +25,10 @@ 
 	 */
 	public $onVisitorSave;
 
-    /**
-     * @var Closure
-     */
-    public $onVisitorReset;
+	/**
+	 * @var Closure
+	 */
+	public $onVisitorReset;
 
 	/**
 	 * @var ProvinceModel
@@ -60,13 +60,13 @@ 
 	 */
 	protected $programFields = [];
 
-    /**
-     * VisitorForm constructor.
-     * @param ProvinceModel     $province
-     * @param ProgramRepository $program
-     * @param BlockModel        $block
-     * @param MeetingModel      $meeting
-     */
+	/**
+	 * VisitorForm constructor.
+	 * @param ProvinceModel     $province
+	 * @param ProgramRepository $program
+	 * @param BlockModel        $block
+	 * @param MeetingModel      $meeting
+	 */
 	public function __construct(
 		ProvinceModel $province,
 		ProgramRepository $program,
@@ -128,15 +128,15 @@ 
 			->getLabelPrototype()->setAttribute('class', 'required');
 		$form->addEmail('email', 'E-mail:')
 			->setRequired(static::MESSAGE_REQUIRED)
-            ->setAttribute('size', 30)
+			->setAttribute('size', 30)
+			->getLabelPrototype()->setAttribute('class', 'required');
+		$form->addTbDatePicker('birthday', 'Datum narození:', null, 16)
+			->setRequired(static::MESSAGE_REQUIRED)
+			->setFormat('d.m.Y')
+			->setAttribute('placeholder', 'dd. mm. rrrr')
+			->setAttribute('id', 'birthday')
+			->setAttribute('class', 'datePicker')
 			->getLabelPrototype()->setAttribute('class', 'required');
-        $form->addTbDatePicker('birthday', 'Datum narození:', null, 16)
-            ->setRequired(static::MESSAGE_REQUIRED)
-            ->setFormat('d.m.Y')
-            ->setAttribute('placeholder', 'dd. mm. rrrr')
-            ->setAttribute('id', 'birthday')
-            ->setAttribute('class', 'datePicker')
-            ->getLabelPrototype()->setAttribute('class', 'required');
 		$form->addText('street', 'Ulice:', 30)
 			->setRequired(static::MESSAGE_REQUIRED)
 			->addRule(Form::MAX_LENGTH, static::MESSAGE_MAX_LENGTH, 30)
@@ -179,53 +179,53 @@ 
 			->setAttribute('placeholder', 'Vaše nabídka na sdílení dobré praxe (co u vás umíte dobře a jste ochotni se o to podělit)');
 		$form->addTextArea('question2', 'Počet a typy lodí:', 50, 8)
 			->setAttribute('placeholder', 'Počet a typy lodí, které sebou přivezete (vyplňte pokud ano)');
-        $form->addText('bill', 'Zaplaceno:', 30)
-            ->setDefaultValue(0);
-        $form->addText('cost', 'Poplatek:', 30)
-            ->setDefaultValue($this->getMeetingModel()->getPrice('cost'));
+		$form->addText('bill', 'Zaplaceno:', 30)
+			->setDefaultValue(0);
+		$form->addText('cost', 'Poplatek:', 30)
+			->setDefaultValue($this->getMeetingModel()->getPrice('cost'));
 
 		$form = $this->buildProgramSwitcher($form);
 
 		$form->addHidden('mid', $this->getMeetingId());
-        $form->addHidden('meeting', $this->getMeetingId());
+		$form->addHidden('meeting', $this->getMeetingId());
 		$form->addHidden('backlink');
 
-        $form->addSubmit('save', 'Uložit')
-            ->setAttribute('class', 'btn-primary')
-            ->onClick[] = [$this, 'processSave'];
-        $form->addSubmit('reset', 'Storno')
-            ->setAttribute('class', 'btn-reset')
-            ->onClick[] = [$this, 'processReset'];
+		$form->addSubmit('save', 'Uložit')
+			->setAttribute('class', 'btn-primary')
+			->onClick[] = [$this, 'processSave'];
+		$form->addSubmit('reset', 'Storno')
+			->setAttribute('class', 'btn-reset')
+			->onClick[] = [$this, 'processReset'];
 
 
-        $form = $this->setupRendering($form);
+		$form = $this->setupRendering($form);
 
 		$form->onSuccess[] = [$this, 'processForm'];
 
 		return $form;
 	}
 
-    /**
-     * @param  SubmitButton $button
-     * @return void
-     */
-    public function processSave(SubmitButton $button)
-    {
-        $visitor = $button->getForm()->getValues();
-
-        $this->onVisitorSave($this, $visitor);
-    }
-
-    /**
-     * @param  SubmitButton $button
-     * @return void
-     */
-    public function processReset(SubmitButton $button)
-    {
-        $visitor = $button->getForm()->getValues();
-
-        $this->onVisitorReset($this, $visitor);
-    }
+	/**
+	 * @param  SubmitButton $button
+	 * @return void
+	 */
+	public function processSave(SubmitButton $button)
+	{
+		$visitor = $button->getForm()->getValues();
+
+		$this->onVisitorSave($this, $visitor);
+	}
+
+	/**
+	 * @param  SubmitButton $button
+	 * @return void
+	 */
+	public function processReset(SubmitButton $button)
+	{
+		$visitor = $button->getForm()->getValues();
+
+		$this->onVisitorReset($this, $visitor);
+	}
 
 	/**
 	 * @param  Form   $form

Spacing +7 added lines, -7 removed lines

@@ -235,7 +235,7 @@ 
 	{
 		$programBlocks = $this->fetchProgramBlocks();
 
-		foreach ($programBlocks as $block) {
+		foreach($programBlocks as $block) {
 
 			$programsInBlock = $this->getProgramRepository()->findByBlockId($block->id);
 
@@ -243,13 +243,13 @@ 
 				0 => 'Nebudu přítomen'
 			];
 
-			foreach ($programsInBlock as $program) {
+			foreach($programsInBlock as $program) {
 				$programs[$program->id] = $program->name;
 			}
 
 			$form->addRadioList(
 				'blck_' . $block->id,
-				$block->day . ', ' . $block->from .' - ' . $block->to .' : ' . $block->name,
+				$block->day . ', ' . $block->from . ' - ' . $block->to . ' : ' . $block->name,
 				$programs
 			)->setDefaultValue(0)
 			->setDisabled($this->filterFilledCapacity($programs));
@@ -269,7 +269,7 @@ 
 			'ano' => 'ano',
 		];
 
-		foreach ($this->fetchMeals() as $name => $label) {
+		foreach($this->fetchMeals() as $name => $label) {
 			$this->setMealField($name);
 			$form->addSelect($name, $label . ':', $yesNoArray);
 		}
@@ -400,7 +400,7 @@ 
 	{
 		$programBlocks = $this->fetchProgramBlocks();
 
-		foreach ($programBlocks as $block) {
+		foreach($programBlocks as $block) {
 			$programFieldName = 'blck_' . $block->id;
 			$this->setProgramField($programFieldName);
 		}
@@ -415,7 +415,7 @@ 
 	{
 		$meals = $this->fetchMeals();
 
-		foreach ($meals as $name => $label) {
+		foreach($meals as $name => $label) {
 			$this->setMealField($name);
 		}
 
@@ -465,7 +465,7 @@ 
 	{
 		return array_keys(
 			array_filter($programs, function($name, $id) {
-				if ($id) {
+				if($id) {
 					$visitorsOnProgram = $this->getProgramRepository()->countVisitors($id);
 					$programCapacity = $this->getProgramRepository()->find($id)->capacity;
 

app/models/VisitorModel.php

Doc Comments +17 added lines, -15 removed lines

@@ -204,11 +204,9 @@ 
 	/**
 	 * Modify a visitor
 	 *
-	 * @param	int		$visitor_id		ID of a visitor
-	 * @param	array	$db_data		Visitor's database data
-	 * @param	array	$meals_data		Data of meals
-	 * @param	array	$programs_data	Program's data
-	 * @return	mixed					TRUE or array of errors
+	 * @param	int		$visitorId		ID of a visitor
+	 * @param	array	$programs	Program's data
+	 * @return	integer					TRUE or array of errors
 	 */
 	public function modify(int $visitorId, array $visitor, array $meals, array $programs)
 	{
@@ -246,10 +244,8 @@ 
 	/**
 	 * Modify a visitor
 	 *
-	 * @param	int		$visitor_id		ID of a visitor
-	 * @param	array	$db_data		Visitor's database data
-	 * @param	array	$meals_data		Data of meals
-	 * @param	array	$programs_data	Program's data
+	 * @param	int		$visitor		ID of a visitor
+	 * @param	array	$programs	Program's data
 	 * @return	mixed					TRUE or array of errors
 	 */
 	public function modifyByGuid($guid, $visitor, $meals, $programs)
@@ -320,7 +316,8 @@ 
 	 * Delete one or multiple record/s
 	 *
 	 * @param	int		ID/s of record
-	 * @return	boolean
+	 * @param integer $id
+	 * @return	integer|null
 	 */
 	public function delete($id)
 	{
@@ -337,7 +334,9 @@ 
 	 *
 	 * @param	int		ID/s of record
 	 * @param 	int 	0 | 1
-	 * @return	boolean
+	 * @param integer $id
+	 * @param string $value
+	 * @return	integer|null
 	 */
 	public function checked($id, $value)
 	{
@@ -407,7 +406,9 @@ 
 	 *
 	 * @param	int	ID/s of visitor
 	 * @param	string	type of payment (pay | advance)
-	 * @return	string	error message or true
+	 * @param integer $ids
+	 * @param string $type
+	 * @return	integer|null	error message or true
 	 */
 	public function payCharge($ids, $type)
 	{
@@ -431,7 +432,8 @@ 
 	 * Get recipients by ids
 	 *
 	 * @param	mixed	ID of visitor
-	 * @return	mixed	result
+	 * @param integer $ids
+	 * @return	\Nette\Database\IRow[]	result
 	 */
 	public function getRecipients($ids)
 	{
@@ -457,7 +459,7 @@ 
 	}
 
 	/**
-	 * @return Row
+	 * @return \Nette\Database\IRow[]
 	 */
 	public function all()
 	{
@@ -523,7 +525,7 @@ 
 
 	/**
 	 * @param	int		ID of visitor
-	 * @return	mixed	result
+	 * @return	\Nette\Database\IRow[]	result
 	 */
 	public function findVisitorPrograms(int $visitorId)
 	{

Unused Use Statements -2 removed lines

@@ -3,10 +3,8 @@
 namespace App\Models;
 
 use Nette\Database\Context;
-use Nette\Utils\Strings;
 use \Exception;
 use DateTime;
-use Tracy\Debugger;
 
 /**
  * Visitor

Spacing +3 added lines, -3 removed lines

@@ -163,7 +163,7 @@ 
 		// visitor's id is empty and i must add one
 		$meals_data['visitor'] = $ID_visitor;
 
-		if($ID_visitor){
+		if($ID_visitor) {
 			// gets data from database
 			$program_blocks = $this->Blocks->getProgramBlocks($DB_data['meeting']);
 
@@ -449,7 +449,7 @@ 
 	 */
 	protected function convertToDateTime($datetime): DateTime
 	{
-		if (is_string($datetime)) {
+		if(is_string($datetime)) {
 			$datetime = new DateTime($datetime);
 		}
 
@@ -499,7 +499,7 @@ 
 			->group('email')
 			->fetchAll();
 
-		foreach($emails as $item){
+		foreach($emails as $item) {
 			$recipientMailAddresses .= $item['email'] . ",\n";
 		}
 

app/presenters/BasePresenter.php

Unused Use Statements +2 added lines, -3 removed lines

@@ -2,10 +2,9 @@
 
 namespace App\Presenters;
 
-use Nette,
-	App\Model;
+use Nette;
+use App\Model;
 use Nette\Utils\ArrayHash;
-use Nette\Utils\Strings;
 use Nette\Http\Request;
 use App\Models\SunlightModel;
 use Nette\Caching\Cache;

Indentation +13 added lines, -13 removed lines

@@ -480,19 +480,19 @@
 		}
 	}
 
-    /**
-     * @param  Nette\Utils\ArrayHash $array
-     * @return self
-     */
-    protected function setBacklinkFromArray(ArrayHash $array): self
-    {
-        if(array_key_exists('backlink', $array) && !empty($array['backlink'])) {
-            $this->setBacklink($array['backlink']);
-            unset($array['backlink']);
-        }
-
-        return $this;
-    }
+	/**
+	 * @param  Nette\Utils\ArrayHash $array
+	 * @return self
+	 */
+	protected function setBacklinkFromArray(ArrayHash $array): self
+	{
+		if(array_key_exists('backlink', $array) && !empty($array['backlink'])) {
+			$this->setBacklink($array['backlink']);
+			unset($array['backlink']);
+		}
+
+		return $this;
+	}
 
 	/**
 	 * Flashes success message

app/presenters/RegistrationPresenter.php

Doc Comments +3 added lines, -4 removed lines

@@ -145,7 +145,7 @@ 
 	/**
 	 * Process data from form
 	 *
-	 * @return void
+	 * @return string
 	 */
 	public function actionCreate($visitor)
 	{
@@ -170,7 +170,7 @@ 
 
 	/**
 	 * @param  string  $guid
-	 * @return void
+	 * @return string
 	 */
 	public function actionUpdate($guid, $visitor)
 	{
@@ -261,7 +261,7 @@ 
 	}
 
 	/**
-	 * @return RegistrationFormControl
+	 * @return RegistrationForm
 	 */
 	protected function createComponentRegistrationForm(): RegistrationForm
 	{
@@ -390,7 +390,6 @@ 
 	}
 
 	/**
-	 * @param EventService $skautisEvent
 	 *
 	 * @return self
 	 */

Unused Use Statements -1 removed lines

@@ -15,7 +15,6 @@
 use App\Components\Forms\Factories\IRegistrationFormFactory;
 use App\Services\SkautIS\EventService;
 use Nette\Utils\ArrayHash;
-use Skautis\Wsdl\WsdlException;
 use App\Models\SettingsModel;
 
 /**

Indentation +33 added lines, -33 removed lines

@@ -55,20 +55,20 @@ 
 	 */
 	protected $skautisEventService;
 
-    /**
-     * @var MealModel
-     */
+	/**
+	 * @var MealModel
+	 */
 	private $mealModel;
 
-    /**
-     * @var ProgramRepository
-     */
+	/**
+	 * @var ProgramRepository
+	 */
 	private $programRepository;
 
-    protected $error = FALSE;
-    protected $hash = NULL;
-    private $user;
-    private $event;
+	protected $error = FALSE;
+	protected $hash = NULL;
+	private $user;
+	private $event;
 
 	/**
 	 * @param MeetingModel       $meetingModel
@@ -107,10 +107,10 @@ 
 	}
 
 	/**
-     * Injector
-     *
-     * @param  IRegistrationFormFactory $factory
-     */
+	 * Injector
+	 *
+	 * @param  IRegistrationFormFactory $factory
+	 */
 	public function injectRegistrationFormFactory(IRegistrationFormFactory $factory)
 	{
 		$this->registrationFormFactory = $factory;
@@ -268,7 +268,7 @@ 
 		$control = $this->registrationFormFactory->create();
 		$control->setMeetingId($this->getMeetingId());
 		$control->onRegistrationSave[] = function(RegistrationForm $control, $visitor) {
-		    $guid = $this->getParameter('guid');
+			$guid = $this->getParameter('guid');
 
 			if($guid) {
 				$guid = $this->actionUpdate($guid, $visitor);
@@ -422,23 +422,23 @@ 
 		return $this;
 	}
 
-    /**
-     * @return ProgramRepository
-     */
-    protected function getProgramRepository(): ProgramRepository
-    {
-        return $this->programRepository;
-    }
-
-    /**
-     * @param  ProgramRepository $repository
-     * @return RegistrationPresenter
-     */
-    protected function setProgramRepository(ProgramRepository $repository): self
-    {
-        $this->programRepository = $repository;
-
-        return $this;
-    }
+	/**
+	 * @return ProgramRepository
+	 */
+	protected function getProgramRepository(): ProgramRepository
+	{
+		return $this->programRepository;
+	}
+
+	/**
+	 * @param  ProgramRepository $repository
+	 * @return RegistrationPresenter
+	 */
+	protected function setProgramRepository(ProgramRepository $repository): self
+	{
+		$this->programRepository = $repository;
+
+		return $this;
+	}
 
 }

Upper-Lower-Casing +2 added lines, -2 removed lines

@@ -65,8 +65,8 @@
      */
 	private $programRepository;
 
-    protected $error = FALSE;
-    protected $hash = NULL;
+    protected $error = false;
+    protected $hash = null;
     private $user;
     private $event;
 

app/presenters/VisitorPresenter.php

Doc Comments +6 added lines, -6 removed lines

@@ -92,7 +92,7 @@ 
 	/**
 	 * Process data from form
 	 *
-	 * @return void
+	 * @return boolean
 	 */
 	public function actionCreate($visitor)
 	{
@@ -114,7 +114,7 @@ 
 	 * Process data from editing
 	 *
 	 * @param  integer 	$id
-	 * @return void
+	 * @return boolean
 	 */
 	public function actionUpdate($id, $visitor)
 	{
@@ -189,7 +189,7 @@ 
 	}
 
 	/**
-	 * @param  integer|string $ids
+	 * @param  integer|string $id
 	 * @return void
 	 */
 	public function actionPay($id)
@@ -218,7 +218,7 @@ 
 	}
 
 	/**
-	 * @param  string|interger $ids
+	 * @param  string|interger $id
 	 * @return void
 	 */
 	public function actionAdvance(int $id = null)
@@ -343,7 +343,7 @@ 
 	}
 
 	/**
-	 * @return VisitorFormControl
+	 * @return VisitorForm
 	 */
 	protected function createComponentVisitorForm(): VisitorForm
 	{
@@ -373,7 +373,7 @@ 
 	}
 
 	/**
-	 * @return Latte
+	 * @return \Nette\Application\UI\ITemplate
 	 */
 	protected function createMailTemplate()
 	{

Indentation +3 added lines, -3 removed lines

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

Spacing +12 added lines, -12 removed lines

@@ -100,10 +100,10 @@ 
 			$guid = $this->getVisitorRepository()->create($visitor);
 			$result = $this->sendRegistrationSummary($visitor, $guid);
 
-			$this->logInfo('Creation of visitor('. $guid .') successfull, result: ' . json_encode($result));
+			$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->logError('Creation of visitor(' . $guid . ') failed, result: ' . $e->getMessage());
 			$this->flashError('Creation of visitor failed, result: ' . $e->getMessage());
 		}
 
@@ -122,10 +122,10 @@ 
 			$result = $this->getVisitorRepository()->update($id, $visitor);
 			$result = $this->sendRegistrationSummary($visitor, $id);
 
-			$this->logInfo('Modification of visitor('. $id .') successfull, result: ' . json_encode($result));
+			$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->logError('Modification of visitor(' . $id . ') failed, result: ' . $e->getMessage());
 			$this->flashFailure('Modification of visitor failed, result: ' . $e->getMessage());
 			$result = false;
 		}
@@ -142,10 +142,10 @@ 
 		try {
 			$result = $this->getVisitorRepository()->delete($id);
 
-			$this->logInfo('Destroying of visitor('. $id .') successfull, result: ' . json_encode($result));
+			$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->logError('Destroying of visitor(' . $id . ') failed, result: ' . $e->getMessage());
 			$this->flashFailure('Destroying of visitor failed, result: ' . $e->getMessage());
 		}
 
@@ -181,7 +181,7 @@ 
 			$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->logError('Sending of e-mail failed, result: ' . $e->getMessage());
 			$this->flashFailure('Sending of e-mail failed, result: ' . $e->getMessage());
 		}
 
@@ -241,7 +241,7 @@ 
 					json_encode($id),
 					$e->getMessage(),
 				]);
-			$this->flashFailure('Zaplacení zálohy pro účastníka s id '.json_encode($id).' neprošlo: '.$e->getMessage());
+			$this->flashFailure('Zaplacení zálohy pro účastníka s id ' . json_encode($id) . ' neprošlo: ' . $e->getMessage());
 		}
 
 		$this->redirect(self::REDIRECT_DEFAULT);
@@ -257,10 +257,10 @@ 
 	{
 		try {
 			$result = $this->getVisitorRepository()->setChecked($id);
-			$this->logInfo('Check of visitor('. $id .') successfull, result: ' . json_encode($result));
+			$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->logError('Check of visitor(' . $id . ') failed, result: ' . $e->getMessage());
 			$this->flashFailure('Check of visitor failed, result: ' . $e->getMessage());
 		}
 
@@ -277,10 +277,10 @@ 
 	{
 		try {
 			$result = $this->getVisitorRepository()->setUnchecked($id);
-			$this->logInfo('Uncheck of visitor('. $id .') successfull, result: ' . json_encode($result));
+			$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->logError('Uncheck of visitor(' . $id . ') failed, result: ' . $e->getMessage());
 			$this->flashFailure('Uncheck of visitor failed, result: ' . $e->getMessage());
 		}
 

app/repositories/ProgramRepository.php

Doc Comments +8 added lines, -8 removed lines

@@ -53,7 +53,7 @@ 
 
 	/**
 	 * @param  int  $id
-	 * @return Nette\Database\Table\ActiveRow
+	 * @return ActiveRow
 	 */
 	public function find(int $id): ActiveRow
 	{
@@ -80,7 +80,7 @@ 
 
 	/**
 	 * @param  int    $programId
-	 * @return array
+	 * @return ActiveRow
 	 */
 	public function findTutor(int $programId): ActiveRow
 	{
@@ -97,7 +97,7 @@ 
 	}
 
 	/**
-	 * @param  Nette\Utils\ArrayHash $program
+	 * @param  ArrayHash $program
 	 * @return boolean
 	 */
 	public function create(ArrayHash $program)
@@ -108,8 +108,8 @@ 
 	}
 
 	/**
-	 * @param  Nette\Utils\ArrayHash $program
-	 * @return boolean
+	 * @param  ArrayHash $program
+	 * @return integer|null
 	 */
 	public function update(int $id, ArrayHash $program)
 	{
@@ -120,7 +120,7 @@ 
 
 	/**
 	 * @param  int    $id
-	 * @return boolean
+	 * @return integer|null
 	 */
 	public function delete(int $id)
 	{
@@ -157,8 +157,8 @@ 
 	}
 
 	/**
-	 * @param  Nette\Utils\ArrayHash $program
-	 * @return Nette\Utils\ArrayHash
+	 * @param  ArrayHash $program
+	 * @return ArrayHash
 	 */
 	protected function transformDisplayInRegValue(ArrayHash $program): ArrayHash
 	{

Indentation +14 added lines, -14 removed lines

@@ -69,13 +69,13 @@ 
 		return $this->getProgramModel()->findByVisitorId($visitorId);
 	}
 
-    /**
-     * @param int $blockId
-     * @return array
-     */
-    public function findByBlockId(int $blockId): array
-    {
-        return $this->getProgramModel()->findByBlockId($blockId);
+	/**
+	 * @param int $blockId
+	 * @return array
+	 */
+	public function findByBlockId(int $blockId): array
+	{
+		return $this->getProgramModel()->findByBlockId($blockId);
 	}
 
 	/**
@@ -127,13 +127,13 @@ 
 		return $this->getProgramModel()->delete($id);
 	}
 
-    /**
-     * @param  int $programId
-     * @return int
-     */
-    public function countVisitors(int $programId): int
-    {
-        return $this->getProgramModel()->countProgramVisitors($programId);
+	/**
+	 * @param  int $programId
+	 * @return int
+	 */
+	public function countVisitors(int $programId): int
+	{
+		return $this->getProgramModel()->countProgramVisitors($programId);
 	}
 
 	/**