BOINC / boinc

html/user/forum_moderate_post_action.php

Switch Indentation +6 added lines, -6 removed lines

@@ -36,12 +36,12 @@
 
 function hide_explanation() {
     switch (post_int("category", true)) {
-        case 1: $c = "Obscene"; break;
-        case 2: $c = "Flame/Hate mail"; break;
-        case 3: $c = "Commercial spam"; break;
-        case 4: $c = "Double post"; break;
-        case 5: $c = "User Request"; break;
-        default: $c = "Other"; break;
+    case 1: $c = "Obscene"; break;
+    case 2: $c = "Flame/Hate mail"; break;
+    case 3: $c = "Commercial spam"; break;
+    case 4: $c = "Double post"; break;
+    case 5: $c = "User Request"; break;
+    default: $c = "Other"; break;
     }
     $x = "\nYour post was categorized as ".$c;
     $x .= mod_comment();

Braces +7 added lines, -7 removed lines

@@ -27,7 +27,7 @@ 
 function mod_comment() {
     $x = "";
     $reason = post_str('reason', true);
-    if ($reason){
+    if ($reason) {
         $x .= "\nModerator comment: $reason
 ";
     }
@@ -62,7 +62,7 @@ 
 
 // See if "action" is provided - either through post or get
 if (!post_str('action', true)) {
-    if (!get_str('action', true)){
+    if (!get_str('action', true)) {
         error_page(tra("You must specify an action..."));
     } else {
         $action = get_str('action');
@@ -72,14 +72,14 @@ 
 }
 
 $explanation = null;
-if ($action=="hide"){
+if ($action=="hide") {
     $result = hide_post($post, $thread, $forum);
     $action_name = "hidden";
     $explanation = hide_explanation();
-} elseif ($action=="unhide"){
+} elseif ($action=="unhide") {
     $result = unhide_post($post, $thread, $forum);
     $action_name = "unhidden";
-} elseif ($action=="delete"){
+} elseif ($action=="delete") {
     $result = delete_post($post, $thread, $forum);
     if (!$result) error_page("Can't delete post");
     page_head("Post deleted");
@@ -96,7 +96,7 @@ 
     }
     page_tail();
     exit;
-} elseif ($action=="move"){
+} elseif ($action=="move") {
     $destid = post_int('threadid');
     $new_thread = BoincThread::lookup_id($destid);
     if (!$new_thread) error_page("No such thread");
@@ -117,7 +117,7 @@ 
 ";
     $explanation .= mod_comment();
     $action_name = "moved to another thread";
-} elseif ($action=="banish_user"){
+} elseif ($action=="banish_user") {
     $auth = false;
     if (defined("MODERATORS_CAN_BANISH") && $user->prefs->privilege(S_MODERATOR)) {
         $auth = true;

Spacing +7 added lines, -7 removed lines

@@ -27,7 +27,7 @@ 
 function mod_comment() {
     $x = "";
     $reason = post_str('reason', true);
-    if ($reason){
+    if ($reason) {
         $x .= "\nModerator comment: $reason
 ";
     }
@@ -62,7 +62,7 @@ 
 
 // See if "action" is provided - either through post or get
 if (!post_str('action', true)) {
-    if (!get_str('action', true)){
+    if (!get_str('action', true)) {
         error_page(tra("You must specify an action..."));
     } else {
         $action = get_str('action');
@@ -72,14 +72,14 @@ 
 }
 
 $explanation = null;
-if ($action=="hide"){
+if ($action == "hide") {
     $result = hide_post($post, $thread, $forum);
     $action_name = "hidden";
     $explanation = hide_explanation();
-} elseif ($action=="unhide"){
+} elseif ($action == "unhide") {
     $result = unhide_post($post, $thread, $forum);
     $action_name = "unhidden";
-} elseif ($action=="delete"){
+} elseif ($action == "delete") {
     $result = delete_post($post, $thread, $forum);
     if (!$result) error_page("Can't delete post");
     page_head("Post deleted");
@@ -96,7 +96,7 @@ 
     }
     page_tail();
     exit;
-} elseif ($action=="move"){
+} elseif ($action == "move") {
     $destid = post_int('threadid');
     $new_thread = BoincThread::lookup_id($destid);
     if (!$new_thread) error_page("No such thread");
@@ -117,7 +117,7 @@ 
 ";
     $explanation .= mod_comment();
     $action_name = "moved to another thread";
-} elseif ($action=="banish_user"){
+} elseif ($action == "banish_user") {
     $auth = false;
     if (defined("MODERATORS_CAN_BANISH") && $user->prefs->privilege(S_MODERATOR)) {
         $auth = true;

html/user/host_edit_form.php

Spacing +3 added lines, -3 removed lines

@@ -50,7 +50,7 @@ 
     if (!hosts_compatible($host, $host2, $detail)) continue;
     $hosts[] = $host2;
     $nhosts++;
-    if ($nhosts==500) break;
+    if ($nhosts == 500) break;
 }
 if ($nhosts == 1) {
     echo "<br>".tra("No hosts are eligible for merging with this one.");
@@ -90,14 +90,14 @@ 
     <script>
         function set_all() {
 ";
-for ($i=1; $i<$nhosts; $i++) {
+for ($i = 1; $i < $nhosts; $i++) {
     echo "document.host_list.id_$i.checked=1;\n";
 }
 echo "
         }
         function clear_all() {
 ";
-for ($i=1; $i<$nhosts; $i++) {
+for ($i = 1; $i < $nhosts; $i++) {
     echo "document.host_list.id_$i.checked=0;\n";
 }
 echo "

html/ops/db_action.php

Indentation +1 added lines, -1 removed lines

@@ -220,7 +220,7 @@
         }
     }
     if ($detail == "low" || $table == "profile") {
-         end_table();
+            end_table();
     }
     _mysql_free_result($result);
 } else {

Spacing +2 added lines, -2 removed lines

@@ -64,7 +64,7 @@ 
 // For display, convert query string characters < and > into 'html form' so
 // that they will be displayed.
 //
-$html_text=str_replace('<', '&lt;', str_replace('>', '&gt;', $main_query));
+$html_text = str_replace('<', '&lt;', str_replace('>', '&gt;', $main_query));
 
 echo "<p>Query: <b>$html_text</b><p>\n";
 
@@ -159,7 +159,7 @@ 
 if ($result) {
     if ($detail == "low") {
         start_table('table-striped');
-        switch($table) {
+        switch ($table) {
         case "result":
             result_short_header();
             break;

html/ops/block_host.php

Spacing +7 added lines, -7 removed lines

@@ -20,16 +20,16 @@ 
 
 // TODO: document; use new DB interface
 
-include_once( "../inc/db.inc" );
-include_once( "../inc/util.inc" );
-include_once( "../inc/db_ops.inc" );
-include_once( "../inc/util_ops.inc" );
-include_once( "../inc/prefs.inc" );
+include_once("../inc/db.inc");
+include_once("../inc/util.inc");
+include_once("../inc/db_ops.inc");
+include_once("../inc/util_ops.inc");
+include_once("../inc/prefs.inc");
 
 db_init();
 
 if (get_int('hostid')) {
-    $hostid = get_int( 'hostid' );
+    $hostid = get_int('hostid');
 } else {
     error_page("no hostid");
 }
@@ -39,7 +39,7 @@ 
 
 admin_page_head($title);
 
-if($hostid > 0) {
+if ($hostid > 0) {
     $result = _mysql_query("UPDATE host SET max_results_day=1 WHERE id=".$hostid);
 }
 

html/ops/problem_host.php

Indentation +5 added lines, -5 removed lines

@@ -116,14 +116,14 @@
 } else {
     $host = BoincHost::lookup_id($hostid);
     if (!$host) {
-    	echo "<h2>No host with that ID</h2>
+        echo "<h2>No host with that ID</h2>
 	 	<center>Please <a href=\"problem_host.php\">try again</a></center>";
     } else {
         $user = BoincUser::lookup_id($host->userid);
-    	echo "<a href=\"problem_host.php\">Do another?</a><br><br>";
-    	send_problem_email($user, $host);
-    	echo "Email to ".$user->email_addr." has been sent.<br>";
-	}
+        echo "<a href=\"problem_host.php\">Do another?</a><br><br>";
+        send_problem_email($user, $host);
+        echo "Email to ".$user->email_addr." has been sent.<br>";
+    }
 }
 
 admin_page_tail();

Spacing +12 added lines, -12 removed lines

@@ -37,12 +37,12 @@ 
     Average credit: ".$host->expavg_credit."
     Average update time: ".time_str($host->expavg_time)."
     IP address: $host->last_ip_addr (same the last $host->nsame_ip_addr times)
-    Domain name: " . $host->domain_name;
+    Domain name: ".$host->domain_name;
     $x = $host->timezone/3600;
-    if ($x >= 0) $x="+$x";
-    $host_content.="
+    if ($x >= 0) $x = "+$x";
+    $host_content .= "
     Local Time = UTC $x hours
-    Number of CPUs: " . $host->p_ncpus."
+    Number of CPUs: ".$host->p_ncpus."
     CPU: $host->p_vendor $host->p_model
     FP ops/sec: ".$host->p_fpops."
     Int ops/sec: ".$host->p_iops."
@@ -50,28 +50,28 @@ 
     Operating System: $host->os_name $host->os_version";
     $x = $host->m_nbytes/(1024*1024);
     $y = round($x, 2);
-    $host_content.="
+    $host_content .= "
     Memory: $y MB";
     $x = $host->m_cache/1024;
     $y = round($x, 2);
-    $host_content.="
+    $host_content .= "
     Cache: $y KB";
     $x = $host->m_swap/(1024*1024);
     $y = round($x, 2);
-    $host_content.="
+    $host_content .= "
     Swap Space: $y MB";
     $x = $host->d_total/(1024*1024*1024);
     $y = round($x, 2);
-    $host_content.="
+    $host_content .= "
     Total Disk Space: $y GB";
     $x = $host->d_free/(1024*1024*1024);
     $y = round($x, 2);
-    $host_content.="
+    $host_content .= "
     Free Disk Space: $y GB
     Avg network bandwidth (upstream): $host->n_bwup bytes/sec
     Avg network bandwidth (downstream): $host->n_bwdown bytes/sec";
     $x = $host->avg_turnaround/86400;
-    $host_content.="
+    $host_content .= "
     Average turnaround: ".round($x, 2)." days
     Number of RPCs: $host->rpc_seqno
     Last RPC: ".time_str($host->rpc_time)."
@@ -88,7 +88,7 @@ 
 installation.  Could you please have a look at it?
 
 Sincerely,
-        The ". PROJECT." team
+        The ".PROJECT." team
 ";
 $body .= "
 
@@ -96,7 +96,7 @@ 
 " . $host_content."
 
 For further information and assistance with ".PROJECT." go to $master_url";
-    echo nl2br($body) . "<br><br>";
+    echo nl2br($body)."<br><br>";
     return send_email($user, $subject, $body);
 
 }

html/ops/clean_cache.php

Indentation +11 added lines, -11 removed lines

@@ -24,17 +24,17 @@
 set_time_limit(0);
 
 function cache_check_diskspace2(){
-   $too_old = 86400;
-   while (1) {
-       $f = disk_free_space("../cache");
-       $u = disk_usage("../cache");
-       echo "free: $f used: $u\n";
-       if ($f > MIN_FREE_SPACE && $u < MAX_CACHE_USAGE) {
-           break;
-       }
-       clean_cache($too_old, "../cache");
-       $too_old/=2;
-   }
+    $too_old = 86400;
+    while (1) {
+        $f = disk_free_space("../cache");
+        $u = disk_usage("../cache");
+        echo "free: $f used: $u\n";
+        if ($f > MIN_FREE_SPACE && $u < MAX_CACHE_USAGE) {
+            break;
+        }
+        clean_cache($too_old, "../cache");
+        $too_old/=2;
+    }
 }
 
 cache_check_diskspace2();

Spacing +2 added lines, -2 removed lines

@@ -23,7 +23,7 @@ 
 
 set_time_limit(0);
 
-function cache_check_diskspace2(){
+function cache_check_diskspace2() {
    $too_old = 86400;
    while (1) {
        $f = disk_free_space("../cache");
@@ -33,7 +33,7 @@ 
            break;
        }
        clean_cache($too_old, "../cache");
-       $too_old/=2;
+       $too_old /= 2;
    }
 }
 

Braces +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
 
 set_time_limit(0);
 
-function cache_check_diskspace2(){
+function cache_check_diskspace2() {
    $too_old = 86400;
    while (1) {
        $f = disk_free_space("../cache");

html/ops/cancel_workunits_action.php

Spacing +8 added lines, -8 removed lines

@@ -38,30 +38,30 @@ 
 if ($cancel && ($cancel == 1)) {
     if ($WU) {
         foreach ($WU as $key => $value) {
-            if($WUs != "") {
-                $WUs = $WUs . ",";
+            if ($WUs != "") {
+                $WUs = $WUs.",";
             }
-            $WUs = $WUs . $value;
+            $WUs = $WUs.$value;
         }
     }
 }
 
 // cancel WUs (if not in rops)
 //
-if($WUs != "") {
+if ($WUs != "") {
     echo "<!--- WUs to cancel: $WUs --->\n";
     if (!in_rops()) {
-        cancel_wus_where("id IN (" . $WUs . ")");
+        cancel_wus_where("id IN (".$WUs.")");
     }
 }
 
 if ($back) {
     if ($back == "errorwus") {
         $args = "?refresh_cache=1";
-        if ($hide_canceled && $hide_canceled=="on") {
+        if ($hide_canceled && $hide_canceled == "on") {
             $args .= "&hide_canceled=on";
         }
-        if ($hide_dlerr && $hide_dlerr=="on") {
+        if ($hide_dlerr && $hide_dlerr == "on") {
             $args .= "&hide_dlerr=on";
         }
         if ($appid) {
@@ -70,7 +70,7 @@ 
         echo "<p><a href=\"errorwus.php$args\">Return to All-error Workunits page</a> (refreshes the cache)</p>";
     } else if ($back == "cancelwus") {
         if ($clause) {
-            $clause=urlencode($clause);
+            $clause = urlencode($clause);
             echo "<p><a href=\"cancel_workunits.php?limit=$limit&uclause=$clause\">";
             echo "Cancel next (max $limit) Workunits</a></p>";
         }

html/ops/revalidate.php

Spacing +2 added lines, -2 removed lines

@@ -21,7 +21,7 @@ 
 // and you want to rerun it on some jobs
 // (e.g. to grant credit to instances previously marked as invalid)
 
-ini_set ("memory_limit", "2G");
+ini_set("memory_limit", "2G");
 
 require_once("../inc/util_ops.inc");
 
@@ -52,7 +52,7 @@ 
     $nresults = 0;
 
     $wus = BoincWorkunit::enum($clause);
-    foreach($wus as $wu) {
+    foreach ($wus as $wu) {
         $results = BoincResult::enum("workunitid=$wu->id");
         $n = 0;
         foreach ($results as $result) {

html/ops/update_profile_pages.php

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;