literat / srazvs

app/services/VisitorService.php

Doc Comments +2 added lines, -3 removed lines

@@ -28,7 +28,6 @@ 
 	 * @param VisitorModel $visitor
 	 * @param MealModel    $meal
 	 * @param BlockModel   $block
-	 * @param Emailer      $emailer
 	 */
 	public function __construct(
 		VisitorModel $visitor,
@@ -42,7 +41,7 @@ 
 
 	/**
 	 * @param  array   $data
-	 * @return string
+	 * @return boolean
 	 */
 	public function create(array $data = [])
 	{
@@ -161,7 +160,7 @@ 
 
 	/**
 	 * @param  VisitorModel $model
-	 * @return VisitorModel
+	 * @return VisitorService
 	 */
 	protected function setVisitorModel(VisitorModel $model)
 	{

Spacing +2 added lines, -2 removed lines

@@ -103,11 +103,11 @@
 		$blocks = $this->getBlockModel()->idsFromCurrentMeeting($data['meeting']);
 
 		$programs = array_map(function($block) use ($data) {
-			if(!array_key_exists('blck_' . $block['id'], $data)) {
+			if(!array_key_exists('blck_'.$block['id'], $data)) {
 				return 0;
 			}
 
-			return $data['blck_' . $block['id']];
+			return $data['blck_'.$block['id']];
 		}, $blocks);
 
 		return $programs;

tests/acceptance/RegistrationCest.php

Spacing +1 added lines, -1 removed lines

@@ -95,7 +95,7 @@
 		$I->seeCurrentUrlMatches($this->succeededRegistrationUrl);
 		$I->see('úspěšně založena');
 		$I->see('Registrace na srazy K + K');
-		foreach ($this->successVisitor['fields'] as $field => $value) {
+		foreach($this->successVisitor['fields'] as $field => $value) {
 			$I->see($value);
 		}
 		$I->see($this->successVisitor['options']['province']);

app/presenters/ProgramPresenter.php

Spacing +16 added lines, -16 removed lines

@@ -85,13 +85,13 @@ 
 		try {
 			$result = $this->getModel()->create($data);
 
-			Debugger::log('Creation of program successfull, result: ' . json_encode($result), Debugger::INFO);
+			Debugger::log('Creation of program successfull, result: '.json_encode($result), Debugger::INFO);
 
 			$this->flashMessage('Položka byla úspěšně vytvořena', 'ok');
 		} catch(Exception $e) {
-			Debugger::log('Creation of program with data ' . json_encode($data) . ' failed, result: ' . $e->getMessage(), Debugger::ERROR);
+			Debugger::log('Creation of program with data '.json_encode($data).' failed, result: '.$e->getMessage(), Debugger::ERROR);
 
-			$this->flashMessage('Záznam se nepodařilo uložit, result: ' . $e->getMessage(), 'error');
+			$this->flashMessage('Záznam se nepodařilo uložit, result: '.$e->getMessage(), 'error');
 		}
 
 		$this->redirect($this->getBacklink() ?: 'Program:listing');
@@ -116,13 +116,13 @@ 
 		try {
 			$result = $this->getModel()->update($id, $data);
 
-			Debugger::log('Modification of program id ' . $id . ' with data ' . json_encode($data) . ' successfull, result: ' . json_encode($result), Debugger::INFO);
+			Debugger::log('Modification of program id '.$id.' with data '.json_encode($data).' successfull, result: '.json_encode($result), Debugger::INFO);
 
 			$this->flashMessage('Položka byla úspěšně upravena.', 'ok');
 		} catch(Exception $e) {
-			Debugger::log('Modification of program id ' . $id . ' failed, result: ' . $e->getMessage(), Debugger::ERROR);
+			Debugger::log('Modification of program id '.$id.' failed, result: '.$e->getMessage(), Debugger::ERROR);
 
-			$this->flashMessage('Modification of program id ' . $id . ' failed, result: ' . $e->getMessage(), 'error');
+			$this->flashMessage('Modification of program id '.$id.' failed, result: '.$e->getMessage(), 'error');
 		}
 
 		$this->redirect($this->getBacklink() ?: 'Program:listing');
@@ -136,11 +136,11 @@ 
 	{
 		try {
 			$result = $this->getModel()->delete($id);
-			Debugger::log('Destroying of program successfull, result: ' . json_encode($result), Debugger::INFO);
+			Debugger::log('Destroying of program successfull, result: '.json_encode($result), Debugger::INFO);
 			$this->flashMessage('Položka byla úspěšně smazána.', 'ok');
 		} catch(Exception $e) {
-			Debugger::log('Destroying of program failed, result: ' .  $e->getMessage(), Debugger::ERROR);
-			$this->flashMessage('Smazání programu se nezdařilo, result: ' . $e->getMessage(), 'error');
+			Debugger::log('Destroying of program failed, result: '.$e->getMessage(), Debugger::ERROR);
+			$this->flashMessage('Smazání programu se nezdařilo, result: '.$e->getMessage(), 'error');
 		}
 
 		$this->redirect('Program:listing');
@@ -156,13 +156,13 @@ 
 			$data = $this->getRequest()->getPost();
 			$result = $this->updateByGuid($id, $data);
 
-			Debugger::log('Modification of program annotation id ' . $id . ' with data ' . json_encode($data) . ' successfull, result: ' . json_encode($result), Debugger::INFO);
+			Debugger::log('Modification of program annotation id '.$id.' with data '.json_encode($data).' successfull, result: '.json_encode($result), Debugger::INFO);
 
 			$this->flashMessage('Položka byla úspěšně upravena.', 'ok');
 		} catch(Exception $e) {
-			Debugger::log('Modification of program annotation guid ' . $id . ' failed, result: ' . $e->getMessage(), Debugger::ERROR);
+			Debugger::log('Modification of program annotation guid '.$id.' failed, result: '.$e->getMessage(), Debugger::ERROR);
 
-			$this->flashMessage('Modification of program annotation guid ' . $id . ' failed, result: ' . $e->getMessage(), 'error');
+			$this->flashMessage('Modification of program annotation guid '.$id.' failed, result: '.$e->getMessage(), 'error');
 		}
 
 		$this->redirect('Program:annotation', $id);
@@ -179,11 +179,11 @@ 
 
 			$this->getEmailer()->tutor($recipients, $tutors->guid, 'program');
 
-			Debugger::log('Sending email to program tutor successfull, result: ' . json_encode($recipients) . ', ' . $tutors->guid, Debugger::INFO);
+			Debugger::log('Sending email to program tutor successfull, result: '.json_encode($recipients).', '.$tutors->guid, Debugger::INFO);
 			$this->flashMessage('Email lektorovi byl odeslán..', 'ok');
 		} catch(Exception $e) {
-			Debugger::log('Sending email to program tutor failed, result: ' .  $e->getMessage(), Debugger::ERROR);
-			$this->flashMessage('Email lektorovi nebyl odeslán, result: ' . $e->getMessage(), 'error');
+			Debugger::log('Sending email to program tutor failed, result: '.$e->getMessage(), Debugger::ERROR);
+			$this->flashMessage('Email lektorovi nebyl odeslán, result: '.$e->getMessage(), 'error');
 		}
 
 		$this->redirect('Program:edit', $id);
@@ -243,7 +243,7 @@ 
 		$template->error_material = '';
 		$template->display_in_reg_checkbox = $this->renderHtmlCheckBox('display_in_reg', 0, 1);
 		$template->block_select = $this->getBlockModel()->renderHtmlSelect(null);
-		$template->selectedCategory	= null;
+		$template->selectedCategory = null;
 	}
 
 	/**

app/services/SkautisAuthenticator.php

Braces +4 added lines, -4 removed lines

@@ -9,11 +9,11 @@
  * Never let user to set his ID himself.

  * @author Hána František <sinacek@gmail.com>

  */

-class SkautisAuthenticator extends Nette\Object implements Nette\Security\IAuthenticator

-{

+class SkautisAuthenticator extends Nette\Object implements Nette\Security\IAuthenticator
+{
 

-	public function authenticate(array $credentials)

-	{

+	public function authenticate(array $credentials)
+	{
 		$data = $credentials[0];

 		return new Nette\Security\Identity($data->ID_User);

 	}

app/services/UserService.php

Spacing +7 added lines, -7 removed lines

@@ -43,7 +43,7 @@ 
 	{
 		$id = __FUNCTION__;
 		// cache by the request
-		if (!($res = $this->load($id))) {
+		if(!($res = $this->load($id))) {
 			$res = $this->save($id, $this->skautis->user->UserDetail());
 		}
 		return $res;
@@ -59,7 +59,7 @@ 
 	public function updateSkautISRole($id)
 	{
 		$unitId = $this->skautis->user->LoginUpdate(array("ID_UserRole" => $id, "ID" => $this->skautis->getToken()));
-		if ($unitId) {
+		if($unitId) {
 			$this->skautis->setRoleId($id);
 			$this->skautis->setUnitId($unitId->ID_Unit);
 		}
@@ -157,17 +157,17 @@ 
 		));
 
 		// returns boolean if certain function for verifying is set
-		if ($ID_Action !== NULL) {
-			if ($res instanceof stdClass) {
+		if($ID_Action !== NULL) {
+			if($res instanceof stdClass) {
 				return false;
 			}
-			if (is_array($res)) {
+			if(is_array($res)) {
 				return true;
 			}
 		}
-		if (is_array($res)) {
+		if(is_array($res)) {
 			$tmp = array();
-			foreach ($res as $v) {
+			foreach($res as $v) {
 				$tmp[$v->ID] = $v;
 			}
 			return $tmp;

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

@@ -148,7 +148,7 @@ 
 	 * @param   type  $ID_Action  tabulka v DB skautisu
 	 * @return  BOOL|stdClass|array
 	 */
-	public function actionVerify($table, $id = NULL, $ID_Action = NULL)
+	public function actionVerify($table, $id = null, $ID_Action = null)
 	{
 		$res = $this->skautis->user->ActionVerify(array(
 			"ID" => $id,
@@ -157,7 +157,7 @@ 
 		));
 
 		// returns boolean if certain function for verifying is set
-		if ($ID_Action !== NULL) {
+		if ($ID_Action !== null) {
 			if ($res instanceof stdClass) {
 				return false;
 			}

app/components/BaseControl.php

Spacing +1 added lines, -1 removed lines

@@ -7,7 +7,7 @@
 abstract class BaseControl extends Control
 {
 
-	const TEMPLATE_DIR = __DIR__ . '/../templates/components';
+	const TEMPLATE_DIR = __DIR__.'/../templates/components';
 	const TEMPLATE_EXT = 'latte';
 
 	/**

app/bootstrap.php

Indentation +3 added lines, -3 removed lines

@@ -40,9 +40,9 @@ 
 	->register();
 
 if ($configurator->isDebugMode()) {
-    $configurator->addConfig(__DIR__ . '/config/config.development.neon');
+	$configurator->addConfig(__DIR__ . '/config/config.development.neon');
 } else {
-    $configurator->addConfig(__DIR__ . '/config/config.production.neon');
+	$configurator->addConfig(__DIR__ . '/config/config.production.neon');
 }
 $configurator->addConfig(__DIR__ . '/config/config.local.neon');
 
@@ -50,7 +50,7 @@ 
  * DI Container and Session
  */
 $configurator->addServices(array(
-    'session.session' => $session,
+	'session.session' => $session,
 ));
 $container = $configurator->createContainer();
 $parameters = $container->getParameters();

Spacing +26 added lines, -26 removed lines

@@ -9,12 +9,12 @@ 
 use Nette\Loaders\RobotLoader;
 use App\Routers\RouterFactory;
 
-require_once __DIR__ . '/../vendor/autoload.php';
+require_once __DIR__.'/../vendor/autoload.php';
 
 define('SESSION_PREFIX', md5('localhost'.'vodni'.'vodni'.'sunlight')."-");
 
 //nastartovani session
-session_name(SESSION_PREFIX . 'session');
+session_name(SESSION_PREFIX.'session');
 
 $requestFatory = new Nette\Http\RequestFactory;
 $httpRequest = $requestFatory->createHttpRequest();
@@ -28,23 +28,23 @@ 
  * Enabling Debugger
  */
 //$configurator->setDebugMode(TRUE);
-$configurator->enableDebugger(__DIR__ . '/../temp/log', 'tomaslitera@outlook.com');
-$configurator->setTempDirectory(__DIR__ . '/../temp');
+$configurator->enableDebugger(__DIR__.'/../temp/log', 'tomaslitera@outlook.com');
+$configurator->setTempDirectory(__DIR__.'/../temp');
 
 /**
  * Autoloading
  */
 $configurator->createRobotLoader()
 	->addDirectory(__DIR__)
-	->addDirectory(__DIR__ . '/../inc')
+	->addDirectory(__DIR__.'/../inc')
 	->register();
 
-if ($configurator->isDebugMode()) {
-    $configurator->addConfig(__DIR__ . '/config/config.development.neon');
+if($configurator->isDebugMode()) {
+    $configurator->addConfig(__DIR__.'/config/config.development.neon');
 } else {
-    $configurator->addConfig(__DIR__ . '/config/config.production.neon');
+    $configurator->addConfig(__DIR__.'/config/config.production.neon');
 }
-$configurator->addConfig(__DIR__ . '/config/config.local.neon');
+$configurator->addConfig(__DIR__.'/config/config.local.neon');
 
 /**
  * DI Container and Session
@@ -56,7 +56,7 @@ 
 $parameters = $container->getParameters();
 
 define('ROOT_DIR', $parameters['wwwDir']);
-define('HTTP_DIR', '//'.$_SERVER['HTTP_HOST'] . '/');
+define('HTTP_DIR', '//'.$_SERVER['HTTP_HOST'].'/');
 
 /**
  * Application's definitions
@@ -65,24 +65,24 @@ 
  */
 
 /* System Directories */
-define('PRJ_DIR',		HTTP_DIR . 'srazvs/');
-define('INC_DIR',		ROOT_DIR . '/inc/');
-define('IMG_DIR',		PRJ_DIR . 'www/images/');
-define('CSS_DIR',		PRJ_DIR . 'www/css/');
-define('JS_DIR',		PRJ_DIR . 'www/js/');
+define('PRJ_DIR', HTTP_DIR.'srazvs/');
+define('INC_DIR', ROOT_DIR.'/inc/');
+define('IMG_DIR', PRJ_DIR.'www/images/');
+define('CSS_DIR', PRJ_DIR.'www/css/');
+define('JS_DIR', PRJ_DIR.'www/js/');
 
 /* Application */
-define('TEMPLATE_DIR',	$parameters['appDir'] . '/templates/');
-define('TPL_DIR',		TEMPLATE_DIR);
+define('TEMPLATE_DIR', $parameters['appDir'].'/templates/');
+define('TPL_DIR', TEMPLATE_DIR);
 
 /* URLs */
-define('BLOCK_DIR',		PRJ_DIR.'block');
-define('PROG_DIR',		PRJ_DIR.'program');
-define('MEET_DIR',		PRJ_DIR.'meeting');
-define('VISIT_DIR',		PRJ_DIR.'visitor');
-define('CAT_DIR',		PRJ_DIR.'category');
-define('EXP_DIR',		PRJ_DIR.'export');
-define('SET_DIR',		PRJ_DIR.'settings');
+define('BLOCK_DIR', PRJ_DIR.'block');
+define('PROG_DIR', PRJ_DIR.'program');
+define('MEET_DIR', PRJ_DIR.'meeting');
+define('VISIT_DIR', PRJ_DIR.'visitor');
+define('CAT_DIR', PRJ_DIR.'category');
+define('EXP_DIR', PRJ_DIR.'export');
+define('SET_DIR', PRJ_DIR.'settings');
 
 /**
  * Connecting to Database
@@ -109,7 +109,7 @@ 
 	die;
 }
 
-$target = $parameters['appDir'] . '/presenters/' . $controllerName . 'Presenter.php';
+$target = $parameters['appDir'].'/presenters/'.$controllerName.'Presenter.php';
 $container->parameters['router'] = $appRequest;
 
 $publicPages = [
@@ -133,7 +133,7 @@ 
 if(file_exists($target)) {
 	// access control
 	if(array_search($controllerName.'.'.$action, $publicPages) === false) {
-		include_once(INC_DIR . 'access.inc.php');
+		include_once(INC_DIR.'access.inc.php');
 	}
 }
 

app/models/BlockModel.php

Spacing +1 added lines, -1 removed lines

@@ -87,7 +87,7 @@
 
 		$html_select = "<select style='width: 300px; font-size: 10px' name='block'>\n";
 
-		foreach($result as $data){
+		foreach($result as $data) {
 			if($data['id'] == $blockId) $selected = "selected";
 			else $selected = "";
 			$html_select .= "<option ".$selected." value='".$data['id']."'>".$data['day'].", ".$data->from->format('%H:%I:%S')." - ".$data->to->format('%H:%I:%S')." : ".$data['name']."</option>\n";

Braces +5 added lines, -2 removed lines

@@ -88,8 +88,11 @@
 		$html_select = "<select style='width: 300px; font-size: 10px' name='block'>\n";
 
 		foreach($result as $data){
-			if($data['id'] == $blockId) $selected = "selected";
-			else $selected = "";
+			if($data['id'] == $blockId) {
+				$selected = "selected";
+			} else {
+				$selected = "";
+			}
 			$html_select .= "<option ".$selected." value='".$data['id']."'>".$data['day'].", ".$data->from->format('%H:%I:%S')." - ".$data->to->format('%H:%I:%S')." : ".$data['name']."</option>\n";
 		}
 		$html_select .= "</select>\n";

app/helpers.php

Spacing +2 added lines, -2 removed lines

@@ -8,7 +8,7 @@ 
 	 * @return void
 	 */
 	function dd() {
-		array_map(function ($x){
+		array_map(function($x) {
 			(\Tracy\Debugger::dump($x));
 		}, func_get_args());
 
@@ -23,7 +23,7 @@ 
 	 * @return string
 	 */
 	function appVersion() {
-		$packagePath = realpath(__DIR__ . '/../package.json');
+		$packagePath = realpath(__DIR__.'/../package.json');
 		$package = json_decode(file_get_contents($packagePath));
 
 		return $package->version;

Braces +4 added lines, -2 removed lines

@@ -7,7 +7,8 @@ 
 	 * @param  mixed
 	 * @return void
 	 */
-	function dd() {
+	function dd()
+	{
 		array_map(function ($x){
 			(\Tracy\Debugger::dump($x));
 		}, func_get_args());
@@ -22,7 +23,8 @@ 
 	 *
 	 * @return string
 	 */
-	function appVersion() {
+	function appVersion()
+	{
 		$packagePath = realpath(__DIR__ . '/../package.json');
 		$package = json_decode(file_get_contents($packagePath));