xoco70 / laravel-tournaments

src/models/Fight.php

Braces +18 added lines, -6 removed lines

@@ -46,7 +46,9 @@ 
      */
     protected static function getActorsToFights(FightersGroup $group)
     {
-        if ($group == null) return null;
+        if ($group == null) {
+            return null;
+        }
         $fighters = $group->getFighters();
         $fighterType = $group->getFighterType();
         if (sizeof($fighters) == 0) {
@@ -186,18 +188,26 @@ 
      */
     public function hasDeterminedParent()
     {
-        if ($this->has2Fighters()) return true;
+        if ($this->has2Fighters()) {
+            return true;
+        }
         foreach ($this->group->children as $child) {
             $fight = $child->fights->get(0);
-            if ($fight->has2Fighters()) return false;
+            if ($fight->has2Fighters()) {
+                return false;
+            }
         }
         return true;
     }
 
     public function shouldBeInFightList()
     {
-        if ($this->belongsToFirstRound() && $this->dontHave2Fighters()) return false;
-        if ($this->has2Fighters()) return true;
+        if ($this->belongsToFirstRound() && $this->dontHave2Fighters()) {
+            return false;
+        }
+        if ($this->has2Fighters()) {
+            return true;
+        }
         // We aint in the first round, and there is 1 or 0 competitor
         // We check children, and see :
         // if there is 2  - 2 fighters -> undetermine, we cannot add it to fight list
@@ -223,7 +233,9 @@ 
     private function belongsToFirstRound()
     {
         $firstRoundFights = $this->group->championship->firstRoundFights->pluck('id')->toArray();
-        if (in_array($this->id, $firstRoundFights)) return true;
+        if (in_array($this->id, $firstRoundFights)) {
+            return true;
+        }
         return false;
     }
 

src/TreeGen/TreeGen.php

Doc Comments +9 added lines, -9 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
@@ -175,9 +175,9 @@ 
     }
 
     /**
-     * @param $area
-     * @param $order
-     * @param $round
+     * @param integer $area
+     * @param integer $order
+     * @param integer $round
      * @return FightersGroup
      */
     protected function saveGroup($area, $order, $round): FightersGroup
@@ -334,8 +334,8 @@ 
 
     /**
      * @param Collection $fighters
-     * @param $frequency
-     * @param $sizeGroupBy
+     * @param integer $frequency
+     * @param integer $sizeGroupBy
      * @param $bye
      * @return Collection
      */
@@ -358,8 +358,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

Spacing +2 added lines, -2 removed lines

@@ -55,7 +55,7 @@ 
     private function getMaxFightersByEntity($userGroups): int
     {
         return $userGroups
-            ->sortByDesc(function ($group) {
+            ->sortByDesc(function($group) {
                 return $group->count();
             })
             ->first()
@@ -93,7 +93,7 @@ 
     protected function getTreeSize($fighterCount, $groupSize)
     {
         $square = collect([1, 2, 4, 8, 16, 32, 64]);
-        $squareMultiplied = $square->map(function ($item) use ($groupSize) {
+        $squareMultiplied = $square->map(function($item) use ($groupSize) {
             return $item * $groupSize;
         });