xoco70 / laravel-tournaments

src/models/Championship.php

Doc Comments +2 added lines, -1 removed lines

@@ -208,7 +208,7 @@ 
 
     /**
      * Return Groups that belongs to a round
-     * @param $round
+     * @param integer $round
      * @return HasMany
      */
     public function groupsByRound($round)
@@ -219,6 +219,7 @@ 
     /**
      * Return Fights that belongs to a round
      *
+     * @param integer $round
      * @return \Illuminate\Database\Eloquent\Relations\HasManyThrough
      */
     public function fightsByRound($round)

src/TreeGen/TreeGen.php

Doc Comments +11 added lines, -6 removed lines

@@ -226,8 +226,10 @@ 
     }
 
     /**
-     * @param $usersByArea
-     * @param $area
+     * @param Collection $usersByArea
+     * @param integer $area
+     * @param integer $round
+     * @param integer $shuffle
      *
      */
     public function generateGroupsForRound($usersByArea, $area, $round, $shuffle)
@@ -247,7 +249,7 @@ 
     /**
      * @param $fighters
      * @param $area
-     * @param $order
+     * @param integer $order
      * @param $round
      * @return FightersGroup
      */
@@ -312,6 +314,9 @@ 
             : new Competitor();
     }
 
+    /**
+     * @param integer $groupSize
+     */
     public function createByeGroup($groupSize): Collection
     {
         $byeFighter = $this->createByeFighter();
@@ -324,7 +329,7 @@ 
 
     /**
      * @param $fighters
-     * @param $fighterGroups
+     * @param Collection $fighterGroups
      * @return Collection
      */
     public function adjustFightersGroupWithByes($fighters, $fighterGroups): Collection
@@ -359,7 +364,7 @@ 
     /**
      * Get the next group on the right ( parent ), final round being the ancestor
      * @param $matchNumber
-     * @param $previousRound
+     * @param Collection $previousRound
      * @return mixed
      */
     private function getParentGroup($matchNumber, $previousRound)
@@ -371,7 +376,7 @@ 
 
     /**
      * Save Groups with their parent info
-     * @param $numRounds
+     * @param integer $numRounds
      * @param $numFightersEliminatory
      */
     private function pushGroups($numRounds, $numFightersEliminatory, $shuffle = true)

Braces +9 added lines, -3 removed lines

@@ -255,7 +255,9 @@ 
     {
 
         $fighters = $fighters->pluck('id');
-        if ($shuffle) $fighters->shuffle();
+        if ($shuffle) {
+            $fighters->shuffle();
+        }
         $group = $this->saveGroup($area, $order, $round, $parent);
 
         // Add all competitors to Pivot Table
@@ -417,10 +419,14 @@ 
     {
         if ($this->championship->hasPreliminary()) {
             $fightersGroup = $fightersByEntity->chunk($this->settings->preliminaryGroupSize);
-            if ($shuffle) $fightersGroup->shuffle();
+            if ($shuffle) {
+                $fightersGroup->shuffle();
+            }
         } elseif ($this->championship->isDirectEliminationType() || $round > 1) {
             $fightersGroup = $fightersByEntity->chunk(2);
-            if ($shuffle) $fightersGroup->shuffle();
+            if ($shuffle) {
+                $fightersGroup->shuffle();
+            }
         } else { // Round Robin
             $fightersGroup = $fightersByEntity->chunk($fightersByEntity->count());
         }

src/models/Fight.php

Braces +9 added lines, -3 removed lines

@@ -289,9 +289,15 @@
      */
     public function getValueToUpdate()
     {
-        if ($this->c1 != null && $this->c2 != null) return null;
-        if ($this->c1 != null) return "c1";
-        if ($this->c2 != null) return "c2";
+        if ($this->c1 != null && $this->c2 != null) {
+            return null;
+        }
+        if ($this->c1 != null) {
+            return "c1";
+        }
+        if ($this->c2 != null) {
+            return "c2";
+        }
         return null;
     }
 }
\ No newline at end of file

src/models/FightersGroup.php

Braces +3 added lines, -1 removed lines

@@ -206,7 +206,9 @@
             foreach ($fightsByRound as $numFight => $fight) {
 
                 $parentGroup = $fight->group->parent;
-                if ($parentGroup == null ) break;
+                if ($parentGroup == null ) {
+                    break;
+                }
                 $parentFight = $parentGroup->fights->get(0);
 
                 // IN this $fight, if it is the first child or the second child --> c1 or c2 in parent

src/TreeGen/DirectEliminationTreeGen.php

Braces +3 added lines, -1 removed lines

@@ -53,7 +53,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