xoco70 / laravel-tournaments

src/models/Championship.php

Braces +3 added lines, -2 removed lines

@@ -157,8 +157,9 @@
 
     public function buildName()
     {
-        if ($this->settings != null && $this->settings->alias != null && $this->settings->alias != '')
-            return $this->settings->alias;
+        if ($this->settings != null && $this->settings->alias != null && $this->settings->alias != '') {
+                    return $this->settings->alias;
+        }
 
         if ($this->hasNoCustomSettings()) {
             return $this->category->name;

Doc Comments +1 added lines, -1 removed lines

@@ -206,7 +206,7 @@
 
     /**
      * Return Groups that belongs to the round in param
-     * @param $round
+     * @param integer $round
      * @return Collection
      */
     public function groupsByRound($round) : Collection

src/models/Team.php

Braces +3 added lines, -1 removed lines

@@ -45,7 +45,9 @@
 
     public function getName()
     {
-        if ($this == null) return "BYE";
+        if ($this == null) {
+            return "BYE";
+        }
         return $this->name;
     }
 

src/TreeGen/DirectEliminationTreeGen.php

Braces +4 added lines, -2 removed lines

@@ -65,7 +65,9 @@ 
         $matches = array_chunk($fighters, 2);
 
         //If there's already a match in the match array, then that means the next round is round 2, so increase the round number
-        if (count($this->brackets)) $roundNumber++;
+        if (count($this->brackets)) {
+            $roundNumber++;
+        }
 
         $countMatches = count($matches);
         //Create the first full round of fighters, some may be blank if waiting on the results of a previous round
@@ -80,7 +82,7 @@ 
                 if ($this->championship->category->isTeam()){
                     $fighter1 = $this->names->get($roundNumber)[0]->fights[$matchNumber-1]->team1;
                     $fighter2 = $this->names->get($roundNumber)[0]->fights[$matchNumber-1]->team2;
-                }else{
+                } else{
                     $fighter1 = $this->names->get($roundNumber)[$matchNumber-1]->fights[0]->competitor1;
                     $fighter2 = $this->names->get($roundNumber)[$matchNumber-1]->fights[0]->competitor2;
                 }

src/models/Competitor.php

Braces +12 added lines, -4 removed lines

@@ -86,15 +86,23 @@
 
     public function getName()
     {
-        if ($this == null) return "BYE";
-        if ($this->user == null) return "BYE";
+        if ($this == null) {
+            return "BYE";
+        }
+        if ($this->user == null) {
+            return "BYE";
+        }
         return $this->user->name;
     }
 
     public function getFullName()
     {
-        if ($this == null) return "BYE";
-        if ($this->user == null) return "BYE";
+        if ($this == null) {
+            return "BYE";
+        }
+        if ($this->user == null) {
+            return "BYE";
+        }
         return $this->user->firstname . " " . $this->user->lastname;
     }
 

src/TreeGen/TreeGen.php

Doc Comments +9 added lines, -5 removed lines

@@ -240,8 +240,9 @@ 
     }
 
     /**
-     * @param $usersByArea
-     * @param $area
+     * @param Collection $usersByArea
+     * @param integer $area
+     * @param integer $round
      *
      */
     public function generateGroupsForRound($usersByArea, $area, $round)
@@ -335,6 +336,9 @@ 
             : new Competitor();
     }
 
+    /**
+     * @param integer $groupSize
+     */
     public function createByeGroup($groupSize): Collection
     {
         $byeFighter = $this->createByeFighter();
@@ -347,7 +351,7 @@ 
 
     /**
      * @param $fighters
-     * @param $fighterGroups
+     * @param Collection $fighterGroups
      * @return Collection
      */
     private function adjustFightersGroupWithByes($fighters, $fighterGroups): Collection
@@ -385,7 +389,7 @@ 
      * @param $roundNumber
      * @param $numRounds
      * @param $matchNumber
-     * @param $previousRound
+     * @param Collection|null $previousRound
      * @return mixed
      */
     private function getParentGroup($roundNumber, $numRounds = 0, $matchNumber, $previousRound)
@@ -399,7 +403,7 @@ 
     }
 
     /**
-     * @param $numRounds
+     * @param integer $numRounds
      * @param $numFightersEliminatory
      */
     private function pushGroups($numRounds, $numFightersEliminatory)