Passed
Branch bigRefactor (c260a1)
by Julien
34:53
created
src/models/Championship.php 1 patch
Braces   +3 added lines, -2 removed lines patch added patch discarded remove patch
@@ -157,8 +157,9 @@
 block discarded – undo
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;
Please login to merge, or discard this patch.
src/models/FightersGroup.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -206,7 +206,9 @@
 block discarded – undo
206 206
             foreach ($fightsByRound as $numFight => $fight) {
207 207
 
208 208
                 $parentGroup = $fight->group->parent;
209
-                if ($parentGroup == null) break;
209
+                if ($parentGroup == null) {
210
+                    break;
211
+                }
210 212
                 $parentFight = $parentGroup->fights->get(0);
211 213
 
212 214
                 // IN this $fight, if it is the first child or the second child --> c1 or c2 in parent
Please login to merge, or discard this patch.
src/models/Competitor.php 1 patch
Braces   +6 added lines, -2 removed lines patch added patch discarded remove patch
@@ -69,8 +69,12 @@
 block discarded – undo
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
 
Please login to merge, or discard this patch.
src/TreeGen/TreeGen.php 1 patch
Doc Comments   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -49,7 +49,7 @@  discard block
 block discarded – undo
49 49
 
50 50
     /**
51 51
      * Get the biggest entity group
52
-     * @param $userGroups
52
+     * @param Collection $userGroups
53 53
      *
54 54
      * @return int
55 55
      */
@@ -106,7 +106,7 @@  discard block
 block discarded – undo
106 106
 
107 107
     /**
108 108
      * Repart BYE in the tree,
109
-     * @param $fighterGroups
109
+     * @param Collection $fighterGroups
110 110
      * @param int $max
111 111
      *
112 112
      * @return Collection
@@ -187,9 +187,9 @@  discard block
 block discarded – undo
187 187
     }
188 188
 
189 189
     /**
190
-     * @param $area
191
-     * @param $order
192
-     * @param $round
190
+     * @param integer $area
191
+     * @param integer $order
192
+     * @param integer $round
193 193
      * @param $parent
194 194
      * @return FightersGroup
195 195
      */
@@ -309,8 +309,8 @@  discard block
 block discarded – undo
309 309
 
310 310
     /**
311 311
      * Chunk Fighters into groups for fighting, and optionnaly shuffle
312
-     * @param $round
313
-     * @param $shuffle
312
+     * @param integer $round
313
+     * @param integer $shuffle
314 314
      * @param $fightersByEntity
315 315
      * @return mixed
316 316
      */
Please login to merge, or discard this patch.