@@ -47,7 +47,7 @@ discard block |
||
47 | 47 | |
48 | 48 | /** |
49 | 49 | * Get the biggest entity group |
50 | - * @param $userGroups |
|
50 | + * @param Collection $userGroups |
|
51 | 51 | * |
52 | 52 | * @return int |
53 | 53 | */ |
@@ -104,7 +104,7 @@ discard block |
||
104 | 104 | |
105 | 105 | /** |
106 | 106 | * Repart BYE in the tree, |
107 | - * @param $fighterGroups |
|
107 | + * @param Collection $fighterGroups |
|
108 | 108 | * @param int $max |
109 | 109 | * |
110 | 110 | * @return Collection |
@@ -185,9 +185,9 @@ discard block |
||
185 | 185 | } |
186 | 186 | |
187 | 187 | /** |
188 | - * @param $area |
|
189 | - * @param $order |
|
190 | - * @param $round |
|
188 | + * @param integer $area |
|
189 | + * @param integer $order |
|
190 | + * @param integer $round |
|
191 | 191 | * @param $parent |
192 | 192 | * @return FightersGroup |
193 | 193 | */ |
@@ -276,10 +276,14 @@ |
||
276 | 276 | function hasDeterminedParent() |
277 | 277 | { |
278 | 278 | |
279 | - if ($this->c1 != null && $this->c2 != null) return true; |
|
279 | + if ($this->c1 != null && $this->c2 != null) { |
|
280 | + return true; |
|
281 | + } |
|
280 | 282 | foreach ($this->group->children as $child) { |
281 | 283 | $fight = $child->fights->get(0); |
282 | - if ($fight->c1 != null && $fight->c2 != null) return false; |
|
284 | + if ($fight->c1 != null && $fight->c2 != null) { |
|
285 | + return false; |
|
286 | + } |
|
283 | 287 | } |
284 | 288 | return true; |
285 | 289 | } |
@@ -203,7 +203,9 @@ |
||
203 | 203 | foreach ($fightsByRound as $fight) { |
204 | 204 | |
205 | 205 | $parentGroup = $fight->group->parent; |
206 | - if ($parentGroup == null) break; |
|
206 | + if ($parentGroup == null) { |
|
207 | + break; |
|
208 | + } |
|
207 | 209 | $parentFight = $parentGroup->fights->get(0); //TODO This Might change when extending to Preliminary |
208 | 210 | |
209 | 211 | // IN this $fight, is c1 or c2 has the info? |
@@ -63,7 +63,7 @@ |
||
63 | 63 | ); |
64 | 64 | } |
65 | 65 | |
66 | - $championship->settings = ChampionshipSettings::createOrUpdate($request, $championship); |
|
66 | + $championship->settings = ChampionshipSettings::createOrUpdate($request, $championship); |
|
67 | 67 | $generation = $championship->chooseGenerationStrategy(); |
68 | 68 | |
69 | 69 | try { |