Passed
Push — vko_test_ubuntu_24.04 ( 4f3057...90f192 )
by Vitalii
22:18 queued 12:17
created
html/ops/host_stats.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -33,7 +33,7 @@  discard block
 block discarded – undo
33 33
     if ($n === false) return '';
34 34
     $m = strpos($p, ']', $n);
35 35
     if ($m === false) return '';
36
-    $x = substr($p, $n+5, $m-$n-5);
36
+    $x = substr($p, $n + 5, $m - $n - 5);
37 37
 
38 38
     $n = strpos($x, ' ');
39 39
     if ($n !== false) return substr($x, 0, $n);
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
47 47
     if ($n === false) return '';
48 48
     $m = strpos($p, ']', $n);
49 49
     if ($m === false) return '';
50
-    $x = substr($p, $n+5, $m-$n-5);
50
+    $x = substr($p, $n + 5, $m - $n - 5);
51 51
     $n = strpos($x, '_');
52 52
     if ($n !== false) return substr($x, 0, $n);
53 53
     $n = strpos($x, 'r');
@@ -57,7 +57,7 @@  discard block
 block discarded – undo
57 57
 
58 58
 function main() {
59 59
     global $max_days;
60
-    $t = time()-$max_days*86400;
60
+    $t = time() - $max_days*86400;
61 61
     $hosts = BoincHost::enum("rpc_time>$t");
62 62
     $descs = [];
63 63
     foreach ($hosts as $host) {
Please login to merge, or discard this patch.
html/inc/db_conn.inc 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -22,12 +22,12 @@  discard block
 block discarded – undo
22 22
 // Intended to be subclassed (e.g., BoincDb, BossaDb)
23 23
 //
24 24
 class DbConn {
25
-    var $db_conn;       // a mysqli object
26
-    var $db_name;       // the DB name
25
+    var $db_conn; // a mysqli object
26
+    var $db_name; // the DB name
27 27
 
28 28
     function init_conn($user, $passwd, $host, $name) {
29 29
         $x = explode(":", $host);
30
-        if (sizeof($x)>1) {
30
+        if (sizeof($x) > 1) {
31 31
             $host = $x[0];
32 32
             $port = $x[1];
33 33
         } else {
@@ -35,7 +35,7 @@  discard block
 block discarded – undo
35 35
         }
36 36
         try {
37 37
             $this->db_conn = @new mysqli($host, $user, $passwd, $name, $port);
38
-        } catch(Exception $e) {
38
+        } catch (Exception $e) {
39 39
             return false;
40 40
         }
41 41
         if (mysqli_connect_error()) {
@@ -136,7 +136,7 @@  discard block
 block discarded – undo
136 136
         return $this->enum_general($classname, $query);
137 137
     }
138 138
 
139
-    function enum($table, $classname, $where_clause=null, $order_clause=null) {
139
+    function enum($table, $classname, $where_clause = null, $order_clause = null) {
140 140
         return self::enum_fields(
141 141
             $table, $classname, '*', $where_clause, $order_clause
142 142
         );
@@ -181,15 +181,15 @@  discard block
 block discarded – undo
181 181
         if ($x) return (double)$x->$field;
182 182
         return false;
183 183
     }
184
-    function count($table, $clause="TRUE") {
184
+    function count($table, $clause = "TRUE") {
185 185
         $query = "select count(*) as total from DBNAME.$table where $clause";
186 186
         return $this->get_int($query, 'total');
187 187
     }
188
-    function sum($table, $field, $clause="") {
188
+    function sum($table, $field, $clause = "") {
189 189
         $query = "select sum($field) as total from DBNAME.$table $clause";
190 190
         return $this->get_double($query, 'total');
191 191
     }
192
-    function max($table, $field, $clause="") {
192
+    function max($table, $field, $clause = "") {
193 193
         $query = "select max($field) as total from DBNAME.$table $clause";
194 194
         return $this->get_double($query, 'total');
195 195
     }
Please login to merge, or discard this patch.
html/inc/util_basic.inc 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -56,10 +56,10 @@  discard block
 block discarded – undo
56 56
     return file_exists("$d/stop_sched");
57 57
 }
58 58
 
59
-function xml_error($num=-1, $msg=null, $file=null, $line=null) {
59
+function xml_error($num = -1, $msg = null, $file = null, $line = null) {
60 60
     global $xml_outer_tag;
61 61
     if (!$msg) {
62
-        switch($num) {
62
+        switch ($num) {
63 63
         case -112: $msg = "Invalid XML"; break;
64 64
         case -136: $msg = "Not found"; break;
65 65
         case -137: $msg = "Name or email address is not unique"; break;
@@ -105,7 +105,7 @@  discard block
 block discarded – undo
105 105
 // If it's a single-tag element, and it's present, just return the tag
106 106
 //
107 107
 function parse_element($xml, $tag) {
108
-    $closetag = "</" . substr($tag,1);
108
+    $closetag = "</".substr($tag, 1);
109 109
     $x = strstr($xml, $tag);
110 110
     if ($x) {
111 111
         if (strstr($tag, "/>")) return $tag;
@@ -121,8 +121,8 @@  discard block
 block discarded – undo
121 121
 
122 122
 function parse_next_element($xml, $tag, &$cursor) {
123 123
     $element = null;
124
-    $closetag = "</" . substr($tag,1);
125
-    $pos = substr($xml,$cursor);
124
+    $closetag = "</".substr($tag, 1);
125
+    $pos = substr($xml, $cursor);
126 126
     $x = strstr($pos, $tag);
127 127
     if ($x) {
128 128
         if (strstr($tag, "/>")) return $tag;
Please login to merge, or discard this patch.
html/user/submit_rpc_handler.php 1 patch
Spacing   +30 added lines, -30 removed lines patch added patch discarded remove patch
@@ -59,7 +59,7 @@  discard block
 block discarded – undo
59 59
     if ($template) {
60 60
         $t = (double)$template->workunit->rsc_fpops_est;
61 61
     }
62
-    foreach($r->batch->job as $job) {
62
+    foreach ($r->batch->job as $job) {
63 63
         $y = (double)$job->rsc_fpops_est;
64 64
         if ($y) {
65 65
             $x += $y;
@@ -85,7 +85,7 @@  discard block
 block discarded – undo
85 85
 function est_elapsed_time($r, $template) {
86 86
     // crude estimate: batch FLOPs / project FLOPS
87 87
     //
88
-    return batch_flop_count($r, $template) / project_flops();
88
+    return batch_flop_count($r, $template)/project_flops();
89 89
 }
90 90
 
91 91
 // if batch-level input template filename was given, read it;
@@ -95,9 +95,9 @@  discard block
 block discarded – undo
95 95
 //
96 96
 function read_input_template($app, $r) {
97 97
     if ((isset($r->batch)) && (isset($r->batch->workunit_template_file)) && ($r->batch->workunit_template_file)) {
98
-        $path = project_dir() . "/templates/".$r->batch->workunit_template_file;
98
+        $path = project_dir()."/templates/".$r->batch->workunit_template_file;
99 99
     } else {
100
-        $path = project_dir() . "/templates/$app->name"."_in";
100
+        $path = project_dir()."/templates/$app->name"."_in";
101 101
     }
102 102
     if (file_exists($path)) {
103 103
         $x = simplexml_load_file($path);
@@ -145,7 +145,7 @@  discard block
 block discarded – undo
145 145
 function validate_batch($jobs, $template) {
146 146
     $i = 0;
147 147
     $n = count($template->file_info);
148
-    foreach($jobs as $job) {
148
+    foreach ($jobs as $job) {
149 149
         $m = count($job->input_files);
150 150
         if ($n != $m) {
151 151
             log_write("wrong # of input files for job $i: need $n, got $m");
@@ -214,7 +214,7 @@  discard block
 block discarded – undo
214 214
 // stage all the files
215 215
 //
216 216
 function stage_files(&$jobs, $user) {
217
-    foreach($jobs as $job) {
217
+    foreach ($jobs as $job) {
218 218
         foreach ($job->input_files as $file) {
219 219
             if ($file->mode != "remote") {
220 220
                 $file->name = stage_file($file, $user);
@@ -227,7 +227,7 @@  discard block
 block discarded – undo
227 227
 //
228 228
 function submit_jobs(
229 229
     $jobs, $job_params, $app, $batch_id, $priority, $app_version_num,
230
-    $input_template_filename,        // batch-level; can also specify per job
230
+    $input_template_filename, // batch-level; can also specify per job
231 231
     $output_template_filename,
232 232
     $user
233 233
 ) {
@@ -237,7 +237,7 @@  discard block
 block discarded – undo
237 237
     // one line per job
238 238
     //
239 239
     $x = "";
240
-    foreach($jobs as $job) {
240
+    foreach ($jobs as $job) {
241 241
         if ($job->name) {
242 242
             $x .= " --wu_name $job->name";
243 243
         }
@@ -272,7 +272,7 @@  discard block
 block discarded – undo
272 272
         $x .= "\n";
273 273
     }
274 274
 
275
-    $cmd = "cd " . project_dir() . "; ./bin/create_work --appname $app->name --batch $batch_id";
275
+    $cmd = "cd ".project_dir()."; ./bin/create_work --appname $app->name --batch $batch_id";
276 276
 
277 277
     if ($user->seti_id) {
278 278
         $cmd .= " --target_user $user->id ";
@@ -380,7 +380,7 @@  discard block
 block discarded – undo
380 380
 //
381 381
 function xml_get_jobs($r) {
382 382
     $jobs = array();
383
-    foreach($r->batch->job as $j) {
383
+    foreach ($r->batch->job as $j) {
384 384
         $job = new StdClass;
385 385
         $job->input_files = array();
386 386
         $job->command_line = (string)$j->command_line;
@@ -431,16 +431,16 @@  discard block
 block discarded – undo
431 431
 //
432 432
 function logical_end_time($r, $jobs, $user, $app) {
433 433
     $total_flops = 0;
434
-    foreach($jobs as $job) {
434
+    foreach ($jobs as $job) {
435 435
         //print_r($job);
436 436
         if ($job->rsc_fpops_est) {
437 437
             $total_flops += $job->rsc_fpops_est;
438 438
         } else if ($job->input_template && $job->input_template->workunit->rsc_fpops_est) {
439
-            $total_flops += (double) $job->input_template->workunit->rsc_fpops_est;
439
+            $total_flops += (double)$job->input_template->workunit->rsc_fpops_est;
440 440
         } else if ($r->batch->job_params->rsc_fpops_est) {
441
-            $total_flops += (double) $r->batch->job_params->rsc_fpops_est;
441
+            $total_flops += (double)$r->batch->job_params->rsc_fpops_est;
442 442
         } else {
443
-            $x = (double) $template->workunit->rsc_fpops_est;
443
+            $x = (double)$template->workunit->rsc_fpops_est;
444 444
             if ($x) {
445 445
                 $total_flops += $x;
446 446
             } else {
@@ -448,7 +448,7 @@  discard block
 block discarded – undo
448 448
             }
449 449
         }
450 450
     }
451
-    $cmd = "cd " . project_dir() . "/bin; ./adjust_user_priority --user $user->id --flops $total_flops --app $app->name";
451
+    $cmd = "cd ".project_dir()."/bin; ./adjust_user_priority --user $user->id --flops $total_flops --app $app->name";
452 452
     $x = exec($cmd);
453 453
     if (!is_numeric($x) || (double)$x == 0) {
454 454
         xml_error(-1, "$cmd returned $x");
@@ -533,15 +533,15 @@  discard block
 block discarded – undo
533 533
     }
534 534
 
535 535
     $job_params = new StdClass;
536
-    $job_params->rsc_disk_bound = (double) $r->batch->job_params->rsc_disk_bound;
537
-    $job_params->rsc_fpops_est = (double) $r->batch->job_params->rsc_fpops_est;
538
-    $job_params->rsc_fpops_bound = (double) $r->batch->job_params->rsc_fpops_bound;
539
-    $job_params->rsc_memory_bound = (double) $r->batch->job_params->rsc_memory_bound;
540
-    $job_params->delay_bound = (double) $r->batch->job_params->delay_bound;
536
+    $job_params->rsc_disk_bound = (double)$r->batch->job_params->rsc_disk_bound;
537
+    $job_params->rsc_fpops_est = (double)$r->batch->job_params->rsc_fpops_est;
538
+    $job_params->rsc_fpops_bound = (double)$r->batch->job_params->rsc_fpops_bound;
539
+    $job_params->rsc_memory_bound = (double)$r->batch->job_params->rsc_memory_bound;
540
+    $job_params->delay_bound = (double)$r->batch->job_params->delay_bound;
541 541
         // could add quorum-related stuff
542 542
 
543
-    $input_template_filename = (string) $r->batch->input_template_filename;
544
-    $output_template_filename = (string) $r->batch->output_template_filename;
543
+    $input_template_filename = (string)$r->batch->input_template_filename;
544
+    $output_template_filename = (string)$r->batch->output_template_filename;
545 545
         // possibly empty
546 546
 
547 547
     submit_jobs(
@@ -634,7 +634,7 @@  discard block
 block discarded – undo
634 634
 }
635 635
 
636 636
 function n_outfiles($wu) {
637
-    $path = project_dir() . "/$wu->output_template_filename";
637
+    $path = project_dir()."/$wu->output_template_filename";
638 638
     $r = simplexml_load_file($path);
639 639
     return count($r->file_info);
640 640
 }
@@ -841,7 +841,7 @@  discard block
 block discarded – undo
841 841
 ";
842 842
         if ($result->server_state == 5) {   // over?
843 843
             $paths = get_outfile_paths($result);
844
-            foreach($paths as $path) {
844
+            foreach ($paths as $path) {
845 845
                 if (is_file($path)) {
846 846
                     $size = filesize($path);
847 847
                     echo "        <outfile>
@@ -885,7 +885,7 @@  discard block
 block discarded – undo
885 885
     } else {
886 886
         $results = BoincResult::enum("workunitid=$job_id");
887 887
         foreach ($results as $r) {
888
-            switch($r->outcome) {
888
+            switch ($r->outcome) {
889 889
             case 1:
890 890
             case 3:
891 891
             case 6:
@@ -1000,8 +1000,8 @@  discard block
 block discarded – undo
1000 1000
     }
1001 1001
 
1002 1002
     $user = check_remote_submit_permissions($r, $app);
1003
-    $in = file_get_contents(project_dir() . "/templates/".$app->name."_in");
1004
-    $out = file_get_contents(project_dir() . "/templates/".$app->name."_out");
1003
+    $in = file_get_contents(project_dir()."/templates/".$app->name."_in");
1004
+    $out = file_get_contents(project_dir()."/templates/".$app->name."_out");
1005 1005
     if ($in === false || $out === false) {
1006 1006
         log_write("template file missing");
1007 1007
         xml_error(-1, "template file missing");
@@ -1013,7 +1013,7 @@  discard block
 block discarded – undo
1013 1013
 
1014 1014
 function ping($r) {
1015 1015
     xml_start_tag("ping");
1016
-    BoincDb::get();     // errors out if DB down or web disabled
1016
+    BoincDb::get(); // errors out if DB down or web disabled
1017 1017
     echo "<success>1</success>
1018 1018
         </ping>
1019 1019
     ";
@@ -1080,9 +1080,9 @@  discard block
 block discarded – undo
1080 1080
 $request_log = parse_config(get_config(), "<remote_submit_request_log>");
1081 1081
 if ($request_log) {
1082 1082
     $log_dir = parse_config(get_config(), "<log_dir>");
1083
-    $request_log = $log_dir . "/" . $request_log;
1083
+    $request_log = $log_dir."/".$request_log;
1084 1084
     if ($file = fopen($request_log, "a")) {
1085
-        fwrite($file, "\n<submit_rpc_handler date=\"" . date(DATE_ATOM) . "\">\n" . $_POST['request'] . "\n</submit_rpc_handler>\n");
1085
+        fwrite($file, "\n<submit_rpc_handler date=\"".date(DATE_ATOM)."\">\n".$_POST['request']."\n</submit_rpc_handler>\n");
1086 1086
         fclose($file);
1087 1087
     }
1088 1088
 }
Please login to merge, or discard this patch.
html/user/get_output2.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -51,7 +51,7 @@  discard block
 block discarded – undo
51 51
     if ($user->authenticator != $auth) die('bad auth');
52 52
 }
53 53
 
54
-function do_result_aux($result, $batch, $file_num=null) {
54
+function do_result_aux($result, $batch, $file_num = null) {
55 55
     $phys_names = get_outfile_phys_names($result);
56 56
     $log_names = get_outfile_log_names($result);
57 57
     if ($file_num !== null) {
@@ -71,7 +71,7 @@  discard block
 block discarded – undo
71 71
         $dir_path = "submit/$batch->id/$result->name";
72 72
         system("rm -r $dir_path");
73 73
         mkdir($dir_path);
74
-        for ($i=0; $i<count($phys_names); $i++) {
74
+        for ($i = 0; $i < count($phys_names); $i++) {
75 75
             $cmd = sprintf('ln -s %s %s/%s',
76 76
                 upload_path($phys_names[$i]),
77 77
                 $dir_path,
@@ -88,7 +88,7 @@  discard block
 block discarded – undo
88 88
     }
89 89
 }
90 90
 
91
-function do_result($result_id, $auth, $file_num=null) {
91
+function do_result($result_id, $auth, $file_num = null) {
92 92
     $result = BoincResult::lookup_id($result_id);
93 93
     if (!$result) die("no result $result_id");
94 94
     $workunit = BoincWorkunit::lookup_id($result->workunitid);
@@ -135,7 +135,7 @@  discard block
 block discarded – undo
135 135
             system($cmd);
136 136
         } else {
137 137
             mkdir(sprintf('%s/%s', $dir_path, $result->name));
138
-            for ($i=0; $i<count($phys_names); $i++) {
138
+            for ($i = 0; $i < count($phys_names); $i++) {
139 139
                 $cmd = sprintf('ln -s %s %s/%s/%s',
140 140
                     upload_path($phys_names[$i]),
141 141
                     $dir_path,
Please login to merge, or discard this patch.
html/user/submit.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -461,7 +461,7 @@  discard block
 block discarded – undo
461 461
     if ($owner) {
462 462
         row2('submitter', $owner->name);
463 463
     }
464
-    row2("application", $app?$app->name:'---');
464
+    row2("application", $app ? $app->name : '---');
465 465
     row2("state", batch_state_string($batch->state));
466 466
     //row2("# jobs", $batch->njobs);
467 467
     //row2("# error jobs", $batch->nerror_jobs);
@@ -523,7 +523,7 @@  discard block
 block discarded – undo
523 523
         $x[] = "Download Results";
524 524
     }
525 525
     row_heading_array($x);
526
-    foreach($wus as $wu) {
526
+    foreach ($wus as $wu) {
527 527
         $resultid = $wu->canonical_resultid;
528 528
         if ($resultid) {
529 529
             $y = '<font color="green">completed</font>';
@@ -532,7 +532,7 @@  discard block
 block discarded – undo
532 532
             $text = "---";
533 533
             if ($batch->state == BATCH_STATE_COMPLETE) {
534 534
                 $y = '<font color="red">failed</font>';
535
-            }   else {
535
+            } else {
536 536
                 $y = "in progress";
537 537
             }
538 538
         }
@@ -595,7 +595,7 @@  discard block
 block discarded – undo
595 595
     $results = BoincResult::enum("workunitid=$wuid");
596 596
     $upload_dir = parse_config(get_config(), "<upload_dir>");
597 597
     $fanout = parse_config(get_config(), "<uldl_dir_fanout>");
598
-    foreach($results as $result) {
598
+    foreach ($results as $result) {
599 599
         $x = [
600 600
             "<a href=result.php?resultid=$result->id>$result->id</a>",
601 601
             state_string($result)
@@ -604,7 +604,7 @@  discard block
 block discarded – undo
604 604
         if ($result->server_state == RESULT_SERVER_STATE_OVER) {
605 605
             $phys_names = get_outfile_phys_names($result);
606 606
             $log_names = get_outfile_log_names($result);
607
-            for ($i=0; $i<count($phys_names); $i++) {
607
+            for ($i = 0; $i < count($phys_names); $i++) {
608 608
                 if ($web_app->assim_move) {
609 609
                     // file is in
610 610
                     // project/results/<batchid>/<wu_name>__file_<log_name>
Please login to merge, or discard this patch.
html/inc/submit_util.inc 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -300,7 +300,7 @@  discard block
 block discarded – undo
300 300
 // get the total size of output files of a batch
301 301
 //
302 302
 function batch_output_file_size($batchid) {
303
-    $batch_td_size=0;
303
+    $batch_td_size = 0;
304 304
     $wus = BoincWorkunit::enum("batch=$batchid");
305 305
     $fanout = parse_config(get_config(), "<uldl_dir_fanout>");
306 306
     $upload_dir = parse_config(get_config(), "<upload_dir>");
@@ -330,7 +330,7 @@  discard block
 block discarded – undo
330 330
 }
331 331
 
332 332
 function boinc_get_wu_output_files_url($user, $wu_id) {
333
-    $auth_str =  md5($user->authenticator.$wu_id);
333
+    $auth_str = md5($user->authenticator.$wu_id);
334 334
     return "get_output.php?cmd=workunit_files&wu_id=$wu_id&auth_str=$auth_str";
335 335
 }
336 336
 
@@ -364,7 +364,7 @@  discard block
 block discarded – undo
364 364
     if (!file_exists($path)) return null;
365 365
     $x = file_get_contents($path);
366 366
     $n = sscanf($x, "%s %d", $md5, $size);
367
-    if ($n != 2 || strlen($md5)!=32) {
367
+    if ($n != 2 || strlen($md5) != 32) {
368 368
         return null;
369 369
     }
370 370
     return [$md5, $size];
Please login to merge, or discard this patch.
html/inc/bootstrap.inc 1 patch
Spacing   +20 added lines, -20 removed lines patch added patch discarded remove patch
@@ -141,12 +141,12 @@  discard block
 block discarded – undo
141 141
         // prefix for links; needed for pages not in top dir
142 142
     $user,
143 143
         // logged-in user, if any
144
-    $fixed=false,
144
+    $fixed = false,
145 145
         // if true, navbar is fixed at top of page.
146 146
         // NOTE: if you do this, you must set a global var $fixed_navbar
147 147
         // to true at compile time
148 148
         // (it needs to be set when page_head() is called).
149
-    $inverse=false
149
+    $inverse = false
150 150
         // white on black?
151 151
 ) {
152 152
     global $master_url;
@@ -215,7 +215,7 @@  discard block
 block discarded – undo
215 215
 // output a panel.
216 216
 // $content_func is a function that generates the panel contents
217 217
 //
218
-function panel($title, $content_func, $class="panel-primary", $body_class="") {
218
+function panel($title, $content_func, $class = "panel-primary", $body_class = "") {
219 219
     echo sprintf('<div class="panel %s">
220 220
         ', $class
221 221
     );
@@ -242,7 +242,7 @@  discard block
 block discarded – undo
242 242
 // $left_width is the width of left column in 1/12 units.
243 243
 // $arg is passed to the functions.
244 244
 //
245
-function grid($top_func, $left_func, $right_func, $left_width=6, $arg=null) {
245
+function grid($top_func, $left_func, $right_func, $left_width = 6, $arg = null) {
246 246
     echo '
247 247
         <div class="container-fluid">
248 248
     ';
@@ -257,7 +257,7 @@  discard block
 block discarded – undo
257 257
             </div>
258 258
         ';
259 259
     }
260
-    $right_width = 12-$left_width;
260
+    $right_width = 12 - $left_width;
261 261
     echo '
262 262
         <div class="row">
263 263
         <div class="col-sm-'.$left_width.'">
@@ -281,7 +281,7 @@  discard block
 block discarded – undo
281 281
 //      use extra = "name=x"
282 282
 //      call forum_focus(x, foo) after defining the field
283 283
 //
284
-function form_start($action, $method='get', $extra='') {
284
+function form_start($action, $method = 'get', $extra = '') {
285 285
     echo sprintf(
286 286
         '<div class="container-fluid">
287 287
         <form class="form-horizontal" method="%s" action="%s" %s>'
@@ -313,7 +313,7 @@  discard block
 block discarded – undo
313 313
 // just the input field
314 314
 //
315 315
 function form_input_text_field(
316
-    $name, $value='', $type='text', $attrs='', $extra=''
316
+    $name, $value = '', $type = 'text', $attrs = '', $extra = ''
317 317
 ) {
318 318
     return sprintf(
319 319
         '<input %s type="%s" class="form-control" name="%s" value="%s">%s',
@@ -324,7 +324,7 @@  discard block
 block discarded – undo
324 324
 // the whole row
325 325
 //
326 326
 function form_input_text(
327
-    $label, $name, $value='', $type='text', $attrs='', $extra=''
327
+    $label, $name, $value = '', $type = 'text', $attrs = '', $extra = ''
328 328
 ) {
329 329
     echo sprintf('
330 330
         <div class="form-group">
@@ -352,7 +352,7 @@  discard block
 block discarded – undo
352 352
     );
353 353
 }
354 354
 
355
-function form_input_textarea($label, $name, $value='', $nrows=4) {
355
+function form_input_textarea($label, $name, $value = '', $nrows = 4) {
356 356
     echo sprintf('
357 357
         <div class="form-group">
358 358
             <label align=right class="%s" for="%s">%s</label>
@@ -368,7 +368,7 @@  discard block
 block discarded – undo
368 368
 
369 369
 // $items is either a string of <option> elements, or an array
370 370
 //
371
-function form_select($label, $name, $items, $selected=null) {
371
+function form_select($label, $name, $items, $selected = null) {
372 372
     echo sprintf('
373 373
         <div class="form-group">
374 374
             <label align=right class="%s" for="%s">%s</label>
@@ -381,7 +381,7 @@  discard block
 block discarded – undo
381 381
         foreach ($items as $i) {
382 382
             echo sprintf(
383 383
                 '<option %s value=%s>%s</option>',
384
-                ($i[0]==$selected)?'selected':'',
384
+                ($i[0] == $selected) ? 'selected' : '',
385 385
                 $i[0], $i[1]
386 386
             );
387 387
         }
@@ -394,7 +394,7 @@  discard block
 block discarded – undo
394 394
 // same, for multiple select.
395 395
 // $selected, if non-null, is a list of selected values
396 396
 //
397
-function form_select_multiple($label, $name, $items, $selected=null) {
397
+function form_select_multiple($label, $name, $items, $selected = null) {
398 398
     echo sprintf('
399 399
         <div class="form-group">
400 400
             <label align=right class="%s" for="%s">%s</label>
@@ -406,7 +406,7 @@  discard block
 block discarded – undo
406 406
     foreach ($items as $i) {
407 407
         echo sprintf(
408 408
             '<option %s value=%s>%s</option>',
409
-            ($selected && in_array($i[0], $selected))?'selected':'',
409
+            ($selected && in_array($i[0], $selected)) ? 'selected' : '',
410 410
             $i[0], $i[1]
411 411
         );
412 412
     }
@@ -415,12 +415,12 @@  discard block
 block discarded – undo
415 415
 
416 416
 // return a list of string for checkbox items
417 417
 //
418
-function checkbox_item_strings($items, $attrs='') {
418
+function checkbox_item_strings($items, $attrs = '') {
419 419
     $x = [];
420 420
     foreach ($items as $i) {
421 421
         $x[] = sprintf('<input %s type="checkbox" name="%s" %s> %s
422 422
             ',
423
-            $attrs, $i[0], $i[2]?"checked":"", $i[1]
423
+            $attrs, $i[0], $i[2] ? "checked" : "", $i[1]
424 424
         );
425 425
     }
426 426
     return $x;
@@ -428,7 +428,7 @@  discard block
 block discarded – undo
428 428
 
429 429
 // $items is list of (name, label, checked)
430 430
 //
431
-function form_checkboxes($label, $items, $attrs='') {
431
+function form_checkboxes($label, $items, $attrs = '') {
432 432
     echo sprintf('
433 433
         <div class="form-group">
434 434
             <label align=right class="%s">%s</label>
@@ -454,7 +454,7 @@  discard block
 block discarded – undo
454 454
         FORM_LEFT_CLASS, $label, FORM_RIGHT_CLASS
455 455
     );
456 456
     foreach ($items as $i) {
457
-        $checked = ($selected == $i[0])?"checked":"";
457
+        $checked = ($selected == $i[0]) ? "checked" : "";
458 458
         echo sprintf('<input type="radio" name="%s" value="%s" %s> %s <br>
459 459
             ',
460 460
             $name, $i[0], $checked, $i[1]
@@ -487,7 +487,7 @@  discard block
 block discarded – undo
487 487
 ';
488 488
 }
489 489
 
490
-function form_submit($text, $attrs='') {
490
+function form_submit($text, $attrs = '') {
491 491
     form_general(
492 492
         "",
493 493
         sprintf(
@@ -497,7 +497,7 @@  discard block
 block discarded – undo
497 497
     );
498 498
 }
499 499
 
500
-function form_checkbox($label, $name, $checked=false) {
500
+function form_checkbox($label, $name, $checked = false) {
501 501
     echo sprintf('
502 502
         <div class="form-group">
503 503
             <label align=right class="%s">%s</label>
@@ -508,6 +508,6 @@  discard block
 block discarded – undo
508 508
     echo sprintf('
509 509
         <input type="checkbox" name="%s" %s>
510 510
         </div>
511
-        ', $name, $checked?"checked":""
511
+        ', $name, $checked ? "checked" : ""
512 512
     );
513 513
 }
Please login to merge, or discard this patch.
html/user/sandbox.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -169,7 +169,7 @@
 block discarded – undo
169 169
     $notice = "";
170 170
     $dir = sandbox_dir($user);
171 171
     $count = count($_FILES['new_file']['tmp_name']);
172
-    for ($i=0; $i<$count; $i++) {
172
+    for ($i = 0; $i < $count; $i++) {
173 173
         $tmp_name = $_FILES['new_file']['tmp_name'][$i];
174 174
         if (!is_uploaded_file($tmp_name)) {
175 175
             error_page("$tmp_name is not uploaded file");
Please login to merge, or discard this patch.