BOINC / boinc

html/user/openid_login.php

Spacing +3 added lines, -3 removed lines

@@ -16,7 +16,7 @@ 
 // You should have received a copy of the GNU Lesser General Public License
 // along with BOINC.  If not, see <http://www.gnu.org/licenses/>.
 
-NOT FINISHED.   DON'T USE
+NOT FINISHED.DON'T USE
 
 require 'openid.php';
 include_once("../inc/boinc_db.inc");
@@ -26,14 +26,14 @@ 
 
 function show_error($str) {
     page_head("Can't create account");
-    echo "$str<br>\n";
+    echo "$str < br > \n";
     page_tail();
     exit();
 }
 
 try {
     $openid = new LightOpenID;
-    echo "<pre>";
+    echo " < pre > ";
     if(!$openid->mode) {
         if(isset($_POST['openid_identifier'])) {
             $openid->identity = $_POST['openid_identifier'];

html/user/prefs_edit.php

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
 $subset = sanitize_tags(get_str("subset"));
 $venue = sanitize_tags(get_str("venue", true));
 $columns = get_int("cols", true);
-$c = $columns?"&cols=$columns":"";
+$c = $columns ? "&cols=$columns" : "";
 check_subset($subset);
 if ($action) {
     check_tokens($user->authenticator);

html/inc/util_basic.inc

Spacing +6 added lines, -6 removed lines

@@ -56,10 +56,10 @@ 
     return file_exists("$d/stop_sched");
 }
 
-function xml_error($num=-1, $msg=null, $file=null, $line=null) {
+function xml_error($num = -1, $msg = null, $file = null, $line = null) {
     global $xml_outer_tag;
     if (!$msg) {
-        switch($num) {
+        switch ($num) {
         case -112: $msg = "Invalid XML"; break;
         case -136: $msg = "Not found"; break;
         case -137: $msg = "Name or email address is not unique"; break;
@@ -105,7 +105,7 @@ 
 // If it's a single-tag element, and it's present, just return the tag
 //
 function parse_element($xml, $tag) {
-    $closetag = "</" . substr($tag,1);
+    $closetag = "</".substr($tag, 1);
     $x = strstr($xml, $tag);
     if ($x) {
         if (strstr($tag, "/>")) return $tag;
@@ -121,8 +121,8 @@ 
 
 function parse_next_element($xml, $tag, &$cursor) {
     $element = null;
-    $closetag = "</" . substr($tag,1);
-    $pos = substr($xml,$cursor);
+    $closetag = "</".substr($tag, 1);
+    $pos = substr($xml, $cursor);
     $x = strstr($pos, $tag);
     if ($x) {
         if (strstr($tag, "/>")) return $tag;
@@ -222,7 +222,7 @@ 
 // whose OSs may have such restrictions.
 //
 function is_valid_filename($x) {
-    if (strlen($x)>255) return false;
+    if (strlen($x) > 255) return false;
     // \w means A-Za-z0-9_
     return preg_match('/^[\w\-.]+$/', $x);
 }

html/user/forum_post.php

Spacing +10 added lines, -10 removed lines

@@ -43,7 +43,7 @@ 
     error_page("Forums are disabled");
 }
 
-if (user_can_create_thread($logged_in_user, $forum)=='no') {
+if (user_can_create_thread($logged_in_user, $forum) == 'no') {
     error_page(tra("Only project admins may create a thread here. However, you may reply to existing threads."));
 }
 check_post_access($logged_in_user, $forum);
@@ -56,10 +56,10 @@ 
 $preview = post_str("preview", true);
 $warning = null;
 
-if ($content && $title && (!$preview)){
+if ($content && $title && (!$preview)) {
     if (post_str('add_signature', true)) {
-        $add_signature = true;    // set a flag and concatenate later
-    }  else {
+        $add_signature = true; // set a flag and concatenate later
+    } else {
         $add_signature = false;
     }
     check_tokens($logged_in_user->authenticator);
@@ -74,14 +74,14 @@ 
             if (post_str('subscribe', true)) {
                 BoincSubscription::replace($logged_in_user->id, $thread->id);
             }
-            header('Location: forum_thread.php?id=' . $thread->id);
+            header('Location: forum_thread.php?id='.$thread->id);
         } else {
             error_page("Can't create thread.  $forum_error");
         }
     }
 }
 
-page_head(tra("Create new thread"),'','','', $bbcode_js);
+page_head(tra("Create new thread"), '', '', '', $bbcode_js);
 show_forum_header($logged_in_user);
 
 if ($warning) {
@@ -122,7 +122,7 @@ 
     $submit_help = "<br /><font color=\"red\">".tra("Remember to add a title")."</font>";
 }
 
-if ($force_title && $title){
+if ($force_title && $title) {
     row2(
         tra("Title"),
         sprintf(
@@ -136,7 +136,7 @@ 
     row2(
         tra("Title").$submit_help,
         sprintf('<input type="text" class="form-control" name="title" value="%s">',
-            $title?htmlspecialchars($title):''
+            $title ?htmlspecialchars($title) : ''
         ),
         null, FORUM_LH_PCT
     );
@@ -147,7 +147,7 @@ 
     sprintf(
         '%s <textarea class="form-control" name="content" rows="12" cols="80">%s</textarea>',
         $bbcode_html,
-        $content?htmlspecialchars($content):''
+        $content ?htmlspecialchars($content) : ''
     ),
     null, FORUM_LH_PCT
 );
@@ -155,7 +155,7 @@ 
 if (!$logged_in_user->prefs->no_signature_by_default) {
     $enable_signature = 'checked="true"';
 } else {
-    $enable_signature='';
+    $enable_signature = '';
 }
 
 if (is_news_forum($forum)) {

html/user/forum_user_posts.php

Spacing +2 added lines, -2 removed lines

@@ -29,7 +29,7 @@ 
 $user = BoincUser::lookup_id($userid);
 if (!$user) error_page('no such user');
 $offset = get_int("offset", true);
-if (!$offset) $offset=0;
+if (!$offset) $offset = 0;
 $items_per_page = 20;
 
 $logged_in_user = get_logged_in_user(false);
@@ -114,7 +114,7 @@ 
 		break;
 	}
     if ($n >= $offset) {
-        show_post_and_context($post, $thread, $forum, $options, $n+1);
+        show_post_and_context($post, $thread, $forum, $options, $n + 1);
     }
     $n++;
 }

html/user/forum_banishment_vote_action.php

Spacing +6 added lines, -6 removed lines

@@ -37,7 +37,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');
@@ -47,10 +47,10 @@ 
 }
 
 $userid = post_int('userid');
-$user=BoincUser::lookup_id($userid);
+$user = BoincUser::lookup_id($userid);
 if (!$user) error_page('No such user');
 
-if ($action!="start"){
+if ($action != "start") {
     error_page("Unknown action");
 }
 
@@ -66,10 +66,10 @@ 
         $mod_category = tra("Other");
 }
 
-if (post_str('reason', true)){
-    start_vote($config,$logged_in_user,$user, $mod_category,post_str("reason"));
+if (post_str('reason', true)) {
+    start_vote($config, $logged_in_user, $user, $mod_category, post_str("reason"));
 } else {
-    start_vote($config,$logged_in_user,$user, $mod_category,"None given");
+    start_vote($config, $logged_in_user, $user, $mod_category, "None given");
 }
 
 ?>

html/user/forum_thread_vote.php

Spacing +1 added lines, -1 removed lines

@@ -30,6 +30,6 @@
 if (!$thread) error_page('No such thread');
 $logged_in_user = get_logged_in_user();
 
-$posts = get_thread_posts($threadid, 0,true);
+$posts = get_thread_posts($threadid, 0, true);
 header("Location: forum_rate.php?choice=p&post=".$posts[0]->id);
 ?>

html/user/forum_moderate_post.php

Spacing +5 added lines, -5 removed lines

@@ -50,7 +50,7 @@ 
 start_table();
 
 $get_reason = true;
-if (get_str('action')=="hide") {
+if (get_str('action') == "hide") {
     //display input that selects reason
     echo "<input type=hidden name=action value=hide>";
     row1(tra("Hide post"));
@@ -67,12 +67,12 @@ 
             )
         )
     );
-} elseif (get_str('action')=="move") {
+} elseif (get_str('action') == "move") {
     row1(tra("Move post"));
     echo "<input type=hidden name=action value=move>";
     row2(tra("Destination thread ID:"), "<input name=\"threadid\">");
     // TODO: display where to move the post as a dropdown instead of having to get ID
-} elseif (get_str('action')=="banish_user") {
+} elseif (get_str('action') == "banish_user") {
     $userid = get_int('userid');
     $user = BoincUser::lookup_id($userid);
     if (!$user) {
@@ -80,7 +80,7 @@ 
     }
     BoincForumPrefs::lookup($user);
     $x = $user->prefs->banished_until;
-    if ($x>time()) {
+    if ($x > time()) {
         error_page(tra("User is already banished"));
     }
     row1(tra("Banish user"));
@@ -98,7 +98,7 @@ 
     echo "<input type=\"hidden\" name=\"id\" value=\"".$postid."\">\n";
     echo "<input type=\"hidden\" name=\"userid\" value=\"".$userid."\">\n";
     echo "<input type=\"hidden\" name=\"confirmed\" value=\"yes\">\n";
-} elseif (get_str('action')=="delete") {
+} elseif (get_str('action') == "delete") {
     echo "<input type=hidden name=action value=delete>";
     row2(
         "Are you sure want to delete this post?  This cannot be undone.",

html/user/forum_search_action.php

Spacing +23 added lines, -23 removed lines

@@ -31,10 +31,10 @@ 
 // Optionally filters by forum, user, time, or hidden if specified.
 //
 function search_thread_titles(
-    $keyword_list, $forum="", $user="", $time="", $limit=200,
-    $sort_style=CREATE_TIME_NEW, $show_hidden = false
-){
-    $search_string="%";
+    $keyword_list, $forum = "", $user = "", $time = "", $limit = 200,
+    $sort_style = CREATE_TIME_NEW, $show_hidden = false
+) {
+    $search_string = "%";
     foreach ($keyword_list as $key => $word) {
         $search_string .= BoincDb::escape_string($word)."%";
     }
@@ -51,7 +51,7 @@ 
     if (!$show_hidden) {
         $query .= " and thread.hidden = 0";
     }
-    switch($sort_style) {
+    switch ($sort_style) {
     case MODIFIED_NEW:
         $query .= ' ORDER BY timestamp DESC';
         break;
@@ -84,11 +84,11 @@ 
 //
 function search_post_content(
     $keyword_list, $forum, $user, $time, $limit, $sort_style, $show_hidden
-){
+) {
     $db = BoincDb::get();
 
-    $search_string="%";
-    foreach ($keyword_list as $key => $word){
+    $search_string = "%";
+    foreach ($keyword_list as $key => $word) {
         $search_string .= BoincDb::escape_string($word)."%";
     }
     $optional_join = "";
@@ -100,20 +100,20 @@ 
     }
     $query = "select post.* from ".$db->db_name.".post".$optional_join." where content like '".$search_string."'";
     if ($forum) {
-        $query.=" and forum = $forum->id";
+        $query .= " and forum = $forum->id";
     }
     if ($user) {
-        $query.=" and post.user = $user->id ";
+        $query .= " and post.user = $user->id ";
     }
     if ($time) {
-        $query.=" and post.timestamp > $time";
+        $query .= " and post.timestamp > $time";
     }
     if (!$show_hidden) {
         $query .= " AND post.hidden = 0";
     }
-    switch($sort_style) {
+    switch ($sort_style) {
     case VIEWS_MOST:
-        $query.= ' ORDER BY views DESC';
+        $query .= ' ORDER BY views DESC';
         break;
     case CREATE_TIME_NEW:
         $query .= ' ORDER by post.timestamp desc';
@@ -128,13 +128,13 @@ 
         $query .= ' ORDER BY post.timestamp DESC';
         break;
     }
-    $query.= " limit $limit";
+    $query .= " limit $limit";
     return BoincPost::enum_general($query);
 }
 
 $logged_in_user = get_logged_in_user(false);
 BoincForumPrefs::lookup($logged_in_user);
-if ($logged_in_user && $logged_in_user->prefs->privilege(S_MODERATOR)){
+if ($logged_in_user && $logged_in_user->prefs->privilege(S_MODERATOR)) {
     $show_hidden_posts = true;
 } else {
     $show_hidden_posts = false;
@@ -147,7 +147,7 @@ 
 $search_max_time = post_int("search_max_time");
 $search_forum = post_int("search_forum");
 $search_sort = post_int("search_sort");
-$search_list = explode(" ",$search_keywords);
+$search_list = explode(" ", $search_keywords);
 if ($search_max_time) {
     $min_timestamp = time() - ($search_max_time*3600*24);
 } else {
@@ -156,7 +156,7 @@ 
 
 $limit = 100;
 
-if ($search_forum==-1){
+if ($search_forum == -1) {
     $forum = null;
 } else if ($search_forum) {
     $forum = BoincForum::lookup_id($search_forum);
@@ -176,11 +176,11 @@ 
 
 // Display the threads
 //
-if (count($threads)){
-    echo "<h3>" . tra("Thread titles matching your query:") . "</h3>";
+if (count($threads)) {
+    echo "<h3>".tra("Thread titles matching your query:")."</h3>";
     start_table('table-striped');
     thread_list_header();
-    foreach ($threads as $thread){
+    foreach ($threads as $thread) {
         if (!$show_hidden_posts) {
             $u = BoincUser::lookup_id($thread->owner);
             if ($u && is_banished($u)) continue;
@@ -200,8 +200,8 @@ 
     $show_hidden_posts
 );
 
-if (count($posts)){
-    echo "<h3>" . tra("Messages matching your query:") . "</h3>";
+if (count($posts)) {
+    echo "<h3>".tra("Messages matching your query:")."</h3>";
     start_table('table-striped');
     row_heading_array(['Info', 'Post'], ['', 'width=70%']);
     $n = 1;
@@ -228,7 +228,7 @@ 
     end_table();
 }
 
-if (!count($threads) && !count($posts)){
+if (!count($threads) && !count($posts)) {
     echo "<p>".tra("Sorry, couldn't find anything matching your search query. You can try to broaden your search by using less words (or less specific words).")."</p>
     <p>"
     .tra("You can also %1 try the same search on Google. %2",