Passed
Push — master ( 27b2a5...f7976d )
by John
04:24
created
app/Http/Controllers/Group/IndexController.php 1 patch
Braces   +10 added lines, -4 removed lines patch added patch discarded remove patch
@@ -43,7 +43,9 @@  discard block
 block discarded – undo
43 43
         $groupModel=new GroupModel();
44 44
         $contestModel=new ContestModel();
45 45
         $basic_info=$groupModel->details($gcode);
46
-        if(empty($basic_info)) return Redirect::route('group.index');
46
+        if(empty($basic_info)) {
47
+            return Redirect::route('group.index');
48
+        }
47 49
         $my_profile=$groupModel->userProfile(Auth::user()->id, $basic_info["gid"]);
48 50
         $clearance=$groupModel->judgeClearance($basic_info["gid"], Auth::user()->id);
49 51
         $member_list=$groupModel->userList($basic_info["gid"]);
@@ -89,9 +91,13 @@  discard block
 block discarded – undo
89 91
     public function analysis($gcode){
90 92
         $groupModel = new GroupModel();
91 93
         $basic_info=$groupModel->details($gcode);
92
-        if(empty($basic_info)) return Redirect::route('group.index');
94
+        if(empty($basic_info)) {
95
+            return Redirect::route('group.index');
96
+        }
93 97
         $clearance=$groupModel->judgeClearance($basic_info["gid"], Auth::user()->id);
94
-        if($clearance < 1) return Redirect::route('group.detail',['gcode' => $gcode]);
98
+        if($clearance < 1) {
99
+            return Redirect::route('group.detail',['gcode' => $gcode]);
100
+        }
95 101
         $group_info = $groupModel->details($gcode);
96 102
         return view('group.settings.analysis', [
97 103
             'page_title'=>"Group Analysis",
@@ -123,7 +129,7 @@  discard block
 block discarded – undo
123 129
                 ),
124 130
                 $gcode . '_Group_Contest_Analysis.xlsx'
125 131
             );
126
-        }else{
132
+        } else{
127 133
             $data = $groupModel->groupMemberPracticeTagStat($group_info['gid']);
128 134
             return Excel::download(
129 135
                 new GroupAnalysisExport(
Please login to merge, or discard this patch.
app/Console/Kernel.php 1 patch
Braces   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -65,12 +65,12 @@
 block discarded – undo
65 65
             foreach ($ret as $gid => $group) {
66 66
                 if(empty($group['result'])){
67 67
                     Log::channel('group_elo')->info('Refreshed Group Elo (Empty) : ('.$gid.')'.$group['name']);
68
-                }else{
68
+                } else{
69 69
                     Log::channel('group_elo')->info('Refreshing Group Elo: ('.$gid.')'.$group['name']);
70 70
                     foreach ($group['result'] as $contest) {
71 71
                         if($contest['ret'] == 'success'){
72 72
                             Log::channel('group_elo')->info('    Elo Clac Successfully : ('.$contest['cid'].')'.$contest['name']);
73
-                        }else{
73
+                        } else{
74 74
                             Log::channel('group_elo')->info('    Elo Clac Faild (Judge Not Over) : ('.$contest['cid'].')'.$contest['name'].'  sids:');
75 75
                             foreach ($contest['submissions'] as $sid) {
76 76
                                 Log::channel('group_elo')->info('        '.$sid['sid']);
Please login to merge, or discard this patch.
app/Babel/Submit/Curl.php 1 patch
Braces   +105 added lines, -21 removed lines patch added patch discarded remove patch
@@ -18,11 +18,31 @@  discard block
 block discarded – undo
18 18
 
19 19
     public function login($all_data)
20 20
     {
21
-        if(isset($all_data["url"]))    $url = $all_data["url"];       else throw new Exception("url is not exist in all_data");
22
-        if(isset($all_data["data"]))   $data = $all_data["data"];     else throw new Exception("data is not exist in all_data");
23
-        if(isset($all_data["oj"]))     $oj = $all_data["oj"];         else throw new Exception("oj is not exist in all_data");
24
-        if(isset($all_data["ret"]))    $ret = $all_data["ret"];       else $ret = 'false';
25
-        if(isset($all_data["handle"])) $handle = $all_data["handle"]; else $handle = "default";
21
+        if(isset($all_data["url"])) {
22
+            $url = $all_data["url"];
23
+        } else {
24
+            throw new Exception("url is not exist in all_data");
25
+        }
26
+        if(isset($all_data["data"])) {
27
+            $data = $all_data["data"];
28
+        } else {
29
+            throw new Exception("data is not exist in all_data");
30
+        }
31
+        if(isset($all_data["oj"])) {
32
+            $oj = $all_data["oj"];
33
+        } else {
34
+            throw new Exception("oj is not exist in all_data");
35
+        }
36
+        if(isset($all_data["ret"])) {
37
+            $ret = $all_data["ret"];
38
+        } else {
39
+            $ret = 'false';
40
+        }
41
+        if(isset($all_data["handle"])) {
42
+            $handle = $all_data["handle"];
43
+        } else {
44
+            $handle = "default";
45
+        }
26 46
 
27 47
         $datapost=curl_init();
28 48
         $headers=array("Expect:");
@@ -54,12 +74,36 @@  discard block
 block discarded – undo
54 74
 
55 75
     public function grab_page($all_data)
56 76
     {
57
-        if(isset($all_data["site"]))    $site = $all_data["site"];          else throw new Exception("site is not exist in all_data");
58
-        if(isset($all_data["oj"]))      $oj = $all_data["oj"];              else throw new Exception("oj is not exist in all_data");
59
-        if(isset($all_data["headers"])) $headers = $all_data["headers"];    else $headers = [];
60
-        if(isset($all_data["handle"]))  $handle = $all_data["handle"];      else $handle = "default";
61
-        if(isset($all_data["follow"]))  $follow = $all_data["follow"];      else $follow = false;
62
-        if(isset($all_data["vcid"]))  $vcid = $all_data["vcid"]."_";      else $vcid = "";
77
+        if(isset($all_data["site"])) {
78
+            $site = $all_data["site"];
79
+        } else {
80
+            throw new Exception("site is not exist in all_data");
81
+        }
82
+        if(isset($all_data["oj"])) {
83
+            $oj = $all_data["oj"];
84
+        } else {
85
+            throw new Exception("oj is not exist in all_data");
86
+        }
87
+        if(isset($all_data["headers"])) {
88
+            $headers = $all_data["headers"];
89
+        } else {
90
+            $headers = [];
91
+        }
92
+        if(isset($all_data["handle"])) {
93
+            $handle = $all_data["handle"];
94
+        } else {
95
+            $handle = "default";
96
+        }
97
+        if(isset($all_data["follow"])) {
98
+            $follow = $all_data["follow"];
99
+        } else {
100
+            $follow = false;
101
+        }
102
+        if(isset($all_data["vcid"])) {
103
+            $vcid = $all_data["vcid"]."_";
104
+        } else {
105
+            $vcid = "";
106
+        }
63 107
 
64 108
         $handle=urlencode($handle);
65 109
 
@@ -86,16 +130,56 @@  discard block
 block discarded – undo
86 130
 
87 131
     public function post_data($all_data)
88 132
     {
89
-        if(isset($all_data["site"]))         $site = $all_data["site"];                 else throw new Exception("site is not exist in all_data");
90
-        if(isset($all_data["data"]))         $data = $all_data["data"];                 else throw new Exception("data is not exist in all_data");
91
-        if(isset($all_data["oj"]))           $oj = $all_data["oj"];                     else throw new Exception("oj is not exist in all_data");
92
-        if(isset($all_data["ret"]))          $ret = $all_data["ret"];                   else $ret = false;
93
-        if(isset($all_data["follow"]))       $follow = $all_data["follow"];             else $follow = true;
94
-        if(isset($all_data["returnHeader"])) $returnHeader = $all_data["returnHeader"]; else $returnHeader = true;
95
-        if(isset($all_data["postJson"]))     $postJson = $all_data["postJson"];         else $postJson = false;
96
-        if(isset($all_data["extraHeaders"])) $extraHeaders = $all_data["extraHeaders"]; else $extraHeaders = [];
97
-        if(isset($all_data["handle"]))       $handle = $all_data["handle"];             else $handle = "default";
98
-        if(isset($all_data["vcid"]))  $vcid = $all_data["vcid"]."_";      else $vcid = "";
133
+        if(isset($all_data["site"])) {
134
+            $site = $all_data["site"];
135
+        } else {
136
+            throw new Exception("site is not exist in all_data");
137
+        }
138
+        if(isset($all_data["data"])) {
139
+            $data = $all_data["data"];
140
+        } else {
141
+            throw new Exception("data is not exist in all_data");
142
+        }
143
+        if(isset($all_data["oj"])) {
144
+            $oj = $all_data["oj"];
145
+        } else {
146
+            throw new Exception("oj is not exist in all_data");
147
+        }
148
+        if(isset($all_data["ret"])) {
149
+            $ret = $all_data["ret"];
150
+        } else {
151
+            $ret = false;
152
+        }
153
+        if(isset($all_data["follow"])) {
154
+            $follow = $all_data["follow"];
155
+        } else {
156
+            $follow = true;
157
+        }
158
+        if(isset($all_data["returnHeader"])) {
159
+            $returnHeader = $all_data["returnHeader"];
160
+        } else {
161
+            $returnHeader = true;
162
+        }
163
+        if(isset($all_data["postJson"])) {
164
+            $postJson = $all_data["postJson"];
165
+        } else {
166
+            $postJson = false;
167
+        }
168
+        if(isset($all_data["extraHeaders"])) {
169
+            $extraHeaders = $all_data["extraHeaders"];
170
+        } else {
171
+            $extraHeaders = [];
172
+        }
173
+        if(isset($all_data["handle"])) {
174
+            $handle = $all_data["handle"];
175
+        } else {
176
+            $handle = "default";
177
+        }
178
+        if(isset($all_data["vcid"])) {
179
+            $vcid = $all_data["vcid"]."_";
180
+        } else {
181
+            $vcid = "";
182
+        }
99 183
 
100 184
         $handle=urlencode($handle);
101 185
 
Please login to merge, or discard this patch.
app/Http/Controllers/Contest/BoardController.php 1 patch
Braces   +12 added lines, -10 removed lines patch added patch discarded remove patch
@@ -37,7 +37,7 @@  discard block
 block discarded – undo
37 37
         if (!$clearance || time() < strtotime($basicInfo['begin_time'])) {
38 38
             if($clearance == 3){
39 39
                 return Redirect::route('contest.admin', ['cid' => $cid]);
40
-            }else{
40
+            } else{
41 41
                 return Redirect::route('contest.detail', ['cid' => $cid]);
42 42
             }
43 43
         }
@@ -83,7 +83,7 @@  discard block
 block discarded – undo
83 83
         if (!$clearance || time() < strtotime($basicInfo['begin_time'])) {
84 84
             if($clearance == 3){
85 85
                 return Redirect::route('contest.admin', ['cid' => $cid]);
86
-            }else{
86
+            } else{
87 87
                 return Redirect::route('contest.detail', ['cid' => $cid]);
88 88
             }
89 89
         }
@@ -116,7 +116,9 @@  discard block
 block discarded – undo
116 116
         }
117 117
 
118 118
         $editor_left_width = $accountModel->getExtra(Auth::user()->id, 'editor_left_width');
119
-        if(empty($editor_left_width)) $editor_left_width='40';
119
+        if(empty($editor_left_width)) {
120
+            $editor_left_width='40';
121
+        }
120 122
 
121 123
         return view('contest.board.editor', [
122 124
             'page_title'=>"Problem Detail",
@@ -153,7 +155,7 @@  discard block
 block discarded – undo
153 155
         if (!$clearance || time() < strtotime($basicInfo['begin_time'])) {
154 156
             if($clearance == 3){
155 157
                 return Redirect::route('contest.admin', ['cid' => $cid]);
156
-            }else{
158
+            } else{
157 159
                 return Redirect::route('contest.detail', ['cid' => $cid]);
158 160
             }
159 161
         }
@@ -168,10 +170,10 @@  discard block
 block discarded – undo
168 170
             if($i != 0) {
169 171
                 if($r['score'] == $contestRank[$i-1]['score'] && $r['penalty'] == $contestRank[$i-1]['penalty']){
170 172
                     $r['rank'] = $contestRank[$i-1]['rank'];
171
-                }else{
173
+                } else{
172 174
                     $r['rank'] = $i + 1;
173 175
                 }
174
-            }else{
176
+            } else{
175 177
                 $r['rank'] = 1;
176 178
             }
177 179
         }
@@ -212,7 +214,7 @@  discard block
 block discarded – undo
212 214
         if (!$clearance || time() < strtotime($basicInfo['begin_time'])) {
213 215
             if($clearance == 3){
214 216
                 return Redirect::route('contest.admin', ['cid' => $cid]);
215
-            }else{
217
+            } else{
216 218
                 return Redirect::route('contest.detail', ['cid' => $cid]);
217 219
             }
218 220
         }
@@ -250,7 +252,7 @@  discard block
 block discarded – undo
250 252
         if (!$clearance || time() < strtotime($basicInfo['begin_time'])) {
251 253
             if($clearance == 3){
252 254
                 return Redirect::route('contest.admin', ['cid' => $cid]);
253
-            }else{
255
+            } else{
254 256
                 return Redirect::route('contest.detail', ['cid' => $cid]);
255 257
             }
256 258
         }
@@ -285,7 +287,7 @@  discard block
 block discarded – undo
285 287
         if (!$clearance || time() < strtotime($basicInfo['begin_time'])) {
286 288
             if($clearance == 3){
287 289
                 return Redirect::route('contest.admin', ['cid' => $cid]);
288
-            }else{
290
+            } else{
289 291
                 return Redirect::route('contest.detail', ['cid' => $cid]);
290 292
             }
291 293
         }
@@ -310,7 +312,7 @@  discard block
 block discarded – undo
310 312
         if (!$clearance || time() < strtotime($basicInfo['begin_time'])) {
311 313
             if($clearance == 3){
312 314
                 return Redirect::route('contest.admin', ['cid' => $cid]);
313
-            }else{
315
+            } else{
314 316
                 return Redirect::route('contest.detail', ['cid' => $cid]);
315 317
             }
316 318
         }
Please login to merge, or discard this patch.