Passed
Push — master ( 89fa2c...180ea8 )
by Christian
01:13 queued 16s
created
html/user/submit_rpc_handler.php 1 patch
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -691,7 +691,8 @@
 block discarded – undo
691 691
         <id>$result->id</id>
692 692
         <state>".state_string($result)."</state>
693 693
 ";
694
-        if ($result->server_state == 5) {   // over?
694
+        if ($result->server_state == 5) {
695
+// over?
695 696
             $paths = get_outfile_paths($result);
696 697
             foreach($paths as $path) {
697 698
                 if (is_file($path)) {
Please login to merge, or discard this patch.
html/user/login_action.php 1 patch
Braces   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -44,7 +44,7 @@  discard block
 block discarded – undo
44 44
         page_tail();
45 45
         exit;
46 46
     }
47
-    if (substr($user->authenticator, 0, 1) == 'x'){
47
+    if (substr($user->authenticator, 0, 1) == 'x') {
48 48
         sleep(LOGIN_FAIL_SLEEP_SEC);
49 49
         error_page("This account has been administratively disabled.");
50 50
     }
@@ -105,7 +105,7 @@  discard block
 block discarded – undo
105 105
             Please <a href=get_passwd.php>try again</a>.
106 106
         ";
107 107
         page_tail();
108
-    } else if (substr($user->authenticator, 0, 1) == 'x'){
108
+    } else if (substr($user->authenticator, 0, 1) == 'x') {
109 109
         sleep(LOGIN_FAIL_SLEEP_SEC);
110 110
         error_page("This account has been administratively disabled.");
111 111
     } else {
Please login to merge, or discard this patch.
html/user/results.php 1 patch
Braces   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -57,7 +57,7 @@
 block discarded – undo
57 57
     $clause = "hostid=$hostid";
58 58
     page_head(tra("$s tasks for computer %1", $host->id));
59 59
     $show_host_link = false;
60
-} else if ($userid){
60
+} else if ($userid) {
61 61
     $user = get_logged_in_user();
62 62
     if ($userid != $user->id) {
63 63
         error_page(tra("No access"));
Please login to merge, or discard this patch.
html/user/sandbox.php 1 patch
Braces   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -52,7 +52,7 @@  discard block
 block discarded – undo
52 52
         </form>
53 53
         <hr>
54 54
     ";
55
-    if (strcmp($err_msg,"")!=0){
55
+    if (strcmp($err_msg,"")!=0) {
56 56
         echo "<p>$err_msg<hr>";
57 57
     }
58 58
     $files = array();
@@ -116,7 +116,7 @@  discard block
 block discarded – undo
116 116
         $s = stat($tmp_name);
117 117
         $size = $s['size'];
118 118
         list($exist, $elf) = sandbox_lf_exist($user, $md5);
119
-        if ($exist){
119
+        if ($exist) {
120 120
             $notice .= "<strong>Notice:</strong> Invalid Upload<br/>";
121 121
             $notice .= "You are trying to upload file  <strong>$name</strong><br/>";
122 122
             $notice .= "Another file <strong>$elf</strong> with the same content (md5: $md5) already exists!<br/>";
@@ -149,9 +149,9 @@  discard block
 block discarded – undo
149 149
         error_page("no such physical file");
150 150
     }
151 151
     $bused = sandbox_file_in_use($user, $name);
152
-    if ($bused){
152
+    if ($bused) {
153 153
         $notice = "<strong>$name</strong> is being used by batch(es), you can not delete it now!<br/>";
154
-    } else{ 
154
+    } else {
155 155
         $notice = "<strong>$name</strong> is not being used by any batch(es) and successfully deleted from your sandbox<br/>";
156 156
         unlink("$dir/$name");
157 157
         unlink($p);
Please login to merge, or discard this patch.
html/user/language_select.php 1 patch
Braces   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -33,8 +33,8 @@
 block discarded – undo
33 33
 }
34 34
 
35 35
 $set_lang = get_str("set_lang", true);
36
-if ($set_lang){
37
-    if (!in_array($set_lang, $languages) && $set_lang!="auto"){
36
+if ($set_lang) {
37
+    if (!in_array($set_lang, $languages) && $set_lang!="auto") {
38 38
         error_page("Language not supported");
39 39
     } else {
40 40
         send_cookie('lang', $set_lang, true);
Please login to merge, or discard this patch.
html/user/get_project_config.php 1 patch
Braces   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -28,10 +28,10 @@
 block discarded – undo
28 28
 //
29 29
 function show_platforms() {
30 30
     $xmlFragment = unserialize(get_cached_data(3600, "project_config_platform_xml"));
31
-    if ($xmlFragment==false){
31
+    if ($xmlFragment==false) {
32 32
         $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", "");
33 33
         $xmlFragment = "    <platforms>";
34
-        foreach ($platforms as $platform){
34
+        foreach ($platforms as $platform) {
35 35
             $xmlFragment .= "
36 36
             <platform>
37 37
                 <platform_name>$platform->name</platform_name>
Please login to merge, or discard this patch.
html/user/forum_search.php 1 patch
Braces   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -56,7 +56,7 @@  discard block
 block discarded – undo
56 56
     </select>");
57 57
 
58 58
 $forumid = null;
59
-if (get_str("forumid",true)){
59
+if (get_str("forumid",true)) {
60 60
     $forumid = get_str("forumid");
61 61
 }
62 62
 $forumlist="<option value=\"-1\">".tra("All")."</option>";
@@ -64,7 +64,7 @@  discard block
 block discarded – undo
64 64
 foreach ($categories as $category) {
65 65
     $forums = BoincForum::enum("parent_type=0 and category=$category->id");
66 66
     foreach ($forums as $forum) {
67
-        if ($forum->id==$forumid){
67
+        if ($forum->id==$forumid) {
68 68
             $forumlist.="<option selected value=\"".$forum->id."\">".$forum->title."</option>";
69 69
         } else {
70 70
             $forumlist.="<option value=\"".$forum->id."\">".$forum->title."</option>";
@@ -76,8 +76,8 @@  discard block
 block discarded – undo
76 76
     '<select class="form-control" name="search_forum">'.$forumlist.'</select');
77 77
 
78 78
 $sortlist = null;
79
-foreach ($thread_sort_styles as $id => $style){
80
-    if ($id == CREATE_TIME_NEW){
79
+foreach ($thread_sort_styles as $id => $style) {
80
+    if ($id == CREATE_TIME_NEW) {
81 81
         $sortlist.="<option selected value=\"".$id."\">".$style."</option>";
82 82
     } else {
83 83
         $sortlist.="<option value=\"".$id."\">".$style."</option>";
Please login to merge, or discard this patch.
html/user/forum_index.php 1 patch
Braces   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -73,7 +73,7 @@
 block discarded – undo
73 73
 
74 74
 show_forum_header($user);
75 75
 
76
-if (defined('FORUM_QA_MERGED_MODE') && FORUM_QA_MERGED_MODE){
76
+if (defined('FORUM_QA_MERGED_MODE') && FORUM_QA_MERGED_MODE) {
77 77
     $categories = BoincCategory::enum("true order by orderID");
78 78
 } else {
79 79
     echo "<p>"
Please login to merge, or discard this patch.
html/user/pm.php 1 patch
Braces   +4 added lines, -2 removed lines patch added patch discarded remove patch
@@ -234,7 +234,8 @@  discard block
 block discarded – undo
234 234
 
235 235
     foreach ($users as $username) {
236 236
         $user = explode(" ", $username);
237
-        if (is_numeric($user[0])) { // user ID is given
237
+        if (is_numeric($user[0])) {
238
+// user ID is given
238 239
             $userid = $user[0];
239 240
             $user = BoincUser::lookup_id($userid);
240 241
             if ($user == null) {
@@ -244,7 +245,8 @@  discard block
 block discarded – undo
244 245
             $users = BoincUser::lookup_name($username);
245 246
             if (count($users) == 0) {
246 247
                 pm_form($replyto, $userid, tra("Could not find user with username %1", $username));
247
-            } elseif (count($users) > 1) { // Non-unique username
248
+            } elseif (count($users) > 1) {
249
+// Non-unique username
248 250
                 pm_form($replyto, $userid, tra("%1 is not a unique username; you will have to use user ID", $username));
249 251
             }
250 252
             $user = $users[0];
Please login to merge, or discard this patch.