BOINC / boinc

html/inc/consent.inc

Spacing +5 added lines, -5 removed lines

@@ -21,7 +21,7 @@ 
 require_once("../inc/boinc_db.inc");
 require_once("../inc/util.inc");
 
-define('CONSENT_TYPE_ENROLL','ENROLL');
+define('CONSENT_TYPE_ENROLL', 'ENROLL');
 
 function check_termsofuse() {
     return defined('TERMSOFUSE_FILE') and file_exists(TERMSOFUSE_FILE);
@@ -32,13 +32,13 @@ 
     $source, $ctime = 0
 ) {
     $mys = BoincDb::escape_string($source);
-    if ($ctime==0) {
+    if ($ctime == 0) {
         $mytime = $user->create_time;
     } else {
         $mytime = $ctime;
     }
     return BoincConsent::insert(
-        "(userid, consent_type_id, consent_time, consent_flag, consent_not_required, source) " .
+        "(userid, consent_type_id, consent_time, consent_flag, consent_not_required, source) ".
         "values($user->id, $consent_type_id, $mytime, $consent_flag, $consent_not_required, '$mys')"
     );
 
@@ -65,10 +65,10 @@ 
 // The integer is the consent_type_id- the id from consent_type table.
 // If the boolean is FALSE, the integer returned is -1.
 //
-function check_consent_type($name, $checkenabled=TRUE) {
+function check_consent_type($name, $checkenabled = TRUE) {
     $name = BoincDb::escape_string($name);
     $ct = BoincConsentType::lookup("shortname = '$name'");
-    if ($ct and ( !$checkenabled or ($ct->enabled)) ) {
+    if ($ct and (!$checkenabled or ($ct->enabled))) {
         return array(TRUE, $ct->id);
     }
     return array(FALSE, -1);

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.",