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;

src/models/Competitor.php

Braces +3 added lines, -1 removed lines

@@ -58,7 +58,9 @@
      */
     private function defaultName()
     {
-        if ($this == null || $this->user == null) return "BYE";
+        if ($this == null || $this->user == null) {
+            return "BYE";
+        }
         return null;
     }
 

src/models/Fight.php

Braces +6 added lines, -2 removed lines

@@ -276,10 +276,14 @@
     function hasDeterminedParent()
     {
 
-        if ($this->c1 != null && $this->c2 != null) return true;
+        if ($this->c1 != null && $this->c2 != null) {
+            return true;
+        }
         foreach ($this->group->children as $child) {
             $fight = $child->fights->get(0);
-            if ($fight->c1 != null && $fight->c2 != null) return false;
+            if ($fight->c1 != null && $fight->c2 != null) {
+                return false;
+            }
         }
         return true;
     }

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?

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
      * @param $parent
      * @return FightersGroup
      */
@@ -338,8 +338,8 @@ 
 
     /**
      * @param Collection $fighters
-     * @param $frequency
-     * @param $sizeGroupBy
+     * @param integer $frequency
+     * @param integer $sizeGroupBy
      * @param $bye
      * @return Collection
      */
@@ -362,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