BOINC / boinc

html/ops/single_job_setup.php

Doc Comments +8 added lines

@@ -107,10 +107,18 @@
     }
 }
 
+/**
+ * @param integer $i
+ *
+ * @return string
+ */
 function app_version_dir($app_name, $i, $platform) {
     return "apps/$app_name/1.$i/$platform";
 }
 
+/**
+ * @param integer $i
+ */
 function make_app_version_dir($app_name, $i, $platform) {
     @mkdir("apps/$app_name");
     @mkdir("apps/$app_name/1.$i");

html/ops/team_export.php

Doc Comments +3 added lines

@@ -58,6 +58,9 @@
     return $x;
 }
 
+/**
+ * @param resource $f
+ */
 function handle_team($team, $f) {
     echo "Team: $team->name\n";
     $user = BoincUser::lookup_id($team->userid);

Spacing +1 added lines, -1 removed lines

@@ -100,7 +100,7 @@
     $f = fopen("temp.xml", "w");
     $teams = BoincTeam::enum(null);
     fwrite($f, "<?xml version=\"1.0\" encoding=\"utf-8\"?>\n<teams>\n");
-    foreach($teams as $team) {
+    foreach ($teams as $team) {
         handle_team($team, $f);
     }
     fwrite($f, "</teams>\n");

html/ops/team_import.php

Doc Comments +9 added lines

@@ -88,6 +88,9 @@ 
     return null;
 }
 
+/**
+ * @param stdClass $t
+ */
 function valid_team($t) {
     if (!$t->id) return false;
     if (!$t->name) return false;
@@ -96,6 +99,9 @@ 
     return true;
 }
 
+/**
+ * @param stdClass $t
+ */
 function update_team($t, $team, $user) {
     global $dry_run;
     if (
@@ -126,6 +132,9 @@ 
     }
 }
 
+/**
+ * @param stdClass $t
+ */
 function insert_case($t, $user) {
     global $master_url;
     global $dry_run;

Spacing +1 added lines, -1 removed lines

@@ -227,7 +227,7 @@
         if ($team->seti_id) {
             if ($team->seti_id == $t->id) {
                 echo "   case 1\n";
-                update_team($t, $team, $user);      // update1 case
+                update_team($t, $team, $user); // update1 case
             } else {
                 echo "   team exists but has wrong seti_id\n";
             }

Braces +3 added lines, -6 removed lines

@@ -63,8 +63,7 @@ 
             $t->user_postal_code = decode($t->user_postal_code);
             $t->user_url = decode($t->user_url);
             return $t;
-        }
-        else if (strstr($s, '<name>')) $t->name = parse_element($s, '<name>');
+        } else if (strstr($s, '<name>')) $t->name = parse_element($s, '<name>');
         else if (strstr($s, '<url>')) $t->url = parse_element($s, '<url>');
         else if (strstr($s, '<type>')) $t->type = parse_element($s, '<type>');
         else if (strstr($s, '<name_html>')) $t->name_html = parse_element($s, '<name_html>');
@@ -73,14 +72,12 @@ 
                 if (strstr($s, '</description>')) break;
                 $t->description .= $s;
             }
-        }
-        else if (strstr($s, '<country>')) $t->country = parse_element($s, '<country>');
+        } else if (strstr($s, '<country>')) $t->country = parse_element($s, '<country>');
         else if (strstr($s, '<id>')) $t->id = parse_element($s, '<id>');
         else if (strstr($s, '<user_email_munged>')) {
             $user_email_munged = parse_element($s, '<user_email_munged>');
             $t->user_email = str_rot13($user_email_munged);
-        }
-        else if (strstr($s, '<user_name>')) $t->user_name = parse_element($s, '<user_name>');
+        } else if (strstr($s, '<user_name>')) $t->user_name = parse_element($s, '<user_name>');
         else if (strstr($s, '<user_country>')) $t->user_country = parse_element($s, '<user_country>');
         else if (strstr($s, '<user_postal_code>')) $t->user_postal_code = parse_element($s, '<user_postal_code>');
         else if (strstr($s, '<user_url>')) $t->user_url = parse_element($s, '<user_url>');

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

@@ -212,7 +212,7 @@
         return;
     }
 
-    echo "Processing $t->name $t->user_email\n";
+    echo "processing $t->name $t->user_email\n";
     $user = BoincUser::lookup_email_addr($t->user_email);
     $team = BoincTeam::lookup_name($t->name);
     if ($team) {

html/ops/update_profile_pages.php

Doc Comments +8 added lines

@@ -98,6 +98,9 @@ 
 // Generates a standard set of links between associated multi-page documents.
 //  All linked files must be of the form "$filename_<page number>.html".
 
+/**
+ * @param integer $currPageNum
+ */
 function write_page_links($filename, $currPageNum, $numPages) {
     echo "<p>Page $currPageNum of $numPages</p>";
 
@@ -332,6 +335,11 @@ 
 // A generalized function to produce some number of pages summarizing a
 // set of user profiles.
 
+/**
+ * @param integer $rowsPerPage
+ * @param integer $colsPerPage
+ * @param string $dir
+ */
 function build_profile_pages(
     $members, $title, $rowsPerPage, $colsPerPage, $dir, $base_filename
 ) {

Indentation +1 added lines, -1 removed lines

@@ -200,7 +200,7 @@
         for ($row = 0; $row < $height; $row++) {
             echo "<tr>";
             for ($col = 0; $col < $width; $col++) {
-	            if ($count < $numIds) {
+                if ($count < $numIds) {
                     echo "<td class=\"bordered\" align=\"center\">
                         <a href=\"".secure_url_base()."view_profile.php?userid=".$userIds[$count]."\"><img src=\"".secure_url_base().IMAGE_URL.$userIds[$count]."_sm.jpg\"></a>
                     </td>";

Spacing +15 added lines, -15 removed lines

@@ -39,7 +39,7 @@ 
 
 // TODO: convert to new DB interface
 
-$alphabet = array('A','B','C','D','E','F','G','H','I','J','K','L','M','N','O','P','Q','R','S','T','U','V','W','X','Y','Z','0','1','2','3','4','5','6','7','8','9');
+$alphabet = array('A', 'B', 'C', 'D', 'E', 'F', 'G', 'H', 'I', 'J', 'K', 'L', 'M', 'N', 'O', 'P', 'Q', 'R', 'S', 'T', 'U', 'V', 'W', 'X', 'Y', 'Z', '0', '1', '2', '3', '4', '5', '6', '7', '8', '9');
 
 
 // Builds a summary table of user profiles.
@@ -53,7 +53,7 @@ 
 function show_user_table($members, $offset, $numToDisplay, $cols) {
     start_table();
 
-    $rows = ceil($numToDisplay / $cols);
+    $rows = ceil($numToDisplay/$cols);
     $count = $offset;
     $numMembers = count($members);
 
@@ -151,7 +151,7 @@ 
     // Build an array of IDs of all users with pictures in their profiles.
     $userIds = array();
     $numIds = 0;
-    foreach($profiles as $profile) {
+    foreach ($profiles as $profile) {
         $user = BoincUser::lookup_id($profile->userid);
         if (!$user) continue; // maybe we should delete the profile if user is non-existent anymore?
         if ($user->name) {
@@ -171,7 +171,7 @@ 
 //        shuffle($userIds);
 //    }
 
-    $numPages = ceil(count($userIds)/($width * $height));
+    $numPages = ceil(count($userIds)/($width*$height));
 
     // Make sure that a page is generated even when no profiles with pictures
     // exist in order to avoid 404 errors from the profile_menu page.
@@ -185,7 +185,7 @@ 
     $count = 0;
 
     for ($page = 1; $page <= $numPages; $page++) {
-        $filename = PROFILE_PATH . "user_gallery_" . $page . ".html";
+        $filename = PROFILE_PATH."user_gallery_".$page.".html";
         open_output_buffer();
 
         page_head("Profile gallery: page $page of $numPages", null, false, "../");
@@ -244,7 +244,7 @@ 
     // of the userids who belong to those countries.
     // Format: array[country][index] = userid.
 
-    foreach($profiles as $profile) {
+    foreach ($profiles as $profile) {
         $user = BoincUser::lookup_id($profile->userid);
         if (!$user) continue; // maybe we should delete the profile if user is non-existent anymore?
         if ($user->country) {
@@ -264,7 +264,7 @@ 
     // TODO: Define a constant for the desired number of rows per page.
 
     foreach ($countries as $country) {
-        $baseFileName =  "profile_country_" . get_legal_filename($country);
+        $baseFileName = "profile_country_".get_legal_filename($country);
         build_profile_pages(
             $countryMembers[$country],
             "User Profiles from $country", 5, 2,
@@ -290,7 +290,7 @@ 
     $numIds = 0;
     $members = array();
 
-    foreach($profiles as $profile) {
+    foreach ($profiles as $profile) {
         $user = BoincUser::lookup_id($profile->userid);
         if (!$user) continue; // maybe we should delete the profile if user is non-existent anymore?
         if ($user->name) {
@@ -335,11 +335,11 @@ 
 function build_profile_pages(
     $members, $title, $rowsPerPage, $colsPerPage, $dir, $base_filename
 ) {
-    $numPerPage = $rowsPerPage * $colsPerPage;
-    $numPages = ceil(count($members) / $numPerPage);
+    $numPerPage = $rowsPerPage*$colsPerPage;
+    $numPages = ceil(count($members)/$numPerPage);
 
     for ($page = 1; $page <= $numPages; $page++) {
-        $filename = $dir . $base_filename . "_" . $page . ".html";
+        $filename = $dir.$base_filename."_".$page.".html";
         open_output_buffer();
 
         $pagetitle = $title.": Page $page of $numPages";
@@ -347,7 +347,7 @@ 
 
         echo "Last updated ", pretty_time_str(time()), "<p>\n";
 
-        $offset = (($page-1) * $rowsPerPage * $colsPerPage);
+        $offset = (($page - 1)*$rowsPerPage*$colsPerPage);
 
         show_user_table($members, $offset, $numPerPage, $colsPerPage);
 
@@ -364,11 +364,11 @@ 
     print_debug_msg("Beginning to build country summary page...");
     $countries = array_keys($countryMembers);
 
-    $filename = PROFILE_PATH . "profile_country.html";
+    $filename = PROFILE_PATH."profile_country.html";
     open_output_buffer();
 
     page_head("User Profiles by Country", null, null, "../");
-    echo "Last updated " . pretty_time_str(time()) . "<p>";
+    echo "Last updated ".pretty_time_str(time())."<p>";
 
     start_table();
     row_heading_array(array("Country", "Profiles"));
@@ -413,7 +413,7 @@ 
 
 $caching = true;
 
-if (@$argv[1]=="-d") $debug=true;
+if (@$argv[1] == "-d") $debug = true;
 
 build_country_pages();
 build_alpha_pages();

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

@@ -180,7 +180,7 @@
         $numPages = 1;
     }
 
-    print_debug_msg("Generating $numPages pages");
+    print_debug_msg("generating $numPages pages");
 
     $count = 0;
 

html/ops/wd_nresults_changing.php

Doc Comments +3 added lines

@@ -23,6 +23,9 @@
 $cli_only = true;
 include_once("util_ops.inc");
 
+/**
+ * @param string $x
+ */
 function fail($x) {
     $f = fopen("error_log", "a");
     if (!$f) return false;

Spacing +3 added lines, -3 removed lines

@@ -41,7 +41,7 @@ 
     return $lines;
 }
 
-function write_count_file($n,$m) {
+function write_count_file($n, $m) {
     $f = fopen("nresults", "w");
     if (!$f) return false;
     $x = sprintf("%d\n%d\n", $n, $m);
@@ -80,12 +80,12 @@ 
     }
     $n = read_count_file();
     if ($n == false) {
-        write_count_file($m,$p);
+        write_count_file($m, $p);
         exit();
     }
     if (trim($n[0]) == $m && trim($n[1]) == $p) {
         fail("Result count hasn't changed\n");
     }
-    write_count_file($m,$p);
+    write_count_file($m, $p);
 
 ?>

html/project.sample/project.inc

Doc Comments +9 added lines

@@ -108,6 +108,10 @@ 
 // If you include any links, prepend URL with $prefix
 // if you need to open divs here close them in project_footer()
 //
+/**
+ * @param string $prefix
+ * @param boolean $is_main
+ */
 function project_banner($title, $prefix, $is_main) {
     if ($is_main) {
         echo '<img class="img-responsive" src="img/water.jpg">';
@@ -119,6 +123,11 @@ 
 }
 //$fixed_navbar = true;
 
+/**
+ * @param boolean $show_return
+ * @param boolean $show_date
+ * @param string $prefix
+ */
 function project_footer($show_return, $show_date, $prefix) {
     // If you include any links, prepend URL with $prefix
     //

Spacing +5 added lines, -5 removed lines

@@ -48,7 +48,7 @@ 
 
 define("FORUM_QA_MERGED_MODE", true);
     // Set to true to merge Message boards and Q&A section
-define ("DISABLE_PROFILES", true);
+define("DISABLE_PROFILES", true);
     // enable profiles only after enabling reCAPTCHA
     // http://boinc.berkeley.edu/trac/wiki/ProtectionFromSpam
 define("USE_STOPFORUMSPAM", true);
@@ -183,15 +183,15 @@ 
     </ol>", PROJECT);
 }
 
-function project_workunit($wu){
+function project_workunit($wu) {
     // shown in the workunit page
 }
 
-function project_user_summary($user){
+function project_user_summary($user) {
     // shown in the user summary page
 }
 
-function project_user_page_private($user){
+function project_user_page_private($user) {
     // shown in the private account page
 }
 
@@ -258,7 +258,7 @@ 
     array("name" => "Uppercase", "short_name" => "UC", "appids" => array(1, 25)),
 );
 
-function project_user_credit($user){
+function project_user_credit($user) {
     global $sub_projects;
     foreach ($sub_projects as $sp) {
         show_app_credit_user($user, $sp["name"], $sp["appids"]);

Braces +4 added lines, -4 removed lines

@@ -183,15 +183,15 @@ 
     </ol>", PROJECT);
 }
 
-function project_workunit($wu){
+function project_workunit($wu) {
     // shown in the workunit page
 }
 
-function project_user_summary($user){
+function project_user_summary($user) {
     // shown in the user summary page
 }
 
-function project_user_page_private($user){
+function project_user_page_private($user) {
     // shown in the private account page
 }
 
@@ -258,7 +258,7 @@ 
     array("name" => "Uppercase", "short_name" => "UC", "appids" => array(1, 25)),
 );
 
-function project_user_credit($user){
+function project_user_credit($user) {
     global $sub_projects;
     foreach ($sub_projects as $sp) {
         show_app_credit_user($user, $sp["name"], $sp["appids"]);

html/project.sample/project_specific_prefs.inc

Doc Comments +16 added lines

@@ -32,6 +32,9 @@ 
 //      Parse form variables into XML, and return it.
 //      Also error-check values, and return errors in $errors->*
 
+/**
+ * @param string $name
+ */
 function option($name, $val) {
     $x = ($name == $val) ?  "selected" : "";
     return "<option name='$name' $x>$name\n";
@@ -90,6 +93,10 @@ 
     return $x;
 }
 
+/**
+ * @param string $name
+ * @param integer $max
+ */
 function number_select($name, $max, $n) {
     $sel = ($n == 0) ? "selected":"";
     $x = "<select class=\"form-control input-sm\" name=$name> <option value=0 $sel>".tra("No limit")."</option>";
@@ -106,6 +113,9 @@ 
     return tra("No limit");
 }
 
+/**
+ * @param stdClass $prefs
+ */
 function project_specific_prefs_edit($prefs, $error=false) {
     global $app_array;
     if (COLOR_PREFS) {
@@ -162,6 +172,9 @@ 
     }
 }
 
+/**
+ * @param boolean $error
+ */
 function project_specific_prefs_parse_form(&$error) {
     global $app_array;
     $x = "";
@@ -216,6 +229,9 @@ 
     return $x;
 }
 
+/**
+ * @param stdClass $prefs
+ */
 function project_specific_prefs_show($prefs, $columns=false) {
     global $app_array;
     if ($columns) {

Spacing +15 added lines, -15 removed lines

@@ -33,7 +33,7 @@ 
 //      Also error-check values, and return errors in $errors->*
 
 function option($name, $val) {
-    $x = ($name == $val) ?  "selected" : "";
+    $x = ($name == $val) ? "selected" : "";
     return "<option name='$name' $x>$name\n";
 }
 
@@ -53,7 +53,7 @@ 
 if (APP_SELECT_PREFS) {
     $app_array = array();
     $apps = BoincApp::enum("deprecated=0");
-    foreach($apps as $app) {
+    foreach ($apps as $app) {
         $app_array[] = array($app->id, $app->user_friendly_name);
     }
 } else {
@@ -91,10 +91,10 @@ 
 }
 
 function number_select($name, $max, $n) {
-    $sel = ($n == 0) ? "selected":"";
+    $sel = ($n == 0) ? "selected" : "";
     $x = "<select class=\"form-control input-sm\" name=$name> <option value=0 $sel>".tra("No limit")."</option>";
-    for ($i=1; $i<=$max; $i++) {
-        $sel = ($n == $i) ? "selected":"";
+    for ($i = 1; $i <= $max; $i++) {
+        $sel = ($n == $i) ? "selected" : "";
         $x .= "<option value=$i $sel>$i</option>";
     }
     $x .= "</select>";
@@ -106,7 +106,7 @@ 
     return tra("No limit");
 }
 
-function project_specific_prefs_edit($prefs, $error=false) {
+function project_specific_prefs_edit($prefs, $error = false) {
     global $app_array;
     if (COLOR_PREFS) {
         $x = $prefs->color_scheme;
@@ -132,15 +132,15 @@ 
             } else {
                 $present = true;
             }
-            $checked = $present?"checked":"";
+            $checked = $present ? "checked" : "";
             $x .= "<input type=checkbox name=app_id_$app_id $checked> $app_name<br>";
         }
         tooltip_row2(APP_SELECT_TOOLTIP, APP_SELECT_DESC, $x);
-        $checked = $prefs->allow_non_preferred_apps?"checked":"";
+        $checked = $prefs->allow_non_preferred_apps ? "checked" : "";
         row2(ACCEPT_ANY_DESC, "<input type=checkbox name=allow_non_preferred_apps $checked>");
     }
     if (NON_GRAPHICAL_PREF) {
-        $checked = $prefs->non_graphical?"checked":"";
+        $checked = $prefs->non_graphical ? "checked" : "";
         row2(
             NON_GRAPHICAL_DESC,
             "<input type=checkbox name=non_graphical $checked>"
@@ -216,11 +216,11 @@ 
     return $x;
 }
 
-function project_specific_prefs_show($prefs, $columns=false) {
+function project_specific_prefs_show($prefs, $columns = false) {
     global $app_array;
     if ($columns) {
         if (COLOR_PREFS) {
-            row_defs(COLOR_DESC,"color_scheme", "", "", $prefs);
+            row_defs(COLOR_DESC, "color_scheme", "", "", $prefs);
         }
         if (GFX_CPU_PREFS) {
             row_defs(MAX_GFX_CPU_PCT_DESC, "max_gfx_cpu_pct", "", "", $prefs);
@@ -254,7 +254,7 @@ 
             row2(ACCEPT_ANY_DESC, $prefs->allow_non_preferred_apps_text);
         }
         if (NON_GRAPHICAL_PREF) {
-            row2(NON_GRAPHICAL_DESC, $prefs->non_graphical?tra("yes"):tra("no"));
+            row2(NON_GRAPHICAL_DESC, $prefs->non_graphical ? tra("yes") : tra("no"));
         }
         if (MAX_JOBS_PREF) {
             row2(tra("Max # jobs"), limit_string($prefs->max_jobs));
@@ -277,12 +277,12 @@ 
         $cursor = 0;
         while ($thisxml = parse_next_element($prefs_xml, "<app_id>", $cursor)) {
             if (is_numeric($thisxml)) {
-                $n = (int) $thisxml;
+                $n = (int)$thisxml;
                 $prefs->app_ids[] = $n;
             }
         }
         $prefs->allow_non_preferred_apps = parse_element($prefs_xml, "<allow_non_preferred_apps>");
-        $prefs->allow_non_preferred_apps_text = $prefs->allow_non_preferred_apps?"yes":"no";
+        $prefs->allow_non_preferred_apps_text = $prefs->allow_non_preferred_apps ? "yes" : "no";
     }
     if (NON_GRAPHICAL_PREF) {
         $prefs->non_graphical = parse_bool($prefs_xml, "non_graphical");
@@ -296,6 +296,6 @@ 
     return $prefs;
 }
 
-$cvs_version_tracker[]="\$Id$";  //Generated automatically - do not edit
+$cvs_version_tracker[] = "\$Id$"; //Generated automatically - do not edit
 
 ?>

html/user/bolt_sched.php

Doc Comments +6 added lines

@@ -28,6 +28,9 @@ 
 require_once("../inc/bolt_util.inc");
 require_once("../inc/util.inc");
 
+/**
+ * @param string $tag
+ */
 function debug_show_state($state, $tag) {
     global $user;
     global $refresh;
@@ -82,6 +85,9 @@ 
 
 // A page is being shown to the user; make a record of it
 //
+/**
+ * @param integer $mode
+ */
 function create_view($iter, $mode, $prev_view_id) {
     global $user;
     global $course;

Spacing +6 added lines, -6 removed lines

@@ -77,7 +77,7 @@ 
 }
 
 function default_mode($item) {
-    return $item->is_exercise()?BOLT_MODE_SHOW:BOLT_MODE_LESSON;
+    return $item->is_exercise() ? BOLT_MODE_SHOW : BOLT_MODE_LESSON;
 }
 
 // A page is being shown to the user; make a record of it
@@ -170,7 +170,7 @@ 
 
 // show an item (lesson, exercise, answer page)
 //
-function show_item($iter, $view_id, $prev_view_id, $mode, $repeat=null) {
+function show_item($iter, $view_id, $prev_view_id, $mode, $repeat = null) {
     global $user;
     global $course;
     global $bolt_ex;
@@ -224,7 +224,7 @@ 
     if ($repeat) {
         $avg = number_format($repeat->avg_score*100, 0);
         echo "<p>Score on this exercise set: $avg%";
-        if ($repeat->flags & REVIEW) {
+        if ($repeat->flags&REVIEW) {
             //echo "<pre>";
             //print_r($repeat);
             //echo "</pre>";
@@ -232,11 +232,11 @@ 
             $r = "<a href=bolt_sched.php?$url_args&action=review&view_id=$view_id&unit_name=$name>Review, then repeat exercises</a>";
             $links[] = $r;
         }
-        if ($repeat->flags & REPEAT) {
+        if ($repeat->flags&REPEAT) {
             $r = "<a href=bolt_sched.php?$url_args&action=repeat&view_id=$view_id>Repeat exercises</a>";
             $links[] = $r;
         }
-        if ($repeat->flags & NEXT) {
+        if ($repeat->flags&NEXT) {
             $links[] = $next;
         }
     } else {
@@ -462,7 +462,7 @@ 
     $bolt_ex->score = 0;
     $bolt_query_string = $item->query_string;
     srand($view_id);
-    ob_start();     // buffer output to avoid showing exercise text
+    ob_start(); // buffer output to avoid showing exercise text
     require($item->filename);
     ob_end_clean();
 

html/user/create_profile.php

Doc Comments +10 added lines

@@ -31,6 +31,10 @@ 
 // If $selection is provided, and if it matches one of the entries in the file,
 // it will be selected by default.
 //
+/**
+ * @param string $name
+ * @param string $filename
+ */
 function show_combo_box($name, $filename, $selection=null) {
     echo "<select name=\"$name\" class=\"form-control\">\n";
 
@@ -185,6 +189,9 @@ 
     show_language_selection($profile);
 }
 
+/**
+ * @param string $name
+ */
 function show_textarea($name, $text) {
     rowify("<textarea name=\"$name\" class=\"form-control\" rows=\"10\">" . $text . "</textarea>");
 }
@@ -311,6 +318,9 @@ 
     page_tail();
 }
 
+/**
+ * @param string $warning
+ */
 function show_profile_form($profile, $warning=null) {
     if ($profile) {
         page_head(tra("Edit your profile"), null, null, null, boinc_recaptcha_get_head_extra());

Spacing +20 added lines, -20 removed lines

@@ -31,7 +31,7 @@ 
 // If $selection is provided, and if it matches one of the entries in the file,
 // it will be selected by default.
 //
-function show_combo_box($name, $filename, $selection=null) {
+function show_combo_box($name, $filename, $selection = null) {
     echo "<select name=\"$name\" class=\"form-control\">\n";
 
     $file = fopen($filename, "r");
@@ -62,14 +62,14 @@ 
 <tr><td colspan=2>
 <table border=0 cellpadding=5
 <tr>
-<td valign=top><a href=\"" . IMAGE_URL . $profile->userid . '.jpg' . "\"><img src=\"" . IMAGE_URL . $profile->userid . '_sm.jpg' . "\"></a>
+<td valign=top><a href=\"" . IMAGE_URL.$profile->userid.'.jpg'."\"><img src=\"".IMAGE_URL.$profile->userid.'_sm.jpg'."\"></a>
 </td>
-<td valign=top>" .tra("%1 Your profile picture is shown to the left.",  $warning) ."
+<td valign=top>" .tra("%1 Your profile picture is shown to the left.", $warning)."
 <p>".
-tra("To replace it, click the \"Browse\" button and select a JPEG or PNG file (%1 or less).", "50KB") ."<br />
+tra("To replace it, click the \"Browse\" button and select a JPEG or PNG file (%1 or less).", "50KB")."<br />
 <input name=picture type=file><br>
 <p>".
-tra("To remove it from your profile, check this box:") . "
+tra("To remove it from your profile, check this box:")."
 <input type=checkbox name=delete_pic>
 <p>
 </td></tr>";
@@ -77,7 +77,7 @@ 
         end_table();
         echo "</td></tr>";
     } else {
-        rowify(tra("If you would like include a picture with your profile, click the \"Browse\" button and select a JPEG or PNG file. Please select images of %1 or less.", "50KB") . "
+        rowify(tra("If you would like include a picture with your profile, click the \"Browse\" button and select a JPEG or PNG file. Please select images of %1 or less.", "50KB")."
 <p>
 <input name=picture type=file>
         ");
@@ -92,7 +92,7 @@ 
     row1(tra("Language"));
     echo "<tr><td>
         <p>" .
-        tra("Select the language in which your profile is written:") . "
+        tra("Select the language in which your profile is written:")."
         <p>
     ";
     if (isset($profile->language)) {
@@ -110,7 +110,7 @@ 
     if ($publickey) {
         table_row(boinc_recaptcha_get_html($publickey));
     }
-    table_row("<p><input class=\"btn btn-default\" type=\"submit\" value=\"".tra("Create/edit profile") ."\" name=\"submit\">");
+    table_row("<p><input class=\"btn btn-default\" type=\"submit\" value=\"".tra("Create/edit profile")."\" name=\"submit\">");
 }
 
 // Returns an array containing:
@@ -124,7 +124,7 @@ 
 
     // Determine if the filetype uploaded is supported.
     // TODO: Change these to constants.
-    switch($size[2]) {
+    switch ($size[2]) {
     case '2':    // JPEG
         $image = imageCreateFromJPEG($fileName);
         break;
@@ -161,7 +161,7 @@ 
 
 function show_description() {
     echo "
-        <p>" .tra("Your %1profile%2 lets you share your opinions and background with the %3 community.", "<b>", "</b>", PROJECT) . "
+        <p>" .tra("Your %1profile%2 lets you share your opinions and background with the %3 community.", "<b>", "</b>", PROJECT)."
         <p>
     ";
 }
@@ -179,14 +179,14 @@ 
     row1(show_profile_heading1());
     rowify(show_profile_question1().html_info());
     show_textarea("response1", $response1);
-    row1( show_profile_heading2());
-    rowify( show_profile_question2().html_info());
+    row1(show_profile_heading2());
+    rowify(show_profile_question2().html_info());
     show_textarea("response2", $response2);
     show_language_selection($profile);
 }
 
 function show_textarea($name, $text) {
-    rowify("<textarea name=\"$name\" class=\"form-control\" rows=\"10\">" . $text . "</textarea>");
+    rowify("<textarea name=\"$name\" class=\"form-control\" rows=\"10\">".$text."</textarea>");
 }
 
 // $profile is null if user doesn't already have a profile.
@@ -233,8 +233,8 @@ 
         $delete_pic = "off";
     }
 
-    if (strlen($response1)==0 &&
-        strlen($response2)==0 &&
+    if (strlen($response1) == 0 &&
+        strlen($response2) == 0 &&
         $delete_pic != "on" &&
         !is_uploaded_file($_FILES['picture']['tmp_name'])
     ) {
@@ -248,7 +248,7 @@ 
         $profile->verification = 0;
     }
 
-    $profile ? $has_picture = $profile->has_picture: $has_picture = false;
+    $profile ? $has_picture = $profile->has_picture : $has_picture = false;
 
     if (is_uploaded_file($_FILES['picture']['tmp_name'])) {
         $has_picture = true;
@@ -263,13 +263,13 @@ 
 
         // Write the original image file to disk.
         // TODO: define a constant for image quality.
-        ImageJPEG($images[0], IMAGE_PATH . $user->id . '.jpg');
-        ImageJPEG($images[1], IMAGE_PATH . $user->id . '_sm.jpg');
+        ImageJPEG($images[0], IMAGE_PATH.$user->id.'.jpg');
+        ImageJPEG($images[1], IMAGE_PATH.$user->id.'_sm.jpg');
     }
     $response1 = sanitize_html($response1);
     $response2 = sanitize_html($response2);
 
-    $has_picture = $has_picture?1:0;
+    $has_picture = $has_picture ? 1 : 0;
     if ($profile) {
         $query = " response1 = '".BoincDb::escape_string($response1)."',"
             ." response2 = '".BoincDb::escape_string($response2)."',"
@@ -311,7 +311,7 @@ 
     page_tail();
 }
 
-function show_profile_form($profile, $warning=null) {
+function show_profile_form($profile, $warning = null) {
     if ($profile) {
         page_head(tra("Edit your profile"), null, null, null, boinc_recaptcha_get_head_extra());
     } else {