BOINC / boinc

html/ops/manage_user.php

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -141,7 +141,7 @@
     global $special_user_bitfield;
     $Nbf = sizeof($special_user_bitfield);
 
-    admin_page_head("Management $user->name");
+    admin_page_head("management $user->name");
 
     if (!defined("POST_REPORT_EMAILS")) {
         echo "<p><font color='RED'>

Braces +4 added lines, -4 removed lines

@@ -39,8 +39,8 @@ 
 
 // Delete a user if they have no credit, results, or posts
 //
-function possibly_delete_user($user){
-    if ($user->total_credit > 0.0){
+function possibly_delete_user($user) {
+    if ($user->total_credit > 0.0) {
         admin_error_page("Cannot delete user: User has credit.");
     }
 
@@ -56,7 +56,7 @@ 
         admin_error_page("Cannot delete user: User has forum posts.");
     }
 
-    if ($user->teamid){
+    if ($user->teamid) {
         user_quit_team($user);
     }
     delete_user($user);
@@ -100,7 +100,7 @@ 
 
             // put a timestamp in wiki to trigger re-validation of credentials
 
-            if (function_exists('touch_wiki_user')){
+            if (function_exists('touch_wiki_user')) {
                 touch_wiki_user($user);
             }
 

Spacing +23 added lines, -23 removed lines

@@ -39,8 +39,8 @@ 
 
 // Delete a user if they have no credit, results, or posts
 //
-function possibly_delete_user($user){
-    if ($user->total_credit > 0.0){
+function possibly_delete_user($user) {
+    if ($user->total_credit > 0.0) {
         admin_error_page("Cannot delete user: User has credit.");
     }
 
@@ -56,7 +56,7 @@ 
         admin_error_page("Cannot delete user: User has forum posts.");
     }
 
-    if ($user->teamid){
+    if ($user->teamid) {
         user_quit_team($user);
     }
     delete_user($user);
@@ -67,8 +67,8 @@ 
 function handle_special_user($user) {
     global $special_user_bitfield;
     $Nbf = sizeof($special_user_bitfield);
-    $bits="";
-    for ($i=0; $i<$Nbf; $i++) {
+    $bits = "";
+    for ($i = 0; $i < $Nbf; $i++) {
         $key = "special_user_$i";
         if (array_key_exists($key, $_POST) && $_POST[$key]) {
             $bits .= "1";
@@ -94,35 +94,35 @@ 
         );
     } else {
         if (is_numeric($dt)) {
-            $t = $dt>0 ? time()+$dt : 0;
+            $t = $dt > 0 ? time() + $dt : 0;
             $q = "UPDATE forum_preferences SET banished_until=$t WHERE userid=$user->id";
             _mysql_query($q);
 
             // put a timestamp in wiki to trigger re-validation of credentials
 
-            if (function_exists('touch_wiki_user')){
+            if (function_exists('touch_wiki_user')) {
                 touch_wiki_user($user);
             }
 
             // Send suspension e-mail to user and administrators
 
-            if ($dt>0) {
-                $subject = PROJECT." posting privileges suspended for ". $user->name;
+            if ($dt > 0) {
+                $subject = PROJECT." posting privileges suspended for ".$user->name;
                 $body = "
-Forum posting privileges for the " .PROJECT. " user \"".$user->name."\"
-have been suspended for " .time_diff($dt). " by ".$g_logged_in_user->name.".
+Forum posting privileges for the " .PROJECT." user \"".$user->name."\"
+have been suspended for " .time_diff($dt)." by ".$g_logged_in_user->name.".
 The reason given was:
 
 $reason
 
-The suspension will end at " .time_str($t)."\n";
+The suspension will end at ".time_str($t)."\n";
             } else {
-                $subject = PROJECT." user ". $user->name. " unsuspended";
+                $subject = PROJECT." user ".$user->name." unsuspended";
                 $body = "
-Forum posting privileges for the " .PROJECT. " user \"".$user->name."\"
+Forum posting privileges for the " .PROJECT." user \"".$user->name."\"
 have been restored by ".$g_logged_in_user->name."\n";
                 if ($reason) {
-                    $body.="The reason given was:\n\n   $reason\n";
+                    $body .= "The reason given was:\n\n   $reason\n";
                 }
             }
 
@@ -186,13 +186,13 @@ 
     row1("Special User Status");
 
     echo "<tr>\n";
-    for ($i=0; $i<$Nbf; $i++) {
+    for ($i = 0; $i < $Nbf; $i++) {
         $bit = substr($user->prefs->special_user, $i, 1);
         echo "<tr><td><input type='checkbox'' name='special_user_".$i."' value='1'";
         if ($bit == 1) {
             echo " checked='checked'";
         }
-        echo ">". $special_user_bitfield[$i] ."</td></tr>\n";
+        echo ">".$special_user_bitfield[$i]."</td></tr>\n";
     }
     echo "</tr>";
 
@@ -217,10 +217,10 @@ 
     if ($user->prefs->banished_until) {
         $dt = $user->prefs->banished_until - time();
         if ($dt > 0) {
-            $x = " Suspended until " . time_str($user->prefs->banished_until)
-                ."<br/> (Expires in " . time_diff($dt) .")" ;
+            $x = " Suspended until ".time_str($user->prefs->banished_until)
+                ."<br/> (Expires in ".time_diff($dt).")";
         } else {
-            $x = " last suspended " . time_str($user->prefs->banished_until);
+            $x = " last suspended ".time_str($user->prefs->banished_until);
         }
         row1($x);
     } else {
@@ -238,11 +238,11 @@ 
     ";
     echo "
         <input type='radio' name='suspend_for' value='172800'> 48 hours  <br/>
-        <input type='radio' name='suspend_for' value='",86400*7,"'> 1 week  <br/>
-        <input type='radio' name='suspend_for' value='",86400*14,"'> 2 weeks  <br/>
+        <input type='radio' name='suspend_for' value='",86400*7, "'> 1 week  <br/>
+        <input type='radio' name='suspend_for' value='",86400*14, "'> 2 weeks  <br/>
     ";
 
-    if ($dt>0) {
+    if ($dt > 0) {
         echo "
             <input type='radio' name='suspend_for' value='-1'>  <b>unsuspend</b>   <br/>";
     }

html/ops/manage_apps.php

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
     $n = post_str("fraction_done_exact", true)?1:0;
     $app->update("fraction_done_exact=$n");
 
-    echo "Application $id updated.
+    echo "application $id updated.
         <p>
         You must restart the project for this to take effect.
     ";

Spacing +14 added lines, -14 removed lines

@@ -25,7 +25,7 @@ 
     $app = BoincApp::lookup_id($id);
     if (!$app) admin_error_page("no such app");
 
-    $n = post_str("deprecated", true)?1:0;
+    $n = post_str("deprecated", true) ? 1 : 0;
     $app->update("deprecated=$n");
 
     $n = post_num("weight");
@@ -37,16 +37,16 @@ 
     $n = post_int("target_nresults");
     $app->update("target_nresults=$n");
 
-    $n = post_str("homogeneous_app_version", true)?1:0;
+    $n = post_str("homogeneous_app_version", true) ? 1 : 0;
     $app->update("homogeneous_app_version=$n");
 
-    $n = post_str("non_cpu_intensive", true)?1:0;
+    $n = post_str("non_cpu_intensive", true) ? 1 : 0;
     $app->update("non_cpu_intensive=$n");
 
-    $n = post_str("beta", true)?1:0;
+    $n = post_str("beta", true) ? 1 : 0;
     $app->update("beta=$n");
 
-    $n = post_str("fraction_done_exact", true)?1:0;
+    $n = post_str("fraction_done_exact", true) ? 1 : 0;
     $app->update("fraction_done_exact=$n");
 
     echo "Application $id updated.
@@ -58,7 +58,7 @@ 
 function add_app() {
     $name = BoincDb::escape_string(post_str('add_name'));
     $user_friendly_name = BoincDb::escape_string(post_str('add_user_friendly_name'));
-    if (empty($name) || empty($user_friendly_name) ) {
+    if (empty($name) || empty($user_friendly_name)) {
         admin_error_page(
             "To add a new application please supply both a brief name and a longer 'user-friendly' name.</font></p>"
         );
@@ -81,11 +81,11 @@ 
         <h2>Edit applications</h2>
     ";
 
-    $app_clause="deprecated=0";
-    $action_url="manage_apps.php";
-    if($all) {
+    $app_clause = "deprecated=0";
+    $action_url = "manage_apps.php";
+    if ($all) {
         $app_clause = "";
-        $action_url="manage_apps.php?all=1";
+        $action_url = "manage_apps.php?all=1";
         echo "<a href=\"manage_apps.php\">Don't show deprecated applications</a>";
     } else {
         echo "<a href=\"manage_apps.php?all=1\">Show deprecated applications</a>";
@@ -117,8 +117,8 @@ 
     $apps = BoincApp::enum($app_clause);
     foreach ($apps as $app) {
         // grey-out deprecated versions
-        $f1=$f2='';
-        if ($app->deprecated==1) {
+        $f1 = $f2 = '';
+        if ($app->deprecated == 1) {
             $f1 = "<font color='GREY'>";
             $f2 = "</font>";
         }
@@ -128,7 +128,7 @@ 
 
         echo "  <TD align='left'>$f1<a href=app_details.php?appid=$app->id>$app->name</a><br> $app->user_friendly_name $f2</TD>\n";
 
-        echo "  <TD align='center'>$f1 " .date_str($app->create_time)."$f2</TD>\n";
+        echo "  <TD align='center'>$f1 ".date_str($app->create_time)."$f2</TD>\n";
 
         $v = $app->weight;
         echo "  <TD align='center'>
@@ -151,7 +151,7 @@ 
         ";
 
         $v = '';
-        if ($app->homogeneous_app_version) $v=' CHECKED ';
+        if ($app->homogeneous_app_version) $v = ' CHECKED ';
         echo "  <TD align='center'>
             <input name='homogeneous_app_version' type='checkbox' $v></TD>
         ";

html/ops/cancel_workunits.php

Braces +2 added lines, -1 removed lines

@@ -100,7 +100,8 @@
     echo "</p>\n";
     echo "</form>\n";
 
-} else { // if ($qclause)
+} else {
+// if ($qclause)
 
     $query = "SELECT id, name FROM workunit WHERE canonical_resultid = 0 AND error_mask = 0 AND $qclause;";
     $db = BoincDb::get(true);

Spacing +3 added lines, -3 removed lines

@@ -29,7 +29,7 @@ 
 admin_page_head("Cancel Jobs");
 
 $limit = get_int('limit', true);
-if (!$limit || $limit==0) {
+if (!$limit || $limit == 0) {
     $limit = 100;
 }
 
@@ -42,9 +42,9 @@ 
 $clause = get_str('clause', true);
 
 if ($minid && $maxid) {
-    $qclause = "id >=" . $minid . " AND id <=" . $maxid;
+    $qclause = "id >=".$minid." AND id <=".$maxid;
 } else if ($list) {
-    $qclause = "id IN (" . $list . ")";
+    $qclause = "id IN (".$list.")";
 } else if ($uclause) {
     $qclause = urldecode($uclause);
 } else if ($clause) {

html/ops/build_po.php

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
 "X-Poedit-SourceCharset: utf-8\\n"
 
 
-HDR;
+hdr;
 
 $out = fopen("en.po", "w");
 

html/user/server_status.php

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -251,7 +251,7 @@
         $pid = file_get_contents($path);
         if ($pid) {
             $pid = trim($pid);
-            $out = Array();
+            $out = array();
             exec("ps -ww $pid", $out);
             foreach ($out as $y) {
                 if (strstr($y, (string)$pid)) return 1;

Spacing +9 added lines, -9 removed lines

@@ -180,7 +180,7 @@ 
             $avg = round($app->info->avg, 2);
             $min = round($app->info->min, 2);
             $max = round($app->info->max, 2);
-            $x = $max?"$avg ($min - $max)":"---";
+            $x = $max ? "$avg ($min - $max)" : "---";
             $u = $app->info->users;
         } else {
             $x = '---';
@@ -275,7 +275,7 @@ 
         $cmd = trim($cmd);
         $x = explode(" ", $cmd);
         $prog = $x[0];
-        $pidname = $prog . '.pid';
+        $pidname = $prog.'.pid';
     }
     $path = "../../pid_$host/$pidname";
     if (is_file($path)) {
@@ -316,12 +316,12 @@ 
     }
     $master_host = $u["host"];
     if ($config->www_host) {
-        $web_host = trim((string) $config->www_host);
+        $web_host = trim((string)$config->www_host);
     } else {
         $web_host = $main_host;
     }
     if ($config->sched_host) {
-        $sched_host = trim((string) $config->sched_host);
+        $sched_host = trim((string)$config->sched_host);
     } else {
         $sched_host = $main_host;
     }
@@ -331,7 +331,7 @@ 
 
     // the upload and download servers are sort of daemons too
     //
-    $url = trim((string) $config->download_url);
+    $url = trim((string)$config->download_url);
     $u = parse_url($url);
     $h = $u["host"];
     if ($h == $master_host) {
@@ -343,14 +343,14 @@ 
     } else {
         $have_remote = true;
     }
-    $url = trim((string) $config->upload_url);
+    $url = trim((string)$config->upload_url);
     $u = parse_url($url);
     $h = $u["host"];
     if ($h == $master_host) {
         $y = new StdClass;
         $y->cmd = "Upload server";
         $y->host = $h;
-        $y->status = !file_exists("../../stop_upload");;
+        $y->status = !file_exists("../../stop_upload"); ;
         $local_daemons[] = $y;
     } else {
         $have_remote = true;
@@ -361,7 +361,7 @@ 
     $y = new StdClass;
     $y->cmd = "Scheduler";
     $y->host = $sched_host;
-    $y->status = !file_exists("../../stop_sched");;
+    $y->status = !file_exists("../../stop_sched"); ;
     $local_daemons[] = $y;
 
     foreach ($daemons->daemon as $d) {
@@ -374,7 +374,7 @@ 
             $disabled_daemons[] = $x;
             continue;
         }
-        $host = $d->host?(string)$d->host:$main_host;
+        $host = $d->host ? (string)$d->host : $main_host;
         if ($host != $web_host) {
             $have_remote = true;
             continue;

html/user/team_change_founder_form.php

Braces +2 added lines, -1 removed lines

@@ -92,7 +92,8 @@
 
 $navailable_users = 0;
 foreach ($users as $user) {
-    if ($user->id != $team->userid) {       //don't show current founder
+    if ($user->id != $team->userid) {
+//don't show current founder
         $user_total_credit = format_credit($user->total_credit);
         $user_expavg_credit = format_credit($user->expavg_credit);
         $selected = ($user->id == $team->ping_user)?"selected":"";

Spacing +1 added lines, -1 removed lines

@@ -95,7 +95,7 @@
     if ($user->id != $team->userid) {       //don't show current founder
         $user_total_credit = format_credit($user->total_credit);
         $user_expavg_credit = format_credit($user->expavg_credit);
-        $selected = ($user->id == $team->ping_user)?"selected":"";
+        $selected = ($user->id == $team->ping_user) ? "selected" : "";
         echo '
             <tr>
             <td><input type="radio" name="userid" value="'.$user->id.'">

html/user/submit_status.php

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
     if (!$batch || $batch->user_id != $user->id) {
         error_page("no batch");
     }
-    page_head("Batch $batch->id");
+    page_head("batch $batch->id");
     $results = BoincResult::enum("batch=$batch->id order by workunitid");
     result_table_start(true, true, null);
     foreach ($results as $result) {

html/user/submit_rpc_handler.php

Braces +2 added lines, -1 removed lines

@@ -691,7 +691,8 @@
         <id>$result->id</id>
         <state>".state_string($result)."</state>
 ";
-        if ($result->server_state == 5) {   // over?
+        if ($result->server_state == 5) {
+// over?
             $paths = get_outfile_paths($result);
             foreach($paths as $path) {
                 if (is_file($path)) {

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -595,7 +595,7 @@
 // return a batch specified by the command, using either ID or name
 //
 function get_batch($r) {
-    $batch = NULL;
+    $batch = null;
     if (!empty($r->batch_id)) {
         $batch_id = (int)($r->batch_id);
         $batch = BoincBatch::lookup_id($batch_id);

Switch Indentation +18 added lines, -18 removed lines

@@ -1102,24 +1102,24 @@
 log_write("----- Handling RPC; command ".$r->getName());
 
 switch ($r->getName()) {
-    case 'abort_batch': handle_abort_batch($r); break;
-    case 'abort_jobs': handle_abort_jobs($r); break;
-    case 'create_batch': create_batch($r); break;
-    case 'estimate_batch': estimate_batch($r); break;
-    case 'get_templates': get_templates($r); break;
-    case 'ping': ping($r); break;
-    case 'query_batch': query_batch($r); break;
-    case 'query_batch2': query_batch2($r); break;
-    case 'query_batches': query_batches($r); break;
-    case 'query_job': query_job($r); break;
-    case 'query_completed_job': query_completed_job($r); break;
-    case 'retire_batch': handle_retire_batch($r); break;
-    case 'set_expire_time': handle_set_expire_time($r); break;
-    case 'submit_batch': submit_batch($r); break;
-    default:
-        log_write("bad command");
-        xml_error(-1, "bad command");
-        break;
+case 'abort_batch': handle_abort_batch($r); break;
+case 'abort_jobs': handle_abort_jobs($r); break;
+case 'create_batch': create_batch($r); break;
+case 'estimate_batch': estimate_batch($r); break;
+case 'get_templates': get_templates($r); break;
+case 'ping': ping($r); break;
+case 'query_batch': query_batch($r); break;
+case 'query_batch2': query_batch2($r); break;
+case 'query_batches': query_batches($r); break;
+case 'query_job': query_job($r); break;
+case 'query_completed_job': query_completed_job($r); break;
+case 'retire_batch': handle_retire_batch($r); break;
+case 'set_expire_time': handle_set_expire_time($r); break;
+case 'submit_batch': submit_batch($r); break;
+default:
+    log_write("bad command");
+    xml_error(-1, "bad command");
+    break;
 }
 
 log_write("RPC done");

Spacing +30 added lines, -30 removed lines

@@ -59,7 +59,7 @@ 
     if ($template) {
         $t = (double)$template->workunit->rsc_fpops_est;
     }
-    foreach($r->batch->job as $job) {
+    foreach ($r->batch->job as $job) {
         $y = (double)$job->rsc_fpops_est;
         if ($y) {
             $x += $y;
@@ -85,7 +85,7 @@ 
 function est_elapsed_time($r, $template) {
     // crude estimate: batch FLOPs / project FLOPS
     //
-    return batch_flop_count($r, $template) / project_flops();
+    return batch_flop_count($r, $template)/project_flops();
 }
 
 // if batch-level input template filename was given, read it;
@@ -95,9 +95,9 @@ 
 //
 function read_input_template($app, $r) {
     if ((isset($r->batch)) && (isset($r->batch->workunit_template_file)) && ($r->batch->workunit_template_file)) {
-        $path = project_dir() . "/templates/".$r->batch->workunit_template_file;
+        $path = project_dir()."/templates/".$r->batch->workunit_template_file;
     } else {
-        $path = project_dir() . "/templates/$app->name"."_in";
+        $path = project_dir()."/templates/$app->name"."_in";
     }
     if (file_exists($path)) {
         $x = simplexml_load_file($path);
@@ -145,7 +145,7 @@ 
 function validate_batch($jobs, $template) {
     $i = 0;
     $n = count($template->file_info);
-    foreach($jobs as $job) {
+    foreach ($jobs as $job) {
         $m = count($job->input_files);
         if ($n != $m) {
             log_write("wrong # of input files for job $i: need $n, got $m");
@@ -214,7 +214,7 @@ 
 // stage all the files
 //
 function stage_files(&$jobs, $user) {
-    foreach($jobs as $job) {
+    foreach ($jobs as $job) {
         foreach ($job->input_files as $file) {
             if ($file->mode != "remote") {
                 $file->name = stage_file($file, $user);
@@ -227,7 +227,7 @@ 
 //
 function submit_jobs(
     $jobs, $job_params, $app, $batch_id, $priority, $app_version_num,
-    $input_template_filename,        // batch-level; can also specify per job
+    $input_template_filename, // batch-level; can also specify per job
     $output_template_filename,
     $user
 ) {
@@ -237,7 +237,7 @@ 
     // one line per job
     //
     $x = "";
-    foreach($jobs as $job) {
+    foreach ($jobs as $job) {
         if ($job->name) {
             $x .= " --wu_name $job->name";
         }
@@ -272,7 +272,7 @@ 
         $x .= "\n";
     }
 
-    $cmd = "cd " . project_dir() . "; ./bin/create_work --appname $app->name --batch $batch_id";
+    $cmd = "cd ".project_dir()."; ./bin/create_work --appname $app->name --batch $batch_id";
 
     if ($user->seti_id) {
         $cmd .= " --target_user $user->id ";
@@ -383,7 +383,7 @@ 
 //
 function xml_get_jobs($r) {
     $jobs = array();
-    foreach($r->batch->job as $j) {
+    foreach ($r->batch->job as $j) {
         $job = new StdClass;
         $job->input_files = array();
         $job->command_line = (string)$j->command_line;
@@ -436,16 +436,16 @@ 
 //
 function logical_end_time($r, $jobs, $user, $app) {
     $total_flops = 0;
-    foreach($jobs as $job) {
+    foreach ($jobs as $job) {
         //print_r($job);
         if ($job->rsc_fpops_est) {
             $total_flops += $job->rsc_fpops_est;
         } else if ($job->input_template && $job->input_template->workunit->rsc_fpops_est) {
-            $total_flops += (double) $job->input_template->workunit->rsc_fpops_est;
+            $total_flops += (double)$job->input_template->workunit->rsc_fpops_est;
         } else if ($r->batch->job_params->rsc_fpops_est) {
-            $total_flops += (double) $r->batch->job_params->rsc_fpops_est;
+            $total_flops += (double)$r->batch->job_params->rsc_fpops_est;
         } else {
-            $x = (double) $template->workunit->rsc_fpops_est;
+            $x = (double)$template->workunit->rsc_fpops_est;
             if ($x) {
                 $total_flops += $x;
             } else {
@@ -453,7 +453,7 @@ 
             }
         }
     }
-    $cmd = "cd " . project_dir() . "/bin; ./adjust_user_priority --user $user->id --flops $total_flops --app $app->name";
+    $cmd = "cd ".project_dir()."/bin; ./adjust_user_priority --user $user->id --flops $total_flops --app $app->name";
     $x = exec($cmd);
     if (!is_numeric($x) || (double)$x == 0) {
         xml_error(-1, "$cmd returned $x");
@@ -538,15 +538,15 @@ 
     }
 
     $job_params = new StdClass;
-    $job_params->rsc_disk_bound = (double) $r->batch->job_params->rsc_disk_bound;
-    $job_params->rsc_fpops_est = (double) $r->batch->job_params->rsc_fpops_est;
-    $job_params->rsc_fpops_bound = (double) $r->batch->job_params->rsc_fpops_bound;
-    $job_params->rsc_memory_bound = (double) $r->batch->job_params->rsc_memory_bound;
-    $job_params->delay_bound = (double) $r->batch->job_params->delay_bound;
+    $job_params->rsc_disk_bound = (double)$r->batch->job_params->rsc_disk_bound;
+    $job_params->rsc_fpops_est = (double)$r->batch->job_params->rsc_fpops_est;
+    $job_params->rsc_fpops_bound = (double)$r->batch->job_params->rsc_fpops_bound;
+    $job_params->rsc_memory_bound = (double)$r->batch->job_params->rsc_memory_bound;
+    $job_params->delay_bound = (double)$r->batch->job_params->delay_bound;
         // could add quorum-related stuff
 
-    $input_template_filename = (string) $r->batch->input_template_filename;
-    $output_template_filename = (string) $r->batch->output_template_filename;
+    $input_template_filename = (string)$r->batch->input_template_filename;
+    $output_template_filename = (string)$r->batch->output_template_filename;
         // possibly empty
 
     submit_jobs(
@@ -639,7 +639,7 @@ 
 }
 
 function n_outfiles($wu) {
-    $path = project_dir() . "/$wu->output_template_filename";
+    $path = project_dir()."/$wu->output_template_filename";
     $r = simplexml_load_file($path);
     return count($r->file_info);
 }
@@ -846,7 +846,7 @@ 
 ";
         if ($result->server_state == 5) {   // over?
             $paths = get_outfile_paths($result);
-            foreach($paths as $path) {
+            foreach ($paths as $path) {
                 if (is_file($path)) {
                     $size = filesize($path);
                     echo "        <outfile>
@@ -890,7 +890,7 @@ 
     } else {
         $results = BoincResult::enum("workunitid=$job_id");
         foreach ($results as $r) {
-            switch($r->outcome) {
+            switch ($r->outcome) {
             case 1:
             case 3:
             case 6:
@@ -1005,8 +1005,8 @@ 
     }
 
     $user = check_remote_submit_permissions($r, $app);
-    $in = file_get_contents(project_dir() . "/templates/".$app->name."_in");
-    $out = file_get_contents(project_dir() . "/templates/".$app->name."_out");
+    $in = file_get_contents(project_dir()."/templates/".$app->name."_in");
+    $out = file_get_contents(project_dir()."/templates/".$app->name."_out");
     if ($in === false || $out === false) {
         log_write("template file missing");
         xml_error(-1, "template file missing");
@@ -1018,7 +1018,7 @@ 
 
 function ping($r) {
     xml_start_tag("ping");
-    BoincDb::get();     // errors out if DB down or web disabled
+    BoincDb::get(); // errors out if DB down or web disabled
     echo "<success>1</success>
         </ping>
     ";
@@ -1092,9 +1092,9 @@ 
 $request_log = parse_config(get_config(), "<remote_submit_request_log>");
 if ($request_log) {
     $log_dir = parse_config(get_config(), "<log_dir>");
-    $request_log = $log_dir . "/" . $request_log;
+    $request_log = $log_dir."/".$request_log;
     if ($file = fopen($request_log, "a")) {
-        fwrite($file, "\n<submit_rpc_handler date=\"" . date(DATE_ATOM) . "\">\n" . $req . "\n</submit_rpc_handler>\n");
+        fwrite($file, "\n<submit_rpc_handler date=\"".date(DATE_ATOM)."\">\n".$req."\n</submit_rpc_handler>\n");
         fclose($file);
     }
 }

html/user/get_project_config.php

Braces +2 added lines, -2 removed lines

@@ -28,10 +28,10 @@
 //
 function show_platforms() {
     $xmlFragment = unserialize(get_cached_data(3600, "project_config_platform_xml"));
-    if ($xmlFragment==false){
+    if ($xmlFragment==false) {
         $platforms = BoincDB::get()->enum_fields("platform, DBNAME.app_version, DBNAME.app", "BoincPlatform", "platform.name, platform.user_friendly_name, plan_class", "app_version.platformid = platform.id and app_version.appid = app.id and app_version.deprecated=0 and app.deprecated=0 group by platform.name, plan_class", "");
         $xmlFragment = "    <platforms>";
-        foreach ($platforms as $platform){
+        foreach ($platforms as $platform) {
             $xmlFragment .= "
             <platform>
                 <platform_name>$platform->name</platform_name>

Indentation +4 added lines, -4 removed lines

@@ -86,11 +86,11 @@ 
 }
 
 if ($disable_account_creation || defined('INVITE_CODES')) {
-	echo "    <account_creation_disabled/>\n";
+    echo "    <account_creation_disabled/>\n";
 }
 
 if (defined('INVITE_CODES')) {
-	echo "    <invite_code_required/>\n";
+    echo "    <invite_code_required/>\n";
 }
 
 echo "    <min_passwd_length>$min_passwd_length</min_passwd_length>\n";
@@ -113,9 +113,9 @@ 
 // the terms_of_use.txt file in the project base directory.
 //
 if (defined('TERMSOFUSE_FILE')) {
-  $tou_file = TERMSOFUSE_FILE;
+    $tou_file = TERMSOFUSE_FILE;
 } else {
-  $tou_file =  "../../terms_of_use.txt";
+    $tou_file =  "../../terms_of_use.txt";
 }
 if (file_exists($tou_file)) {
     $terms_of_use = trim(file_get_contents($tou_file));

Spacing +3 added lines, -3 removed lines

@@ -30,10 +30,10 @@ 
 //
 function show_platforms() {
     $xmlFragment = unserialize(get_cached_data(3600, "project_config_platform_xml"));
-    if ($xmlFragment==false){
+    if ($xmlFragment == false) {
         $platforms = BoincDB::get()->enum_fields("platform, DBNAME.app_version, DBNAME.app", "BoincPlatform", "platform.name, platform.user_friendly_name, plan_class", "app_version.platformid = platform.id and app_version.appid = app.id and app_version.deprecated=0 and app.deprecated=0 group by platform.name, plan_class", "");
         $xmlFragment = "    <platforms>";
-        foreach ($platforms as $platform){
+        foreach ($platforms as $platform) {
             $xmlFragment .= "
             <platform>
                 <platform_name>$platform->name</platform_name>
@@ -115,7 +115,7 @@ 
 if (defined('TERMSOFUSE_FILE')) {
   $tou_file = TERMSOFUSE_FILE;
 } else {
-  $tou_file =  "../../terms_of_use.txt";
+  $tou_file = "../../terms_of_use.txt";
 }
 if (file_exists($tou_file)) {
     $terms_of_use = trim(file_get_contents($tou_file));