@@ -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; |
@@ -69,8 +69,12 @@ |
||
69 | 69 | */ |
70 | 70 | private function defaultName() |
71 | 71 | { |
72 | - if ($this == null) return "BYE"; |
|
73 | - if ($this->user == null) return "BYE"; |
|
72 | + if ($this == null) { |
|
73 | + return "BYE"; |
|
74 | + } |
|
75 | + if ($this->user == null) { |
|
76 | + return "BYE"; |
|
77 | + } |
|
74 | 78 | return null; |
75 | 79 | } |
76 | 80 |
@@ -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 | */ |
@@ -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? |