mattheworres / phpdraft

api/Domain/Validators/LoginUserValidator.php

Spacing +30 added lines, -30 removed lines

@@ -24,7 +24,7 @@ 
     $name = $request->get('_name');
     $recaptcha = $request->get('_recaptcha');
 
-    if(empty($password)
+    if (empty($password)
       || empty($confirmPassword)
       || empty($emailAddress)
       || empty($name)
@@ -41,7 +41,7 @@ 
 
     $this->validateEmailAddress($emailAddress, $errors, $valid);
 
-    if(!$this->app['phpdraft.LoginUserRepository']->NameIsUnique($name)) {
+    if (!$this->app['phpdraft.LoginUserRepository']->NameIsUnique($name)) {
       $errors[] = "Name already taken.";
       $valid = false;
     }
@@ -58,14 +58,14 @@ 
     $emailAddress = $request->get('_email');
     $verificationToken = $this->app['phpdraft.SaltService']->UrlDecodeSalt($request->get('_verificationToken'));
 
-    if(strlen($verificationToken) != 16) {
+    if (strlen($verificationToken) != 16) {
       $errors[] = "Verification token invalid.";
       $valid = false;
     }
 
     $this->validateEmailAddress($emailAddress, $errors, $valid);
 
-    if(!$this->app['phpdraft.LoginUserRepository']->VerificationMatches($emailAddress, $verificationToken)) {
+    if (!$this->app['phpdraft.LoginUserRepository']->VerificationMatches($emailAddress, $verificationToken)) {
       $errors[] = "Verification token invalid.";
       $valid = false;
     }
@@ -99,18 +99,18 @@ 
     $valid = true;
     $errors = array();
 
-    if(empty($emailAddress)
+    if (empty($emailAddress)
       || empty($verificationToken)) {
       $errors[] = "One or more missing fields";
       $valid = false;
     }
 
-    if(strlen($verificationToken) != 16) {
+    if (strlen($verificationToken) != 16) {
       $errors[] = "Verification token invalid.";
       $valid = false;
     }
 
-    if(!$this->app['phpdraft.LoginUserRepository']->VerificationMatches($emailAddress, $verificationToken)) {
+    if (!$this->app['phpdraft.LoginUserRepository']->VerificationMatches($emailAddress, $verificationToken)) {
       $errors[] = "Verification token invalid.";
       $valid = false;
     }
@@ -131,7 +131,7 @@ 
     $confirmPassword = $request->get('_confirmPassword');
     $verificationToken = $this->app['phpdraft.SaltService']->UrlDecodeSalt($request->get('_verificationToken'));
 
-    if(empty($emailAddress)
+    if (empty($emailAddress)
       || empty($password)
       || empty($confirmPassword)
       || empty($verificationToken)) {
@@ -139,12 +139,12 @@ 
       $valid = false;
     }
 
-    if(strlen($verificationToken) != 16) {
+    if (strlen($verificationToken) != 16) {
       $errors[] = "Verification token invalid.";
       $valid = false;
     }
 
-    if(!$this->app['phpdraft.LoginUserRepository']->VerificationMatches($emailAddress, $verificationToken)) {
+    if (!$this->app['phpdraft.LoginUserRepository']->VerificationMatches($emailAddress, $verificationToken)) {
       $errors[] = "Verification token invalid.";
       $valid = false;
     }
@@ -172,7 +172,7 @@ 
 
     $currentUser = $this->app['phpdraft.LoginUserService']->GetCurrentUser();
 
-    if(empty($currentUser) || $currentUser == null) {
+    if (empty($currentUser) || $currentUser == null) {
       $valid = false;
       $errors[] = "Invalid user.";
 
@@ -181,38 +181,38 @@ 
     }
 
     //Password required to make any changes
-    if(empty($password) || !$this->app['security.encoder.digest']->isPasswordValid($currentUser->password, $password, $currentUser->salt)) {
+    if (empty($password) || !$this->app['security.encoder.digest']->isPasswordValid($currentUser->password, $password, $currentUser->salt)) {
       $errors[] = "Incorrect password entered.";
       $valid = false;
     }
 
     //Need to verify new email
-    if(empty($emailAddress)) {
+    if (empty($emailAddress)) {
       $errors[] = "Email address is missing.";
       $valid = false;
-    } else if(!$this->app['phpdraft.StringsEqual']($emailAddress, $currentUser->email)) {
+    } else if (!$this->app['phpdraft.StringsEqual']($emailAddress, $currentUser->email)) {
       $this->validateEmailAddress($emailAddress, $errors, $valid);
 
       $this->validateUniqueEmail($emailAddress, $errors, $valid);
     }
 
     //Need to verify new password, ensure old password is correct
-    if(!empty($newPassword)) {
+    if (!empty($newPassword)) {
       $this->validatePasswordLength($newPassword, $errors, $valid);
 
       $this->validatePasswordsMatch($newPassword, $newConfirmedPassword, $errors, $valid);
     }
 
     //If the name has changed, ensure the new one is valid and unique
-    if($currentUser->name != $name) {
-      if(empty($name)) {
+    if ($currentUser->name != $name) {
+      if (empty($name)) {
         $errors[] = "Name is required.";
         $valid = false;
       }
 
       $this->validateNameLength($name, $errors, $valid);
 
-      if(!$this->app['phpdraft.LoginUserRepository']->NameIsUnique($name)) {
+      if (!$this->app['phpdraft.LoginUserRepository']->NameIsUnique($name)) {
         $errors[] = "Name already taken.";
         $valid = false;
       }
@@ -227,7 +227,7 @@ 
 
     $loadedUser = $this->app['phpdraft.LoginUserRepository']->LoadById($user->id);
 
-    if($user->id == 0 || empty($loadedUser)) {
+    if ($user->id == 0 || empty($loadedUser)) {
       $valid = false;
       $errors[] = "Invalid user.";
 
@@ -236,21 +236,21 @@ 
     }
 
     //Need to verify new email
-    if(empty($user->email)) {
+    if (empty($user->email)) {
       $errors[] = "Email address is missing.";
       $valid = false;
-    } else if(!$this->app['phpdraft.StringsEqual']($user->email, $loadedUser->email)) {
+    } else if (!$this->app['phpdraft.StringsEqual']($user->email, $loadedUser->email)) {
       $this->validateEmailAddress($user->email, $errors, $valid);
 
       $this->validateUniqueEmail($user->email, $errors, $valid);
     }
 
-    if(strlen($user->name) > 100) {
+    if (strlen($user->name) > 100) {
       $errors[] = "Name is above maximum length";
       $valid = false;
     }
 
-    if(!$this->app['phpdraft.LoginUserRepository']->NameIsUnique($user->name, $user->id)) {
+    if (!$this->app['phpdraft.LoginUserRepository']->NameIsUnique($user->name, $user->id)) {
       $errors[] = "Name already taken.";
       $valid = false;
     }
@@ -259,47 +259,47 @@ 
   }
 
   private function validatePasswordsMatch($password1, $password2, &$errors, &$valid) {
-    if(!$this->app['phpdraft.StringsEqual']($password1, $password2)) {
+    if (!$this->app['phpdraft.StringsEqual']($password1, $password2)) {
       $errors[] = "Password values do not match.";
       $valid = false;
     }
   }
 
   private function validateUniqueEmail($emailAddress, &$errors, &$valid) {
-    if(!$this->app['phpdraft.LoginUserRepository']->EmailIsUnique($emailAddress)) {
+    if (!$this->app['phpdraft.LoginUserRepository']->EmailIsUnique($emailAddress)) {
       $errors[] = "Email already registered.";
       $valid = false;
     }
   }
 
   private function validateEmailExists($emailAddress, &$errors, &$valid) {
-    if(!$this->app['phpdraft.LoginUserRepository']->EmailExists($emailAddress)) {
+    if (!$this->app['phpdraft.LoginUserRepository']->EmailExists($emailAddress)) {
       $errors[] = "Email invalid.";
       $valid = false;
     }
   }
 
   private function validateEmailAddress($emailAddress, &$errors, &$valid) {
-    if(!$this->app['phpdraft.EmailValidator']->isValid($emailAddress) || strlen($emailAddress) > 255) {
+    if (!$this->app['phpdraft.EmailValidator']->isValid($emailAddress) || strlen($emailAddress) > 255) {
       $errors[] = "Email invalid.";
       $valid = false;
     }
   }
 
   private function validatePasswordLength($password, &$errors, &$valid) {
-    if(strlen($password) < 8) {
+    if (strlen($password) < 8) {
       $errors[] = "Password is below minimum length.";
       $valid = false;
     }
 
-    if(strlen($password) > 255) {
+    if (strlen($password) > 255) {
       $errors[] = "Password is above maximum length.";
       $valid = false;
     }
   }
 
   private function validateNameLength($name, &$errors, &$valid) {
-    if(strlen($name) > 100) {
+    if (strlen($name) > 100) {
       $errors[] = "Name is above maximum length";
       $valid = false;
     }

api/Domain/Validators/ManagerValidator.php

Spacing +22 added lines, -22 removed lines

@@ -20,29 +20,29 @@ 
 
     try {
       $current_manager_count = $this->app['phpdraft.ManagerRepository']->GetNumberOfCurrentManagers($draft_id);
-    } catch(\Exception $e) {
+    } catch (\Exception $e) {
       $errors[] = $e->getMessage();
       $valid = false;
       $current_manager_count = 0;
     }
     
-    if(empty($manager->draft_id)
+    if (empty($manager->draft_id)
       || empty($manager->manager_name)) {
       $errors[] = "One or more missing fields.";
       $valid = false;
     }
 
-    if(!empty($manager->manager_name) && strlen($manager->manager_name) > 255) {
+    if (!empty($manager->manager_name) && strlen($manager->manager_name) > 255) {
       $errors[] = "Manager name is above maximum length.";
       $valid = false;
     }
 
-    if($current_manager_count > 19) {
+    if ($current_manager_count > 19) {
       $errors[] = "Unable to add more managers - 20 is the maximum number of managers allowed.";
       $valid = false;
     }
 
-    if(!$this->app['phpdraft.ManagerRepository']->NameIsUnique($manager->manager_name)) {
+    if (!$this->app['phpdraft.ManagerRepository']->NameIsUnique($manager->manager_name)) {
       $errors[] = "Manager name '$manager->manager_name' already exists - please choose another name.";
       $valid = false;
     }
@@ -56,34 +56,34 @@ 
     $current_manager_count = $this->app['phpdraft.ManagerRepository']->GetNumberOfCurrentManagers($draft_id);
     $numberOfNewManagers = count($managersArray);
     $maxNumberOfManagers = 20 - $current_manager_count;
-    $numberOfManagersOver = $numberOfNewManagers-$maxNumberOfManagers;
+    $numberOfManagersOver = $numberOfNewManagers - $maxNumberOfManagers;
     $uniqueManagersCount = count(array_unique($managersArray));
 
-    if($numberOfNewManagers > $maxNumberOfManagers) {
+    if ($numberOfNewManagers > $maxNumberOfManagers) {
       $manager_noun = $numberOfManagersOver > 1 ? "managers" : "manager";
       $errors[] = "Unable to add $numberOfNewManagers new managers - the maximum number of managers is 20. Draft currently has $current_manager_count, so please remove $numberOfManagersOver $manager_noun to continue.";
       $valid = false;
     }
 
-    if($numberOfNewManagers != $uniqueManagersCount) {
+    if ($numberOfNewManagers != $uniqueManagersCount) {
       $errors[] = "Two or more managers had the same name - please ensure all managers have unique names.";
       $valid = false;
     }
 
     $manager_number = 1;
-    foreach($managersArray as $manager) {
-      if(empty($manager->draft_id)
+    foreach ($managersArray as $manager) {
+      if (empty($manager->draft_id)
         || empty($manager->manager_name)) {
         $errors[] = "One or more missing fields for manager #$manager_number.";
         $valid = false;
       }
 
-      if(!empty($manager->manager_name) && strlen($manager->manager_name) > 255) {
+      if (!empty($manager->manager_name) && strlen($manager->manager_name) > 255) {
         $errors[] = "Manager #$manager_number's name is above the maximum length.";
         $valid = false;
       }
 
-      if(!$this->app['phpdraft.ManagerRepository']->NameIsUnique($manager->manager_name, $draft_id)) {
+      if (!$this->app['phpdraft.ManagerRepository']->NameIsUnique($manager->manager_name, $draft_id)) {
         $errors[] = "Manager name '$manager->manager_name' already exists - please choose another name.";
         $valid = false;
       }
@@ -100,26 +100,26 @@ 
     $current_manager_count = $this->app['phpdraft.ManagerRepository']->GetNumberOfCurrentManagers($draft_id);
     $ids_given = count($managersIdArray);
 
-    if(count($managersIdArray) == 0) {
+    if (count($managersIdArray) == 0) {
       $errors[] = "One or more missing fields.";
       $valid = false;
     }
 
-    if($ids_given > 20) {
+    if ($ids_given > 20) {
       $errors[] = "Too many managers given - maximum number of managers is 20";
       $valid = false;
     }
 
-    if($ids_given != $current_manager_count) {
+    if ($ids_given != $current_manager_count) {
       $manager_noun = $current_manager_count > 1 ? "managers" : "manager";
       $errors[] = "Incorrect number of managers given. Draft #$draft_id has $current_manager_count $manager_noun, but $ids_given were received.";
       $valid = false;
     }
 
     //Save ourself a little processing if we've been handled a faulty request:
-    if($valid = true) {
-      foreach($managersIdArray as $manager_id) {
-        if(!$this->app['phpdraft.ManagerRepository']->ManagerExists($manager_id, $draft_id)) {
+    if ($valid = true) {
+      foreach ($managersIdArray as $manager_id) {
+        if (!$this->app['phpdraft.ManagerRepository']->ManagerExists($manager_id, $draft_id)) {
           $errors[] = "Manager #$manager_id does not exist.";
           $valid = false;
         }
@@ -133,24 +133,24 @@ 
     $valid = true;
     $errors = array();
     
-    if(empty($manager->manager_id)
+    if (empty($manager->manager_id)
       || empty($manager->draft_id)
       || empty($manager->manager_name)) {
       $errors[] = "One or more missing fields.";
       $valid = false;
     }
 
-    if($draft->draft_id != $manager->draft_id) {
+    if ($draft->draft_id != $manager->draft_id) {
       $errors[] = "Manager does not belong to draft #$draft_id";
       $valid = false;
     }
 
-    if(!empty($manager->manager_name) && strlen($manager->manager_name) > 255) {
+    if (!empty($manager->manager_name) && strlen($manager->manager_name) > 255) {
       $errors[] = "Manager name is above maximum length.";
       $valid = false;
     }
 
-    if(!$this->app['phpdraft.ManagerRepository']->NameIsUnique($manager->manager_name, $draft->draft_id, $manager->manager_id)) {
+    if (!$this->app['phpdraft.ManagerRepository']->NameIsUnique($manager->manager_name, $draft->draft_id, $manager->manager_id)) {
       $errors[] = "Manager name '$manager->manager_name' already exists - please choose another name.";
       $valid = false;
     }

api/Domain/Validators/PickValidator.php

Spacing +14 added lines, -14 removed lines

@@ -20,7 +20,7 @@ 
     $teams = $this->app['phpdraft.DraftDataRepository']->GetTeams($draft->draft_sport);
     $positions = $this->app['phpdraft.DraftDataRepository']->GetPositions($draft->draft_sport);
 
-    if(empty($pick->first_name)
+    if (empty($pick->first_name)
       || empty($pick->last_name)
       || empty($pick->team)
       || empty($pick->position)) {
@@ -28,32 +28,32 @@ 
       $valid = false;
     }
 
-    if($pick->draft_id != $draft->draft_id) {
+    if ($pick->draft_id != $draft->draft_id) {
       $errors[] = "Pick does not belong to draft #$draft->draft_id.";
       $valid = false;
     }
 
-    if(strlen($pick->first_name) > 255) {
+    if (strlen($pick->first_name) > 255) {
       $errors[] = "First name is above maximum length.";
       $valid = false;
     }
 
-    if(strlen($pick->last_name) > 255) {
+    if (strlen($pick->last_name) > 255) {
       $errors[] = "Last name is above maximum length.";
       $valid = false;
     }
 
-    if($draft->draft_current_pick != $pick->player_pick) {
+    if ($draft->draft_current_pick != $pick->player_pick) {
       $errors[] = "Pick #$pick->player_pick is not the current pick for draft #$draft->draft_id.";
       $valid = false;
     }
 
-    if(!array_key_exists($pick->team, $teams)) {
+    if (!array_key_exists($pick->team, $teams)) {
       $errors[] = "Team $pick->team is an invalid value.";
       $valid = false;
     }
 
-    if(!array_key_exists($pick->position, $positions)) {
+    if (!array_key_exists($pick->position, $positions)) {
       $errors[] = "Position $pick->position is an invalid value.";
       $valid = false;
     }
@@ -67,7 +67,7 @@ 
     $teams = $this->app['phpdraft.DraftDataRepository']->GetTeams($draft->draft_sport);
     $positions = $this->app['phpdraft.DraftDataRepository']->GetPositions($draft->draft_sport);
 
-    if(empty($pick->first_name)
+    if (empty($pick->first_name)
       || empty($pick->last_name)
       || empty($pick->team)
       || empty($pick->position)) {
@@ -75,27 +75,27 @@ 
       $valid = false;
     }
 
-    if($pick->draft_id != $draft->draft_id) {
+    if ($pick->draft_id != $draft->draft_id) {
       $errors[] = "Pick does not belong to draft #$draft->draft_id.";
       $valid = false;
     }
 
-    if(strlen($pick->first_name) > 255) {
+    if (strlen($pick->first_name) > 255) {
       $errors[] = "First name is above maximum length.";
       $valid = false;
     }
 
-    if(strlen($pick->last_name) > 255) {
+    if (strlen($pick->last_name) > 255) {
       $errors[] = "Last name is above maximum length.";
       $valid = false;
     }
 
-    if(!array_key_exists($pick->team, $teams)) {
+    if (!array_key_exists($pick->team, $teams)) {
       $errors[] = "Team $pick->team is an invalid value.";
       $valid = false;
     }
 
-    if(!array_key_exists($pick->position, $positions)) {
+    if (!array_key_exists($pick->position, $positions)) {
       $errors[] = "Position $pick->position is an invalid value.";
       $valid = false;
     }
@@ -107,7 +107,7 @@ 
     $valid = true;
     $errors = array();
 
-    if($pick->draft_id != $draft->draft_id) {
+    if ($pick->draft_id != $draft->draft_id) {
       $errors[] = "Pick does not belong to draft #$draft->draft_id.";
       $valid = false;
     }

api/Domain/Validators/DepthChartPositionValidator.php

Spacing +6 added lines, -6 removed lines

@@ -17,7 +17,7 @@ 
     $errors = array();
     $draft_sports = $this->app['phpdraft.DraftDataRepository']->GetSports();
 
-    if(strlen($draft_sport) < 3 || strlen($draft_sport) > 4 || strlen($draft_sports[$draft_sport]) == 0) {
+    if (strlen($draft_sport) < 3 || strlen($draft_sport) > 4 || strlen($draft_sports[$draft_sport]) == 0) {
       $errors[] = "Draft sport is an invalid value.";
       $valid = false;
     }
@@ -29,28 +29,28 @@ 
     $valid = true;
     $errors = array();
 
-    if(count($depthChartPositionCreateModel->positions) !== count(array_unique($depthChartPositionCreateModel->positions))) {
+    if (count($depthChartPositionCreateModel->positions) !== count(array_unique($depthChartPositionCreateModel->positions))) {
       $valid = false;
       $errors[] = "One or more of the positions are not unique.";
     }
 
     $rounds = 0;
 
-    foreach($depthChartPositionCreateModel->positions as $depthChartPosition) {
+    foreach ($depthChartPositionCreateModel->positions as $depthChartPosition) {
       $rounds += (int)$depthChartPosition->slots;
     }
 
-    if($rounds == 0) {
+    if ($rounds == 0) {
       $valid = false;
       $errors[] = "The depth chart positions must specify at least 1 slot";
     }
 
-    if($rounds > 30) {
+    if ($rounds > 30) {
       $valid = false;
       $errors[] = "The depth chart positions cannot specify more than 30 slots in total.";
     }
 
-    if(count($depthChartPositionCreateModel->positions) > 30) {
+    if (count($depthChartPositionCreateModel->positions) > 30) {
       $valid = false;
       $errors[] = "Too many depth chart positions have been provided - 30 is the maximum.";
     }

api/Domain/Validators/DraftValidator.php

Spacing +25 added lines, -25 removed lines

@@ -21,11 +21,11 @@ 
     $current_user = $this->app['phpdraft.LoginUserService']->GetUserFromHeaderToken($request);
     $draft_password = $request->headers->get(DRAFT_PASSWORD_HEADER, '');
 
-    if($current_user != null && ($draft->commish_id == $current_user->id || $this->app['phpdraft.LoginUserService']->CurrentUserIsAdmin($current_user))) {
+    if ($current_user != null && ($draft->commish_id == $current_user->id || $this->app['phpdraft.LoginUserService']->CurrentUserIsAdmin($current_user))) {
       return true;
     }
 
-    if(empty($draft->draft_password) || ($draft->draft_password == $draft_password)) {
+    if (empty($draft->draft_password) || ($draft->draft_password == $draft_password)) {
       return true;
     }
 
@@ -33,11 +33,11 @@ 
   }
 
   public function IsDraftEditableForUser(Draft $draft, LoginUser $current_user) {
-    if(!empty($current_user) && !empty($draft) && $draft->commish_id == $current_user->id) {
+    if (!empty($current_user) && !empty($draft) && $draft->commish_id == $current_user->id) {
       return true;
     }
 
-    if(!empty($current_user) && $this->app['phpdraft.LoginUserService']->CurrentUserIsAdmin($current_user)) {
+    if (!empty($current_user) && $this->app['phpdraft.LoginUserService']->CurrentUserIsAdmin($current_user)) {
       return true;
     }
 
@@ -50,7 +50,7 @@ 
     $draft_sports = $this->app['phpdraft.DraftDataRepository']->GetSports();
     $draft_styles = $this->app['phpdraft.DraftDataRepository']->GetStyles();
 
-    if(empty($draft->commish_id)
+    if (empty($draft->commish_id)
       || empty($draft->draft_name)
       || empty($draft->draft_sport)
       || empty($draft->draft_style)) {
@@ -58,33 +58,33 @@ 
       $valid = false;
     }
 
-    if(!empty($draft->draft_password) && strlen($draft->draft_password) > 255) {
+    if (!empty($draft->draft_password) && strlen($draft->draft_password) > 255) {
       $errors[] = "Password is above maximum length.";
       $valid = false;
     }
 
-    if(strlen($draft->draft_name) > 255) {
+    if (strlen($draft->draft_name) > 255) {
       $errors[] = "Draft name is above maximum length";
       $valid = false;
     }
 
-    if(strlen($draft->draft_sport) < 3 || strlen($draft->draft_sport) > 4 || strlen($draft_sports[$draft->draft_sport]) == 0) {
+    if (strlen($draft->draft_sport) < 3 || strlen($draft->draft_sport) > 4 || strlen($draft_sports[$draft->draft_sport]) == 0) {
       $errors[] = "Draft sport is an invalid value.";
       $valid = false;
     }
 
-    if(!array_key_exists($draft->draft_style, $draft_styles)) {
+    if (!array_key_exists($draft->draft_style, $draft_styles)) {
 
       $errors[] = "Draft style is an invalid value.";
       $valid = false;
     }
 
-    if(!$this->app['phpdraft.DraftRepository']->NameIsUnique($draft->draft_name, $draft->draft_id)) {
+    if (!$this->app['phpdraft.DraftRepository']->NameIsUnique($draft->draft_name, $draft->draft_id)) {
       $errors[] = "Draft name is already taken, please choose a different name.";
       $valid = false;
     }
 
-    if($draft->draft_rounds < 1 || $draft->draft_rounds > 30) {
+    if ($draft->draft_rounds < 1 || $draft->draft_rounds > 30) {
       $errors[] = "Invalid number of draft rounds.";
       $valid = false;
     }
@@ -97,27 +97,27 @@ 
     $errors = array();
     $draft_statuses = $this->app['phpdraft.DraftDataRepository']->GetStatuses();
 
-    if($old_status == "complete") {
+    if ($old_status == "complete") {
       $valid = false;
       $errors[] = "The draft is completed, therefore its status cannot be changed.";
     }
 
-    if($draft->draft_status == "complete") {
+    if ($draft->draft_status == "complete") {
       $valid = false;
       $errors[] = "You cannot set the draft as completed manually.";
     }
 
-    if(empty($draft->draft_status)) {
+    if (empty($draft->draft_status)) {
       $errors[] = "One or more missing fields.";
       $valid = false;
     }
 
-    if(!array_key_exists($draft->draft_status, $draft_statuses)) {
+    if (!array_key_exists($draft->draft_status, $draft_statuses)) {
       $errors[] = "Draft status is an invalid value.";
       $valid = false;
     }
 
-    if($draft->draft_status == "in_progress" && !$this->app['phpdraft.ManagerRepository']->DraftHasManagers($draft->draft_id)) {
+    if ($draft->draft_status == "in_progress" && !$this->app['phpdraft.ManagerRepository']->DraftHasManagers($draft->draft_id)) {
       $valid = false;
       $errors[] = "A draft must have at least 2 managers before it can begin.";
     }
@@ -132,7 +132,7 @@ 
     $current_status_text = strtolower($draft_statuses[$draft->draft_status]);
     $is_setting_up = $this->app['phpdraft.DraftService']->DraftSettingUp($draft);
 
-    if(!$is_setting_up) {
+    if (!$is_setting_up) {
       $valid = false;
       $errors[] = "Unable to work on draft #$draft->draft_id: draft is $current_status_text";
     }
@@ -148,7 +148,7 @@ 
     $is_setting_up = $this->app['phpdraft.DraftService']->DraftSettingUp($draft);
     $is_in_progress = $this->app['phpdraft.DraftService']->DraftInProgress($draft);
 
-    if(!$is_setting_up && !$is_in_progress) {
+    if (!$is_setting_up && !$is_in_progress) {
       $valid = false;
       $errors[] = "Unable to work on draft #$draft->draft_id: draft is $current_status_text";
     }
@@ -163,7 +163,7 @@ 
     $current_status_text = strtolower($draft_statuses[$draft->draft_status]);
     $is_in_progress = $this->app['phpdraft.DraftService']->DraftInProgress($draft);
 
-    if(!$is_in_progress) {
+    if (!$is_in_progress) {
       $valid = false;
       $errors[] = "Unable to work on draft #$draft->draft_id: draft is $current_status_text";
     }
@@ -179,7 +179,7 @@ 
     $is_in_progress = $this->app['phpdraft.DraftService']->DraftInProgress($draft);
     $is_complete = $this->app['phpdraft.DraftService']->DraftComplete($draft);
 
-    if(!$is_complete && !$is_in_progress) {
+    if (!$is_complete && !$is_in_progress) {
       $valid = false;
       $errors[] = "Unable to work on draft #$draft->draft_id: draft is $current_status_text";
     }
@@ -196,7 +196,7 @@ 
     $is_complete = $this->app['phpdraft.DraftService']->DraftComplete($draft);
     $isLessThanTenMinutes = false;
 
-    if($is_complete) {
+    if ($is_complete) {
       $utc_timezone = new \DateTimeZone("UTC");
       $now_utc = new \DateTime(null, $utc_timezone);
       $end_time = new \DateTime($draft->draft_end_time, $utc_timezone);
@@ -204,17 +204,17 @@ 
       $end_seconds = $end_time->getTimestamp();
       $seconds_elapsed = $end_seconds - $now_seconds;
 
-      if($seconds_elapsed < 600) {
+      if ($seconds_elapsed < 600) {
         $isLessThanTenMinutes = true;
       }
     }
 
-    if(!$is_complete && !$is_in_progress) {
+    if (!$is_complete && !$is_in_progress) {
       $valid = false;
       $errors[] = "Unable to work on draft #$draft->draft_id: draft is $current_status_text";
     }
 
-    if($is_complete && !$isLessThanTenMinutes) {
+    if ($is_complete && !$isLessThanTenMinutes) {
       $valid = false;
       $errors[] = "Unable to work on draft #$draft->draft_id: draft is complete and beyond the ten minute grace period.";
     }
@@ -229,7 +229,7 @@ 
     $current_status_text = strtolower($draft_statuses[$draft->draft_status]);
     $is_complete = $this->app['phpdraft.DraftService']->DraftComplete($draft);
 
-    if(!$is_complete) {
+    if (!$is_complete) {
       $valid = false;
       $errors[] = "Unable to work on draft #$draft->draft_id: draft is $current_status_text";
     }

api/Domain/Services/TradeService.php

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
 
       $response->success = true;
       $response->trade = $trade;
-    } catch(\Exception $e) {
+    } catch (\Exception $e) {
       $response->success = false;
       $response->errors = array($e->getMessage());
     }

api/Domain/Services/DraftService.php

Braces +3 added lines, -3 removed lines

@@ -34,7 +34,7 @@ 
 
       $response->success = true;
       $response->draft = $draft;
-    }catch(\Exception $e) {
+    } catch(\Exception $e) {
       $response->success = false;
       $response->errors = array($e->getMessage());
     }
@@ -55,7 +55,7 @@ 
 
       $response->success = true;
       $response->draft = $draft;
-    }catch(\Exception $e) {
+    } catch(\Exception $e) {
       $response->success = false;
       $response->errors = array($e->getMessage());
     }
@@ -85,7 +85,7 @@ 
 
       $response->success = true;
       $response->draft = $draft;
-    }catch(\Exception $e) {
+    } catch(\Exception $e) {
       $response->success = false;
       $response->errors = array($e->getMessage());
     }

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/Services/RoundTimeService.php

Spacing +3 added lines, -3 removed lines

@@ -24,7 +24,7 @@ 
 
       $response->success = true;
       $response->roundTimes = $roundTimes;
-    }catch(\Exception $e) {
+    } catch (\Exception $e) {
       $response->success = false;
       $response->errors = array($e->getMessage());
     }
@@ -39,7 +39,7 @@ 
       $current_pick = $this->app['phpdraft.PickRepository']->GetCurrentPick($draft);
       $last_pick = $this->app['phpdraft.PickRepository']->GetPreviousPick($draft);
       $current_round_picktime = $this->app['phpdraft.RoundTimeRepository']->LoadByRound($draft);
-    } catch(\Exception $e) {
+    } catch (\Exception $e) {
       $response->success = false;
       $response->errors = array($e->getMessage());
     }
@@ -47,7 +47,7 @@ 
     $response->success = true;
     $response->timer_enabled = $current_round_picktime != null;
 
-    if(!$response->timer_enabled) {
+    if (!$response->timer_enabled) {
       return $response;
     }
 

Braces +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
 
       $response->success = true;
       $response->roundTimes = $roundTimes;
-    }catch(\Exception $e) {
+    } catch(\Exception $e) {
       $response->success = false;
       $response->errors = array($e->getMessage());
     }

api/Domain/Services/ManagerService.php

Spacing +5 added lines, -5 removed lines

@@ -21,7 +21,7 @@ 
 
       $response->success = true;
       $response->manager = $manager;
-    }catch(\Exception $e) {
+    } catch (\Exception $e) {
       $response->success = false;
       $response->errors = array($e->getMessage());
     }
@@ -37,7 +37,7 @@ 
 
       $response->success = true;
       $response->managers = $managers;
-    }catch(\Exception $e) {
+    } catch (\Exception $e) {
       $response->success = false;
       $response->errors = array($e->getMessage());
     }
@@ -52,7 +52,7 @@ 
       $this->app['phpdraft.ManagerRepository']->ReorderManagers($managersIdArray);
       
       $response->success = true;
-    }catch(\Exception $e) {
+    } catch (\Exception $e) {
       $response->success = false;
       $response->errors = array($e->getMessage());
     }
@@ -87,7 +87,7 @@ 
       $managers = $this->app['phpdraft.ManagerRepository']->GetManagersByDraftOrder($draft_id);
       $managersIdArray = array();
 
-      foreach($managers as $manager) {
+      foreach ($managers as $manager) {
         $managersIdArray[] = $manager->manager_id;
       }
 
@@ -97,7 +97,7 @@ 
 
       $response->managers = $managers;
       $response->success = true;
-    } catch(\Exception $e) {
+    } catch (\Exception $e) {
       $response->success = false;
       $response->errors = array($e->getMessage());
     }

Braces +3 added lines, -3 removed lines

@@ -21,7 +21,7 @@ 
 
       $response->success = true;
       $response->manager = $manager;
-    }catch(\Exception $e) {
+    } catch(\Exception $e) {
       $response->success = false;
       $response->errors = array($e->getMessage());
     }
@@ -37,7 +37,7 @@ 
 
       $response->success = true;
       $response->managers = $managers;
-    }catch(\Exception $e) {
+    } catch(\Exception $e) {
       $response->success = false;
       $response->errors = array($e->getMessage());
     }
@@ -52,7 +52,7 @@ 
       $this->app['phpdraft.ManagerRepository']->ReorderManagers($managersIdArray);
       
       $response->success = true;
-    }catch(\Exception $e) {
+    } catch(\Exception $e) {
       $response->success = false;
       $response->errors = array($e->getMessage());
     }