@@ -58,7 +58,9 @@ |
||
58 | 58 | */ |
59 | 59 | private function defaultName() |
60 | 60 | { |
61 | - if ($this == null || $this->user == null) return "BYE"; |
|
61 | + if ($this == null || $this->user == null) { |
|
62 | + return "BYE"; |
|
63 | + } |
|
62 | 64 | return null; |
63 | 65 | } |
64 | 66 |
@@ -48,7 +48,7 @@ discard block |
||
48 | 48 | |
49 | 49 | /** |
50 | 50 | * Get the biggest entity group |
51 | - * @param $userGroups |
|
51 | + * @param Collection $userGroups |
|
52 | 52 | * |
53 | 53 | * @return int |
54 | 54 | */ |
@@ -107,7 +107,7 @@ discard block |
||
107 | 107 | |
108 | 108 | /** |
109 | 109 | * Repart BYE in the tree, |
110 | - * @param $fighterGroups |
|
110 | + * @param Collection $fighterGroups |
|
111 | 111 | * @param int $max |
112 | 112 | * |
113 | 113 | * @return Collection |
@@ -175,9 +175,9 @@ discard block |
||
175 | 175 | } |
176 | 176 | |
177 | 177 | /** |
178 | - * @param $area |
|
179 | - * @param $order |
|
180 | - * @param $round |
|
178 | + * @param integer $area |
|
179 | + * @param integer $order |
|
180 | + * @param integer $round |
|
181 | 181 | * @param $parent |
182 | 182 | * @return FightersGroup |
183 | 183 | */ |
@@ -338,8 +338,8 @@ discard block |
||
338 | 338 | |
339 | 339 | /** |
340 | 340 | * @param Collection $fighters |
341 | - * @param $frequency |
|
342 | - * @param $sizeGroupBy |
|
341 | + * @param integer $frequency |
|
342 | + * @param integer $sizeGroupBy |
|
343 | 343 | * @param $bye |
344 | 344 | * @return Collection |
345 | 345 | */ |
@@ -362,8 +362,8 @@ discard block |
||
362 | 362 | /** |
363 | 363 | * @param $frequency |
364 | 364 | * @param $sizeGroupBy |
365 | - * @param $count |
|
366 | - * @param $byeCount |
|
365 | + * @param integer $count |
|
366 | + * @param integer $byeCount |
|
367 | 367 | * @return bool |
368 | 368 | */ |
369 | 369 | private function shouldInsertBye($frequency, $sizeGroupBy, $count, $byeCount): bool |
@@ -49,7 +49,9 @@ |
||
49 | 49 | { |
50 | 50 | foreach ($fightsByRound as $fight) { |
51 | 51 | $parentGroup = $fight->group->parent; |
52 | - if ($parentGroup == null) break; |
|
52 | + if ($parentGroup == null) { |
|
53 | + break; |
|
54 | + } |
|
53 | 55 | $parentFight = $parentGroup->fights->get(0); //TODO This Might change when extending to Preliminary |
54 | 56 | |
55 | 57 | // IN this $fight, is c1 or c2 has the info? |
@@ -63,7 +63,9 @@ discard block |
||
63 | 63 | */ |
64 | 64 | private static function getActorsToFights(Championship $championship, FightersGroup $group = null) |
65 | 65 | { |
66 | - if ($group == null) return null; |
|
66 | + if ($group == null) { |
|
67 | + return null; |
|
68 | + } |
|
67 | 69 | $fighters = $group->getFighters(); |
68 | 70 | $fighterType = $group->getFighterType(); |
69 | 71 | if (sizeof($fighters) == 0) { |
@@ -274,10 +276,14 @@ discard block |
||
274 | 276 | function hasDeterminedParent() |
275 | 277 | { |
276 | 278 | |
277 | - if ($this->c1 != null && $this->c2 != null) return true; |
|
279 | + if ($this->c1 != null && $this->c2 != null) { |
|
280 | + return true; |
|
281 | + } |
|
278 | 282 | foreach ($this->group->children as $child) { |
279 | 283 | $fight = $child->fights->get(0); |
280 | - if ($fight->c1 != null && $fight->c2 != null) return false; |
|
284 | + if ($fight->c1 != null && $fight->c2 != null) { |
|
285 | + return false; |
|
286 | + } |
|
281 | 287 | } |
282 | 288 | return true; |
283 | 289 | } |
@@ -169,8 +169,9 @@ |
||
169 | 169 | |
170 | 170 | public function buildName() |
171 | 171 | { |
172 | - if ($this->settings != null && $this->settings->alias != null) |
|
173 | - return $this->settings->alias; |
|
172 | + if ($this->settings != null && $this->settings->alias != null) { |
|
173 | + return $this->settings->alias; |
|
174 | + } |
|
174 | 175 | |
175 | 176 | if ($this->hasNoCustomSettings()) { |
176 | 177 | return $this->category->name; |