@@ -157,8 +157,9 @@ |
||
| 157 | 157 | |
| 158 | 158 | public function buildName() |
| 159 | 159 | { |
| 160 | - if ($this->settings != null && $this->settings->alias != null && $this->settings->alias != '') |
|
| 161 | - return $this->settings->alias; |
|
| 160 | + if ($this->settings != null && $this->settings->alias != null && $this->settings->alias != '') { |
|
| 161 | + return $this->settings->alias; |
|
| 162 | + } |
|
| 162 | 163 | |
| 163 | 164 | if ($this->hasNoCustomSettings()) { |
| 164 | 165 | return $this->category->name; |
@@ -203,7 +203,9 @@ |
||
| 203 | 203 | $fightsByRound = $championship->fightsByRound($numRound)->with('group.parent')->get(); |
| 204 | 204 | foreach ($fightsByRound as $fight) { |
| 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? |
@@ -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 | */ |
@@ -105,7 +105,7 @@ discard block |
||
| 105 | 105 | |
| 106 | 106 | /** |
| 107 | 107 | * Repart BYE in the tree, |
| 108 | - * @param $fighterGroups |
|
| 108 | + * @param Collection $fighterGroups |
|
| 109 | 109 | * @param int $max |
| 110 | 110 | * |
| 111 | 111 | * @return Collection |
@@ -163,7 +163,6 @@ discard block |
||
| 163 | 163 | * @param Collection $usersByArea |
| 164 | 164 | * @param integer $area |
| 165 | 165 | * @param integer $round |
| 166 | - * @param integer $shuffle |
|
| 167 | 166 | * |
| 168 | 167 | */ |
| 169 | 168 | public function generateGroupsForRound($usersByArea, $area, $round) |
@@ -186,9 +185,9 @@ discard block |
||
| 186 | 185 | } |
| 187 | 186 | |
| 188 | 187 | /** |
| 189 | - * @param $area |
|
| 190 | - * @param $order |
|
| 191 | - * @param $round |
|
| 188 | + * @param integer $area |
|
| 189 | + * @param integer $order |
|
| 190 | + * @param integer $round |
|
| 192 | 191 | * @param $parent |
| 193 | 192 | * @return FightersGroup |
| 194 | 193 | */ |
@@ -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 | |