BOINC / boinc

html/user/team_admins.php

Spacing +3 added lines, -3 removed lines

@@ -58,7 +58,7 @@ 
     ;
     $admins = BoincTeamAdmin::enum("teamid=$teamid");
     start_table();
-    if (count($admins)==0) {
+    if (count($admins) == 0) {
         row1(tra("There are currently no Team Admins"));
     } else {
         row1(tra("Current Team Admins"), 3);
@@ -96,7 +96,7 @@ 
 
 function add_admin($team) {
     $email_addr = get_str('email_addr');
-    $email_addr =  BoincDb::escape_string($email_addr);
+    $email_addr = BoincDb::escape_string($email_addr);
     $user = BoincUser::lookup("email_addr='$email_addr'");
     if (!$user) error_page(tra("no such user"));
     if ($user->teamid != $team->id) error_page(tra("User is not member of team"));
@@ -115,7 +115,7 @@ 
 require_founder_login($user, $team);
 
 $action = get_str('action', true);
-switch($action) {
+switch ($action) {
 case 'remove':
     check_tokens($user->authenticator);
     remove_admin($team);

html/user/show_coproc.php

Spacing +1 added lines, -1 removed lines

@@ -92,7 +92,7 @@
         echo $x;
         break;
     }
-    echo "</td><td align=right>".format_credit_large($y->credit),"</td><td align=right>$y->nresults</td></tr>\n";
+    echo "</td><td align=right>".format_credit_large($y->credit), "</td><td align=right>$y->nresults</td></tr>\n";
 }
 
 $mode = get_str('mode', true);

html/user/account_finish_action.php

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
 $auth = post_str("auth");
 $name = post_str("name");
 
-if (strlen($name)==0) {
+if (strlen($name) == 0) {
     error_page(tra("You must supply a name for your account"));
 }
 if ($name != sanitize_tags($name)) {

html/user/openid.php

Indentation +19 added lines, -19 removed lines

@@ -47,11 +47,11 @@ 
 class LightOpenID
 {
     public $returnUrl
-         , $required = array()
-         , $optional = array()
-         , $verify_peer = null
-         , $capath = null
-         , $cainfo = null;
+            , $required = array()
+            , $optional = array()
+            , $verify_peer = null
+            , $capath = null
+            , $cainfo = null;
     private $identity, $claimed_id;
     protected $server, $version, $trustRoot, $aliases, $identifier_select = false
             , $ax = false, $sreg = false, $data, $setup_url = null;
@@ -261,8 +261,8 @@ 
                         $parsed_url = parse_url($this->claimed_id);
                         $this->identity =
                         $this->claimed_id = $parsed_url['scheme'] . '://'
-                                          . $parsed_url['host']
-                                          . $headers[$name];
+                                            . $parsed_url['host']
+                                            . $headers[$name];
                     }
                 }
             }
@@ -301,14 +301,14 @@ 
 
         $url = $parts + $url;
         $url = $url['scheme'] . '://'
-             . (empty($url['username'])?''
+                . (empty($url['username'])?''
                  :(empty($url['password'])? "{$url['username']}@"
                  :"{$url['username']}:{$url['password']}@"))
-             . $url['host']
-             . (empty($url['port'])?'':":{$url['port']}")
-             . (empty($url['path'])?'':$url['path'])
-             . (empty($url['query'])?'':"?{$url['query']}")
-             . (empty($url['fragment'])?'':"#{$url['fragment']}");
+                . $url['host']
+                . (empty($url['port'])?'':":{$url['port']}")
+                . (empty($url['path'])?'':$url['path'])
+                . (empty($url['query'])?'':"?{$url['query']}")
+                . (empty($url['fragment'])?'':"#{$url['fragment']}");
         return $url;
     }
 
@@ -533,7 +533,7 @@ 
 
     protected function authUrl_v1($immediate)
     {
-	$returnUrl = $this->returnUrl;
+    $returnUrl = $this->returnUrl;
         # If we have an openid.delegate that is different from our claimed id,
         # we need to somehow preserve the claimed id between requests.
         # The simplest way is to just send it along with the return_to url.
@@ -549,7 +549,7 @@ 
             ) + $this->sregParams();
 
         return $this->build_url(parse_url($this->server)
-                               , array('query' => http_build_query($params, '', '&')));
+                                , array('query' => http_build_query($params, '', '&')));
     }
 
     protected function authUrl_v2($immediate)
@@ -574,14 +574,14 @@ 
 
         if ($this->identifier_select) {
             $params['openid.identity'] = $params['openid.claimed_id']
-                 = 'http://specs.openid.net/auth/2.0/identifier_select';
+                    = 'http://specs.openid.net/auth/2.0/identifier_select';
         } else {
             $params['openid.identity'] = $this->identity;
             $params['openid.claimed_id'] = $this->claimed_id;
         }
 
         return $this->build_url(parse_url($this->server)
-                               , array('query' => http_build_query($params, '', '&')));
+                                , array('query' => http_build_query($params, '', '&')));
     }
 
     /**
@@ -638,7 +638,7 @@ 
             # If it's an OpenID 1 provider, and we've got claimed_id,
             # we have to append it to the returnUrl, like authUrl_v1 does.
             $this->returnUrl .= (strpos($this->returnUrl, '?') ? '&' : '?')
-                             .  'openid.claimed_id=' . $this->claimed_id;
+                                .  'openid.claimed_id=' . $this->claimed_id;
         }
 
         if ($this->data['openid_return_to'] != $this->returnUrl) {
@@ -707,7 +707,7 @@ 
                 continue;
             }
             $key = substr($this->data['openid_' . $alias . '_type_' . $key],
-                          strlen('http://axschema.org/'));
+                            strlen('http://axschema.org/'));
             $attributes[$key] = $value;
         }
         return $attributes;

Braces +24 added lines, -42 removed lines

@@ -44,8 +44,7 @@ 
  * @copyright Copyright (c) 2010, Mewp
  * @license http://www.opensource.org/licenses/mit-license.php MIT
  */
-class LightOpenID
-{
+class LightOpenID {
     public $returnUrl
          , $required = array()
          , $optional = array()
@@ -67,8 +66,7 @@ 
         'pref/timezone'           => 'timezone',
         );
 
-    function __construct()
-    {
+    function __construct() {
         $this->trustRoot = (!empty($_SERVER['HTTPS']) ? 'https' : 'http') . '://' . $_SERVER['HTTP_HOST'];
         $uri = rtrim(preg_replace('#((?<=\?)|&)openid\.[^&]+#', '', $_SERVER['REQUEST_URI']), '?');
         $this->returnUrl = $this->trustRoot . $uri;
@@ -80,8 +78,7 @@ 
         }
     }
 
-    function __set($name, $value)
-    {
+    function __set($name, $value) {
         switch ($name) {
         case 'identity':
             if (strlen($value = trim((String) $value))) {
@@ -102,8 +99,7 @@ 
         }
     }
 
-    function __get($name)
-    {
+    function __get($name) {
         switch ($name) {
         case 'identity':
             # We return claimed_id instead of identity,
@@ -124,8 +120,7 @@ 
      * @param $url url to check
      * @return true, if the server exists; false otherwise
      */
-    function hostExists($url)
-    {
+    function hostExists($url) {
         if (strpos($url, '/') === false) {
             $server = $url;
         } else {
@@ -139,8 +134,7 @@ 
         return !!gethostbynamel($server);
     }
 
-    protected function request_curl($url, $method='GET', $params=array())
-    {
+    protected function request_curl($url, $method='GET', $params=array()) {
         $params = http_build_query($params, '', '&');
         $curl = curl_init($url . ($method == 'GET' && $params ? '?' . $params : ''));
         curl_setopt($curl, CURLOPT_FOLLOWLOCATION, true);
@@ -195,8 +189,7 @@ 
         return $response;
     }
 
-    protected function request_streams($url, $method='GET', $params=array())
-    {
+    protected function request_streams($url, $method='GET', $params=array()) {
         if(!$this->hostExists($url)) {
             throw new ErrorException('Invalid request.');
         }
@@ -285,16 +278,14 @@ 
         return file_get_contents($url, false, $context);
     }
 
-    protected function request($url, $method='GET', $params=array())
-    {
+    protected function request($url, $method='GET', $params=array()) {
         if(function_exists('curl_init') && !ini_get('safe_mode') && !ini_get('open_basedir')) {
             return $this->request_curl($url, $method, $params);
         }
         return $this->request_streams($url, $method, $params);
     }
 
-    protected function build_url($url, $parts)
-    {
+    protected function build_url($url, $parts) {
         if (isset($url['query'], $parts['query'])) {
             $parts['query'] = $url['query'] . '&' . $parts['query'];
         }
@@ -316,8 +307,7 @@ 
      * Helper function used to scan for <meta>/<link> tags and extract information
      * from them
      */
-    protected function htmlTag($content, $tag, $attrName, $attrValue, $valueName)
-    {
+    protected function htmlTag($content, $tag, $attrName, $attrValue, $valueName) {
         preg_match_all("#<{$tag}[^>]*$attrName=['\"].*?$attrValue.*?['\"][^>]*$valueName=['\"](.+?)['\"][^>]*/?>#i", $content, $matches1);
         preg_match_all("#<{$tag}[^>]*$valueName=['\"](.+?)['\"][^>]*$attrName=['\"].*?$attrValue.*?['\"][^>]*/?>#i", $content, $matches2);
 
@@ -331,8 +321,7 @@ 
      * @return String OP Endpoint (i.e. OpenID provider address).
      * @throws ErrorException
      */
-    function discover($url)
-    {
+    function discover($url) {
         if (!$url) throw new ErrorException('No identity supplied.');
         # Use xri.net proxy to resolve i-name identities
         if (!preg_match('#^https?:#', $url)) {
@@ -465,8 +454,7 @@ 
         throw new ErrorException('Endless redirection!');
     }
 
-    protected function sregParams()
-    {
+    protected function sregParams() {
         $params = array();
         # We always use SREG 1.1, even if the server is advertising only support for 1.0.
         # That's because it's fully backwards compatibile with 1.0, and some providers
@@ -492,8 +480,7 @@ 
         return $params;
     }
 
-    protected function axParams()
-    {
+    protected function axParams() {
         $params = array();
         if ($this->required || $this->optional) {
             $params['openid.ns.ax'] = 'http://openid.net/srv/ax/1.0';
@@ -531,8 +518,7 @@ 
         return $params;
     }
 
-    protected function authUrl_v1($immediate)
-    {
+    protected function authUrl_v1($immediate) {
 	$returnUrl = $this->returnUrl;
         # If we have an openid.delegate that is different from our claimed id,
         # we need to somehow preserve the claimed id between requests.
@@ -552,8 +538,7 @@ 
                                , array('query' => http_build_query($params, '', '&')));
     }
 
-    protected function authUrl_v2($immediate)
-    {
+    protected function authUrl_v2($immediate) {
         $params = array(
             'openid.ns'          => 'http://specs.openid.net/auth/2.0',
             'openid.mode'        => $immediate ? 'checkid_immediate' : 'checkid_setup',
@@ -590,8 +575,7 @@ 
      * @param String $select_identifier Whether to request OP to select identity for an user in OpenID 2. Does not affect OpenID 1.
      * @throws ErrorException
      */
-    function authUrl($immediate = false)
-    {
+    function authUrl($immediate = false) {
         if ($this->setup_url && !$immediate) return $this->setup_url;
         if (!$this->server) $this->discover($this->identity);
 
@@ -606,8 +590,7 @@ 
      * @return Bool Whether the verification was successful.
      * @throws ErrorException
      */
-    function validate()
-    {
+    function validate() {
         # If the request was using immediate mode, a failure may be reported
         # by presenting user_setup_url (for 1.1) or reporting
         # mode 'setup_needed' (for 2.0). Also catching all modes other than
@@ -668,12 +651,12 @@ 
         return preg_match('/is_valid\s*:\s*true/i', $response);
     }
 
-    protected function getAxAttributes()
-    {
+    protected function getAxAttributes() {
         $alias = null;
         if (isset($this->data['openid_ns_ax'])
             && $this->data['openid_ns_ax'] != 'http://openid.net/srv/ax/1.0'
-        ) { # It's the most likely case, so we'll check it before
+        ) {
+# It's the most likely case, so we'll check it before
             $alias = 'ax';
         } else {
             # 'ax' prefix is either undefined, or points to another extension,
@@ -713,8 +696,7 @@ 
         return $attributes;
     }
 
-    protected function getSregAttributes()
-    {
+    protected function getSregAttributes() {
         $attributes = array();
         $sreg_to_ax = array_flip(self::$ax_to_sreg);
         foreach ($this->data as $key => $value) {
@@ -741,11 +723,11 @@ 
      *     * @return Array Array of attributes with keys being the AX schema names, e.g. 'contact/email'
      * @see http://www.axschema.org/types/
      */
-    function getAttributes()
-    {
+    function getAttributes() {
         if (isset($this->data['openid_ns'])
             && $this->data['openid_ns'] == 'http://specs.openid.net/auth/2.0'
-        ) { # OpenID 2.0
+        ) {
+# OpenID 2.0
             # We search for both AX and SREG attributes, with AX taking precedence.
             return $this->getAxAttributes() + $this->getSregAttributes();
         }

Spacing +64 added lines, -64 removed lines

@@ -69,13 +69,13 @@ 
 
     function __construct()
     {
-        $this->trustRoot = (!empty($_SERVER['HTTPS']) ? 'https' : 'http') . '://' . $_SERVER['HTTP_HOST'];
+        $this->trustRoot = (!empty($_SERVER['HTTPS']) ? 'https' : 'http').'://'.$_SERVER['HTTP_HOST'];
         $uri = rtrim(preg_replace('#((?<=\?)|&)openid\.[^&]+#', '', $_SERVER['REQUEST_URI']), '?');
-        $this->returnUrl = $this->trustRoot . $uri;
+        $this->returnUrl = $this->trustRoot.$uri;
 
         $this->data = $_POST + $_GET; # OPs may send data as POST or GET.
 
-        if(!function_exists('curl_init') && !in_array('https', stream_get_wrappers())) {
+        if (!function_exists('curl_init') && !in_array('https', stream_get_wrappers())) {
             throw new ErrorException('You must have either https wrappers or curl enabled.');
         }
     }
@@ -84,7 +84,7 @@ 
     {
         switch ($name) {
         case 'identity':
-            if (strlen($value = trim((String) $value))) {
+            if (strlen($value = trim((String)$value))) {
                 if (preg_match('#^xri:/*#i', $value, $m)) {
                     $value = substr($value, strlen($m[0]));
                 } elseif (!preg_match('/^(?:[=@+\$!\(]|https?:)/i', $value)) {
@@ -139,23 +139,23 @@ 
         return !!gethostbynamel($server);
     }
 
-    protected function request_curl($url, $method='GET', $params=array())
+    protected function request_curl($url, $method = 'GET', $params = array())
     {
         $params = http_build_query($params, '', '&');
-        $curl = curl_init($url . ($method == 'GET' && $params ? '?' . $params : ''));
+        $curl = curl_init($url.($method == 'GET' && $params ? '?'.$params : ''));
         curl_setopt($curl, CURLOPT_FOLLOWLOCATION, true);
         curl_setopt($curl, CURLOPT_HEADER, false);
         curl_setopt($curl, CURLOPT_SSL_VERIFYPEER, false);
         curl_setopt($curl, CURLOPT_RETURNTRANSFER, true);
         curl_setopt($curl, CURLOPT_HTTPHEADER, array('Accept: application/xrds+xml, */*'));
 
-        if($this->verify_peer !== null) {
+        if ($this->verify_peer !== null) {
             curl_setopt($curl, CURLOPT_SSL_VERIFYPEER, $this->verify_peer);
-            if($this->capath) {
+            if ($this->capath) {
                 curl_setopt($curl, CURLOPT_CAPATH, $this->capath);
             }
 
-            if($this->cainfo) {
+            if ($this->cainfo) {
                 curl_setopt($curl, CURLOPT_CAINFO, $this->cainfo);
             }
         }
@@ -171,17 +171,17 @@ 
         }
         $response = curl_exec($curl);
 
-        if($method == 'HEAD') {
+        if ($method == 'HEAD') {
             $headers = array();
-            foreach(explode("\n", $response) as $header) {
-                $pos = strpos($header,':');
+            foreach (explode("\n", $response) as $header) {
+                $pos = strpos($header, ':');
                 $name = strtolower(trim(substr($header, 0, $pos)));
-                $headers[$name] = trim(substr($header, $pos+1));
+                $headers[$name] = trim(substr($header, $pos + 1));
             }
 
             # Updating claimed_id in case of redirections.
             $effective_url = curl_getinfo($curl, CURLINFO_EFFECTIVE_URL);
-            if($effective_url != $url) {
+            if ($effective_url != $url) {
                 $this->identity = $this->claimed_id = $effective_url;
             }
 
@@ -195,14 +195,14 @@ 
         return $response;
     }
 
-    protected function request_streams($url, $method='GET', $params=array())
+    protected function request_streams($url, $method = 'GET', $params = array())
     {
-        if(!$this->hostExists($url)) {
+        if (!$this->hostExists($url)) {
             throw new ErrorException('Invalid request.');
         }
 
         $params = http_build_query($params, '', '&');
-        switch($method) {
+        switch ($method) {
         case 'GET':
             $opts = array(
                 'http' => array(
@@ -211,7 +211,7 @@ 
                     'ignore_errors' => true,
                 )
             );
-            $url = $url . ($params ? '?' . $params : '');
+            $url = $url.($params ? '?'.$params : '');
             break;
         case 'POST':
             $opts = array(
@@ -236,31 +236,31 @@ 
                 ))
             );
 
-            $url = $url . ($params ? '?' . $params : '');
-            $headers_tmp = get_headers ($url);
-            if(!$headers_tmp) {
+            $url = $url.($params ? '?'.$params : '');
+            $headers_tmp = get_headers($url);
+            if (!$headers_tmp) {
                 return array();
             }
 
             # Parsing headers.
             $headers = array();
-            foreach($headers_tmp as $header) {
-                $pos = strpos($header,':');
+            foreach ($headers_tmp as $header) {
+                $pos = strpos($header, ':');
                 $name = strtolower(trim(substr($header, 0, $pos)));
-                $headers[$name] = trim(substr($header, $pos+1));
+                $headers[$name] = trim(substr($header, $pos + 1));
 
                 # Following possible redirections. The point is just to have
                 # claimed_id change with them, because get_headers() will
                 # follow redirections automatically.
                 # We ignore redirections with relative paths.
                 # If any known provider uses them, file a bug report.
-                if($name == 'location') {
-                    if(strpos($headers[$name], 'http') === 0) {
+                if ($name == 'location') {
+                    if (strpos($headers[$name], 'http') === 0) {
                         $this->identity = $this->claimed_id = $headers[$name];
-                    } elseif($headers[$name][0] == '/') {
+                    } elseif ($headers[$name][0] == '/') {
                         $parsed_url = parse_url($this->claimed_id);
                         $this->identity =
-                        $this->claimed_id = $parsed_url['scheme'] . '://'
+                        $this->claimed_id = $parsed_url['scheme'].'://'
                                           . $parsed_url['host']
                                           . $headers[$name];
                     }
@@ -272,7 +272,7 @@ 
             return $headers;
         }
 
-        if($this->verify_peer) {
+        if ($this->verify_peer) {
             $opts += array('ssl' => array(
                 'verify_peer' => true,
                 'capath'      => $this->capath,
@@ -280,14 +280,14 @@ 
             ));
         }
 
-        $context = stream_context_create ($opts);
+        $context = stream_context_create($opts);
 
         return file_get_contents($url, false, $context);
     }
 
-    protected function request($url, $method='GET', $params=array())
+    protected function request($url, $method = 'GET', $params = array())
     {
-        if(function_exists('curl_init') && !ini_get('safe_mode') && !ini_get('open_basedir')) {
+        if (function_exists('curl_init') && !ini_get('safe_mode') && !ini_get('open_basedir')) {
             return $this->request_curl($url, $method, $params);
         }
         return $this->request_streams($url, $method, $params);
@@ -296,19 +296,19 @@ 
     protected function build_url($url, $parts)
     {
         if (isset($url['query'], $parts['query'])) {
-            $parts['query'] = $url['query'] . '&' . $parts['query'];
+            $parts['query'] = $url['query'].'&'.$parts['query'];
         }
 
         $url = $parts + $url;
-        $url = $url['scheme'] . '://'
-             . (empty($url['username'])?''
-                 :(empty($url['password'])? "{$url['username']}@"
+        $url = $url['scheme'].'://'
+             . (empty($url['username']) ? ''
+                 :(empty($url['password']) ? "{$url['username']}@"
                  :"{$url['username']}:{$url['password']}@"))
              . $url['host']
-             . (empty($url['port'])?'':":{$url['port']}")
-             . (empty($url['path'])?'':$url['path'])
-             . (empty($url['query'])?'':"?{$url['query']}")
-             . (empty($url['fragment'])?'':"#{$url['fragment']}");
+             . (empty($url['port']) ? '' : ":{$url['port']}")
+             . (empty($url['path']) ? '' : $url['path'])
+             . (empty($url['query']) ? '' : "?{$url['query']}")
+             . (empty($url['fragment']) ? '' : "#{$url['fragment']}");
         return $url;
     }
 
@@ -322,7 +322,7 @@ 
         preg_match_all("#<{$tag}[^>]*$valueName=['\"](.+?)['\"][^>]*$attrName=['\"].*?$attrValue.*?['\"][^>]*/?>#i", $content, $matches2);
 
         $result = array_merge($matches1[1], $matches2[1]);
-        return empty($result)?false:$result[0];
+        return empty($result) ?false:$result[0];
     }
 
     /**
@@ -348,7 +348,7 @@ 
         $yadis = true;
 
         # We'll jump a maximum of 5 times, to avoid endless redirections.
-        for ($i = 0; $i < 5; $i ++) {
+        for ($i = 0; $i < 5; $i++) {
             if ($yadis) {
                 $headers = $this->request($url, 'HEAD');
 
@@ -370,12 +370,12 @@ 
                     $content = $this->request($url, 'GET');
 
                     preg_match_all('#<Service.*?>(.*?)</Service>#s', $content, $m);
-                    foreach($m[1] as $content) {
-                        $content = ' ' . $content; # The space is added, so that strpos doesn't return 0.
+                    foreach ($m[1] as $content) {
+                        $content = ' '.$content; # The space is added, so that strpos doesn't return 0.
 
                         # OpenID 2
                         $ns = preg_quote('http://specs.openid.net/auth/2.0/');
-                        if(preg_match('#<Type>\s*'.$ns.'(server|signon)\s*</Type>#s', $content, $type)) {
+                        if (preg_match('#<Type>\s*'.$ns.'(server|signon)\s*</Type>#s', $content, $type)) {
                             if ($type[1] == 'server') $this->identifier_select = true;
 
                             preg_match('#<URI.*?>(.*)</URI>#', $content, $server);
@@ -384,7 +384,7 @@ 
                                 return false;
                             }
                             # Does the server advertise support for either AX or SREG?
-                            $this->ax   = (bool) strpos($content, '<Type>http://openid.net/srv/ax/1.0</Type>');
+                            $this->ax   = (bool)strpos($content, '<Type>http://openid.net/srv/ax/1.0</Type>');
                             $this->sreg = strpos($content, '<Type>http://openid.net/sreg/1.0</Type>')
                                        || strpos($content, '<Type>http://openid.net/extensions/sreg/1.1</Type>');
 
@@ -498,33 +498,33 @@ 
         if ($this->required || $this->optional) {
             $params['openid.ns.ax'] = 'http://openid.net/srv/ax/1.0';
             $params['openid.ax.mode'] = 'fetch_request';
-            $this->aliases  = array();
+            $this->aliases = array();
             $counts   = array();
             $required = array();
             $optional = array();
-            foreach (array('required','optional') as $type) {
+            foreach (array('required', 'optional') as $type) {
                 foreach ($this->$type as $alias => $field) {
                     if (is_int($alias)) $alias = strtr($field, '/', '_');
-                    $this->aliases[$alias] = 'http://axschema.org/' . $field;
+                    $this->aliases[$alias] = 'http://axschema.org/'.$field;
                     if (empty($counts[$alias])) $counts[$alias] = 0;
                     $counts[$alias] += 1;
                     ${$type}[] = $alias;
                 }
             }
             foreach ($this->aliases as $alias => $ns) {
-                $params['openid.ax.type.' . $alias] = $ns;
+                $params['openid.ax.type.'.$alias] = $ns;
             }
             foreach ($counts as $alias => $count) {
                 if ($count == 1) continue;
-                $params['openid.ax.count.' . $alias] = $count;
+                $params['openid.ax.count.'.$alias] = $count;
             }
 
             # Don't send empty ax.requied and ax.if_available.
             # Google and possibly other providers refuse to support ax when one of these is empty.
-            if($required) {
+            if ($required) {
                 $params['openid.ax.required'] = implode(',', $required);
             }
-            if($optional) {
+            if ($optional) {
                 $params['openid.ax.if_available'] = implode(',', $optional);
             }
         }
@@ -537,8 +537,8 @@ 
         # If we have an openid.delegate that is different from our claimed id,
         # we need to somehow preserve the claimed id between requests.
         # The simplest way is to just send it along with the return_to url.
-        if($this->identity != $this->claimed_id) {
-            $returnUrl .= (strpos($returnUrl, '?') ? '&' : '?') . 'openid.claimed_id=' . $this->claimed_id;
+        if ($this->identity != $this->claimed_id) {
+            $returnUrl .= (strpos($returnUrl, '?') ? '&' : '?').'openid.claimed_id='.$this->claimed_id;
         }
 
         $params = array(
@@ -612,15 +612,15 @@ 
         # by presenting user_setup_url (for 1.1) or reporting
         # mode 'setup_needed' (for 2.0). Also catching all modes other than
         # id_res, in order to avoid throwing errors.
-        if(isset($this->data['openid_user_setup_url'])) {
+        if (isset($this->data['openid_user_setup_url'])) {
             $this->setup_url = $this->data['openid_user_setup_url'];
             return false;
         }
-        if($this->mode != 'id_res') {
+        if ($this->mode != 'id_res') {
             return false;
         }
 
-        $this->claimed_id = isset($this->data['openid_claimed_id'])?$this->data['openid_claimed_id']:$this->data['openid_identity'];
+        $this->claimed_id = isset($this->data['openid_claimed_id']) ? $this->data['openid_claimed_id'] : $this->data['openid_identity'];
         $params = array(
             'openid.assoc_handle' => $this->data['openid_assoc_handle'],
             'openid.signed'       => $this->data['openid_signed'],
@@ -638,7 +638,7 @@ 
             # If it's an OpenID 1 provider, and we've got claimed_id,
             # we have to append it to the returnUrl, like authUrl_v1 does.
             $this->returnUrl .= (strpos($this->returnUrl, '?') ? '&' : '?')
-                             .  'openid.claimed_id=' . $this->claimed_id;
+                             .  'openid.claimed_id='.$this->claimed_id;
         }
 
         if ($this->data['openid_return_to'] != $this->returnUrl) {
@@ -656,8 +656,8 @@ 
             # In such case, validation would fail, since we'd send different data than OP
             # wants to verify. stripslashes() should solve that problem, but we can't
             # use it when magic_quotes is off.
-            $value = $this->data['openid_' . str_replace('.','_',$item)];
-            $params['openid.' . $item] = (version_compare(PHP_VERSION, '5.4.0') < 0 && get_magic_quotes_gpc()) ? stripslashes($value) : $value;
+            $value = $this->data['openid_'.str_replace('.', '_', $item)];
+            $params['openid.'.$item] = (version_compare(PHP_VERSION, '5.4.0') < 0 && get_magic_quotes_gpc()) ? stripslashes($value) : $value;
 
         }
 
@@ -695,18 +695,18 @@ 
 
         $attributes = array();
         foreach ($this->data as $key => $value) {
-            $keyMatch = 'openid_' . $alias . '_value_';
+            $keyMatch = 'openid_'.$alias.'_value_';
             if (substr($key, 0, strlen($keyMatch)) != $keyMatch) {
                 continue;
             }
             $key = substr($key, strlen($keyMatch));
-            if (!isset($this->data['openid_' . $alias . '_type_' . $key])) {
+            if (!isset($this->data['openid_'.$alias.'_type_'.$key])) {
                 # OP is breaking the spec by returning a field without
                 # associated ns. This shouldn't happen, but it's better
                 # to check, than cause an E_NOTICE.
                 continue;
             }
-            $key = substr($this->data['openid_' . $alias . '_type_' . $key],
+            $key = substr($this->data['openid_'.$alias.'_type_'.$key],
                           strlen('http://axschema.org/'));
             $attributes[$key] = $value;
         }

html/user/apps.php

Spacing +3 added lines, -3 removed lines

@@ -48,7 +48,7 @@ 
             echo "        <beta/>\n";
         }
     } else {
-        $b = $app->beta?" (beta test)":"";
+        $b = $app->beta ? " (beta test)" : "";
         echo "
             <tr><th class=\"bg-primary\" colspan=4>$app->user_friendly_name$b</th></tr>
             <tr>
@@ -61,7 +61,7 @@ 
     }
     foreach ($platforms as $platform) {
         $avs = latest_avs_app_platform($app->id, $platform->id);
-        foreach($avs as $av) {
+        foreach ($avs as $av) {
             $create_time_f = pretty_time_str($av->create_time);
             if ($xml) {
                 echo "    <version>\n";
@@ -83,7 +83,7 @@ 
                 $gf = $av->expavg_credit/200;
                 $total_gf += $gf;
                 $gf = number_format($gf, 0);
-                $b = $av->beta?" (beta test)":"";
+                $b = $av->beta ? " (beta test)" : "";
                 echo "<tr>
                     <td>$platform->user_friendly_name</td>
                     <td>$version_num_f$b</td>

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);
 }