mattheworres / phpdraft

deploy/_deploy_tasks.php

Indentation +18 added lines, -18 removed lines

@@ -7,13 +7,13 @@ 
     'deploy:prepare',
     'deploy:lock',
     'deploy:release',
-	'phpdraft:upload_files',
-	'phpdraft:remote_yarn',
+  'phpdraft:upload_files',
+  'phpdraft:remote_yarn',
     'phpdraft:remote_composer',
     'phpdraft:breakpoint',
     'phpdraft:migrate',
-	'deploy:symlink',
-	//'phpdraft:restart_fpm',
+  'deploy:symlink',
+  //'phpdraft:restart_fpm',
     'deploy:unlock',
     'cleanup',
     'success'
@@ -32,13 +32,13 @@ 
     $settings_location = ask("What is the absolute (local) path of where your settings files are located?",
         "~/phpdraft_settings");
 
-	runLocally("cp $settings_location/.htaccess .htaccess");
+  runLocally("cp $settings_location/.htaccess .htaccess");
     runLocally("cp $settings_location/appsettings.php appsettings.php");
-	runLocally("cp $settings_location/config.js js/config.js");
+  runLocally("cp $settings_location/config.js js/config.js");
     runLocally("cp $settings_location/deploy.php deploy.php");
-	runLocally("cp $settings_location/index.html index.html");
-	runLocally("cp $settings_location/phinx.yml phinx.yml");
-	runLocally("cp $settings_location/web.config web.config");
+  runLocally("cp $settings_location/index.html index.html");
+  runLocally("cp $settings_location/phinx.yml phinx.yml");
+  runLocally("cp $settings_location/web.config web.config");
 
     writeln("\n\n<info>Success! I imported those settings for you, you should be ready to deploy now.</info>\n");
 });
@@ -75,12 +75,12 @@ 
         'db',
         'fonts',
         'images',
-		'js',
+    'js',
         '.htaccess',
         'appsettings.php',
         'composer.json',
-		'composer.lock',
-		'package.json',
+    'composer.lock',
+    'package.json',
         'phinx.yml',
         'deploy',
         'index.html',
@@ -95,7 +95,7 @@ 
 
 desc('Install NPM dependencies remotely');
 task('phpdraft:remote_yarn', function() {
-	cd('{{release_path}}');
+  cd('{{release_path}}');
 
     run('yarn --production');
 
@@ -143,9 +143,9 @@ 
 
 desc('Run Phinx migrations');
 task('phpdraft:migrate', function() {
-	cd('{{release_path}}');
+  cd('{{release_path}}');
 
-	set('phpdraft_rollback_required', true);
+  set('phpdraft_rollback_required', true);
 
     run('php deploy/phinx.phar migrate -e production');
 
@@ -156,9 +156,9 @@ 
 task('phpdraft:rollback', function() {
     writeln('<comment>Rolling back database migrations...</comment>');
     if(get("phpdraft_rollback_required") == true) {
-       cd('{{release_path}}');
-       run('php deploy/phinx.phar rollback -e production');
-       cd('{{deploy_path}}');
+        cd('{{release_path}}');
+        run('php deploy/phinx.phar rollback -e production');
+        cd('{{deploy_path}}');
     }
 })->setPrivate();
 

Spacing +3 added lines, -3 removed lines

@@ -55,8 +55,8 @@ 
         'vendor'
     ];
 
-    foreach($items_required_for_deploy as $file) {
-        if(file_exists($file) !== true) {
+    foreach ($items_required_for_deploy as $file) {
+        if (file_exists($file) !== true) {
             writeln("<error>File or directory $file does not exist, cannot deploy.</error>\n");
             writeln("<comment>Ensure you have downloaded a compiled release from https://github.com/mattheworres/phpdraft/releases</comment>\n");
             writeln("<comment>Or, if you are building from sourcecode, consult the wiki on how to properly prepare a release.</comment>\n");
@@ -155,7 +155,7 @@ 
 desc('Rollback migrations on failure');
 task('phpdraft:rollback', function() {
     writeln('<comment>Rolling back database migrations...</comment>');
-    if(get("phpdraft_rollback_required") == true) {
+    if (get("phpdraft_rollback_required") == true) {
        cd('{{release_path}}');
        run('php deploy/phinx.phar rollback -e production');
        cd('{{deploy_path}}');

deploy/_setup_tasks.php

Indentation +8 added lines, -8 removed lines

@@ -13,12 +13,12 @@ 
 
 desc('PHP Draft: Offer to backup current settings for future upgrade imports');
 task('backup', function() {
-	if (file_exists("js/config.js") == false) {
-		writeln("<error>Looks like js/config.js doesn't exist - I can't back up your settings.</error>\n");
-		writeln("<comment>Ensure you have downloaded a compiled release from https://github.com/mattheworres/phpdraft/releases</comment>\n");
-		writeln("<comment>Or, if you are building from sourcecode, consult the wiki on how to properly prepare a release.</comment>\n");
-		throw new \Exception("PHP Draft is not in a exportable state (use downloads from Releases on Github)");
-	}
+  if (file_exists("js/config.js") == false) {
+    writeln("<error>Looks like js/config.js doesn't exist - I can't back up your settings.</error>\n");
+    writeln("<comment>Ensure you have downloaded a compiled release from https://github.com/mattheworres/phpdraft/releases</comment>\n");
+    writeln("<comment>Or, if you are building from sourcecode, consult the wiki on how to properly prepare a release.</comment>\n");
+    throw new \Exception("PHP Draft is not in a exportable state (use downloads from Releases on Github)");
+  }
 
   writeln("\n\n<info>Looking great! Hey, if you want I can back up these snazzy settings files "
     ."I just created for you. This will make it WAY easier to update PHP Draft in the future!</info>\n");
@@ -35,8 +35,8 @@ 
     runLocally("cp js/config.js $backup_location/config.js");
     runLocally("cp phinx.yml $backup_location/phinx.yml");
     runLocally("cp deploy.php $backup_location/deploy.php");
-		runLocally("cp .htaccess $backup_location/.htaccess");
-		runLocally("cp index.html $backup_location/index.html");
+    runLocally("cp .htaccess $backup_location/.htaccess");
+    runLocally("cp index.html $backup_location/index.html");
     runLocally("cp web.config $backup_location/web.config");
   }
 });

Spacing +4 added lines, -4 removed lines

@@ -24,7 +24,7 @@ 
     ."I just created for you. This will make it WAY easier to update PHP Draft in the future!</info>\n");
   $answer = askConfirmation("Should I back up your settings for you?", false);
 
-  if($answer == true) {
+  if ($answer == true) {
     writeln("\n\n<info>Awesome! Wherever you tell me to back your settings up to, you should make sure "
       ."that only your user has permissions to view the file (if the server you're installing on "
       ."is more important, you should probably ensure your disk is encrypted, too).</info>\n");
@@ -48,7 +48,7 @@ 
 
   $answer = askConfirmation($message, false);
 
-  if($answer == false) {
+  if ($answer == false) {
     throw new \Exception("Quitting setup at the user's request. BYE FELICIA.");
   }
 })->setPrivate();
@@ -146,8 +146,8 @@ 
     'deploy.php'
   ];
 
-  foreach(get('phpdraft_setup_variables') as $property => $value) {
-    foreach($files_to_replace_in as $file) {
+  foreach (get('phpdraft_setup_variables') as $property => $value) {
+    foreach ($files_to_replace_in as $file) {
       //TODO: Is there something more platform agnostic we can use?
       runLocally("sed -i '' 's|{phpdraft.$property}|$value|g' $file $file");
     }

api/config/_app.php

Spacing +12 added lines, -12 removed lines

@@ -1,26 +1,26 @@
 <?php 
 
-require_once dirname(__FILE__).'/../../vendor/autoload.php';
+require_once dirname(__FILE__) . '/../../vendor/autoload.php';
 
 $app = new Silex\Application();
 
 //Handles settings users can define
 //Please see README.md for instructions on how to setup
-require_once dirname(__FILE__).'/../../appsettings.php';
+require_once dirname(__FILE__) . '/../../appsettings.php';
 
 $app['debug'] = DEBUG_MODE;
 
-require_once dirname(__FILE__).'/_database.php';      //Sets up database connections
-require_once dirname(__FILE__).'/_log.php';           //Sets up logging
+require_once dirname(__FILE__) . '/_database.php'; //Sets up database connections
+require_once dirname(__FILE__) . '/_log.php'; //Sets up logging
 
 //Registrations with Pimple DI
-require_once dirname(__FILE__).'/_registerServices.php';
-require_once dirname(__FILE__).'/_registerRepositories.php';
-require_once dirname(__FILE__).'/_registerValidators.php';
-require_once dirname(__FILE__).'/_registerFactories.php';
-
-require_once dirname(__FILE__).'/_middlewares.php';   //Defines middleware handlers for shared logic, like ensuring a draft is editable
-require_once dirname(__FILE__).'/_router.php';        //Sets up controller routing
-require_once dirname(__FILE__).'/_security.php';      //Sets up Symfony-based security & user authentication
+require_once dirname(__FILE__) . '/_registerServices.php';
+require_once dirname(__FILE__) . '/_registerRepositories.php';
+require_once dirname(__FILE__) . '/_registerValidators.php';
+require_once dirname(__FILE__) . '/_registerFactories.php';
+
+require_once dirname(__FILE__) . '/_middlewares.php'; //Defines middleware handlers for shared logic, like ensuring a draft is editable
+require_once dirname(__FILE__) . '/_router.php'; //Sets up controller routing
+require_once dirname(__FILE__) . '/_security.php'; //Sets up Symfony-based security & user authentication
 
 return $app;
\ No newline at end of file

api/Domain/Validators/TradeValidator.php

Spacing +8 added lines, -8 removed lines

@@ -19,7 +19,7 @@ 
     $valid = true;
     $errors = array();
 
-    if($draft->draft_id != $manager->draft_id) {
+    if ($draft->draft_id != $manager->draft_id) {
       $errors[] = "Manager does not belong to draft #$draft->draft_id";
       $valid = false;
     }
@@ -33,37 +33,37 @@ 
     $assetCount = count($trade->trade_assets);
     $uniqueAssetCount = count(array_unique($trade->trade_assets));
 
-    if(empty($trade->trade_round) || $trade->trade_round < 0 || $trade->trade_round > $draft->draft_rounds) {
+    if (empty($trade->trade_round) || $trade->trade_round < 0 || $trade->trade_round > $draft->draft_rounds) {
       $errors[] = "Invalid value for trade round.";
       $valid = false;
     }
 
-    foreach($trade->trade_assets as $trade_asset) {
-      if($trade_asset->oldmanager_id != $trade->manager1_id 
+    foreach ($trade->trade_assets as $trade_asset) {
+      if ($trade_asset->oldmanager_id != $trade->manager1_id 
         && $trade_asset->oldmanager_id != $trade->manager2_id) {
         $player_id = $trade_asset->player->player_id;
         $errors[] = "Asset #$player_id does not belong to either manager.";
         $valid = false;
       }
 
-      if($trade_asset->player->draft_id != $draft->draft_id) {
+      if ($trade_asset->player->draft_id != $draft->draft_id) {
         $player_id = $trade_asset->player->player_id;
         $errors[] = "Asset #$player_id does not belong to draft #$draft->draft_id";
         $valid = false;
       }
     }
 
-    if($assetCount != $uniqueAssetCount) {
+    if ($assetCount != $uniqueAssetCount) {
       $errors[] = "One or more of the trade assets are duplicate.";
       $valid = false;
     }
 
-    if($draft->draft_id != $trade->manager1->draft_id) {
+    if ($draft->draft_id != $trade->manager1->draft_id) {
       $errors[] = "Manager 1 does not belong to draft #$draft->draft_id";
       $valid = false;
     }
 
-    if($draft->draft_id != $trade->manager2->draft_id) {
+    if ($draft->draft_id != $trade->manager2->draft_id) {
       $errors[] = "Manager 2 does not belong to draft #$draft->draft_id";
       $valid = false;
     }

api/Domain/Validators/ProPlayerValidator.php

Spacing +3 added lines, -3 removed lines

@@ -21,19 +21,19 @@
     $errors = array();
     $sports = $this->app['phpdraft.DraftDataRepository']->GetSports();
 
-    if(empty($sport)) {
+    if (empty($sport)) {
       $errors[] = "One or more missing fields.";
       $valid = false;
     }
 
-    if(!array_key_exists($sport, $sports)) {
+    if (!array_key_exists($sport, $sports)) {
       $errors[] = "Sport $sport is an invalid value.";
       $valid = false;
     }
 
     if (!isset($file)) {
       $valid = false;
-      $errors[] =  "Must upload a CSV file";
+      $errors[] = "Must upload a CSV file";
     } else {
       $fileErrorCode = $file->getError();
 

api/Domain/Services/DraftService.php

Spacing +8 added lines, -8 removed lines

@@ -28,13 +28,13 @@ 
     try {
       $draft = $this->app['phpdraft.DraftRepository']->Create($draft);
 
-      if($draft->using_depth_charts) {
+      if ($draft->using_depth_charts) {
         $depth_charts = $this->app['phpdraft.DepthChartPositionRepository']->Save($depthChartModel, $draft->draft_id);
       }
 
       $response->success = true;
       $response->draft = $draft;
-    }catch(\Exception $e) {
+    } catch (\Exception $e) {
       $response->success = false;
       $response->errors = array($e->getMessage());
     }
@@ -48,14 +48,14 @@ 
     try {
       $draft = $this->app['phpdraft.DraftRepository']->Update($draft);
 
-      if($draft->using_depth_charts == 1) {
+      if ($draft->using_depth_charts == 1) {
         $this->app['phpdraft.DepthChartPositionRepository']->DeleteAllDepthChartPositions($draft->draft_id);
         $depth_charts = $this->app['phpdraft.DepthChartPositionRepository']->Save($depthChartModel, $draft->draft_id);
       }
 
       $response->success = true;
       $response->draft = $draft;
-    }catch(\Exception $e) {
+    } catch (\Exception $e) {
       $response->success = false;
       $response->errors = array($e->getMessage());
     }
@@ -70,7 +70,7 @@ 
       $draft = $this->app['phpdraft.DraftRepository']->UpdateStatus($draft);
 
       //If we know we're moving from undrafted to in progress, perform the necessary setup steps:
-      if($draft->draft_status != $old_status && $draft->draft_status == "in_progress") {
+      if ($draft->draft_status != $old_status && $draft->draft_status == "in_progress") {
         //Delete all trades
         $this->app['phpdraft.TradeRepository']->DeleteAllTrades($draft->draft_id);
         //Delete all picks
@@ -85,7 +85,7 @@ 
 
       $response->success = true;
       $response->draft = $draft;
-    }catch(\Exception $e) {
+    } catch (\Exception $e) {
       $response->success = false;
       $response->errors = array($e->getMessage());
     }
@@ -111,7 +111,7 @@ 
       $this->app['phpdraft.DraftRepository']->DeleteDraft($draft->draft_id);
 
       $response->success = true;
-    } catch(\Exception $e) {
+    } catch (\Exception $e) {
       $response->success = false;
       $response->errors = array($e->getMessage());
     }
@@ -125,7 +125,7 @@ 
     try {
       $response->draft_statistics = $this->app['phpdraft.DraftStatsRepository']->LoadDraftStats($draft_id);
       $response->success = true;
-    } catch(\Exception $e) {
+    } catch (\Exception $e) {
       $message = $e->getMessage();
       $response->success = false;
       $response->errors[] = $message;

api/Domain/Repositories/DraftStatsRepository.php

Spacing +17 added lines, -17 removed lines

@@ -21,7 +21,7 @@ 
 
     $load_stmt->setFetchMode(\PDO::FETCH_INTO, $stats);
 
-    if(!$load_stmt->execute()) {
+    if (!$load_stmt->execute()) {
       throw new \Exception("Unable to load draft stats");
     }
 
@@ -29,7 +29,7 @@ 
       return null;
     }
 
-    if(!$load_stmt->fetch()) {
+    if (!$load_stmt->fetch()) {
       throw new \Exception("Error while loading draft stats");
     }
 
@@ -103,7 +103,7 @@ 
     $insert_stmt->bindValue(':least_drafted_position', $draft_stats->least_drafted_position);
     $insert_stmt->bindValue(':least_drafted_position_count', $draft_stats->least_drafted_position_count);
 
-    if(!$insert_stmt->execute()) {
+    if (!$insert_stmt->execute()) {
       throw new \Exception("Unable to insert new draft stats row.");
     }
 
@@ -118,7 +118,7 @@ 
     $delete_stmt = $this->app['db']->prepare("DELETE FROM draft_stats WHERE draft_id = ?");
     $delete_stmt->bindParam(1, $draft_id);
 
-    if(!$delete_stmt->execute()) {
+    if (!$delete_stmt->execute()) {
       throw new \Exception("Unable to delete existing stats rows.");
     }
   }
@@ -170,7 +170,7 @@ 
     $row = $stmt->fetch();
 
     $stats->shortest_avg_pick_manager_name = $row['manager_name'];
-    $stats->shortest_avg_pick_seconds = (int) $row['pick_average'];
+    $stats->shortest_avg_pick_seconds = (int)$row['pick_average'];
   }
 
   private function _LoadSlowestPick($draft_id, DraftStats &$stats) {
@@ -190,7 +190,7 @@ 
     $row = $stmt->fetch();
 
     $stats->longest_single_pick_manager_name = $row['manager_name'];
-    $stats->longest_single_pick_seconds = (int) $row['pick_max'];
+    $stats->longest_single_pick_seconds = (int)$row['pick_max'];
   }
 
   private function _LoadFastestPick($draft_id, DraftStats &$stats) {
@@ -210,7 +210,7 @@ 
     $row = $stmt->fetch();
 
     $stats->shortest_single_pick_manager_name = $row['manager_name'];
-    $stats->shortest_single_pick_seconds = (int) $row['pick_min'];
+    $stats->shortest_single_pick_seconds = (int)$row['pick_min'];
   }
 
   private function _LoadAveragePickTime($draft_id, DraftStats &$stats) {
@@ -225,7 +225,7 @@ 
 
     $row = $stmt->fetch();
 
-    $stats->average_pick_seconds = (int) $row['pick_average'];
+    $stats->average_pick_seconds = (int)$row['pick_average'];
   }
 
   private function _LoadRoundTimes(Draft $draft, DraftStats &$stats) {
@@ -243,8 +243,8 @@ 
 
     $row = $stmt->fetch();
 
-    $stats->longest_round = (int) $row['player_round'];
-    $stats->longest_round_seconds = (int) $row['round_time'];
+    $stats->longest_round = (int)$row['player_round'];
+    $stats->longest_round_seconds = (int)$row['round_time'];
 
     //Stupid that I can't just re-use the above statement. All that changes is DESC to ASC. Stupid.
     $stmt = $this->app['db']->prepare("SELECT DISTINCT p.player_round, sum( p.pick_duration ) AS round_time
@@ -261,8 +261,8 @@ 
 
     $row = $stmt->fetch();
 
-    $stats->shortest_round = (int) $row['player_round'];
-    $stats->shortest_round_seconds = (int) $row['round_time'];
+    $stats->shortest_round = (int)$row['player_round'];
+    $stats->shortest_round_seconds = (int)$row['round_time'];
 
     $stmt = $this->app['db']->prepare("SELECT p.player_round, sum( p.pick_duration ) / ? AS avg_round_time
     FROM players p
@@ -279,7 +279,7 @@ 
 
     $row = $stmt->fetch();
 
-    $stats->average_round_seconds = (int) $row['avg_round_time'];
+    $stats->average_round_seconds = (int)$row['avg_round_time'];
   }
 
   private function _LoadTeamSuperlatives($draft_id, DraftStats &$stats, $teams) {
@@ -298,7 +298,7 @@ 
     $row = $stmt->fetch();
 
     $stats->most_drafted_team = isset($row['team']) ? $teams[$row['team']] : "";
-    $stats->most_drafted_team_count = (int) $row['team_occurences'];
+    $stats->most_drafted_team_count = (int)$row['team_occurences'];
 
     $stmt = $this->app['db']->prepare("SELECT DISTINCT p.team, count(team) as team_occurences
     FROM players p
@@ -315,7 +315,7 @@ 
     $row = $stmt->fetch();
 
     $stats->least_drafted_team = isset($row['team']) ? $teams[$row['team']] : "";
-    $stats->least_drafted_team_count = (int) $row['team_occurences'];
+    $stats->least_drafted_team_count = (int)$row['team_occurences'];
   }
 
   private function _LoadPositionSuperlatives($draft_id, DraftStats &$stats, $positions) {
@@ -334,7 +334,7 @@ 
     $row = $stmt->fetch();
 
     $stats->most_drafted_position = isset($row['position']) ? $positions[$row['position']] : "";
-    $stats->most_drafted_position_count = (int) $row['position_occurences'];
+    $stats->most_drafted_position_count = (int)$row['position_occurences'];
 
     $stmt = $this->app['db']->prepare("SELECT DISTINCT p.position, count(position) as position_occurences
     FROM players p
@@ -351,6 +351,6 @@ 
     $row = $stmt->fetch();
 
     $stats->least_drafted_position = isset($row['position']) ? $positions[$row['position']] : "";
-    $stats->least_drafted_position_count = (int) $row['position_occurences'];
+    $stats->least_drafted_position_count = (int)$row['position_occurences'];
   }
 }

api/Domain/Repositories/LoginUserRepository.php

Spacing +12 added lines, -12 removed lines

@@ -85,11 +85,11 @@ 
 
     $users = array();
 
-    if(!$load_stmt->execute()) {
+    if (!$load_stmt->execute()) {
       throw new \Exception("Unable to load users.");
     }
 
-    while($user = $load_stmt->fetch()) {
+    while ($user = $load_stmt->fetch()) {
       
       $users[] = $this->_ScrubUser($user);
     }
@@ -117,7 +117,7 @@ 
       throw new \Exception("Unable to create user.");
     }
 
-    $user->id = (int) $this->app['db']->lastInsertId();
+    $user->id = (int)$this->app['db']->lastInsertId();
 
     return $user;
   }
@@ -155,7 +155,7 @@ 
 
     $result = $update_stmt->execute();
 
-    if($result == false) {
+    if ($result == false) {
       throw new \Exception("Unable to erase verification key for user.");
     }
 
@@ -166,7 +166,7 @@ 
     $delete_stmt = $this->app['db']->prepare("DELETE FROM users WHERE id = ?");
     $delete_stmt->bindParam(1, $user->id);
 
-    if(!$delete_stmt->execute()) {
+    if (!$delete_stmt->execute()) {
       throw new \Exception("Unable to delete user #$user->id");
     }
 
@@ -175,7 +175,7 @@ 
 
   public function NameIsUnique($name, $id = null) {
     $name = strtolower($name);
-    if($id == null) {
+    if ($id == null) {
       $name_stmt = $this->app['db']->prepare("SELECT name FROM users WHERE name LIKE ?");
       $name_stmt->bindParam(1, $name);
     } else {
@@ -184,7 +184,7 @@ 
       $name_stmt->bindParam(2, $id);
     }
 
-    if(!$name_stmt->execute()) {
+    if (!$name_stmt->execute()) {
       throw new \Exception(sprintf('Name %s is invalid', $name));
     }
 
@@ -192,7 +192,7 @@ 
   }
 
   public function EmailExists($email, $id = null) {
-    if($id == null) {
+    if ($id == null) {
       $email_stmt = $this->app['db']->prepare("SELECT email FROM users WHERE email = ?");
       $email_stmt->bindParam(1, $email);
     } else {
@@ -213,7 +213,7 @@ 
     $email_stmt = $this->app['db']->prepare("SELECT email FROM users WHERE email = ? LIMIT 1");
     $email_stmt->bindParam(1, $email);
 
-    if(!$email_stmt->execute()) {
+    if (!$email_stmt->execute()) {
       throw new \Exception(sprintf('Email %s is invalid', $email));
     }
 
@@ -229,11 +229,11 @@ 
 
     $users = array();
 
-    if(!$search_stmt->execute()) {
+    if (!$search_stmt->execute()) {
       throw new \Exception("Unable to load users");
     }
 
-    while($user = $search_stmt->fetch()) {
+    while ($user = $search_stmt->fetch()) {
       unset($user->enabled);
       unset($user->email);
       unset($user->password);
@@ -253,7 +253,7 @@ 
     $verification_stmt->bindParam(1, $email);
     $verification_stmt->bindParam(2, $verificationKey);
 
-    if(!$verification_stmt->execute()) {
+    if (!$verification_stmt->execute()) {
       throw new \Exception('Verification is invalid.');
     }
 

Braces +18 added lines, -12 removed lines

@@ -25,11 +25,13 @@ 
     $load_stmt->setFetchMode(\PDO::FETCH_INTO, $user);
     $load_stmt->bindParam(1, $email);
 
-    if (!$load_stmt->execute())
-      throw new \Exception(sprintf('Email "%s" does not exist.', $email));
+    if (!$load_stmt->execute()) {
+          throw new \Exception(sprintf('Email "%s" does not exist.', $email));
+    }
 
-    if (!$load_stmt->fetch())
-      throw new \Exception(sprintf('Email "%s" does not exist.', $email));
+    if (!$load_stmt->fetch()) {
+          throw new \Exception(sprintf('Email "%s" does not exist.', $email));
+    }
 
     return $user;
   }
@@ -43,11 +45,13 @@ 
     $load_stmt->setFetchMode(\PDO::FETCH_INTO, $user);
     $load_stmt->bindParam(1, $id);
 
-    if (!$load_stmt->execute())
-      throw new \Exception(sprintf('User #%s does not exist.', $id));
+    if (!$load_stmt->execute()) {
+          throw new \Exception(sprintf('User #%s does not exist.', $id));
+    }
 
-    if (!$load_stmt->fetch())
-      throw new \Exception(sprintf('User #%s does not exist.', $id));
+    if (!$load_stmt->fetch()) {
+          throw new \Exception(sprintf('User #%s does not exist.', $id));
+    }
 
     return $user;
   }
@@ -61,11 +65,13 @@ 
     $load_stmt->setFetchMode(\PDO::FETCH_INTO, $user);
     $load_stmt->bindParam(1, $id);
 
-    if (!$load_stmt->execute())
-      throw new \Exception(sprintf('User #%s does not exist.', $id));
+    if (!$load_stmt->execute()) {
+          throw new \Exception(sprintf('User #%s does not exist.', $id));
+    }
 
-    if (!$load_stmt->fetch())
-      throw new \Exception(sprintf('User #%s does not exist.', $id));
+    if (!$load_stmt->fetch()) {
+          throw new \Exception(sprintf('User #%s does not exist.', $id));
+    }
 
     unset($user->enabled);
     unset($user->email);

tests/api/Domain/Validators/RoundTimeValidatorTest.php

Spacing +1 added lines, -1 removed lines

@@ -8,7 +8,7 @@
 
 class RoundTimeValidatorTest extends TestCase {
   function setUp() {
-    $this->app = require dirname(__FILE__).'/../../../../api/config/_app.php';
+    $this->app = require dirname(__FILE__) . '/../../../../api/config/_app.php';
     $this->roundTimeCreateModel = new RoundTimeCreateModel();
     $this->sut = new RoundTimeValidator($this->app);
   }