xoco70 / laravel-tournaments

src/TreeGen/TreeGen.php

Doc Comments +9 added lines, -10 removed lines

@@ -48,7 +48,7 @@ 
 
     /**
      * Get the biggest entity group
-     * @param $userGroups
+     * @param Collection $userGroups
      *
      * @return int
      */
@@ -107,7 +107,7 @@ 
 
     /**
      * Repart BYE in the tree,
-     * @param $fighterGroups
+     * @param Collection $fighterGroups
      * @param int $max
      *
      * @return Collection
@@ -153,7 +153,6 @@ 
      * @param Collection $usersByArea
      * @param integer $area
      * @param integer $round
-     * @param integer $shuffle
      *
      */
     public function generateGroupsForRound($usersByArea, $area, $round)
@@ -176,9 +175,9 @@ 
     }
 
     /**
-     * @param $area
-     * @param $order
-     * @param $round
+     * @param integer $area
+     * @param integer $order
+     * @param integer $round
      * @param $parent
      * @return FightersGroup
      */
@@ -339,8 +338,8 @@ 
 
     /**
      * @param Collection $fighters
-     * @param $frequency
-     * @param $sizeGroupBy
+     * @param integer $frequency
+     * @param integer $sizeGroupBy
      * @param $bye
      * @return Collection
      */
@@ -363,8 +362,8 @@ 
     /**
      * @param $frequency
      * @param $sizeGroupBy
-     * @param $count
-     * @param $byeCount
+     * @param integer $count
+     * @param integer $byeCount
      * @return bool
      */
     private function shouldInsertBye($frequency, $sizeGroupBy, $count, $byeCount): bool

src/models/FightersGroup.php

Braces +3 added lines, -1 removed lines

@@ -201,7 +201,9 @@
             $fightsByRound = $championship->fightsByRound($numRound)->with('group.parent', 'group.children')->get();
             foreach ($fightsByRound as $fight) {
                 $parentGroup = $fight->group->parent;
-                if ($parentGroup == null) break;
+                if ($parentGroup == null) {
+                    break;
+                }
                 $parentFight = $parentGroup->fights->get(0); //TODO This Might change when extending to Preliminary
 
                 // IN this $fight, is c1 or c2 has the info?