xoco70 / laravel-tournaments

src/TreeGen/TreeGen.php

Doc Comments +13 added lines, -7 removed lines

@@ -22,8 +22,14 @@ 
     abstract protected function generateFights();
 
 
+    /**
+     * @param string|null $fighterToUpdate
+     */
     abstract protected function addFighterToGroup(FightersGroup $group, $fighter, $fighterToUpdate);
 
+    /**
+     * @return Collection
+     */
     abstract protected function getByeGroup($fighters);
 
     abstract protected function getNumRounds($fightersCount);
@@ -61,7 +67,7 @@ 
     /**
      * Get the biggest entity group.
      *
-     * @param $userGroups
+     * @param Collection $userGroups
      *
      * @return int
      */
@@ -130,7 +136,7 @@ 
     /**
      * Repart BYE in the tree,.
      *
-     * @param $fighterGroups
+     * @param Collection $fighterGroups
      * @param int $max
      *
      * @return Collection
@@ -288,7 +294,7 @@ 
     /**
      * Attach a parent to every child for nestedSet Navigation.
      *
-     * @param $numFightersElim
+     * @param integer $numFightersElim
      */
     protected function addParentToChildren($numFightersElim)
     {
@@ -312,8 +318,8 @@ 
 
     /**
      * @param Collection $fighters
-     * @param $frequency
-     * @param $sizeGroupBy
+     * @param integer $frequency
+     * @param integer $sizeGroupBy
      * @param $bye
      *
      * @return Collection
@@ -338,8 +344,8 @@ 
     /**
      * @param $frequency
      * @param $sizeGroupBy
-     * @param $count
-     * @param $byeCount
+     * @param integer $count
+     * @param integer $byeCount
      *
      * @return bool
      */

Spacing +4 added lines, -4 removed lines

@@ -68,7 +68,7 @@ 
     private function getMaxFightersByEntity($userGroups): int
     {
         return $userGroups
-            ->sortByDesc(function ($group) {
+            ->sortByDesc(function($group) {
                 return $group->count();
             })
             ->first()
@@ -107,7 +107,7 @@ 
     protected function getTreeSize($fighterCount, $groupSize)
     {
         $squareMultiplied = collect([1, 2, 4, 8, 16, 32, 64])
-            ->map(function ($item) use ($groupSize) {
+            ->map(function($item) use ($groupSize) {
                 return $item * $groupSize;
             }); // [4, 8, 16, 32, 64,...]
 
@@ -165,7 +165,7 @@ 
         $sizeGroupBy = count($byeGroup);
 
         $frequency = $sizeGroupBy != 0
-            ? (int) floor($sizeFighters / $sizeGroupBy)
+            ? (int)floor($sizeFighters / $sizeGroupBy)
             : -1;
 
         // Create Copy of $competitors
@@ -355,7 +355,7 @@ 
     {
         // Delete previous fight for this championship
         $arrGroupsId = $this->championship->fightersGroups()->get()->pluck('id');
-        if (sizeof($arrGroupsId) > 0){
+        if (sizeof($arrGroupsId) > 0) {
             Fight::destroy($arrGroupsId);
         }
     }

src/TreeGen/PlayOffTreeGen.php

Doc Comments +4 added lines, -1 removed lines

@@ -29,7 +29,7 @@ 
      *
      * @param $fightersByEntity
      *
-     * @return mixed
+     * @return Collection
      */
     protected function chunkAndShuffle(Collection $fightersByEntity)
     {
@@ -104,6 +104,9 @@ 
         $this->generateGroupsForRound($fighters, 1);
     }
 
+    /**
+     * @param integer $round
+     */
     protected function generateGroupsForRound(Collection $fightersByArea, $round)
     {
         $fightersId = $fightersByArea->pluck('id');