BOINC / boinc

drupal/sites/default/boinc/modules/boinccore/boinccore.admin.inc

Indentation +84 added lines, -84 removed lines

@@ -7,185 +7,185 @@ 
 */
 
 /**
-  * The BOINC community form allows the setting of requirements for interaction
-  * in the community sections of the site.
-  */
+ * The BOINC community form allows the setting of requirements for interaction
+ * in the community sections of the site.
+ */
 function boinccore_admin_community(&$form_state) {
-  $form = array();
-  $default = array(
+    $form = array();
+    $default = array(
     'min_credit_to_post' => variable_get('boinc_comment_min_credit', 0),
     'penalty_period' => variable_get('boinc_penalty_period', 7*24*60*60),
     'moderator_mailing_list' => variable_get('boinc_moderator_mailing_list', ''),
     'moderator_force_individual_notifications' => variable_get('boinc_moderator_force_individual_notifications', 0),
-  );
+    );
   
-  // Define the form
-  $form['boinc_comment_min_credit'] = array(
+    // Define the form
+    $form['boinc_comment_min_credit'] = array(
     '#type' => 'textfield',
     '#title' => t('Credit required for community participation'),
     '#default_value' => $default['min_credit_to_post'],
     '#description' => t('Set a minimum threshold of credit a user must have
       earned before being allowed to create a profile, post comments or participate in forums.'),
-  );
-  $form['boinc_penalty_period'] = array(
+    );
+    $form['boinc_penalty_period'] = array(
     '#type' => 'textfield',
     '#title' => t('Penalty period'),
     '#default_value' => $default['penalty_period'],
     '#description' => t('The amount of time (in seconds) that user privileges
       are revoked when a user is banned from community participation.'),
-  );
-  $form['boinc_moderator_mailing_list'] = array(
+    );
+    $form['boinc_moderator_mailing_list'] = array(
     '#type' => 'textfield',
     '#title' => t('Moderator mailing list'),
     '#default_value' => $default['moderator_mailing_list'],
     '#description' => t('As an alternative to sending moderator notifications
       to individual users with the moderator role, enter a mailing list address
       here to be the only target of the notification emails.'),
-  );
-  $form['boinc_moderator_force_individual_notifications'] = array(
+    );
+    $form['boinc_moderator_force_individual_notifications'] = array(
     '#type' => 'checkbox',
     '#title' => t('Always send individual notifications'),
     '#default_value' => $default['moderator_force_individual_notifications'],
     '#description' => t('Continue to send individual moderator notification
       emails even when a mailing list is provided.'),
-  );
+    );
   
-  $form['tools'] = array(
+    $form['tools'] = array(
     '#title' => t('Tools'),
     '#type' => 'fieldset',
     '#description' => '',
     '#collapsible' => TRUE,
     '#collapsed' => FALSE,
     '#attributes' => array('class' => '')
-  );
-  $form['tools']['reset_uotd'] = array(
+    );
+    $form['tools']['reset_uotd'] = array(
     '#type' => 'submit',
     '#value' => t('Force new UOTD'),
     '#submit' => array('boinccore_reset_user_of_the_day'),
-  );
+    );
   
-  return system_settings_form($form);
+    return system_settings_form($form);
 }
 
 /**
-  * Validate the BOINC community form.
-  */
+ * Validate the BOINC community form.
+ */
 function boinccore_admin_community_validate($form, &$form_state) {
-  $values = $form_state['values'];
-  if (!is_numeric($values['boinc_comment_min_credit'])) {
+    $values = $form_state['values'];
+    if (!is_numeric($values['boinc_comment_min_credit'])) {
     form_set_error('boinc_comment_min_credit', t('The minimum credit value must be a number.'));
-  }
+    }
 }
 
 /**
-  * Handle post-validation submission of BOINC community form.
-  */
+ * Handle post-validation submission of BOINC community form.
+ */
 function boinccore_admin_community_submit($form, &$form_state) {
-  drupal_set_message(t('BOINC community settings have been updated.'));
+    drupal_set_message(t('BOINC community settings have been updated.'));
 }
 
 /**
  * Force a reset of the user of the day
  */
 function boinccore_reset_user_of_the_day() {
-  if (boincuser_select_user_of_the_day()) {
+    if (boincuser_select_user_of_the_day()) {
     drupal_set_message(t('The user of the day has been reset.'));
-  }
+    }
 }
 
 /**
-  * The standard page generator form provides an interface to create standard
-  * pages, such as a Help or About page.
-  */
+ * The standard page generator form provides an interface to create standard
+ * pages, such as a Help or About page.
+ */
 function boinccore_admin_default_content(&$form_state) {
-  $form = array();
+    $form = array();
   
-  // Define the form
-  $form['page'] = array(
+    // Define the form
+    $form['page'] = array(
     '#type' => 'select',
     '#title' => t('Generate default content for'),
     '#options' => array(
-      0 => t('all standard pages'),
+        0 => t('all standard pages'),
 //      'about' => t('About page'),
-      'help' => t('Help page'),
+        'help' => t('Help page'),
     ),
-  );
-  $form['submit'] = array(
+    );
+    $form['submit'] = array(
     '#type' => 'submit',
     '#value' => 'Generate content',
-  );
+    );
   
-  return $form;
+    return $form;
 }
 
 /**
-  * Generate content based on the selection made on the form
-  */
+ * Generate content based on the selection made on the form
+ */
 function boinccore_admin_default_content_submit($form, &$form_state) {
-  $option = $form_state['values']['page'];
-  $page_list = boinccore_admin_default_content_generate($option);
-  $pages_generated = count($page_list);
-  if ($pages_generated > 0) {
+    $option = $form_state['values']['page'];
+    $page_list = boinccore_admin_default_content_generate($option);
+    $pages_generated = count($page_list);
+    if ($pages_generated > 0) {
     switch ($pages_generated) {
     case 1:
       $page_list = reset($page_list);
-      break;
+        break;
     case 2:
       $page_list = implode(' and ', $page_list);
-      break;
+        break;
     default:
       end($page_list);
-      $page_list[key($page_list)] = 'and ' . $page_list[key($page_list)];
-      $page_list = implode(', ', $page_list);
+        $page_list[key($page_list)] = 'and ' . $page_list[key($page_list)];
+        $page_list = implode(', ', $page_list);
     }
     drupal_set_message(t('Default content has been generated for the @listed
       @pages.', array(
         '@listed' => $page_list, 
         '@pages' => format_plural($pages_generated, 'page', 'pages'),
-      )
+        )
     ));
-  }
-  else {
+    }
+    else {
     if ($option) {
-      drupal_set_message(t('The @specified page already exists.',
+        drupal_set_message(t('The @specified page already exists.',
         array('@specified' => $option)), 'warning');
     }
     else {
-      drupal_set_message(t('All pages already exist, so none can be recreated
+        drupal_set_message(t('All pages already exist, so none can be recreated
         automatically.'), 'warning');
     }
     drupal_set_message(t('To recreate a page, first delete the existing page (or
       remove its URL alias), then try this function again.'), 'warning');
-  }
+    }
 }
 
 /**
-  * 
-  */
+ * 
+ */
 function boinccore_admin_default_content_generate($pages = array()) {
-  if (!$pages) {
+    if (!$pages) {
     // If not specified, attempt to generate all pages by default
     $pages = array(
-      'help',
+        'help',
 //      'about',
     );
-  }
-  elseif (!is_array($pages)) {
+    }
+    elseif (!is_array($pages)) {
     $pages = array($pages);
-  }
-  $pages_generated = array();
-  if (in_array('about', $pages)) {
+    }
+    $pages_generated = array();
+    if (in_array('about', $pages)) {
     // Create the default About page, if it doesn't exist
     // TODO: what is the default About page...?
     if (!drupal_lookup_path('source', 'about')) {
-      drupal_set_message('The about page doesn\'t yet have a definition...', 'warning');
-      //$pages_generated[] = 'about';
+        drupal_set_message('The about page doesn\'t yet have a definition...', 'warning');
+        //$pages_generated[] = 'about';
+    }
     }
-  }
-  if (in_array('help', $pages)) {
+    if (in_array('help', $pages)) {
     // Create the default Help page, if it doesn't exist
     if (!drupal_lookup_path('source', 'help')) {
-      $node = array(
+        $node = array(
         'type' => 'page',
         'title' => t('Help'),
         'uid' => 1,
@@ -198,7 +198,7 @@ 
         'comment' => 0, // disable comments
         'moderate' => 0,
         'body' => preg_replace('/>\s+</', '><',
-          '<p>Use this page to provide help and support for your site\'s
+            '<p>Use this page to provide help and support for your site\'s
           visitors:</p>
           <ul>
             <li>Add a link to your primary forum for getting help</li>
@@ -209,21 +209,21 @@ 
         'sticky' => 0,
         'format' => 4,
         //'teaser' => $teaser,
-      );
-      $node = (object) $node; // node_save requires an object
-      node_save($node);
+        );
+        $node = (object) $node; // node_save requires an object
+        node_save($node);
       
-      $item = array(
+        $item = array(
         'link_path' => drupal_get_normal_path('help'),
         'link_title' => t('Help'),
         'menu_name' => 'primary-links',
         'weight' => 50,
-      );
-      menu_link_save($item);
-      $pages_generated[] = 'help';
+        );
+        menu_link_save($item);
+        $pages_generated[] = 'help';
+    }
     }
-  }
-  // Refresh the cache with these changes
-  menu_cache_clear_all();
-  return $pages_generated;
+    // Refresh the cache with these changes
+    menu_cache_clear_all();
+    return $pages_generated;
 }

Spacing +3 added lines, -3 removed lines

@@ -135,7 +135,7 @@ 
       break;
     default:
       end($page_list);
-      $page_list[key($page_list)] = 'and ' . $page_list[key($page_list)];
+      $page_list[key($page_list)] = 'and '.$page_list[key($page_list)];
       $page_list = implode(', ', $page_list);
     }
     drupal_set_message(t('Default content has been generated for the @listed
@@ -189,7 +189,7 @@ 
         'type' => 'page',
         'title' => t('Help'),
         'uid' => 1,
-        'status' => 1,  // published
+        'status' => 1, // published
         'promote' => 0, // not promoted to front page
         'path' => 'help',
         'pathauto_perform_alias' => FALSE,
@@ -210,7 +210,7 @@ 
         'format' => 4,
         //'teaser' => $teaser,
       );
-      $node = (object) $node; // node_save requires an object
+      $node = (object)$node; // node_save requires an object
       node_save($node);
       
       $item = array(

Braces +3 added lines, -6 removed lines

@@ -144,13 +144,11 @@ 
         '@pages' => format_plural($pages_generated, 'page', 'pages'),
       )
     ));
-  }
-  else {
+  } else {
     if ($option) {
       drupal_set_message(t('The @specified page already exists.',
         array('@specified' => $option)), 'warning');
-    }
-    else {
+    } else {
       drupal_set_message(t('All pages already exist, so none can be recreated
         automatically.'), 'warning');
     }
@@ -169,8 +167,7 @@ 
       'help',
 //      'about',
     );
-  }
-  elseif (!is_array($pages)) {
+  } elseif (!is_array($pages)) {
     $pages = array($pages);
   }
   $pages_generated = array();

Upper-Lower-Casing +3 added lines, -3 removed lines

@@ -54,8 +54,8 @@ 
     '#title' => t('Tools'),
     '#type' => 'fieldset',
     '#description' => '',
-    '#collapsible' => TRUE,
-    '#collapsed' => FALSE,
+    '#collapsible' => true,
+    '#collapsed' => false,
     '#attributes' => array('class' => '')
   );
   $form['tools']['reset_uotd'] = array(
@@ -192,7 +192,7 @@ 
         'status' => 1,  // published
         'promote' => 0, // not promoted to front page
         'path' => 'help',
-        'pathauto_perform_alias' => FALSE,
+        'pathauto_perform_alias' => false,
         'created' => time(),
         'changed' => time(),
         'comment' => 0, // disable comments

drupal/sites/default/boinc/modules/comment_form_block/comment_form_block.module

Indentation +17 added lines, -17 removed lines

@@ -9,40 +9,40 @@ 
  * Implements hook_block() for Drupal 6 compatibility
  */
 function comment_form_block_block($op = 'list', $delta = 0, $edit = array()) {
-  switch ($op) {
-  case 'view':
+    switch ($op) {
+    case 'view':
     //$block = node_comment_block_block_view($delta);
     $block = array();
     if ($delta == 'comment_form') {
-      if (arg(0) == 'node' && is_numeric($nid = arg(1)) && !arg(2)) {
+        if (arg(0) == 'node' && is_numeric($nid = arg(1)) && !arg(2)) {
         global $user;
         $node = node_load($nid);
         if ($node->comment !== 0 AND user_access('post comments', $user)) {
-          $block['subject'] = NULL; // This should be NULL otherwise there will be duplicate h2 elements.
-          $block['content'] = '<h2>' . bts('Post new comment') . '</h2>';
-          $block['content'] .= drupal_get_form('comment_form', array('nid' => $nid)); 
+            $block['subject'] = NULL; // This should be NULL otherwise there will be duplicate h2 elements.
+            $block['content'] = '<h2>' . bts('Post new comment') . '</h2>';
+            $block['content'] .= drupal_get_form('comment_form', array('nid' => $nid)); 
+        }
         }
-      }
     }
     break;
-  case 'list':
+    case 'list':
     $blocks = comment_form_block_block_info();
     return $blocks;
     break;
-  default:
+    default:
   }
-  return $block;
+    return $block;
 }
 
 /**
  * Implements hook_block_info().
  */
 function comment_form_block_block_info() {
-  $blocks['comment_form'] = array(
+    $blocks['comment_form'] = array(
     'info' => bts('Comment form'),
-  );
+    );
 
-  return $blocks;
+    return $blocks;
 }
 
 /**
@@ -81,10 +81,10 @@ 
  * Implements hook_form_FORM_ID_alter() for block_admin_configure.
  */
 function comment_form_block_form_block_admin_configure_alter(&$form, &$form_state) {
-  if (isset($form['delta'])) {
+    if (isset($form['delta'])) {
     if ($form['delta']['#value'] == 'comment_form') {
-      $form['settings']['title']['#disabled'] = TRUE;
-      $form['settings']['title']['#description'] = t('The title for this block cannot be overridden.');
+        $form['settings']['title']['#disabled'] = TRUE;
+        $form['settings']['title']['#description'] = t('The title for this block cannot be overridden.');
+    }
     }
-  }
 }

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
         $node = node_load($nid);
         if ($node->comment !== 0 AND user_access('post comments', $user)) {
           $block['subject'] = NULL; // This should be NULL otherwise there will be duplicate h2 elements.
-          $block['content'] = '<h2>' . bts('Post new comment') . '</h2>';
+          $block['content'] = '<h2>'.bts('Post new comment').'</h2>';
           $block['content'] .= drupal_get_form('comment_form', array('nid' => $nid)); 
         }
       }

Upper-Lower-Casing +3 added lines, -3 removed lines

@@ -17,8 +17,8 @@ 
       if (arg(0) == 'node' && is_numeric($nid = arg(1)) && !arg(2)) {
         global $user;
         $node = node_load($nid);
-        if ($node->comment !== 0 AND user_access('post comments', $user)) {
-          $block['subject'] = NULL; // This should be NULL otherwise there will be duplicate h2 elements.
+        if ($node->comment !== 0 and user_access('post comments', $user)) {
+          $block['subject'] = null; // This should be NULL otherwise there will be duplicate h2 elements.
           $block['content'] = '<h2>' . bts('Post new comment') . '</h2>';
           $block['content'] .= drupal_get_form('comment_form', array('nid' => $nid)); 
         }
@@ -83,7 +83,7 @@ 
 function comment_form_block_form_block_admin_configure_alter(&$form, &$form_state) {
   if (isset($form['delta'])) {
     if ($form['delta']['#value'] == 'comment_form') {
-      $form['settings']['title']['#disabled'] = TRUE;
+      $form['settings']['title']['#disabled'] = true;
       $form['settings']['title']['#description'] = t('The title for this block cannot be overridden.');
     }
   }

drupal/sites/default/boinc/modules/flag_comment_notify/flag_comment_notify.module

Indentation +32 added lines, -32 removed lines

@@ -26,16 +26,16 @@ 
  * Implementation of hook_comment(); 
  */
 function flag_comment_notify_comment(&$a1, $op) {
-  switch ($op) {
-  case 'insert':
+    switch ($op) {
+    case 'insert':
     module_load_include('inc', 'drupal_queue', 'drupal_queue');
     $queue = DrupalQueue::get('flag_comment_notify');
     $subscribed_users = array_keys((array) flag_get_content_flags('node', $a1['nid'], 'subscriptions'));
     foreach ($subscribed_users as $uid) {
-      $queue->createItem(array('uid' => $uid, 'cid' => (int) $a1['cid']));
+        $queue->createItem(array('uid' => $uid, 'cid' => (int) $a1['cid']));
     }
     break;
-  default:
+    default:
   }
 }
 
@@ -43,12 +43,12 @@ 
  * Implementation of hook_cron_queue_info()
  */
 function flag_comment_notify_cron_queue_info() {
-  $queues = array();
-  $queues['flag_comment_notify'] = array(
+    $queues = array();
+    $queues['flag_comment_notify'] = array(
     'worker callback' => 'flag_comment_notify_send_notification',
     'time' => 60,
-  );
-  return $queues;
+    );
+    return $queues;
 }
 
 
@@ -57,44 +57,44 @@ 
  *  *  *  *  *  *  *  *  *  *  *  *  *  *  *  *  *  *  *  *  *  *  *  *  *  *  *  */
 
 function flag_comment_notify_send_notification($data) {
-  $account = user_load($data['uid']);
-  $comment = _comment_load($data['cid']);
-  $node = node_load($comment->nid);
-  $author = user_load($comment->uid);
-  if ($account->mail AND $node->nid) {
+    $account = user_load($data['uid']);
+    $comment = _comment_load($data['cid']);
+    $node = node_load($comment->nid);
+    $author = user_load($comment->uid);
+    if ($account->mail AND $node->nid) {
     // Don't send notifications to the author of the comment
     if ($account->uid != $author->uid) {
-      $params['account'] = $account;
-      $params['comment'] = $comment;
-      $params['node'] = $node;
-      $params['author'] = $author;
-      drupal_mail('flag_comment_notify', 'comment_posted', $account->mail,
+        $params['account'] = $account;
+        $params['comment'] = $comment;
+        $params['node'] = $node;
+        $params['author'] = $author;
+        drupal_mail('flag_comment_notify', 'comment_posted', $account->mail,
         user_preferred_language($account), $params);
     }
-  }
+    }
 }
 
 
 function flag_comment_notify_mail($key, &$message, $params) {
-  $language = $message['language'];
-  $variables = user_mail_tokens($params['account'], $language);
-  $variables['!comment_url'] = url(
+    $language = $message['language'];
+    $variables = user_mail_tokens($params['account'], $language);
+    $variables['!comment_url'] = url(
     "goto/comment/{$params['comment']->cid}",
     array(
-      'absolute' => TRUE,
+        'absolute' => TRUE,
     )
-  );
-  $variables['!topic_name'] = $params['node']->title;
-  $variables['!author'] = $params['author']->name;
-  switch($key) {
-  case 'comment_posted':
+    );
+    $variables['!topic_name'] = $params['node']->title;
+    $variables['!author'] = $params['author']->name;
+    switch($key) {
+    case 'comment_posted':
     $message['subject'] = bts('!site: comment posted to "!topic_name"', $variables,
-      $language->language);
+        $language->language);
     $message['body'][] = bts('!author has posted a reply to "!topic_name".',
-      $variables, $language->language);
+        $variables, $language->language);
     $message['body'][] = bts("To view this topic at !site, click here: \n!comment_url",
-      $variables, $language->language);
+        $variables, $language->language);
     break;
-  default:
+    default:
   }
 }

Spacing +3 added lines, -3 removed lines

@@ -30,9 +30,9 @@ 
   case 'insert':
     module_load_include('inc', 'drupal_queue', 'drupal_queue');
     $queue = DrupalQueue::get('flag_comment_notify');
-    $subscribed_users = array_keys((array) flag_get_content_flags('node', $a1['nid'], 'subscriptions'));
+    $subscribed_users = array_keys((array)flag_get_content_flags('node', $a1['nid'], 'subscriptions'));
     foreach ($subscribed_users as $uid) {
-      $queue->createItem(array('uid' => $uid, 'cid' => (int) $a1['cid']));
+      $queue->createItem(array('uid' => $uid, 'cid' => (int)$a1['cid']));
     }
     break;
   default:
@@ -86,7 +86,7 @@ 
   );
   $variables['!topic_name'] = $params['node']->title;
   $variables['!author'] = $params['author']->name;
-  switch($key) {
+  switch ($key) {
   case 'comment_posted':
     $message['subject'] = bts('!site: comment posted to "!topic_name"', $variables,
       $language->language);

Upper-Lower-Casing +2 added lines, -2 removed lines

@@ -61,7 +61,7 @@ 
   $comment = _comment_load($data['cid']);
   $node = node_load($comment->nid);
   $author = user_load($comment->uid);
-  if ($account->mail AND $node->nid) {
+  if ($account->mail and $node->nid) {
     // Don't send notifications to the author of the comment
     if ($account->uid != $author->uid) {
       $params['account'] = $account;
@@ -81,7 +81,7 @@ 
   $variables['!comment_url'] = url(
     "goto/comment/{$params['comment']->cid}",
     array(
-      'absolute' => TRUE,
+      'absolute' => true,
     )
   );
   $variables['!topic_name'] = $params['node']->title;

drupal/sites/default/boinc/modules/boincimport/includes/import_team.php

Indentation +46 added lines, -46 removed lines

@@ -11,56 +11,56 @@ 
  * at a time and avoid exhausting memory.
  */
 
-  require_once('./includes/bootstrap.inc');
-  drupal_bootstrap(DRUPAL_BOOTSTRAP_FULL);
-  require_boinc('db');
+    require_once('./includes/bootstrap.inc');
+    drupal_bootstrap(DRUPAL_BOOTSTRAP_FULL);
+    require_boinc('db');
   
-  // Parse arguments
-  $team_id = isset($argv[1]) ? $argv[1] : null;
-  $team_type_tid = isset($argv[2]) ? $argv[2] : null;
-  $input_format = isset($argv[3]) ? $argv[3] : null;
+    // Parse arguments
+    $team_id = isset($argv[1]) ? $argv[1] : null;
+    $team_type_tid = isset($argv[2]) ? $argv[2] : null;
+    $input_format = isset($argv[3]) ? $argv[3] : null;
   
-  $count = 0;
+    $count = 0;
   
-  // Get teams from BOINC
-  db_set_active('boinc');
-  $boincteam = db_fetch_object(db_query('SELECT * FROM team WHERE id=%d', array($team_id)));
-  $boincteam_members = db_query('SELECT id FROM user WHERE teamid=%d', array($team_id));
-  $boincteam_admin = (int) db_result(db_query('SELECT userid FROM team_admin WHERE teamid=%d', array($team_id)));
-  db_set_active('default');
+    // Get teams from BOINC
+    db_set_active('boinc');
+    $boincteam = db_fetch_object(db_query('SELECT * FROM team WHERE id=%d', array($team_id)));
+    $boincteam_members = db_query('SELECT id FROM user WHERE teamid=%d', array($team_id));
+    $boincteam_admin = (int) db_result(db_query('SELECT userid FROM team_admin WHERE teamid=%d', array($team_id)));
+    db_set_active('default');
   
-  $team_exists = db_query('SELECT team_id FROM {boincteam} WHERE team_id = %d', $boincteam->id);
-  // FIXME: $team_exists==FALSE should be handled as an error and return an error code!
+    $team_exists = db_query('SELECT team_id FROM {boincteam} WHERE team_id = %d', $boincteam->id);
+    // FIXME: $team_exists==FALSE should be handled as an error and return an error code!
 
-  if ($team_exists != FALSE && db_fetch_object($team_exists) == FALSE) {
+    if ($team_exists != FALSE && db_fetch_object($team_exists) == FALSE) {
     $boincteam->description = _boincimport_text_sanitize($boincteam->description);
     $teaser = node_teaser($boincteam->description);
     
     // Construct the team as an organic group node
     $node = array(
-      'type' => 'team',
-      'title' => $boincteam->name,
-      'body' => $boincteam->description,
-      'teaser' => $teaser,
-      'uid' => boincuser_lookup_uid($boincteam->userid),
-      'path' => null,
-      'status' => 1,  // published or not - always publish
-      'promote' => 0,
-      'created' => $boincteam->create_time,
-      'comment' => 0,  // comments disabled
-      'moderate' => 0,
-      'sticky' => 0,
-      'format' => $input_format
+        'type' => 'team',
+        'title' => $boincteam->name,
+        'body' => $boincteam->description,
+        'teaser' => $teaser,
+        'uid' => boincuser_lookup_uid($boincteam->userid),
+        'path' => null,
+        'status' => 1,  // published or not - always publish
+        'promote' => 0,
+        'created' => $boincteam->create_time,
+        'comment' => 0,  // comments disabled
+        'moderate' => 0,
+        'sticky' => 0,
+        'format' => $input_format
     );
     
     // Use pathauto function, if available, to clean up the path
     if (module_exists('pathauto')) {
-      module_load_include('inc', 'pathauto', 'pathauto');
-      $node['path'] = pathauto_cleanstring($boincteam->name);
+        module_load_include('inc', 'pathauto', 'pathauto');
+        $node['path'] = pathauto_cleanstring($boincteam->name);
     }
     else {
-      echo 'Pathauto module is required!';
-      exit;
+        echo 'Pathauto module is required!';
+        exit;
     }
     
     $node = (object) $node; // node_save requires an object form
@@ -72,22 +72,22 @@ 
     
     // Save the team IDs to a BOINC <--> Drupal reference table.
     db_query('INSERT INTO {boincteam} (team_id, nid) VALUES (%d, %d)', $boincteam->id, $node->nid);
-  }
+    }
   
-  // Determine team membership
-  db_set_active('boinc');
-  $boincteam_member_ids = array();
-  while ($boincuser = db_fetch_object($boincteam_members)) $boincteam_member_ids[] = $boincuser->id;
-  db_set_active('default');
-  if ($boincteam_member_ids) {
+    // Determine team membership
+    db_set_active('boinc');
+    $boincteam_member_ids = array();
+    while ($boincuser = db_fetch_object($boincteam_members)) $boincteam_member_ids[] = $boincuser->id;
+    db_set_active('default');
+    if ($boincteam_member_ids) {
     $team_members = db_query('SELECT uid FROM {boincuser} WHERE boinc_id IN(%s)', implode(',', $boincteam_member_ids));
     $team_admin = (int) db_result(db_query('SELECT uid FROM {boincuser} WHERE boinc_id=%d', $boincteam_admin));
     
     while ($drupal_user = db_fetch_object($team_members)) {
-      // Add action to take on member accounts?
-      $count++;
+        // Add action to take on member accounts?
+        $count++;
+    }
     }
-  }
   
-  echo $count;
-  
\ No newline at end of file
+    echo $count;
+    
\ No newline at end of file

Spacing +5 added lines, -5 removed lines

@@ -26,7 +26,7 @@ 
   db_set_active('boinc');
   $boincteam = db_fetch_object(db_query('SELECT * FROM team WHERE id=%d', array($team_id)));
   $boincteam_members = db_query('SELECT id FROM user WHERE teamid=%d', array($team_id));
-  $boincteam_admin = (int) db_result(db_query('SELECT userid FROM team_admin WHERE teamid=%d', array($team_id)));
+  $boincteam_admin = (int)db_result(db_query('SELECT userid FROM team_admin WHERE teamid=%d', array($team_id)));
   db_set_active('default');
   
   $team_exists = db_query('SELECT team_id FROM {boincteam} WHERE team_id = %d', $boincteam->id);
@@ -44,10 +44,10 @@ 
       'teaser' => $teaser,
       'uid' => boincuser_lookup_uid($boincteam->userid),
       'path' => null,
-      'status' => 1,  // published or not - always publish
+      'status' => 1, // published or not - always publish
       'promote' => 0,
       'created' => $boincteam->create_time,
-      'comment' => 0,  // comments disabled
+      'comment' => 0, // comments disabled
       'moderate' => 0,
       'sticky' => 0,
       'format' => $input_format
@@ -63,7 +63,7 @@ 
       exit;
     }
     
-    $node = (object) $node; // node_save requires an object form
+    $node = (object)$node; // node_save requires an object form
     
     $node->taxonomy[] = taxonomy_get_term($team_type_tid);
     
@@ -81,7 +81,7 @@ 
   db_set_active('default');
   if ($boincteam_member_ids) {
     $team_members = db_query('SELECT uid FROM {boincuser} WHERE boinc_id IN(%s)', implode(',', $boincteam_member_ids));
-    $team_admin = (int) db_result(db_query('SELECT uid FROM {boincuser} WHERE boinc_id=%d', $boincteam_admin));
+    $team_admin = (int)db_result(db_query('SELECT uid FROM {boincuser} WHERE boinc_id=%d', $boincteam_admin));
     
     while ($drupal_user = db_fetch_object($team_members)) {
       // Add action to take on member accounts?

Braces +4 added lines, -3 removed lines

@@ -57,8 +57,7 @@ 
     if (module_exists('pathauto')) {
       module_load_include('inc', 'pathauto', 'pathauto');
       $node['path'] = pathauto_cleanstring($boincteam->name);
-    }
-    else {
+    } else {
       echo 'Pathauto module is required!';
       exit;
     }
@@ -77,7 +76,9 @@ 
   // Determine team membership
   db_set_active('boinc');
   $boincteam_member_ids = array();
-  while ($boincuser = db_fetch_object($boincteam_members)) $boincteam_member_ids[] = $boincuser->id;
+  while ($boincuser = db_fetch_object($boincteam_members)) {
+      $boincteam_member_ids[] = $boincuser->id;
+  }
   db_set_active('default');
   if ($boincteam_member_ids) {
     $team_members = db_query('SELECT uid FROM {boincuser} WHERE boinc_id IN(%s)', implode(',', $boincteam_member_ids));

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
   $team_exists = db_query('SELECT team_id FROM {boincteam} WHERE team_id = %d', $boincteam->id);
   // FIXME: $team_exists==FALSE should be handled as an error and return an error code!
 
-  if ($team_exists != FALSE && db_fetch_object($team_exists) == FALSE) {
+  if ($team_exists != false && db_fetch_object($team_exists) == false) {
     $boincteam->description = _boincimport_text_sanitize($boincteam->description);
     $teaser = node_teaser($boincteam->description);
     

drupal/sites/default/boinc/modules/boincimport/includes/import_subscriptions.php

Indentation +19 added lines, -19 removed lines

@@ -12,33 +12,33 @@
  */
 
 
-  require_once('./includes/bootstrap.inc');
-  drupal_bootstrap(DRUPAL_BOOTSTRAP_FULL);
-  require_boinc('db');
+    require_once('./includes/bootstrap.inc');
+    drupal_bootstrap(DRUPAL_BOOTSTRAP_FULL);
+    require_boinc('db');
   
-  // Parse arguments
-  $record_offset = isset($argv[1]) ? $argv[1] : 0;
-  $chunk_size = isset($argv[2]) ? $argv[2] : 100;
+    // Parse arguments
+    $record_offset = isset($argv[1]) ? $argv[1] : 0;
+    $chunk_size = isset($argv[2]) ? $argv[2] : 100;
   
-  // Construct sql conditions
-  $limit = sprintf('LIMIT %d,%d', $record_offset, $chunk_size);
+    // Construct sql conditions
+    $limit = sprintf('LIMIT %d,%d', $record_offset, $chunk_size);
   
-  $total_count = 0;
+    $total_count = 0;
   
-  // Get the users with subscriptions to import
-  db_set_active('boinc');
-  $subscribed_boinc_users = db_query("
+    // Get the users with subscriptions to import
+    db_set_active('boinc');
+    $subscribed_boinc_users = db_query("
     SELECT DISTINCT userid FROM {subscriptions}
     ORDER BY userid ASC %s", $limit
-  );
-  db_set_active('default');
+    );
+    db_set_active('default');
   
-  // Import subscriptions
-  while ($boinc_subscription = db_fetch_object($subscribed_boinc_users)) {
+    // Import subscriptions
+    while ($boinc_subscription = db_fetch_object($subscribed_boinc_users)) {
     $uid = get_drupal_id($boinc_subscription->userid);
     $count = boincuser_pull_subscriptions($uid);
     $total_count += $count;
     echo "\nuser: {$uid}; boinc_id: {$boinc_subscription->userid}; {$count} subscriptions";
-  }
-  echo "\n";
-  echo $total_count;
\ No newline at end of file
+    }
+    echo "\n";
+    echo $total_count;
\ No newline at end of file

drupal/sites/default/boinc/modules/boincimport/includes/import_users.php

Indentation +22 added lines, -22 removed lines

@@ -12,26 +12,26 @@ 
  */
 
 
-  require_once('./includes/bootstrap.inc');
-  drupal_bootstrap(DRUPAL_BOOTSTRAP_FULL);
-  require_boinc('db');
+    require_once('./includes/bootstrap.inc');
+    drupal_bootstrap(DRUPAL_BOOTSTRAP_FULL);
+    require_boinc('db');
   
-  // Parse arguments
-  $import_lurkers = isset($argv[1]) ? $argv[1] : false;
-  $record_offset = isset($argv[2]) ? $argv[2] : 0;
-  $chunk_size = isset($argv[3]) ? $argv[3] : 100;
+    // Parse arguments
+    $import_lurkers = isset($argv[1]) ? $argv[1] : false;
+    $record_offset = isset($argv[2]) ? $argv[2] : 0;
+    $chunk_size = isset($argv[3]) ? $argv[3] : 100;
   
-  // Construct sql conditions
-  $limit = sprintf('LIMIT %d,%d', $record_offset, $chunk_size);
+    // Construct sql conditions
+    $limit = sprintf('LIMIT %d,%d', $record_offset, $chunk_size);
   
-  $count = 0;
+    $count = 0;
   
-  db_set_active('boinc');
-  if ($import_lurkers) {
+    db_set_active('boinc');
+    if ($import_lurkers) {
     $boinc_accounts = db_query('SELECT id FROM user ORDER BY id %s', $limit);
-  }
-  //else $boinc_accounts = db_query('SELECT DISTINCT user AS boinc_id FROM post ORDER BY boinc_id %s', $limit);
-  else {
+    }
+    //else $boinc_accounts = db_query('SELECT DISTINCT user AS boinc_id FROM post ORDER BY boinc_id %s', $limit);
+    else {
     // Need to import any user who is currently ignored in order to keep them
     // ignored... not particularly clean (ignored users are stored in a string)
     $ignored_user_list = array();
@@ -42,7 +42,7 @@ 
       ORDER BY userid ASC"
     );
     while ($ignoring_user = db_fetch_object($ignoring_users)) {
-      $ignored_user_list = $ignored_user_list + array_fill_keys(explode('|', trim($ignoring_user->ignorelist, '|')), 1);
+        $ignored_user_list = $ignored_user_list + array_fill_keys(explode('|', trim($ignoring_user->ignorelist, '|')), 1);
     }
     $ignored_user_list = array_keys($ignored_user_list);
     $boinc_accounts = db_query("
@@ -58,14 +58,14 @@ 
         (SELECT DISTINCT senderid FROM {private_messages})
       ) AS usersToImport ORDER BY id ASC %s", implode(',', $ignored_user_list), $limit
     );
-  }
-  db_set_active('default');
+    }
+    db_set_active('default');
   
-  while ($boinc_account = db_fetch_object($boinc_accounts)) {
+    while ($boinc_account = db_fetch_object($boinc_accounts)) {
     // Grab the BOINC user object and create a Drupal user from it
     if (boincuser_register_make_drupal_user($boinc_account->id)) {
-      $count++;
+        $count++;
+    }
     }
-  }
   
-  echo $count;
\ No newline at end of file
+    echo $count;
\ No newline at end of file

drupal/sites/default/boinc/modules/boincimport/boincimport.pages.inc

Indentation +185 added lines, -185 removed lines

@@ -5,45 +5,45 @@ 
  * Callback admin/boinc/import/settings
  */
 function boincimport_admin_settings() {
-  global $conf ;
-  global $db_url;
-  global $base_url;
-  $stage = variable_get('boincimport_config_stage', 0); 
+    global $conf ;
+    global $db_url;
+    global $base_url;
+    $stage = variable_get('boincimport_config_stage', 0); 
   
-  // Start with a quick sanity check on the BOINC environment
-  boinc_get_path();
+    // Start with a quick sanity check on the BOINC environment
+    boinc_get_path();
   
-  //We'll be checking that the setup is complete.
-  //If it is not complete, the migration will not go ahead.
-  $ready_for_migration = 1;
+    //We'll be checking that the setup is complete.
+    //If it is not complete, the migration will not go ahead.
+    $ready_for_migration = 1;
 
-  // Checking that the migration has been tested on a copy first.
-  $form['test'] = array(
+    // Checking that the migration has been tested on a copy first.
+    $form['test'] = array(
     '#type' => 'fieldset',
     '#title' => t('Test on copy first'),
     '#collapsible' => TRUE,
     '#collapsed' => TRUE,
-  );
+    );
     
-  $output = '';
-  if (!variable_get('boincimport_tested', 0)) {
+    $output = '';
+    if (!variable_get('boincimport_tested', 0)) {
     $form['test']['#collapsed']= FALSE;
     $output = '<span class="marker">';
     $ready_for_migration = 0;
-  }
-  $output .= '<p>' . t('The migration will modify and even destroy some data. You
+    }
+    $output .= '<p>' . t('The migration will modify and even destroy some data. You
     must test this module on a copy of your data first, several times if necessary.
     Only use this module on your live data when you are familiar with the procedure
     and you\'re confident everything will run smoothly.') . '</p><p>' . t('Also, you
     should make a full backup of your database just prior to performing the
     migration on the live data. It is better to turn off both the BOINC site and the
     Drupal site during the migration.') . '</p>';
-  if (!variable_get('boincimport_tested', 0)) {
+    if (!variable_get('boincimport_tested', 0)) {
     $output .= '</span>';
-  }
-  $form['test']['intro'] = array('#value' => $output);
+    }
+    $form['test']['intro'] = array('#value' => $output);
 
-  $form['test']['boincimport_tested'] = array(
+    $form['test']['boincimport_tested'] = array(
     '#type' => 'checkbox',
     '#title' => t('I am running a trial migration on a disposable copy of the database
       OR I have already done so and I am running this on my live site, confident that
@@ -51,64 +51,64 @@ 
     '#return_value' => 1,
     '#default_value' => variable_get('boincimport_tested', 0),
     '#description' => '',
-  );
+    );
   
-  // Where is BOINC data?
-  $pre = variable_get('boincimport_table_prefix', '');
-  $tables = array($pre .'forum', $pre .'post', $pre .'thread', $pre .'user');
-  $db = (is_array($db_url) AND isset($db_url['boinc'])) ? 'boinc' : 'default';
-  $result = _boincimport_check_tables($tables, $db, 0);
+    // Where is BOINC data?
+    $pre = variable_get('boincimport_table_prefix', '');
+    $tables = array($pre .'forum', $pre .'post', $pre .'thread', $pre .'user');
+    $db = (is_array($db_url) AND isset($db_url['boinc'])) ? 'boinc' : 'default';
+    $result = _boincimport_check_tables($tables, $db, 0);
   
-  $config_path = conf_path();
-  $form['boinc_data'] = array(
+    $config_path = conf_path();
+    $form['boinc_data'] = array(
     '#type' => 'fieldset',
     '#title' => t('Location of BOINC data'),
     '#collapsible' => TRUE,
     '#collapsed' => TRUE,
-  );
-  $form['boinc_data']['boincimport_db_configured'] = array(
+    );
+    $form['boinc_data']['boincimport_db_configured'] = array(
     '#type' => 'checkbox',
     '#title' => t('There is a \'boinc\' database connection string configured in %file',
-      array('%file' => "{$config_path}/settings.php")),
+        array('%file' => "{$config_path}/settings.php")),
     '#return_value' => 1,
     '#default_value' => variable_get('boincimport_db_configured', 1),
     '#description' => '',
     '#disabled' => ($result['result']) ? TRUE : FALSE
-  );
-  if (variable_get('boincimport_db_configured', 1) != 1) {
+    );
+    if (variable_get('boincimport_db_configured', 1) != 1) {
     $default_db_url = is_array($db_url) ? end($db_url) : $db_url;
     $url = variable_get('boincimport_db_url', $default_db_url);
     if (!preg_match('{(mysql|pgsql):\/\/(.+):(.*?)@(.+)/(\w+)}i', $url)) {
-      $output = '<p class="marker">'. t('Set the address of the BOINC database
+        $output = '<p class="marker">'. t('Set the address of the BOINC database
         according to the following format:') . '<br /><pre>
         mysql://boinc_username:boinc_password@localhost/boinc_database
         </pre></p>';
-      $ready_for_migration = 0;
-      $form['boinc_data']['#collapsed'] = FALSE;
-      $form['boinc_data']['db_connect_string_note'] = array('#value' => $output);
+        $ready_for_migration = 0;
+        $form['boinc_data']['#collapsed'] = FALSE;
+        $form['boinc_data']['db_connect_string_note'] = array('#value' => $output);
     }
 
     $form['boinc_data']['boincimport_db_url'] = array(
-      '#type' => 'textfield',
-      '#title' => t('BOINC database location'),
-      '#default_value' => variable_get('boincimport_db_url', $default_db_url),
-      '#size' => 80,
-      '#maxlength' => 255,
-      '#description' => '',
+        '#type' => 'textfield',
+        '#title' => t('BOINC database location'),
+        '#default_value' => variable_get('boincimport_db_url', $default_db_url),
+        '#size' => 80,
+        '#maxlength' => 255,
+        '#description' => '',
     );
-  }
+    }
 
-  $output = '<p>' . t('Note that due to a limitation within Drupal core, we cannot
+    $output = '<p>' . t('Note that due to a limitation within Drupal core, we cannot
     import messages from one database type to another database type. E.g, we
     cannot import BOINC from a myqsl DB, to a postgresql DB. You need to import
     the data into the same type of database first (import all the BOINC data, as is,
     from mysql to postgresql).') . '</p><p>' . t('If both BOINC and Drupal are on
     postgresql, see !this_issue.', array('!this_issue' => l(t('this issue'),
     'http://drupal.org/node/64877', array('attributes' => array('target' => '_blank'))))) . '</p>';
-  $form['boinc_data']['note'] = array('#value' => $output);
+    $form['boinc_data']['note'] = array('#value' => $output);
   
-  // BOINC table prefix
-  $form['boinc_data']['boincimport_table_prefix'] = array(
+    // BOINC table prefix
+    $form['boinc_data']['boincimport_table_prefix'] = array(
     '#type' => 'textfield',
     '#title' => t('BOINC table prefix'),
     '#default_value' => variable_get('boincimport_table_prefix', ''),
@@ -116,30 +116,30 @@ 
     '#maxlength' => 255,
     '#description' => t('What prefix is used for your BOINC tables?'),
     '#disabled' => ($result['result']) ? TRUE : FALSE
-  );
+    );
   
-  $output = $result['html'];
-  if ($result['result'] != 1) {
+    $output = $result['html'];
+    if ($result['result'] != 1) {
     $ready_for_migration = 0;
     $form['boinc_data']['#collapsed']= FALSE;
     $output .= '<span class="marker">';
     $output .= t('Some tables couldn\'t be found. Are the database settings correct?
       Please correct the errors so that the BOINC tables can be accessed.');
     $output .= '</span>';
-  }
+    }
 
-  $form['boinc_data']['result'] = array('#value' => $output);
+    $form['boinc_data']['result'] = array('#value' => $output);
   
-  // USERS settings
-  $form['users'] = array(
+    // USERS settings
+    $form['users'] = array(
     '#type' => 'fieldset',
     '#title' => t('User import'),
     '#collapsible' => TRUE,
     '#collapsed' => TRUE,
-  );
+    );
 
-  // User import batch size
-  $form['users']['boincimport_user_batch_size'] = array(
+    // User import batch size
+    $form['users']['boincimport_user_batch_size'] = array(
     '#type' => 'textfield',
     '#title' => t('Number of accounts to process at a time'),
     '#default_value' => variable_get('boincimport_user_batch_size', 50),
@@ -148,10 +148,10 @@ 
     '#description' => t('Due to memory intensive API calls, this value may have to
       be adjusted to avoid exhausting system resources. The default value is a
       balance between performance and reliability.')
-  );
+    );
 
-  // Import users who have never posted?
-  $form['users']['boincimport_import_lurkers'] = array(
+    // Import users who have never posted?
+    $form['users']['boincimport_import_lurkers'] = array(
     '#type' => 'checkbox',
     '#title' => t('Import users who have never posted?'),
     '#return_value' => 1,
@@ -160,199 +160,199 @@ 
       involvement? This may not be desired, as it can take a very long time to
       import tens or hundreds of thousands of accounts. (note that users who
       are not imported now will still be imported on-the-fly upon login)'),
-  );
+    );
 
-  // Input format settings
-  $form['input'] = array(
+    // Input format settings
+    $form['input'] = array(
     '#type' => 'fieldset',
     '#title' => t('Input format settings'),
     '#collapsible' => TRUE,
     '#collapsed' => TRUE
-  );
-  $output = '<p>'. t('Select which !input_format should be used for the imported
+    );
+    $output = '<p>'. t('Select which !input_format should be used for the imported
     messages. It is highly-recommended that this value be left as it is, as the
     Full HTML and PHP code formats can be very dangerous!', 
     array('!input_format' => l(t('input format'), 'admin/settings/filters'))) .'</p>';
-  $form['input']['intro'] = array('#value' => $output);
+    $form['input']['intro'] = array('#value' => $output);
   
-  $formats = db_query('SELECT format, name FROM {filter_formats}');
-  while ($format = db_fetch_array($formats)) {
+    $formats = db_query('SELECT format, name FROM {filter_formats}');
+    while ($format = db_fetch_array($formats)) {
     $input_formats[$format['format']]= $format['name'];
-  }
+    }
   
-  // By default, use the format named "Rich text"; fall back to Filtered HTML
-  $default_input_format = db_result(db_query("
+    // By default, use the format named "Rich text"; fall back to Filtered HTML
+    $default_input_format = db_result(db_query("
     SELECT format FROM {filter_formats} WHERE name = '%s'", 'Rich text'));
-  if (!$default_input_format) $default_input_format = 1;
+    if (!$default_input_format) $default_input_format = 1;
   
-  $input_format = variable_get('boincimport_input_format', $default_input_format);
-  $form['input']['boincimport_input_format'] = array(
+    $input_format = variable_get('boincimport_input_format', $default_input_format);
+    $form['input']['boincimport_input_format'] = array(
     '#type' => 'radios',
     '#title' => t('Input format'),
     '#default_value' => $input_format,
     '#options' => $input_formats
-  );
-  if (!$input_format) {
+    );
+    if (!$input_format) {
     $ready_for_migration = 0;
     $form['input']['#collapsed']= FALSE;
-  }
-  $output = '<p><em>'. t('This setting will be applied to all posts and private
+    }
+    $output = '<p><em>'. t('This setting will be applied to all posts and private
     messages.') .'</em></p>';
-  $form['input']['result'] = array('#value' => $output);
+    $form['input']['result'] = array('#value' => $output);
   
-  // BBcode settings
-  $form['bbcode'] = array(
+    // BBcode settings
+    $form['bbcode'] = array(
     '#type' => 'fieldset',
     '#title' => t('BBcode settings'),
     '#collapsible' => TRUE,
     '#collapsed' => TRUE,
-  );
+    );
   
-  $output = '<p>' . t('BOINC posts are often formatted in BBcode. During the
+    $output = '<p>' . t('BOINC posts are often formatted in BBcode. During the
     import process, BBcode can be handled in two diffent ways') . ':</p>';
-  $output .= '<ul><li>' . t('Leave the BBcode as is and use the !bbcode_module
+    $output .= '<ul><li>' . t('Leave the BBcode as is and use the !bbcode_module
     after the migration to filter the imported posts. In this case, the module
     is not required during the migration, but will need to be installed and
     configured later.',
     array('!bbcode_module' => l(t('bbcode module'), 'http://drupal.org/project/bbcode',
     array('attributes' => array('target' => '_blank'))))) . '</li>';
-  $output .= '<li>' . t('Filter the BBcode now and replace it with plain HTML.
+    $output .= '<li>' . t('Filter the BBcode now and replace it with plain HTML.
     The BBcode module will be required during the migration process, but can
     be disabled afterwards.') . '</li></ul>';
-  $output .= '<p>' . t('An appropriate input filter should be chosen, as well,
+    $output .= '<p>' . t('An appropriate input filter should be chosen, as well,
     as it will affect how the post is displayed. For example, if BBcode is
     removed and replaced by HTML, some HTML tags will be stripped by the
     "Filtered HTML" input format unless they are first added to the list of
     allowed tags in that format. See the !input_formats page.',
     array('!input_formats' => l(t('input formats'), 'admin/settings/filters'))) . '</p>';
     
-  $form['bbcode']['intro'] = array('#value' => $output);
-  $form['bbcode']['boincimport_bbcode'] = array(
+    $form['bbcode']['intro'] = array('#value' => $output);
+    $form['bbcode']['boincimport_bbcode'] = array(
     '#type' => 'radios',
     '#title' => t('BBcode handling'),
     '#default_value' => variable_get('boincimport_bbcode', 0),
     '#options' => array(t('Leave BBcode as is.'), t('Transform BBcode into HTML.')),
     '#description' => '',
-  );
+    );
   
-  $module_status = _boincimport_check_module('bbcode');
-  $bbcode_disabled = ($module_status['result'] == 0) ? true : false;
-  $output = $module_status['html'];
+    $module_status = _boincimport_check_module('bbcode');
+    $bbcode_disabled = ($module_status['result'] == 0) ? true : false;
+    $output = $module_status['html'];
   
-  if (variable_get('boincimport_bbcode', 0)) {
+    if (variable_get('boincimport_bbcode', 0)) {
     if ($bbcode_disabled) {
-      $ready_for_migration = 0;
-      $form['bbcode']['#collapsed'] = FALSE;
-      $output .= '<p><span class="marker">';
-      $output .= t('The !bbcode_module is required to transform BBcode into HTML.
+        $ready_for_migration = 0;
+        $form['bbcode']['#collapsed'] = FALSE;
+        $output .= '<p><span class="marker">';
+        $output .= t('The !bbcode_module is required to transform BBcode into HTML.
         You first need to !enable_bbcode', array(
-          '!bbcode_module' => l(t('bbcode module'), 'http://drupal.org/project/bbcode', array('attributes' => array('target' => '_blank'))),
-          '!enable_bbcode' => l(t('enable the bbcode module'), 'admin/build/modules')));
-      $output .= '</span></p>';
+            '!bbcode_module' => l(t('bbcode module'), 'http://drupal.org/project/bbcode', array('attributes' => array('target' => '_blank'))),
+            '!enable_bbcode' => l(t('enable the bbcode module'), 'admin/build/modules')));
+        $output .= '</span></p>';
     }
     $sql = "SELECT COUNT(*) FROM {filters} WHERE module = 'bbcode' AND format = %d";
     $result = db_query($sql, $input_format);
     if (!db_result($result)) {
-      $form['bbcode']['#collapsed'] = FALSE;
-      $output .= '<p class="marker">'. t('You need to !enable_bbcode_input and
+        $form['bbcode']['#collapsed'] = FALSE;
+        $output .= '<p class="marker">'. t('You need to !enable_bbcode_input and
         then !configure_bbcode_options', array(
-          '!enable_bbcode_input' =>  l(t('enable BBcode in the selected input format'), "admin/settings/filters/{$input_format}"),
-          '!configure_bbcode_options' => l(t('configure BBcode options'), "admin/settings/filters/{$input_format}/configure"))) .'</p>';
+            '!enable_bbcode_input' =>  l(t('enable BBcode in the selected input format'), "admin/settings/filters/{$input_format}"),
+            '!configure_bbcode_options' => l(t('configure BBcode options'), "admin/settings/filters/{$input_format}/configure"))) .'</p>';
     }
     $output .= '<p>'. t('For best results, !disable_all_options', 
     array('!disable_all_options' => l(t('all four BBcode configure options should be disabled'), "admin/settings/filters/{$input_format}/configure"))) . '.</p>';
-  } else {
+    } else {
     if ($bbcode_disabled) {
-      $form['bbcode']['#collapsed'] = FALSE;
-      $output .= '<p>'. t('The !bbcode_module is recommended, but not required.
+        $form['bbcode']['#collapsed'] = FALSE;
+        $output .= '<p>'. t('The !bbcode_module is recommended, but not required.
         You may !enable_bbcode after having installed it.', array(
-          '!bbcode_module' => l(t('bbcode module'), 'http://drupal.org/project/bbcode', array('attributes' => array('target' => '_blank'))),
-          '!enable_bbcode' => l(t('enable the bbcode module'), 'admin/build/modules'))) .'</p>';
+            '!bbcode_module' => l(t('bbcode module'), 'http://drupal.org/project/bbcode', array('attributes' => array('target' => '_blank'))),
+            '!enable_bbcode' => l(t('enable the bbcode module'), 'admin/build/modules'))) .'</p>';
     }
     $output .= '<p>' . t('Don\'t forget to !enable_bbcode_input and then !configure_bbcode_options.', array(
-      '!enable_bbcode_input' =>  l(t('enable BBcode in the selected input format'), "admin/settings/filters/{$input_format}"),
-      '!configure_bbcode_options' => l(t('configure BBcode options'), "admin/settings/filters/{$input_format}/configure"))) . '</p>';
+        '!enable_bbcode_input' =>  l(t('enable BBcode in the selected input format'), "admin/settings/filters/{$input_format}"),
+        '!configure_bbcode_options' => l(t('configure BBcode options'), "admin/settings/filters/{$input_format}/configure"))) . '</p>';
     $output .= '<p>' . t('You may want to enable it for other input formats, too.') . '</p>';
-  }
-  $output .= '<p><em>'. t('This setting will be applied to all posts, private
+    }
+    $output .= '<p><em>'. t('This setting will be applied to all posts, private
     messages and the users\' signature.') .'</em></p>';
     
-  $form['bbcode']['result'] = array('#value' => $output);
+    $form['bbcode']['result'] = array('#value' => $output);
   
-  // URL transform settings
-  $form['url_transforms'] = array(
+    // URL transform settings
+    $form['url_transforms'] = array(
     '#type' => 'fieldset',
     '#title' => t('URL transforms'),
     '#collapsible' => TRUE,
     '#collapsed' => TRUE,
-  );
-  if (!variable_get('boincimport_base_url_boinc', 0)) {
+    );
+    if (!variable_get('boincimport_base_url_boinc', 0)) {
     $form['url_transforms']['#collapsed']= FALSE;
     $output = '<span class="marker">';
     $ready_for_migration = 0;
-  }
+    }
 
-  // List of BOINC domains possible in URLs to transform
-  $form['url_transforms']['boincimport_base_url_boinc'] = array(
+    // List of BOINC domains possible in URLs to transform
+    $form['url_transforms']['boincimport_base_url_boinc'] = array(
     '#type' => 'textarea',
     '#title' => t('BOINC base URLs to transform'),
     '#default_value' => variable_get('boincimport_base_url_boinc', ''),
     '#description' => t('List URL bases that should be transformed from
       BOINC format to Drupal format (enter one domain per line, including http://).'),
-  );
+    );
 
-  // The Drupal domain to use in URL transform results
-  $form['url_transforms']['boincimport_base_url_drupal'] = array(
+    // The Drupal domain to use in URL transform results
+    $form['url_transforms']['boincimport_base_url_drupal'] = array(
     '#type' => 'textfield',
     '#title' => t('Target Drupal base URL'),
     '#default_value' => variable_get('boincimport_base_url_drupal', ''),
     '#description' => t('Specify the URL base to use in transform results. If
       left blank, the base_url configured in this Drupal environment will be
       used (currently: %url)', array('%url' =>$base_url)),
-  );
+    );
   
-  // Are we ready for migration?
-  $form['migration'] = array(
+    // Are we ready for migration?
+    $form['migration'] = array(
     '#type' => 'fieldset',
     '#title' => 'Migration',
     '#collapsible' => TRUE,
     '#collapsed' => FALSE,
-  );
-  variable_set('boincimport_ready', $ready_for_migration);
-  if ($ready_for_migration) {
+    );
+    variable_set('boincimport_ready', $ready_for_migration);
+    if ($ready_for_migration) {
     $output = '<p>'. t('Check the settings above one more time. If everything
       is ok, !proceed.', array('!proceed' => l(t('you can now proceed with the migration'), 'admin/boinc/import/process'))) .'</p>';
-  } else {
+    } else {
     $output = '<p class="marker">'. t('You are not ready for the migration yet.
       Please check the settings above.') .'</p>';
-  }
+    }
   
-  $form['migration']['info'] = array('#value' => $output);
-  return system_settings_form($form); 
+    $form['migration']['info'] = array('#value' => $output);
+    return system_settings_form($form); 
 }
 
 /**
  * Callback admin/boinc/import/post_configuration
  */
 function boincimport_post_configuration() {
-  $output = '<p>'. t('This page will help to tie up any loose ends following
+    $output = '<p>'. t('This page will help to tie up any loose ends following
     the BOINC data import process. Before continuing, it may be best to have
     already:') .'</p>';
-  $output .= '<ul>';
-  $output .= '<li>'. t('Imported users, teams, and forums using the BOINC
+    $output .= '<ul>';
+    $output .= '<li>'. t('Imported users, teams, and forums using the BOINC
     import module') .'</li>';
-  $output .= '<li>'. t('Imported any additional data using add on modules if
+    $output .= '<li>'. t('Imported any additional data using add on modules if
     available.') .'</li>';
-  $output .= '</ul>';
-  //User Set up
-  $output .= '<h2>'. t('User settings') .'</h2>';  
-  $output .= '<p>'. t('Make sure your !user_settings are correct. Specifically,
+    $output .= '</ul>';
+    //User Set up
+    $output .= '<h2>'. t('User settings') .'</h2>';  
+    $output .= '<p>'. t('Make sure your !user_settings are correct. Specifically,
     you can enable or disable signatures there. ', array(
-      '!user_settings' => l(t('user settings'), 'admin/user/settings')
+        '!user_settings' => l(t('user settings'), 'admin/user/settings')
     )) .'</p>'; 
-  $output .= '<p>' . t('Note that avatar/picture support is not tied to user
+    $output .= '<p>' . t('Note that avatar/picture support is not tied to user
     settings Picture support at this time.') . '</p>';
-  /*$output .= '<p>' . t('In order to activate avatar/picture support, you need to
+    /*$output .= '<p>' . t('In order to activate avatar/picture support, you need to
     enable pictures on this page and also:') . '</p>';
   $output .= '<ul>';
   $output .= '<li>'. t('Tick the checkboxes to allow "User pictures in posts" and/or
@@ -365,73 +365,73 @@ 
     )) .'</li>';
   $output .= '</ul>'; */
   
-  // Forum Set up
-  $output .= '<h2>'. t('Forum settings') .'</h2>';
-  $output .= '<p>'. t('Check any additional changes to make on the
+    // Forum Set up
+    $output .= '<h2>'. t('Forum settings') .'</h2>';
+    $output .= '<p>'. t('Check any additional changes to make on the
     !forum_settings page.', array(
-      '!forum_settings' =>  l(t('forum settings'), 'admin/content/forum/settings')
+        '!forum_settings' =>  l(t('forum settings'), 'admin/content/forum/settings')
     )) .'</p>';
-  $output .= '<p>'. t('It also may be a good idea to look at the !forum_list to
+    $output .= '<p>'. t('It also may be a good idea to look at the !forum_list to
     verify that all necessary containers and forums are in place.', array(
-      '!forum_list' =>  l(t('forum list'), 'admin/content/forum')
+        '!forum_list' =>  l(t('forum list'), 'admin/content/forum')
     )) .'</p>';
-  /*$output .= '<p>'. t('If need be, set up the private forums as explained in the
+    /*$output .= '<p>'. t('If need be, set up the private forums as explained in the
     !documentation.', array(
       '!documentation' => l(t('documentation'), 'http://drupal.org/node/58969', array('attributes' => array('target' => '_blank')))
     )) .'</p>';
     */
-  /*
+    /*
   // Redirection from old URLs:
   $output .= '<h2>'. t('Redirection from BOINC forum URLs to new Drupal forum URLs') .'</h2>';
   $output .= '<p>'. t('Enable the boincforum_redirect.module. This way, users
     accessing URLs from your old BOINC installation will be redirected to your new
     Drupal forum.') .'</p>';
   */
-  // BBcode
-  $bbcode = variable_get('boincimport_bbcode', 0);
-  if ($bbcode) {
+    // BBcode
+    $bbcode = variable_get('boincimport_bbcode', 0);
+    if ($bbcode) {
     $output .= '<h2>'. t('Remove BBcode module') .'</h2>';
     $output .= '<p>'. t('The BBcode has been cleaned from the posts. You shouldn\'t
       need the bbcode module anymore. You can disable it.') .'</p>';
-  } else {
+    } else {
     $output .= '<h2>'. t('Configure BBcode module') .'</h2>';
     $output .= '<p>'. t('Most likely, there are many BOINC forum postings
       formatted in BBcode. It would be a good idea to verify that BBcode is
       being formatted correctly in the imported posts. Have a look around
       !the_forum.', array(
         '!the_forum' =>  l(t('the forum'), 'community')
-      )) . '</p>';
-  }
+        )) . '</p>';
+    }
 
-  return $output;
+    return $output;
 }
 
 /**
  * Callback admin/boinc/import
  */
 function boincimport_main() {
-  // Start with a quick sanity check on the BOINC environment
-  boinc_get_path();
-  // If that's ok, on with the show
-  $output .= t('You may want to !check_documentation before starting.', array(
+    // Start with a quick sanity check on the BOINC environment
+    boinc_get_path();
+    // If that's ok, on with the show
+    $output .= t('You may want to !check_documentation before starting.', array(
     '!check_documentation' => l(t('check the documentation'), 'http://boinc.berkeley.edu/trac/wiki/InstallDrupal', array('attributes' =>array('target' => '_blank')))
-  ));
-  $output = '<ol><li>';
-  $output .= l(t('Configure the migration'), 'admin/boinc/import/settings');
-  $output .= '</li>';
-  $output .= '<li>';
-  $output .= l(t('Execute the migration'), 'admin/boinc/import/process');
-  $output .= '</li>';
-  $output .= '<li>';
-  $output .= l(t('Post migration configuration'), 'admin/boinc/import/post_configuration');
-  $output .= '</li>';
-  $output .= '<li>';
-  $output .= t('!cleanup (Accessing this link will delete all the saved BOINC forum
+    ));
+    $output = '<ol><li>';
+    $output .= l(t('Configure the migration'), 'admin/boinc/import/settings');
+    $output .= '</li>';
+    $output .= '<li>';
+    $output .= l(t('Execute the migration'), 'admin/boinc/import/process');
+    $output .= '</li>';
+    $output .= '<li>';
+    $output .= l(t('Post migration configuration'), 'admin/boinc/import/post_configuration');
+    $output .= '</li>';
+    $output .= '<li>';
+    $output .= t('!cleanup (Accessing this link will delete all the saved BOINC forum
     conversion settings: click only when you\'re done with all the above.)', array(
-      '!cleanup' => l(t('Cleanup'), 'admin/boinc/import/cleanup')
+        '!cleanup' => l(t('Cleanup'), 'admin/boinc/import/cleanup')
     ));
-  $output .= '</li>';
-  /*$output .= '<li>';
+    $output .= '</li>';
+    /*$output .= '<li>';
   $output .= t('Enable the boincforum_redirect module for seamless redirection
     from the old forums to the new. This module requires !clean_urls to be active.', array(
       '!clean_urls' => l(t('Clean URLs'), 'admin/settings/clean-urls')
@@ -442,14 +442,14 @@ 
     uninstall BOINC forum - This will delete tables necessary for redirection. You can,
     however, deactivate the module safely.');
   $output .= '</li>';*/
-  $output .= '</ol>';
+    $output .= '</ol>';
 
-  // process will die() if the link to the BOINC database is wrong:
-  $output .= '<p>'. t('If the BOINC data is in another database, but you made a
+    // process will die() if the link to the BOINC database is wrong:
+    $output .= '<p>'. t('If the BOINC data is in another database, but you made a
     mistake while setting up the data base url, you may be unable to access the
     setting page due to some limitations of the core of Drupal. !reset_db_url.', array(
-      '!reset_db_url' => l(t('Click here to reset the database url'), 'admin/boinc/import/reset')
+        '!reset_db_url' => l(t('Click here to reset the database url'), 'admin/boinc/import/reset')
     )) .'</p>';
 
-  return $output;
+    return $output;
 }

Spacing +66 added lines, -66 removed lines

@@ -5,7 +5,7 @@ 
  * Callback admin/boinc/import/settings
  */
 function boincimport_admin_settings() {
-  global $conf ;
+  global $conf;
   global $db_url;
   global $base_url;
   $stage = variable_get('boincimport_config_stage', 0); 
@@ -27,17 +27,17 @@ 
     
   $output = '';
   if (!variable_get('boincimport_tested', 0)) {
-    $form['test']['#collapsed']= FALSE;
+    $form['test']['#collapsed'] = FALSE;
     $output = '<span class="marker">';
     $ready_for_migration = 0;
   }
-  $output .= '<p>' . t('The migration will modify and even destroy some data. You
+  $output .= '<p>'.t('The migration will modify and even destroy some data. You
     must test this module on a copy of your data first, several times if necessary.
     Only use this module on your live data when you are familiar with the procedure
-    and you\'re confident everything will run smoothly.') . '</p><p>' . t('Also, you
+    and you\'re confident everything will run smoothly.').'</p><p>'.t('Also, you
     should make a full backup of your database just prior to performing the
     migration on the live data. It is better to turn off both the BOINC site and the
-    Drupal site during the migration.') . '</p>';
+    Drupal site during the migration.').'</p>';
   if (!variable_get('boincimport_tested', 0)) {
     $output .= '</span>';
   }
@@ -55,7 +55,7 @@ 
   
   // Where is BOINC data?
   $pre = variable_get('boincimport_table_prefix', '');
-  $tables = array($pre .'forum', $pre .'post', $pre .'thread', $pre .'user');
+  $tables = array($pre.'forum', $pre.'post', $pre.'thread', $pre.'user');
   $db = (is_array($db_url) AND isset($db_url['boinc'])) ? 'boinc' : 'default';
   $result = _boincimport_check_tables($tables, $db, 0);
   
@@ -79,8 +79,8 @@ 
     $default_db_url = is_array($db_url) ? end($db_url) : $db_url;
     $url = variable_get('boincimport_db_url', $default_db_url);
     if (!preg_match('{(mysql|pgsql):\/\/(.+):(.*?)@(.+)/(\w+)}i', $url)) {
-      $output = '<p class="marker">'. t('Set the address of the BOINC database
-        according to the following format:') . '<br /><pre>
+      $output = '<p class="marker">'.t('Set the address of the BOINC database
+        according to the following format:').'<br /><pre>
         mysql://boinc_username:boinc_password@localhost/boinc_database
         </pre></p>';
       $ready_for_migration = 0;
@@ -98,13 +98,13 @@ 
     );
   }
 
-  $output = '<p>' . t('Note that due to a limitation within Drupal core, we cannot
+  $output = '<p>'.t('Note that due to a limitation within Drupal core, we cannot
     import messages from one database type to another database type. E.g, we
     cannot import BOINC from a myqsl DB, to a postgresql DB. You need to import
     the data into the same type of database first (import all the BOINC data, as is,
-    from mysql to postgresql).') . '</p><p>' . t('If both BOINC and Drupal are on
+    from mysql to postgresql).').'</p><p>'.t('If both BOINC and Drupal are on
     postgresql, see !this_issue.', array('!this_issue' => l(t('this issue'),
-    'http://drupal.org/node/64877', array('attributes' => array('target' => '_blank'))))) . '</p>';
+    'http://drupal.org/node/64877', array('attributes' => array('target' => '_blank'))))).'</p>';
   $form['boinc_data']['note'] = array('#value' => $output);
   
   // BOINC table prefix
@@ -121,7 +121,7 @@ 
   $output = $result['html'];
   if ($result['result'] != 1) {
     $ready_for_migration = 0;
-    $form['boinc_data']['#collapsed']= FALSE;
+    $form['boinc_data']['#collapsed'] = FALSE;
     $output .= '<span class="marker">';
     $output .= t('Some tables couldn\'t be found. Are the database settings correct?
       Please correct the errors so that the BOINC tables can be accessed.');
@@ -169,15 +169,15 @@ 
     '#collapsible' => TRUE,
     '#collapsed' => TRUE
   );
-  $output = '<p>'. t('Select which !input_format should be used for the imported
+  $output = '<p>'.t('Select which !input_format should be used for the imported
     messages. It is highly-recommended that this value be left as it is, as the
     Full HTML and PHP code formats can be very dangerous!', 
-    array('!input_format' => l(t('input format'), 'admin/settings/filters'))) .'</p>';
+    array('!input_format' => l(t('input format'), 'admin/settings/filters'))).'</p>';
   $form['input']['intro'] = array('#value' => $output);
   
   $formats = db_query('SELECT format, name FROM {filter_formats}');
   while ($format = db_fetch_array($formats)) {
-    $input_formats[$format['format']]= $format['name'];
+    $input_formats[$format['format']] = $format['name'];
   }
   
   // By default, use the format named "Rich text"; fall back to Filtered HTML
@@ -194,10 +194,10 @@ 
   );
   if (!$input_format) {
     $ready_for_migration = 0;
-    $form['input']['#collapsed']= FALSE;
+    $form['input']['#collapsed'] = FALSE;
   }
-  $output = '<p><em>'. t('This setting will be applied to all posts and private
-    messages.') .'</em></p>';
+  $output = '<p><em>'.t('This setting will be applied to all posts and private
+    messages.').'</em></p>';
   $form['input']['result'] = array('#value' => $output);
   
   // BBcode settings
@@ -208,23 +208,23 @@ 
     '#collapsed' => TRUE,
   );
   
-  $output = '<p>' . t('BOINC posts are often formatted in BBcode. During the
-    import process, BBcode can be handled in two diffent ways') . ':</p>';
-  $output .= '<ul><li>' . t('Leave the BBcode as is and use the !bbcode_module
+  $output = '<p>'.t('BOINC posts are often formatted in BBcode. During the
+    import process, BBcode can be handled in two diffent ways').':</p>';
+  $output .= '<ul><li>'.t('Leave the BBcode as is and use the !bbcode_module
     after the migration to filter the imported posts. In this case, the module
     is not required during the migration, but will need to be installed and
     configured later.',
     array('!bbcode_module' => l(t('bbcode module'), 'http://drupal.org/project/bbcode',
-    array('attributes' => array('target' => '_blank'))))) . '</li>';
-  $output .= '<li>' . t('Filter the BBcode now and replace it with plain HTML.
+    array('attributes' => array('target' => '_blank'))))).'</li>';
+  $output .= '<li>'.t('Filter the BBcode now and replace it with plain HTML.
     The BBcode module will be required during the migration process, but can
-    be disabled afterwards.') . '</li></ul>';
-  $output .= '<p>' . t('An appropriate input filter should be chosen, as well,
+    be disabled afterwards.').'</li></ul>';
+  $output .= '<p>'.t('An appropriate input filter should be chosen, as well,
     as it will affect how the post is displayed. For example, if BBcode is
     removed and replaced by HTML, some HTML tags will be stripped by the
     "Filtered HTML" input format unless they are first added to the list of
     allowed tags in that format. See the !input_formats page.',
-    array('!input_formats' => l(t('input formats'), 'admin/settings/filters'))) . '</p>';
+    array('!input_formats' => l(t('input formats'), 'admin/settings/filters'))).'</p>';
     
   $form['bbcode']['intro'] = array('#value' => $output);
   $form['bbcode']['boincimport_bbcode'] = array(
@@ -254,28 +254,28 @@ 
     $result = db_query($sql, $input_format);
     if (!db_result($result)) {
       $form['bbcode']['#collapsed'] = FALSE;
-      $output .= '<p class="marker">'. t('You need to !enable_bbcode_input and
+      $output .= '<p class="marker">'.t('You need to !enable_bbcode_input and
         then !configure_bbcode_options', array(
           '!enable_bbcode_input' =>  l(t('enable BBcode in the selected input format'), "admin/settings/filters/{$input_format}"),
-          '!configure_bbcode_options' => l(t('configure BBcode options'), "admin/settings/filters/{$input_format}/configure"))) .'</p>';
+          '!configure_bbcode_options' => l(t('configure BBcode options'), "admin/settings/filters/{$input_format}/configure"))).'</p>';
     }
-    $output .= '<p>'. t('For best results, !disable_all_options', 
-    array('!disable_all_options' => l(t('all four BBcode configure options should be disabled'), "admin/settings/filters/{$input_format}/configure"))) . '.</p>';
+    $output .= '<p>'.t('For best results, !disable_all_options', 
+    array('!disable_all_options' => l(t('all four BBcode configure options should be disabled'), "admin/settings/filters/{$input_format}/configure"))).'.</p>';
   } else {
     if ($bbcode_disabled) {
       $form['bbcode']['#collapsed'] = FALSE;
-      $output .= '<p>'. t('The !bbcode_module is recommended, but not required.
+      $output .= '<p>'.t('The !bbcode_module is recommended, but not required.
         You may !enable_bbcode after having installed it.', array(
           '!bbcode_module' => l(t('bbcode module'), 'http://drupal.org/project/bbcode', array('attributes' => array('target' => '_blank'))),
-          '!enable_bbcode' => l(t('enable the bbcode module'), 'admin/build/modules'))) .'</p>';
+          '!enable_bbcode' => l(t('enable the bbcode module'), 'admin/build/modules'))).'</p>';
     }
-    $output .= '<p>' . t('Don\'t forget to !enable_bbcode_input and then !configure_bbcode_options.', array(
+    $output .= '<p>'.t('Don\'t forget to !enable_bbcode_input and then !configure_bbcode_options.', array(
       '!enable_bbcode_input' =>  l(t('enable BBcode in the selected input format'), "admin/settings/filters/{$input_format}"),
-      '!configure_bbcode_options' => l(t('configure BBcode options'), "admin/settings/filters/{$input_format}/configure"))) . '</p>';
-    $output .= '<p>' . t('You may want to enable it for other input formats, too.') . '</p>';
+      '!configure_bbcode_options' => l(t('configure BBcode options'), "admin/settings/filters/{$input_format}/configure"))).'</p>';
+    $output .= '<p>'.t('You may want to enable it for other input formats, too.').'</p>';
   }
-  $output .= '<p><em>'. t('This setting will be applied to all posts, private
-    messages and the users\' signature.') .'</em></p>';
+  $output .= '<p><em>'.t('This setting will be applied to all posts, private
+    messages and the users\' signature.').'</em></p>';
     
   $form['bbcode']['result'] = array('#value' => $output);
   
@@ -287,7 +287,7 @@ 
     '#collapsed' => TRUE,
   );
   if (!variable_get('boincimport_base_url_boinc', 0)) {
-    $form['url_transforms']['#collapsed']= FALSE;
+    $form['url_transforms']['#collapsed'] = FALSE;
     $output = '<span class="marker">';
     $ready_for_migration = 0;
   }
@@ -320,11 +320,11 @@ 
   );
   variable_set('boincimport_ready', $ready_for_migration);
   if ($ready_for_migration) {
-    $output = '<p>'. t('Check the settings above one more time. If everything
-      is ok, !proceed.', array('!proceed' => l(t('you can now proceed with the migration'), 'admin/boinc/import/process'))) .'</p>';
+    $output = '<p>'.t('Check the settings above one more time. If everything
+      is ok, !proceed.', array('!proceed' => l(t('you can now proceed with the migration'), 'admin/boinc/import/process'))).'</p>';
   } else {
-    $output = '<p class="marker">'. t('You are not ready for the migration yet.
-      Please check the settings above.') .'</p>';
+    $output = '<p class="marker">'.t('You are not ready for the migration yet.
+      Please check the settings above.').'</p>';
   }
   
   $form['migration']['info'] = array('#value' => $output);
@@ -335,23 +335,23 @@ 
  * Callback admin/boinc/import/post_configuration
  */
 function boincimport_post_configuration() {
-  $output = '<p>'. t('This page will help to tie up any loose ends following
+  $output = '<p>'.t('This page will help to tie up any loose ends following
     the BOINC data import process. Before continuing, it may be best to have
-    already:') .'</p>';
+    already:').'</p>';
   $output .= '<ul>';
-  $output .= '<li>'. t('Imported users, teams, and forums using the BOINC
-    import module') .'</li>';
-  $output .= '<li>'. t('Imported any additional data using add on modules if
-    available.') .'</li>';
+  $output .= '<li>'.t('Imported users, teams, and forums using the BOINC
+    import module').'</li>';
+  $output .= '<li>'.t('Imported any additional data using add on modules if
+    available.').'</li>';
   $output .= '</ul>';
   //User Set up
-  $output .= '<h2>'. t('User settings') .'</h2>';  
-  $output .= '<p>'. t('Make sure your !user_settings are correct. Specifically,
+  $output .= '<h2>'.t('User settings').'</h2>';  
+  $output .= '<p>'.t('Make sure your !user_settings are correct. Specifically,
     you can enable or disable signatures there. ', array(
       '!user_settings' => l(t('user settings'), 'admin/user/settings')
-    )) .'</p>'; 
-  $output .= '<p>' . t('Note that avatar/picture support is not tied to user
-    settings Picture support at this time.') . '</p>';
+    )).'</p>'; 
+  $output .= '<p>'.t('Note that avatar/picture support is not tied to user
+    settings Picture support at this time.').'</p>';
   /*$output .= '<p>' . t('In order to activate avatar/picture support, you need to
     enable pictures on this page and also:') . '</p>';
   $output .= '<ul>';
@@ -366,15 +366,15 @@ 
   $output .= '</ul>'; */
   
   // Forum Set up
-  $output .= '<h2>'. t('Forum settings') .'</h2>';
-  $output .= '<p>'. t('Check any additional changes to make on the
+  $output .= '<h2>'.t('Forum settings').'</h2>';
+  $output .= '<p>'.t('Check any additional changes to make on the
     !forum_settings page.', array(
       '!forum_settings' =>  l(t('forum settings'), 'admin/content/forum/settings')
-    )) .'</p>';
-  $output .= '<p>'. t('It also may be a good idea to look at the !forum_list to
+    )).'</p>';
+  $output .= '<p>'.t('It also may be a good idea to look at the !forum_list to
     verify that all necessary containers and forums are in place.', array(
       '!forum_list' =>  l(t('forum list'), 'admin/content/forum')
-    )) .'</p>';
+    )).'</p>';
   /*$output .= '<p>'. t('If need be, set up the private forums as explained in the
     !documentation.', array(
       '!documentation' => l(t('documentation'), 'http://drupal.org/node/58969', array('attributes' => array('target' => '_blank')))
@@ -390,17 +390,17 @@ 
   // BBcode
   $bbcode = variable_get('boincimport_bbcode', 0);
   if ($bbcode) {
-    $output .= '<h2>'. t('Remove BBcode module') .'</h2>';
-    $output .= '<p>'. t('The BBcode has been cleaned from the posts. You shouldn\'t
-      need the bbcode module anymore. You can disable it.') .'</p>';
+    $output .= '<h2>'.t('Remove BBcode module').'</h2>';
+    $output .= '<p>'.t('The BBcode has been cleaned from the posts. You shouldn\'t
+      need the bbcode module anymore. You can disable it.').'</p>';
   } else {
-    $output .= '<h2>'. t('Configure BBcode module') .'</h2>';
-    $output .= '<p>'. t('Most likely, there are many BOINC forum postings
+    $output .= '<h2>'.t('Configure BBcode module').'</h2>';
+    $output .= '<p>'.t('Most likely, there are many BOINC forum postings
       formatted in BBcode. It would be a good idea to verify that BBcode is
       being formatted correctly in the imported posts. Have a look around
       !the_forum.', array(
         '!the_forum' =>  l(t('the forum'), 'community')
-      )) . '</p>';
+      )).'</p>';
   }
 
   return $output;
@@ -445,11 +445,11 @@ 
   $output .= '</ol>';
 
   // process will die() if the link to the BOINC database is wrong:
-  $output .= '<p>'. t('If the BOINC data is in another database, but you made a
+  $output .= '<p>'.t('If the BOINC data is in another database, but you made a
     mistake while setting up the data base url, you may be unable to access the
     setting page due to some limitations of the core of Drupal. !reset_db_url.', array(
       '!reset_db_url' => l(t('Click here to reset the database url'), 'admin/boinc/import/reset')
-    )) .'</p>';
+    )).'</p>';
 
   return $output;
 }

Upper-Lower-Casing +25 added lines, -25 removed lines

@@ -21,13 +21,13 @@ 
   $form['test'] = array(
     '#type' => 'fieldset',
     '#title' => t('Test on copy first'),
-    '#collapsible' => TRUE,
-    '#collapsed' => TRUE,
+    '#collapsible' => true,
+    '#collapsed' => true,
   );
     
   $output = '';
   if (!variable_get('boincimport_tested', 0)) {
-    $form['test']['#collapsed']= FALSE;
+    $form['test']['#collapsed']= false;
     $output = '<span class="marker">';
     $ready_for_migration = 0;
   }
@@ -56,15 +56,15 @@ 
   // Where is BOINC data?
   $pre = variable_get('boincimport_table_prefix', '');
   $tables = array($pre .'forum', $pre .'post', $pre .'thread', $pre .'user');
-  $db = (is_array($db_url) AND isset($db_url['boinc'])) ? 'boinc' : 'default';
+  $db = (is_array($db_url) and isset($db_url['boinc'])) ? 'boinc' : 'default';
   $result = _boincimport_check_tables($tables, $db, 0);
   
   $config_path = conf_path();
   $form['boinc_data'] = array(
     '#type' => 'fieldset',
     '#title' => t('Location of BOINC data'),
-    '#collapsible' => TRUE,
-    '#collapsed' => TRUE,
+    '#collapsible' => true,
+    '#collapsed' => true,
   );
   $form['boinc_data']['boincimport_db_configured'] = array(
     '#type' => 'checkbox',
@@ -73,7 +73,7 @@ 
     '#return_value' => 1,
     '#default_value' => variable_get('boincimport_db_configured', 1),
     '#description' => '',
-    '#disabled' => ($result['result']) ? TRUE : FALSE
+    '#disabled' => ($result['result']) ? true : false
   );
   if (variable_get('boincimport_db_configured', 1) != 1) {
     $default_db_url = is_array($db_url) ? end($db_url) : $db_url;
@@ -84,7 +84,7 @@ 
         mysql://boinc_username:boinc_password@localhost/boinc_database
         </pre></p>';
       $ready_for_migration = 0;
-      $form['boinc_data']['#collapsed'] = FALSE;
+      $form['boinc_data']['#collapsed'] = false;
       $form['boinc_data']['db_connect_string_note'] = array('#value' => $output);
     }
 
@@ -115,13 +115,13 @@ 
     '#size' => 20,
     '#maxlength' => 255,
     '#description' => t('What prefix is used for your BOINC tables?'),
-    '#disabled' => ($result['result']) ? TRUE : FALSE
+    '#disabled' => ($result['result']) ? true : false
   );
   
   $output = $result['html'];
   if ($result['result'] != 1) {
     $ready_for_migration = 0;
-    $form['boinc_data']['#collapsed']= FALSE;
+    $form['boinc_data']['#collapsed']= false;
     $output .= '<span class="marker">';
     $output .= t('Some tables couldn\'t be found. Are the database settings correct?
       Please correct the errors so that the BOINC tables can be accessed.');
@@ -134,8 +134,8 @@ 
   $form['users'] = array(
     '#type' => 'fieldset',
     '#title' => t('User import'),
-    '#collapsible' => TRUE,
-    '#collapsed' => TRUE,
+    '#collapsible' => true,
+    '#collapsed' => true,
   );
 
   // User import batch size
@@ -166,8 +166,8 @@ 
   $form['input'] = array(
     '#type' => 'fieldset',
     '#title' => t('Input format settings'),
-    '#collapsible' => TRUE,
-    '#collapsed' => TRUE
+    '#collapsible' => true,
+    '#collapsed' => true
   );
   $output = '<p>'. t('Select which !input_format should be used for the imported
     messages. It is highly-recommended that this value be left as it is, as the
@@ -194,7 +194,7 @@ 
   );
   if (!$input_format) {
     $ready_for_migration = 0;
-    $form['input']['#collapsed']= FALSE;
+    $form['input']['#collapsed']= false;
   }
   $output = '<p><em>'. t('This setting will be applied to all posts and private
     messages.') .'</em></p>';
@@ -204,8 +204,8 @@ 
   $form['bbcode'] = array(
     '#type' => 'fieldset',
     '#title' => t('BBcode settings'),
-    '#collapsible' => TRUE,
-    '#collapsed' => TRUE,
+    '#collapsible' => true,
+    '#collapsed' => true,
   );
   
   $output = '<p>' . t('BOINC posts are often formatted in BBcode. During the
@@ -242,7 +242,7 @@ 
   if (variable_get('boincimport_bbcode', 0)) {
     if ($bbcode_disabled) {
       $ready_for_migration = 0;
-      $form['bbcode']['#collapsed'] = FALSE;
+      $form['bbcode']['#collapsed'] = false;
       $output .= '<p><span class="marker">';
       $output .= t('The !bbcode_module is required to transform BBcode into HTML.
         You first need to !enable_bbcode', array(
@@ -253,7 +253,7 @@ 
     $sql = "SELECT COUNT(*) FROM {filters} WHERE module = 'bbcode' AND format = %d";
     $result = db_query($sql, $input_format);
     if (!db_result($result)) {
-      $form['bbcode']['#collapsed'] = FALSE;
+      $form['bbcode']['#collapsed'] = false;
       $output .= '<p class="marker">'. t('You need to !enable_bbcode_input and
         then !configure_bbcode_options', array(
           '!enable_bbcode_input' =>  l(t('enable BBcode in the selected input format'), "admin/settings/filters/{$input_format}"),
@@ -263,7 +263,7 @@ 
     array('!disable_all_options' => l(t('all four BBcode configure options should be disabled'), "admin/settings/filters/{$input_format}/configure"))) . '.</p>';
   } else {
     if ($bbcode_disabled) {
-      $form['bbcode']['#collapsed'] = FALSE;
+      $form['bbcode']['#collapsed'] = false;
       $output .= '<p>'. t('The !bbcode_module is recommended, but not required.
         You may !enable_bbcode after having installed it.', array(
           '!bbcode_module' => l(t('bbcode module'), 'http://drupal.org/project/bbcode', array('attributes' => array('target' => '_blank'))),
@@ -283,11 +283,11 @@ 
   $form['url_transforms'] = array(
     '#type' => 'fieldset',
     '#title' => t('URL transforms'),
-    '#collapsible' => TRUE,
-    '#collapsed' => TRUE,
+    '#collapsible' => true,
+    '#collapsed' => true,
   );
   if (!variable_get('boincimport_base_url_boinc', 0)) {
-    $form['url_transforms']['#collapsed']= FALSE;
+    $form['url_transforms']['#collapsed']= false;
     $output = '<span class="marker">';
     $ready_for_migration = 0;
   }
@@ -315,8 +315,8 @@ 
   $form['migration'] = array(
     '#type' => 'fieldset',
     '#title' => 'Migration',
-    '#collapsible' => TRUE,
-    '#collapsed' => FALSE,
+    '#collapsible' => true,
+    '#collapsed' => false,
   );
   variable_set('boincimport_ready', $ready_for_migration);
   if ($ready_for_migration) {

drupal/sites/default/boinc/modules/boinctranslate/boinctranslate.admin.inc

Indentation +132 added lines, -132 removed lines

@@ -7,85 +7,85 @@ 
 */
 
 /**
-  * The BOINC translation settings form allows configuration of BTS integration
-  * (and other translation related settings)
-  */
+ * The BOINC translation settings form allows configuration of BTS integration
+ * (and other translation related settings)
+ */
 function boinctranslate_admin_settings(&$form_state) {
-  $form = array();
-  $initialized = FALSE;
-  $import_enabled = FALSE;
-  $default = array(
+    $form = array();
+    $initialized = FALSE;
+    $import_enabled = FALSE;
+    $default = array(
     'transifex_user' => variable_get('boinc_translate_transifex_user', ''),
     'transifex_pass' => variable_get('boinc_translate_transifex_pass', ''),
     'transifex_boinc_name' => variable_get(
-      'boinc_translate_transifex_standard_name', 'boinc'
+        'boinc_translate_transifex_standard_name', 'boinc'
     ),
     'transifex_boinc_resources' => variable_get(
-      'boinc_translate_transifex_standard_resources',
-      "project-generic\nweb"
+        'boinc_translate_transifex_standard_resources',
+        "project-generic\nweb"
     ),
     'transifex_boinc_drupal_resource' => variable_get(
-      'boinc_translate_transifex_boinc_drupal_resource',
-      'drupal'
+        'boinc_translate_transifex_boinc_drupal_resource',
+        'drupal'
     ), 
     'transifex_project_name' => variable_get(
-      'boinc_translate_transifex_project_name', ''
+        'boinc_translate_transifex_project_name', ''
     ),
     'transifex_project_resources' => variable_get(
-      'boinc_translate_transifex_project_resources', ''
+        'boinc_translate_transifex_project_resources', ''
     ), 
-  );
+    );
   
-  if ($default['transifex_user'] AND $default['transifex_pass']) {
+    if ($default['transifex_user'] AND $default['transifex_pass']) {
     if (trim($default['transifex_boinc_resources'])
     OR trim($default['transifex_project_resources'])) {
-      $import_enabled = TRUE;
+        $import_enabled = TRUE;
     }
     if ($default['transifex_boinc_name']
     AND $default['transifex_project_name']
     AND trim($default['transifex_boinc_resources'])
     AND trim($default['transifex_project_resources'])) {
-      $initialized = TRUE;
+        $initialized = TRUE;
+    }
     }
-  }
   
-  // Define the form
+    // Define the form
   
-  $form['transifex'] = array(
+    $form['transifex'] = array(
     '#title' => t('Transifex settings'),
     '#type' => 'fieldset',
     '#description' => '',
     '#collapsible' => TRUE,
     '#collapsed' => $initialized,
     '#attributes' => array('class' => '')
-  );
-  $form['transifex']['boinc_translate_transifex_user'] = array(
+    );
+    $form['transifex']['boinc_translate_transifex_user'] = array(
     '#type' => 'textfield',
     '#title' => t('User name'),
     '#default_value' => $default['transifex_user'],
-  );
-  $form['transifex']['boinc_translate_transifex_pass'] = array(
+    );
+    $form['transifex']['boinc_translate_transifex_pass'] = array(
     '#type' => 'password',
     '#title' => t('Password'),
     '#attributes' => array(
-      'placeholder' => $default['transifex_pass'] ? '********' : '',
+        'placeholder' => $default['transifex_pass'] ? '********' : '',
     ),
-  );
-  $form['transifex']['boinc_translate_transifex_standard_name'] = array(
+    );
+    $form['transifex']['boinc_translate_transifex_standard_name'] = array(
     '#type' => 'textfield',
     '#title' => t('BOINC Transifex project'),
     '#default_value' => $default['transifex_boinc_name'],
     '#description' => t('The portion of the Transifex URL that identifies BOINC.'),
-  );
-  $form['transifex']['boinc_translate_transifex_boinc_drupal_resource'] = array(
+    );
+    $form['transifex']['boinc_translate_transifex_boinc_drupal_resource'] = array(
     '#type' => 'textfield',
     '#title' => t('Official BOINC Drupal resource'),
     '#default_value' => $default['transifex_boinc_drupal_resource'],
     '#description' => t('The portion of the Transifex URL that identifies
       the standard BOINC resource to use for translation of the public
       content defined in the Drupal system.'),
-  );
-  $form['transifex']['boinc_translate_transifex_standard_resources'] = array(
+    );
+    $form['transifex']['boinc_translate_transifex_standard_resources'] = array(
     '#type' => 'textarea',
     '#title' => t('Additional BOINC resources'),
     '#default_value' => $default['transifex_boinc_resources'],
@@ -96,14 +96,14 @@ 
       by any matching translations found in resources further down the list.
       Note that strings from the Official BOINC Drupal resource will override
       any strings from these additional resources.'),
-  );
-  $form['transifex']['boinc_translate_transifex_project_name'] = array(
+    );
+    $form['transifex']['boinc_translate_transifex_project_name'] = array(
     '#type' => 'textfield',
     '#title' => t('Project-specific Transifex project'),
     '#default_value' => $default['transifex_project_name'],
     '#description' => t('The portion of the Transifex URL that identifies this project.'),
-  );
-  $form['transifex']['boinc_translate_transifex_project_resources'] = array(
+    );
+    $form['transifex']['boinc_translate_transifex_project_resources'] = array(
     '#type' => 'textarea',
     '#title' => t('Project-specific resources'),
     '#default_value' => $default['transifex_project_resources'],
@@ -116,211 +116,211 @@ 
       NOTE: The first resource listed here is considered the primary resource
       for project-specific translations and will be updated by the "Update
       project-specific resources" button below!'),
-  );
-  $form['transifex']['buttons']['submit'] = array(
+    );
+    $form['transifex']['buttons']['submit'] = array(
     '#type' => 'submit',
     '#value' => t('Save configuration'),
-  );
+    );
   
-  $form['tools'] = array(
+    $form['tools'] = array(
     '#title' => t('Tools'),
     '#type' => 'fieldset',
     '#description' => '',
     '#collapsible' => TRUE,
     '#collapsed' => FALSE,
     '#attributes' => array('class' => '')
-  );
-  $form['tools']['initialize_languages'] = array(
+    );
+    $form['tools']['initialize_languages'] = array(
     '#type' => 'button',
     '#value' => t('Install official BOINC languages'),
     '#executes_submit_callback' => TRUE,
     '#submit' => array('boinctranslate_admin_settings_initialize_languages'),
     '#disabled' => !$initialized,
-  );
-  $form['tools']['export_now'] = array(
+    );
+    $form['tools']['export_now'] = array(
     '#type' => 'button',
     '#value' => t('Update project-specific resources'),
     '#executes_submit_callback' => TRUE,
     '#submit' => array('boinctranslate_admin_settings_export_now'),
     '#disabled' => !$import_enabled,
-  );
-  $form['tools']['import_now'] = array(
+    );
+    $form['tools']['import_now'] = array(
     '#type' => 'button',
     '#value' => t('Import all translations'),
     '#executes_submit_callback' => TRUE,
     '#submit' => array('boinctranslate_admin_settings_import_now'),
     '#disabled' => !$import_enabled,
-  );
-  if (user_access('update official BOINC translations')) {
+    );
+    if (user_access('update official BOINC translations')) {
     $form['tools']['update_official_boinc'] = array(
-      '#type' => 'button',
-      '#value' => t('Update official BOINC translations'),
-      '#executes_submit_callback' => TRUE,
-      '#submit' => array('boinctranslate_admin_settings_update_official_boinc'),
-      '#disabled' => !$import_enabled,
+        '#type' => 'button',
+        '#value' => t('Update official BOINC translations'),
+        '#executes_submit_callback' => TRUE,
+        '#submit' => array('boinctranslate_admin_settings_update_official_boinc'),
+        '#disabled' => !$import_enabled,
     );
-  }
-  $form['tools']['download_boinc_pot'] = array(
+    }
+    $form['tools']['download_boinc_pot'] = array(
     '#type' => 'button',
     '#value' => t('Download official BOINC .pot'),
     '#executes_submit_callback' => TRUE,
     '#submit' => array('boinctranslate_admin_settings_download_boinc_pot'),
     '#disabled' => !$import_enabled,
-  );
-  $form['tools']['download_project_pot'] = array(
+    );
+    $form['tools']['download_project_pot'] = array(
     '#type' => 'button',
     '#value' => t('Download project-specific .pot'),
     '#executes_submit_callback' => TRUE,
     '#submit' => array('boinctranslate_admin_settings_download_project_pot'),
     '#disabled' => !$import_enabled,
-  );
+    );
   
-  $form['#submit'][] = 'system_settings_form_submit';
-  $form['#theme'] = 'system_settings_form'; 
+    $form['#submit'][] = 'system_settings_form_submit';
+    $form['#theme'] = 'system_settings_form'; 
   
-  return $form;
+    return $form;
 }
 
 function boinctranslate_admin_settings_export_now() {
-  drupal_goto('admin/boinc/translation/export');
+    drupal_goto('admin/boinc/translation/export');
 }
 
 function boinctranslate_admin_settings_import_now() {
-  drupal_goto('admin/boinc/translation/import');
+    drupal_goto('admin/boinc/translation/import');
 }
 
 function boinctranslate_admin_settings_initialize_languages() {
-  drupal_goto('admin/boinc/translation/initialize-languages');
+    drupal_goto('admin/boinc/translation/initialize-languages');
 }
 
 function boinctranslate_admin_settings_update_official_boinc() {
-  drupal_goto('admin/boinc/translation/update-official-boinc');
+    drupal_goto('admin/boinc/translation/update-official-boinc');
 }
 
 function boinctranslate_admin_settings_download_boinc_pot() {
-  drupal_goto('admin/boinc/translation/download-pot/boinc');
+    drupal_goto('admin/boinc/translation/download-pot/boinc');
 }
 
 function boinctranslate_admin_settings_download_project_pot() {
-  drupal_goto('admin/boinc/translation/download-pot/project');
+    drupal_goto('admin/boinc/translation/download-pot/project');
 }
 
 
 /**
-  * Validate the BOINC translation settings form.
-  */
+ * Validate the BOINC translation settings form.
+ */
 function boinctranslate_admin_settings_validate($form, &$form_state) {
-  $values = $form_state['values'];
-  $errors = array();
-  $api_base_url = 'https://www.transifex.com/api/2';
+    $values = $form_state['values'];
+    $errors = array();
+    $api_base_url = 'https://www.transifex.com/api/2';
   
-  if (!$values['boinc_translate_transifex_user']) {
+    if (!$values['boinc_translate_transifex_user']) {
     form_set_error('boinc_translate_transifex_user', t('User name is required.'));
-  }
-  if (!$values['boinc_translate_transifex_pass']) {
+    }
+    if (!$values['boinc_translate_transifex_pass']) {
     if (!variable_get('boinc_translate_transifex_pass', '')) {
-      form_set_error('boinc_translate_transifex_pass', t('Password is required.'));
+        form_set_error('boinc_translate_transifex_pass', t('Password is required.'));
     }
     else {
-      unset($form_state['values']['boinc_translate_transifex_pass']);
+        unset($form_state['values']['boinc_translate_transifex_pass']);
+    }
     }
-  }
-  if (!$values['boinc_translate_transifex_standard_name']) {
+    if (!$values['boinc_translate_transifex_standard_name']) {
     form_set_error('boinc_translate_transifex_standard_name', 
-      t('BOINC Transifex project name is required.')
+        t('BOINC Transifex project name is required.')
     );
-  }
-  if (!$values['boinc_translate_transifex_standard_resources']) {
+    }
+    if (!$values['boinc_translate_transifex_standard_resources']) {
     form_set_error('boinc_translate_transifex_standard_resources',
-      t('At least one BOINC Transifex project resource is required.')
+        t('At least one BOINC Transifex project resource is required.')
     );
-  }
-  if (!$values['boinc_translate_transifex_project_resources']) {
+    }
+    if (!$values['boinc_translate_transifex_project_resources']) {
     form_set_error('boinc_translate_transifex_project_resources',
-      t('At least one project-specific Transifex resource is required.')
+        t('At least one project-specific Transifex resource is required.')
     );
-  }
+    }
   
-  $username = $values['boinc_translate_transifex_user'];
-  $password = ($values['boinc_translate_transifex_pass']) ? $values['boinc_translate_transifex_pass'] : variable_get('boinc_translate_transifex_pass', '');
-  $boinc_name = $values['boinc_translate_transifex_standard_name'];
-  $boinc_resources = boinctranslate_parse_resources(
+    $username = $values['boinc_translate_transifex_user'];
+    $password = ($values['boinc_translate_transifex_pass']) ? $values['boinc_translate_transifex_pass'] : variable_get('boinc_translate_transifex_pass', '');
+    $boinc_name = $values['boinc_translate_transifex_standard_name'];
+    $boinc_resources = boinctranslate_parse_resources(
     $values['boinc_translate_transifex_standard_resources']
-  );
+    );
   
-  if ($username AND $password AND $boinc_name AND $boinc_resources) {
+    if ($username AND $password AND $boinc_name AND $boinc_resources) {
     // Test authentication
     $authenticated = FALSE;
     $path = "project/{$boinc_name}/resource/{$boinc_resources[0]}/translation/en";
     $response = boinctranslate_transifex_request($path, NULL, TRUE, FALSE, $username, $password);
     
     if ($response) {
-      if ($response == '401 UNAUTHORIZED') {
+        if ($response == '401 UNAUTHORIZED') {
         form_set_error(
-          'boinc_translate_transifex_pass',
-          t('Transifex authentication failed.')
+            'boinc_translate_transifex_pass',
+            t('Transifex authentication failed.')
         );
-      }
-      else {
+        }
+        else {
         $authenticated = TRUE;
-      }
+        }
     }
     
     if ($authenticated) {
-      // Prepare list of resources to validate
-      $transifex_resources = array(
+        // Prepare list of resources to validate
+        $transifex_resources = array(
         'boinc' => $boinc_resources,
-      );
-      // Parse project-specific resources
-      $project_name = trim($values['boinc_translate_transifex_project_name']);
-      $project_resources = boinctranslate_parse_resources(
+        );
+        // Parse project-specific resources
+        $project_name = trim($values['boinc_translate_transifex_project_name']);
+        $project_resources = boinctranslate_parse_resources(
         $values['boinc_translate_transifex_project_resources']
-      );
-      if ($project_name AND $project_resources) {
+        );
+        if ($project_name AND $project_resources) {
         $transifex_resources[$project_name] = $project_resources;
-      }
-      elseif ($project_name AND !$project_resources) {
+        }
+        elseif ($project_name AND !$project_resources) {
         drupal_set_message(
-          t('No project-specific resources were provided'),
-          'warning'
+            t('No project-specific resources were provided'),
+            'warning'
         );
-      }
-      elseif ($project_resources AND !$project_name) {
+        }
+        elseif ($project_resources AND !$project_name) {
         drupal_set_message(
-          t('No project-specific Transifex project name was provided'),
-          'warning'
+            t('No project-specific Transifex project name was provided'),
+            'warning'
         );
-      }
+        }
       
-      // Try to access the given resources
-      foreach ($transifex_resources as $project => $resources) {
+        // Try to access the given resources
+        foreach ($transifex_resources as $project => $resources) {
         foreach ($resources as $resource) {
           
-          $path = "project/{$project}/resource/{$resource}/translation/en";
-          $response = boinctranslate_transifex_request($path);
+            $path = "project/{$project}/resource/{$resource}/translation/en";
+            $response = boinctranslate_transifex_request($path);
           
-          if ($response == '404 NOT FOUND') {
+            if ($response == '404 NOT FOUND') {
             form_set_error(
-              'boinc_translate_transifex_' . ($project == $boinc_name ? 'standard' : 'project') . '_resources',
-              t('Resource %name not found in %project.',
+                'boinc_translate_transifex_' . ($project == $boinc_name ? 'standard' : 'project') . '_resources',
+                t('Resource %name not found in %project.',
                 array(
-                  '%name' => $resource,
-                  '%project' => $project,
+                    '%name' => $resource,
+                    '%project' => $project,
+                )
                 )
-              )
             );
-          }
+            }
         }
-      }
+        }
+    }
     }
-  }
 }
 
 /**
-  * Handle post-validation submission of BOINC translation settings form.
-  */
+ * Handle post-validation submission of BOINC translation settings form.
+ */
 function boinctranslate_admin_settings_submit($form, &$form_state) {
-  drupal_set_message(t('BOINC translation settings have been updated.'));
+    drupal_set_message(t('BOINC translation settings have been updated.'));
 }
 
 

Spacing +1 added lines, -1 removed lines

@@ -301,7 +301,7 @@
           
           if ($response == '404 NOT FOUND') {
             form_set_error(
-              'boinc_translate_transifex_' . ($project == $boinc_name ? 'standard' : 'project') . '_resources',
+              'boinc_translate_transifex_'.($project == $boinc_name ? 'standard' : 'project').'_resources',
               t('Resource %name not found in %project.',
                 array(
                   '%name' => $resource,

Braces +4 added lines, -8 removed lines

@@ -220,8 +220,7 @@ 
   if (!$values['boinc_translate_transifex_pass']) {
     if (!variable_get('boinc_translate_transifex_pass', '')) {
       form_set_error('boinc_translate_transifex_pass', t('Password is required.'));
-    }
-    else {
+    } else {
       unset($form_state['values']['boinc_translate_transifex_pass']);
     }
   }
@@ -260,8 +259,7 @@ 
           'boinc_translate_transifex_pass',
           t('Transifex authentication failed.')
         );
-      }
-      else {
+      } else {
         $authenticated = TRUE;
       }
     }
@@ -278,14 +276,12 @@ 
       );
       if ($project_name AND $project_resources) {
         $transifex_resources[$project_name] = $project_resources;
-      }
-      elseif ($project_name AND !$project_resources) {
+      } elseif ($project_name AND !$project_resources) {
         drupal_set_message(
           t('No project-specific resources were provided'),
           'warning'
         );
-      }
-      elseif ($project_resources AND !$project_name) {
+      } elseif ($project_resources AND !$project_name) {
         drupal_set_message(
           t('No project-specific Transifex project name was provided'),
           'warning'

Upper-Lower-Casing +25 added lines, -25 removed lines

@@ -12,8 +12,8 @@ 
   */
 function boinctranslate_admin_settings(&$form_state) {
   $form = array();
-  $initialized = FALSE;
-  $import_enabled = FALSE;
+  $initialized = false;
+  $import_enabled = false;
   $default = array(
     'transifex_user' => variable_get('boinc_translate_transifex_user', ''),
     'transifex_pass' => variable_get('boinc_translate_transifex_pass', ''),
@@ -36,16 +36,16 @@ 
     ), 
   );
   
-  if ($default['transifex_user'] AND $default['transifex_pass']) {
+  if ($default['transifex_user'] and $default['transifex_pass']) {
     if (trim($default['transifex_boinc_resources'])
-    OR trim($default['transifex_project_resources'])) {
-      $import_enabled = TRUE;
+    or trim($default['transifex_project_resources'])) {
+      $import_enabled = true;
     }
     if ($default['transifex_boinc_name']
-    AND $default['transifex_project_name']
-    AND trim($default['transifex_boinc_resources'])
-    AND trim($default['transifex_project_resources'])) {
-      $initialized = TRUE;
+    and $default['transifex_project_name']
+    and trim($default['transifex_boinc_resources'])
+    and trim($default['transifex_project_resources'])) {
+      $initialized = true;
     }
   }
   
@@ -55,7 +55,7 @@ 
     '#title' => t('Transifex settings'),
     '#type' => 'fieldset',
     '#description' => '',
-    '#collapsible' => TRUE,
+    '#collapsible' => true,
     '#collapsed' => $initialized,
     '#attributes' => array('class' => '')
   );
@@ -126,28 +126,28 @@ 
     '#title' => t('Tools'),
     '#type' => 'fieldset',
     '#description' => '',
-    '#collapsible' => TRUE,
-    '#collapsed' => FALSE,
+    '#collapsible' => true,
+    '#collapsed' => false,
     '#attributes' => array('class' => '')
   );
   $form['tools']['initialize_languages'] = array(
     '#type' => 'button',
     '#value' => t('Install official BOINC languages'),
-    '#executes_submit_callback' => TRUE,
+    '#executes_submit_callback' => true,
     '#submit' => array('boinctranslate_admin_settings_initialize_languages'),
     '#disabled' => !$initialized,
   );
   $form['tools']['export_now'] = array(
     '#type' => 'button',
     '#value' => t('Update project-specific resources'),
-    '#executes_submit_callback' => TRUE,
+    '#executes_submit_callback' => true,
     '#submit' => array('boinctranslate_admin_settings_export_now'),
     '#disabled' => !$import_enabled,
   );
   $form['tools']['import_now'] = array(
     '#type' => 'button',
     '#value' => t('Import all translations'),
-    '#executes_submit_callback' => TRUE,
+    '#executes_submit_callback' => true,
     '#submit' => array('boinctranslate_admin_settings_import_now'),
     '#disabled' => !$import_enabled,
   );
@@ -155,7 +155,7 @@ 
     $form['tools']['update_official_boinc'] = array(
       '#type' => 'button',
       '#value' => t('Update official BOINC translations'),
-      '#executes_submit_callback' => TRUE,
+      '#executes_submit_callback' => true,
       '#submit' => array('boinctranslate_admin_settings_update_official_boinc'),
       '#disabled' => !$import_enabled,
     );
@@ -163,14 +163,14 @@ 
   $form['tools']['download_boinc_pot'] = array(
     '#type' => 'button',
     '#value' => t('Download official BOINC .pot'),
-    '#executes_submit_callback' => TRUE,
+    '#executes_submit_callback' => true,
     '#submit' => array('boinctranslate_admin_settings_download_boinc_pot'),
     '#disabled' => !$import_enabled,
   );
   $form['tools']['download_project_pot'] = array(
     '#type' => 'button',
     '#value' => t('Download project-specific .pot'),
-    '#executes_submit_callback' => TRUE,
+    '#executes_submit_callback' => true,
     '#submit' => array('boinctranslate_admin_settings_download_project_pot'),
     '#disabled' => !$import_enabled,
   );
@@ -248,11 +248,11 @@ 
     $values['boinc_translate_transifex_standard_resources']
   );
   
-  if ($username AND $password AND $boinc_name AND $boinc_resources) {
+  if ($username and $password and $boinc_name and $boinc_resources) {
     // Test authentication
-    $authenticated = FALSE;
+    $authenticated = false;
     $path = "project/{$boinc_name}/resource/{$boinc_resources[0]}/translation/en";
-    $response = boinctranslate_transifex_request($path, NULL, TRUE, FALSE, $username, $password);
+    $response = boinctranslate_transifex_request($path, null, true, false, $username, $password);
     
     if ($response) {
       if ($response == '401 UNAUTHORIZED') {
@@ -262,7 +262,7 @@ 
         );
       }
       else {
-        $authenticated = TRUE;
+        $authenticated = true;
       }
     }
     
@@ -276,16 +276,16 @@ 
       $project_resources = boinctranslate_parse_resources(
         $values['boinc_translate_transifex_project_resources']
       );
-      if ($project_name AND $project_resources) {
+      if ($project_name and $project_resources) {
         $transifex_resources[$project_name] = $project_resources;
       }
-      elseif ($project_name AND !$project_resources) {
+      elseif ($project_name and !$project_resources) {
         drupal_set_message(
           t('No project-specific resources were provided'),
           'warning'
         );
       }
-      elseif ($project_resources AND !$project_name) {
+      elseif ($project_resources and !$project_name) {
         drupal_set_message(
           t('No project-specific Transifex project name was provided'),
           'warning'

drupal/sites/default/boinc/modules/boincuser/boincuser.module

Indentation +608 added lines, -608 removed lines

@@ -27,44 +27,44 @@ 
  * with defined URL paths
  */
 function boincuser_menu() {
-  $items['account/posts'] = array(
+    $items['account/posts'] = array(
     'title' => 'Recent posts',
     'description' => '',
     'page callback' => 'boincuser_goto_recent_posts',
     'access callback' => 'user_is_logged_in',
     'type' => MENU_CALLBACK,
-  );
-  $items['account/profile'] = array(
+    );
+    $items['account/profile'] = array(
     'title' => '',
     'description' => '',
     'page callback' => 'boincuser_view_profile',
     'access arguments' => array('access content'),
     'type' => MENU_NORMAL_ITEM
-  );
-  $items['account/profile/view'] = array(
+    );
+    $items['account/profile/view'] = array(
     'title' => 'View',
     'description' => 'Show a user profile',
     'page callback' => 'boincuser_view_profile',
     'access arguments' => array('access content'),
     'type' => MENU_DEFAULT_LOCAL_TASK,
     'weight' => 0
-  );
-  $items['account/profile/edit'] = array(
+    );
+    $items['account/profile/edit'] = array(
     'title' => 'Edit',
     'description' => 'Edit a user profile',
     'page callback' => 'boincuser_edit_profile',
     'access arguments' => array('access content'),
     'type' => MENU_LOCAL_TASK,
     'weight' => 5
-  );
-  $items['account/team'] = array(
+    );
+    $items['account/team'] = array(
     'title' => 'User team',
     'description' => '',
     'page callback' => 'boincuser_goto_team',
     'access callback' => 'user_is_logged_in',
     'type' => MENU_CALLBACK,
-  );
-  $items['moderate/profile/%user/approve'] = array(
+    );
+    $items['moderate/profile/%user/approve'] = array(
     'title' => 'Profile approval',
     'description' => 'Approve profile content',
     'page callback' => 'boincuser_moderate_profile_approve',
@@ -72,8 +72,8 @@ 
     'access arguments' => array('edit any profile content'),
     'type' => MENU_CALLBACK,
     'weight' => 5
-  );
-  $items['moderate/profile/%user/edit'] = array(
+    );
+    $items['moderate/profile/%user/edit'] = array(
     'title' => 'Profile editor',
     'description' => 'Edit a user profile',
     'page callback' => 'boincuser_edit_profile',
@@ -81,8 +81,8 @@ 
     'access arguments' => array('edit any profile content'),
     'type' => MENU_CALLBACK,
     'weight' => 5
-  );
-  $items['moderate/profile/%/reject'] = array(
+    );
+    $items['moderate/profile/%/reject'] = array(
     'title' => bts('Reject profile'),
     'description' => 'Reject profile content',
     'page callback' => 'drupal_get_form',
@@ -90,52 +90,52 @@ 
     'access arguments' => array('edit any profile content'),
     'type' => MENU_CALLBACK,
     'weight' => 5
-  );
-  $items['moderate/user/%/ban'] = array(
+    );
+    $items['moderate/user/%/ban'] = array(
     'title' => bts('Ban user'),
     'description' => 'Ban a user from using community features',
     'page callback' => 'drupal_get_form',
     'page arguments' => array('boincuser_moderate_user_ban_form', 2),
     'access callback' => 'boincuser_moderate_community_access',
     'type' => MENU_CALLBACK,
-  );
-  $items['join'] = array(
+    );
+    $items['join'] = array(
     'title' => '',
     'description' => '',
     'page callback' => 'join_page',
     'access arguments' => array('access content'),
     'type' => MENU_NORMAL_ITEM
-  );
-  $items['join/new'] = array(
+    );
+    $items['join/new'] = array(
     'title' => bts("I'm new"),
     'page callback' => 'join_page',
     'page arguments' => array(1),
     'access arguments' => array('access content'),
     'type' => MENU_DEFAULT_LOCAL_TASK,
     'weight' => 0
-  );
-  $items['join/boinc'] = array(
+    );
+    $items['join/boinc'] = array(
     'title' => bts("I'm a BOINC user"),
     'page callback' => 'join_page',
     'page arguments' => array(1),
     'access arguments' => array('access content'),
     'type' => MENU_LOCAL_TASK,
     'weight' => 5
-  );
-  $items['user/login/auth'] = array(
+    );
+    $items['user/login/auth'] = array(
     'title' => bts('Authenticator login'),
     'description' => 'Log in using a user authenticator',
     'page callback' => 'drupal_get_form',
     'page arguments' => array('boincuser_authloginform'),
     'access arguments' => array('access content'),
     'type' => MENU_CALLBACK,
-  );
-  $items['user_control'] = array(
+    );
+    $items['user_control'] = array(
     'page callback' => 'boincuser_control',
     'access arguments' => array('access user profiles'),
     'type' => MENU_CALLBACK
-  );
-  $items['admin/boinc'] = array(
+    );
+    $items['admin/boinc'] = array(
     'title' => 'BOINC configuration',
     'position' => 'right',
     'weight' => -8,
@@ -143,8 +143,8 @@ 
     'access arguments' => array('administer site configuration'),
     'file' => 'system.admin.inc',
     'file path' => drupal_get_path('module', 'system'),
-  );
-  $items['admin/boinc/environment'] = array(
+    );
+    $items['admin/boinc/environment'] = array(
     'title' => 'Environment: General',
     'description' => 'Set paths to BOINC functions and any other necessary
       variables that establish a BOINC environment.',
@@ -153,8 +153,8 @@ 
     'access arguments' => array('administer site configuration'),
     'type' => MENU_NORMAL_ITEM,
     'file' => 'boincuser.admin.inc'
-  );
-  $items['admin/boinc/scheduler'] = array(
+    );
+    $items['admin/boinc/scheduler'] = array(
     'title' => 'Environment: Scheduling server URLs',
     'description' => 'Set BOINC scheduler options.',
     'page callback' => 'drupal_get_form',
@@ -162,8 +162,8 @@ 
     'access arguments' => array('administer site configuration'),
     'type' => MENU_NORMAL_ITEM,
     'file' => 'boincuser.admin.inc'
-  );
-  $items['admin/boinc/other'] = array(
+    );
+    $items['admin/boinc/other'] = array(
     'title' => 'Other Options',
     'description' => 'Set other BOINC options.',
     'page callback' => 'drupal_get_form',
@@ -171,42 +171,42 @@ 
     'access arguments' => array('administer site configuration'),
     'type' => MENU_NORMAL_ITEM,
     'file' => 'boincuser.admin.inc'
-  );
+    );
   
-  $items['create_account.php'] = array(
+    $items['create_account.php'] = array(
     'title' => 'Create Account RPC',
     'description' => 'RPC for creating user accounts.',
     'page callback' => 'boincuser_create_account',
     'access callback' => TRUE,
     'type' => MENU_CALLBACK
-  );
-  return $items;
+    );
+    return $items;
 }
 
 /**
  *
  */
 function boincuser_init() {
-  // Skip this check for charts, which are loaded separately
-  // (may get duplicate or unexpected messages otherwise)
-  if (substr($_GET['q'], 0, 7) == 'charts/') {
+    // Skip this check for charts, which are loaded separately
+    // (may get duplicate or unexpected messages otherwise)
+    if (substr($_GET['q'], 0, 7) == 'charts/') {
     return;
-  }
+    }
 
-  // If admin user, do some basic site functionality checks
-  if (user_access('administer site configuration')) {
+    // If admin user, do some basic site functionality checks
+    if (user_access('administer site configuration')) {
     // Ensure we have a configured BOINC environment
     boinc_get_path();
     boinc_get_scheduler_tags();
-  }
+    }
   
-  // Check credits for the verified contributor role
-  boincuser_check_credit_requirements();
+    // Check credits for the verified contributor role
+    boincuser_check_credit_requirements();
   
-  if (module_exists('boincteam')) {
+    if (module_exists('boincteam')) {
     // Display any persistent team messages
     boincteam_show_messages();
-  }
+    }
 }
 
 /**
@@ -214,8 +214,8 @@ 
  * Drupal user operations
  */
 function boincuser_user($op, &$edit, &$account, $category = NULL) {
-  // Handle BOINC integration for users with UID > 1 (skip anonymous and admin)
-  if ($account->uid > 1) {
+    // Handle BOINC integration for users with UID > 1 (skip anonymous and admin)
+    if ($account->uid > 1) {
     switch($op) {
     case 'load':
       // User loading; insert BOINC data into the user object
@@ -223,11 +223,11 @@ 
         SELECT boinc_id, penalty_expiration
         FROM {boincuser} WHERE uid = %d",
         $account->uid
-      ));
-      $account->boincuser_id = $drupal_user->boinc_id;
-      $account->boincuser_penalty_expiration = $drupal_user->penalty_expiration;
-      db_set_active('boinc');
-      $boinc_user = db_fetch_object(db_query("
+        ));
+        $account->boincuser_id = $drupal_user->boinc_id;
+        $account->boincuser_penalty_expiration = $drupal_user->penalty_expiration;
+        db_set_active('boinc');
+        $boinc_user = db_fetch_object(db_query("
         SELECT
           name,
           authenticator,
@@ -241,27 +241,27 @@ 
         FROM {user}
         WHERE id = %d",
         $account->boincuser_id
-      ));
-      $account->boincuser_name = $boinc_user->name;
-      $account->boincuser_account_key = $boinc_user->authenticator;
-      $account->boincuser_weak_auth = md5($boinc_user->authenticator . $boinc_user->passwd_hash);
-      $account->boincuser_total_credit = round($boinc_user->total_credit);
-      $account->boincuser_expavg_credit = round($boinc_user->expavg_credit);
-      $account->boincuser_expavg_time = round($boinc_user->expavg_time);
-      $account->boincuser_cpid = md5($boinc_user->cross_project_id . $account->mail);
-      $account->boincuser_default_pref_set = $boinc_user->venue;
-      $account->boincteam_id = $boinc_user->teamid;
-      db_set_active('default');
-      // Set Drupal team ID
-      $account->team = NULL;
-      if ($account->boincteam_id) {
+        ));
+        $account->boincuser_name = $boinc_user->name;
+        $account->boincuser_account_key = $boinc_user->authenticator;
+        $account->boincuser_weak_auth = md5($boinc_user->authenticator . $boinc_user->passwd_hash);
+        $account->boincuser_total_credit = round($boinc_user->total_credit);
+        $account->boincuser_expavg_credit = round($boinc_user->expavg_credit);
+        $account->boincuser_expavg_time = round($boinc_user->expavg_time);
+        $account->boincuser_cpid = md5($boinc_user->cross_project_id . $account->mail);
+        $account->boincuser_default_pref_set = $boinc_user->venue;
+        $account->boincteam_id = $boinc_user->teamid;
+        db_set_active('default');
+        // Set Drupal team ID
+        $account->team = NULL;
+        if ($account->boincteam_id) {
         $account->team = db_result(db_query("
           SELECT nid FROM {boincteam} WHERE team_id = %d",
-          $account->boincteam_id
+            $account->boincteam_id
         ));
-      }
-      // Set post count
-      $account->post_count = db_result(db_query("
+        }
+        // Set post count
+        $account->post_count = db_result(db_query("
         SELECT COUNT(*) +
         (
           SELECT COUNT(*) FROM {node}
@@ -274,8 +274,8 @@ 
         WHERE comments.uid = '%d'
         AND node.status = 1",
         $account->uid, $account->uid
-      ));
-      break;
+        ));
+        break;
       
     case 'view':
       // SAMPLE: Add BOINC data to the user profile
@@ -303,95 +303,95 @@ 
           if (!boincuser_register_validate($edit)) {
             // BOINC user validation failed for registration; set an error accordingly
             form_set_error('mail', bts('An account already exists for @email. Log in or request password assistance to access your @project account.', array('@email' => $edit['mail'], '@project' => PROJECT)));
-          }
-          else {
+            }
+            else {
             // Save profile information for use during Insert
             $_SESSION['profileInfo'] = array(
-              'country' => $edit['field_country'][0]['value'],
-              'zip' => $edit['field_zip'][0]['value'],                  
-              'url' => $edit['field_url'][0]['value'],
-              'background' => $edit['field_background'][0]['value'],
-              'opinions' => $edit['field_opinions'][0]['value']
+                'country' => $edit['field_country'][0]['value'],
+                'zip' => $edit['field_zip'][0]['value'],                  
+                'url' => $edit['field_url'][0]['value'],
+                'background' => $edit['field_background'][0]['value'],
+                'opinions' => $edit['field_opinions'][0]['value']
             );
             // With BOINC validation passed, make sure name is unique
             $edit['name'] = find_unique_name($edit['boincuser_name']);
-          }
-          break;
+            }
+            break;
           
         case 'user_account':
           // Validate data before updating user account info
           boincuser_account_validate($edit, $account);
-          break;
+            break;
         
         default:
           
         }
         // We don't want to save validation source, so remove it
         $edit['validation_source'] = null;
-      }
-      break;
+        }
+        break;
       
     case 'insert':
       // New user being added to the system
       $profile_info = $_SESSION['profileInfo'];
-      $imported = $_SESSION['importedUser'];
-      unset($_SESSION['profileInfo']);
-      unset($_SESSION['importedUser']);
+        $imported = $_SESSION['importedUser'];
+        unset($_SESSION['profileInfo']);
+        unset($_SESSION['importedUser']);
       
-      watchdog(
+        watchdog(
         'boincuser',
         'Creating user account for %email_addr',
         array('%email_addr' => $edit['mail']),
         WATCHDOG_NOTICE
-      );
+        );
       
-      // NOTE: Registrations are normally handled via the create_account RPC,
-      // so the following will not run (kept in case form registration is used)
+        // NOTE: Registrations are normally handled via the create_account RPC,
+        // so the following will not run (kept in case form registration is used)
       
-      // If from the registration form, create a BOINC user and relationships
+        // If from the registration form, create a BOINC user and relationships
       
-      if ($profile_info and !$imported) {
+        if ($profile_info and !$imported) {
         // Create a BOINC account unless importing from BOINC
         $user_params = array(
-          'email_addr' => $edit['mail'],
-          'name' => $edit['boincuser_name'],
-          'passwd_hash' => md5($edit['pass'].$edit['mail']),
-          'country' => $profile_info['country'],
-          'postal_code' => $profile_info['zip']
+            'email_addr' => $edit['mail'],
+            'name' => $edit['boincuser_name'],
+            'passwd_hash' => md5($edit['pass'].$edit['mail']),
+            'country' => $profile_info['country'],
+            'postal_code' => $profile_info['zip']
         );
         $boinc_user = boincuser_register_make_user($user_params);
         if (!$boinc_user) {
-          // Account exists with this email addr
-          form_set_error('email', bts('Error creating BOINC account.'));
-          return;
+            // Account exists with this email addr
+            form_set_error('email', bts('Error creating BOINC account.'));
+            return;
         }
         // Add user to community role by default (not banned)
         $unrestricted_role = array_search('community member', user_roles(true)); 
         $edit['roles'] = array(
-          $unrestricted_role => ''
+            $unrestricted_role => ''
         );
         // Set profile data in BOINC db
         db_set_active('boinc');
         // TODO: add language, image support; error handling
         $reference = db_query("INSERT INTO {profile} SET userid='%d', response1='%s', response2='%s', verification=0", $boinc_user->id, $profile_info['background'], $profile_info['opinions']);
         if (!$reference) {
-          drupal_set_message(t('Error creating BOINC profile.'), 'error');
+            drupal_set_message(t('Error creating BOINC profile.'), 'error');
         }
         $reference = db_query("UPDATE {user} SET url='%s' WHERE id='%d'", $profile_info['url'], $boinc_user->id);
         if (!$reference) {
-          drupal_set_message(t('Error updating BOINC account.'), 'error');
+            drupal_set_message(t('Error updating BOINC account.'), 'error');
         }
         db_set_active('default');
         // Cross reference Drupal account with BOINC
         $reference = db_query("INSERT INTO {boincuser} SET uid='%d', boinc_id='%d'", $account->uid, $boinc_user->id);
         if (!$reference) {
-          drupal_set_message(t('Error connecting BOINC account.'), 'error');
-          return;
+            drupal_set_message(t('Error connecting BOINC account.'), 'error');
+            return;
         }
         // Don't save custom fields to the Drupal user object
         $edit['boincuser_name'] = null;
-      }
-      break;
+        }
+        break;
       
     case 'update':
       if (isset($edit['update_source'])) {
@@ -401,9 +401,9 @@ 
         case 'user_account':
           // Ensure that BOINC data is altered
           $boinc_user = BoincUser::lookup_id($account->boincuser_id);
-          $changing_email = ($edit['mail'] AND $edit['mail'] != $boinc_user->email_addr) ? true : false;
-          $changing_pass = ($edit['pass']) ? true : false;
-          if ($changing_email OR $changing_pass) {
+            $changing_email = ($edit['mail'] AND $edit['mail'] != $boinc_user->email_addr) ? true : false;
+            $changing_pass = ($edit['pass']) ? true : false;
+            if ($changing_email OR $changing_pass) {
             // Set password hash appropriately
             $passwd = ($edit['pass']) ? $edit['pass'] : $edit['current_pass'];
             $passwd_hash = md5($passwd.$edit['mail']);
@@ -412,29 +412,29 @@ 
             $result = $boinc_user->update(
                 "email_addr='{$email_addr}', passwd_hash='{$passwd_hash}'"
             );
-          }
-          break;
+            }
+            break;
         case 'user_profile':
           if ($edit['boincuser_name'] != $boinc_user->name) {
             $boincuser_name = $edit['boincuser_name'];
             $result = $boinc_user->update(
                 "name='{$boincuser_name}'"
             );
-          }
-          break;
+            }
+            break;
         default:
         }
         // We don't want to save update source or duplicate custom fields, so
         // remove them before continuing to core Drupal routines
         $edit['update_source'] = null;
         $edit['boincuser_name'] = null;
-      }
-      break;
+        }
+        break;
       
     default:
       
     }
-  }
+    }
 }
 
 /**
@@ -442,13 +442,13 @@ 
  * Obsolete in Drupal 7...
  */
 function boincuser_nodeapi(&$node, $op, $a3 = null, $a4 = null) {
-  // In Drupal 7, these operation cases will all exist as their own hooks,
-  // so let's approximate that here so that this function can simply be removed
-  // upon migration to 7
-  switch($op) {
-  case 'update':
+    // In Drupal 7, these operation cases will all exist as their own hooks,
+    // so let's approximate that here so that this function can simply be removed
+    // upon migration to 7
+    switch($op) {
+    case 'update':
     boincuser_node_update($node);
-  }
+    }
 }
 
 /**
@@ -456,8 +456,8 @@ 
  * is updated (forward compatible to Drupal 7)
  */
 function boincuser_node_update($node) {
-  switch($node->type) {
-  case 'profile':
+    switch($node->type) {
+    case 'profile':
     // Update the BOINC database directly
     $account = user_load($node->uid);
     // Save user account data
@@ -469,11 +469,11 @@ 
       UPDATE user
       SET country = '%s', postal_code = '%s', url = '%s', has_profile = 1
       WHERE id = %d",
-      $country, $postal_code, $url, $account->boincuser_id
+        $country, $postal_code, $url, $account->boincuser_id
     );
     db_set_active('default');
     if (!$account_updated) {
-      drupal_set_message(t('Error saving BOINC account info.'), 'error');
+        drupal_set_message(t('Error saving BOINC account info.'), 'error');
     }
     // Save profile data
     $response1 = $node->field_background[0]['value'];
@@ -484,16 +484,16 @@ 
       SET userid = %d, response1 = '%s', response2 = '%s'
       ON DUPLICATE KEY UPDATE
         response1 = '%s', response2 = '%s'",
-      $account->boincuser_id, $response1, $response2,
-      $response1, $response2
+        $account->boincuser_id, $response1, $response2,
+        $response1, $response2
     );
     db_set_active('default');
     if (!$profile_updated) {
-      drupal_set_message(t('Error saving BOINC profile.'), 'error');
+        drupal_set_message(t('Error saving BOINC profile.'), 'error');
     }
     break;
     
-  default:
+    default:
     
   }
 }
@@ -502,19 +502,19 @@ 
 * Implementation of hook_views_api()
 */
 function boincuser_views_api() {
-  return array(
+    return array(
     'api' => 2.0,
     'path' => drupal_get_path('module', 'boincuser')
-  );
+    );
 }
 
 /**
 * Implementation of hook_form_alter()
 */
 function boincuser_form_alter(&$form, $form_state, $form_id) {
-  global $user;
-  switch ($form_id) {
-  case 'flag_confirm':
+    global $user;
+    switch ($form_id) {
+    case 'flag_confirm':
     // The URL seems to be the only way to put any kind of context to this
     // request!
     $action = arg(2);
@@ -523,25 +523,25 @@ 
     
     // Wrap action buttons for styling consistency
     $form['form control tabs prefix'] = array(
-      '#value' => '<ul class="form-control tab-list">',
-      '#weight' => 1001,
+        '#value' => '<ul class="form-control tab-list">',
+        '#weight' => 1001,
     );
     
     switch ($flag_type) {
     case 'friend':
       $friend_id = $form['content_id']['#value'];
-      $flag = flag_get_flag('friend');
-      $friend_status = flag_friend_determine_friend_status($flag, $friend_id, $user->uid);
+        $flag = flag_get_flag('friend');
+        $friend_status = flag_friend_determine_friend_status($flag, $friend_id, $user->uid);
       
-      // General friend form overrides
-      $form['flag_friend_submit']['#prefix'] = '<li class="first tab">';
-      $form['flag_friend_submit']['#value'] = bts('Send request');
-      $form['flag_friend_submit']['#type'] = 'submit';
-      $form['flag_friend_submit']['#suffix'] = '</li>';
-      $form['flag_friend_submit']['#weight'] = 1002;
+        // General friend form overrides
+        $form['flag_friend_submit']['#prefix'] = '<li class="first tab">';
+        $form['flag_friend_submit']['#value'] = bts('Send request');
+        $form['flag_friend_submit']['#type'] = 'submit';
+        $form['flag_friend_submit']['#suffix'] = '</li>';
+        $form['flag_friend_submit']['#weight'] = 1002;
       
-      switch ($friend_status) {
-      case FLAG_FRIEND_BOTH:
+        switch ($friend_status) {
+        case FLAG_FRIEND_BOTH:
       case FLAG_FRIEND_FLAGGED:
         unset($form['actions']);
         $form['flag_friend_submit']['#value'] = bts('Remove friend');
@@ -550,53 +550,53 @@ 
         $form['#submit'][] = 'boincuser_fix_unfriend_form_submit';
         $form['#submit'][] = $final_handler;
         break;
-      case FLAG_FRIEND_PENDING:
+        case FLAG_FRIEND_PENDING:
         unset($form['actions']);
         $form['flag_friend_submit']['#value'] = bts('Remove request');
         break;
-      case FLAG_FRIEND_APPROVAL:
+        case FLAG_FRIEND_APPROVAL:
         if ($action == 'flag') {
-          $form['flag_friend_submit']['#value'] = bts('Approve request');
+            $form['flag_friend_submit']['#value'] = bts('Approve request');
         }
         elseif ($action == 'unflag') {
-          unset($form['actions']);
-          $form['flag_friend_submit']['#value'] = bts('Deny request');
+            unset($form['actions']);
+            $form['flag_friend_submit']['#value'] = bts('Deny request');
         }
         break;
-      case FLAG_FRIEND_UNFLAGGED:
+        case FLAG_FRIEND_UNFLAGGED:
       default:
         $user_links[] = array(
-          'title' => bts('Add as friend'),
-          'href' => "flag/confirm/flag/friend/{$account->uid}"
+            'title' => bts('Add as friend'),
+            'href' => "flag/confirm/flag/friend/{$account->uid}"
         );
-      }
-      break;
+        }
+        break;
       
     default:
     }
     
     $form['cancel'] = array(
-      '#value' => '<li class="tab">' . l(bts('Cancel'), $cancel_dest) . '</li>',
-      '#weight' => 1004,
+        '#value' => '<li class="tab">' . l(bts('Cancel'), $cancel_dest) . '</li>',
+        '#weight' => 1004,
     );
     $form['form control tabs suffix'] = array(
-      '#value' => '</ul>',
-      '#weight' => 1010,
+        '#value' => '</ul>',
+        '#weight' => 1010,
     );
     
     break;
     
-  // General node edit form
-  case 'news_node_form':
+    // General node edit form
+    case 'news_node_form':
     $form['separator_bottom'] = array(
-      '#value' => '<div class="separator buttons"></div>',
-      '#weight' => 999,
+        '#value' => '<div class="separator buttons"></div>',
+        '#weight' => 999,
     );
     
     // Wrap action buttons for styling consistency
     $form['buttons']['form control tabs prefix'] = array(
-      '#value' => '<ul class="form-control tab-list">',
-      '#weight' => 1001,
+        '#value' => '<ul class="form-control tab-list">',
+        '#weight' => 1001,
     );
     $form['buttons']['submit']['#prefix'] = '<li class="first tab">';
     $form['buttons']['submit']['#value'] = bts('Save changes');
@@ -609,15 +609,15 @@ 
     $form['buttons']['preview_changes']['#suffix'] = '</li>';
     $form['buttons']['preview_changes']['#weight'] = 1004;
     $form['buttons']['cancel'] = array(
-      '#value' => '<li class="tab">' . l(bts('Cancel'), "node/{$form['nid']['#value']}") . '</li>',
-      '#weight' => 1005,
+        '#value' => '<li class="tab">' . l(bts('Cancel'), "node/{$form['nid']['#value']}") . '</li>',
+        '#weight' => 1005,
     );
     $form['buttons']['delete']['#prefix'] = '<li class="tab">';
     $form['buttons']['delete']['#suffix'] = '</li>';
     $form['buttons']['delete']['#weight'] = 1006;
     $form['buttons']['form control tabs suffix'] = array(
-      '#value' => '</ul>',
-      '#weight' => 1010,
+        '#value' => '</ul>',
+        '#weight' => 1010,
     );
     
     // Preview is ugly, unset until it works
@@ -625,40 +625,40 @@ 
 
     break;
   
-  case 'node_delete_confirm':
+    case 'node_delete_confirm':
     $form['separator_bottom'] = array(
-      '#value' => '<div class="separator buttons"></div>',
-      '#weight' => 999,
+        '#value' => '<div class="separator buttons"></div>',
+        '#weight' => 999,
     );
     
     // Wrap action buttons for styling consistency
     $form['actions']['form control tabs prefix'] = array(
-      '#value' => '<ul class="form-control tab-list">',
-      '#weight' => 1001,
+        '#value' => '<ul class="form-control tab-list">',
+        '#weight' => 1001,
     );
     $form['actions']['submit']['#prefix'] = '<li class="first tab">';
     $form['actions']['submit']['#value'] = bts('Delete');
     $form['actions']['submit']['#suffix'] = '</li>';
     $form['actions']['submit']['#weight'] = 1002;
     $form['actions']['cancel'] = array(
-      '#value' => '<li class="tab">' . l(bts('Cancel'), "node/{$form['nid']['#value']}") . '</li>',
-      '#weight' => 1005,
+        '#value' => '<li class="tab">' . l(bts('Cancel'), "node/{$form['nid']['#value']}") . '</li>',
+        '#weight' => 1005,
     );
     $form['actions']['form control tabs suffix'] = array(
-      '#value' => '</ul>',
-      '#weight' => 1010,
+        '#value' => '</ul>',
+        '#weight' => 1010,
     );
     $form['#redirect'] = 'account/profile';
     break;
     
-  case 'privatemsg_new':
+    case 'privatemsg_new':
     
     $form['privatemsg']['body']['#title'] = '';
     
     // Wrap action buttons for styling consistency
     $form['privatemsg']['form control tabs prefix'] = array(
-      '#value' => '<ul class="form-control tab-list">',
-      '#weight' => 1001,
+        '#value' => '<ul class="form-control tab-list">',
+        '#weight' => 1001,
     );
     $form['privatemsg']['submit']['#prefix'] = '<li class="first tab">';
     $form['privatemsg']['submit']['#value'] = bts('Send message');
@@ -668,26 +668,26 @@ 
     $form['privatemsg']['preview']['#suffix'] = '</li>';
     $form['privatemsg']['preview']['#weight'] = 1003;
     $form['privatemsg']['cancel'] = array(
-      '#value' => '<li class="tab">' . l(bts('Cancel'), $_GET['q']) . '</li>',
-      '#weight' => 1004,
+        '#value' => '<li class="tab">' . l(bts('Cancel'), $_GET['q']) . '</li>',
+        '#weight' => 1004,
     );
     $form['privatemsg']['form control tabs suffix'] = array(
-      '#value' => '</ul>',
-      '#weight' => 1010,
+        '#value' => '</ul>',
+        '#weight' => 1010,
     );
     
     unset($form['privatemsg']['recipient_display']);
     
     break;
     
-  // Login form
-  case 'user_login':
+    // Login form
+    case 'user_login':
   case 'user_login_block':
     drupal_set_title(bts('Login'));
     // Replace name with email in login form
     unset($form['name']);
     array_unshift($form, array(
-      'email' => array(
+        'email' => array(
         '#type' => 'textfield',
         '#title' => bts('Email address'),
         '#size' => ($form_id == 'user_login_block') ? 15 : 60,
@@ -695,18 +695,18 @@ 
         '#required' => TRUE,
         '#attributes' => array('tabindex' => '1'),
         '#description' => bts('Enter your @s email address.', array('@s' => variable_get('site_name', 'BOINC')))
-      ),
-      'validation_source' => array(
+        ),
+        'validation_source' => array(
         '#type' => 'hidden',
         '#value' => 'user_login'
-      )
+        )
     ));
     $form['#redirect'] = 'home';
     
     // Wrap action buttons for styling consistency
     $form['buttons']['form control tabs prefix'] = array(
-      '#value' => '<ul class="form-control tab-list">',
-      '#weight' => 1001,
+        '#value' => '<ul class="form-control tab-list">',
+        '#weight' => 1001,
     );
     $form['buttons']['submit'] = $form['submit'];
     $form['buttons']['submit']['#prefix'] = '<li class="first tab">';
@@ -714,8 +714,8 @@ 
     $form['buttons']['submit']['#suffix'] = '</li>';
     $form['buttons']['submit']['#weight'] = 1002;
     $form['buttons']['form control tabs suffix'] = array(
-      '#value' => '</ul>',
-      '#weight' => 1010,
+        '#value' => '</ul>',
+        '#weight' => 1010,
     );
     unset($form['submit']);
     
@@ -724,23 +724,23 @@ 
     isset($form['buttons']['submit']['#attributes']) ? array_push($form['buttons']['submit']['#attributes'], array('tabindex' => '3')) : $form['buttons']['submit']['#attributes'] = array('tabindex' => '3');
     // If the user login form is being submitted, use BOINC validation handler.
     if (isset($form_state['post']['email']) and isset($form_state['post']['pass'])) {
-      // Find the local validation function's entry so we can replace it.
-      $array_key = array_search('user_login_authenticate_validate', $form['#validate']);
-      if ($array_key === FALSE) {
+        // Find the local validation function's entry so we can replace it.
+        $array_key = array_search('user_login_authenticate_validate', $form['#validate']);
+        if ($array_key === FALSE) {
         // Could not find it. Some other module must have run form_alter().
         // We will simply add our validation just before the final validator.
         $final_validator = array_pop($form['#validate']);
         $form['#validate'][] = 'boincuser_login_validate';
         $form['#validate'][] = $final_validator;
-      } else {
+        } else {
         // Replace the local validation function with BOINC validation
         $form['#validate'][$array_key] = 'boincuser_login_validate';
-      }
+        }
     }
     break;
     
-  // User credentials form
-  case 'user_profile_form':
+    // User credentials form
+    case 'user_profile_form':
     
     // Use the displaly name as the title, not the username
     $account = user_load($form['#uid']);
@@ -750,7 +750,7 @@ 
     // coming from the password reset function
     $reset_pass = (strpos($_SERVER['HTTP_REFERER'], "/user/reset/{$form['#uid']}") === FALSE) ? 0 : 1;
     if ($reset_pass) {
-      $_SESSION['reset_pass'] = 1;
+        $_SESSION['reset_pass'] = 1;
     }
     
     $form['account']['name']['#size'] = 40;
@@ -759,50 +759,50 @@ 
     $form['account']['pass']['#size'] = 17;
     
     if (!$reset_pass AND ($user->uid == $account->uid OR !user_access('administer users'))) {
-      // Add a password authenticator, required to change email or pw
-      $form['account']['current_pass'] = array(
+        // Add a password authenticator, required to change email or pw
+        $form['account']['current_pass'] = array(
         '#type' => 'password',
         '#title' => bts('Enter your password to save changes'),
         '#description' => bts('Enter your current password if changing your email
           address or password.'),
         '#size' => 17,
         '#attributes' => array(
-          'autocomplete' => 'off',
+            'autocomplete' => 'off',
         ),
-      );
+        );
     }
     
     // Add account keys, CPID, etc
     $form['account']['boincuser_id'] = array(
-      '#value' => '
+        '#value' => '
         <div class="form-item">
           <label>' . bts('BOINC user ID') . '</label>
           <span>' . $account->boincuser_id . '</span>
         </div>',
     );
     $form['account']['user_id'] = array(
-      '#value' => '
+        '#value' => '
         <div class="form-item">
           <label>' . bts('Drupal user ID') . '</label>
           <span>' . $account->uid . '</span>
         </div>',
     );
     $form['account']['account_key'] = array(
-      '#value' => '
+        '#value' => '
         <div class="form-item">
           <label>' . bts('Account key') . '</label>
           <span>' . $account->boincuser_account_key . '</span>
         </div>',
     );
     $form['account']['weak_account_key'] = array(
-      '#value' => '
+        '#value' => '
         <div class="form-item">
           <label>' . bts('Weak account key') . '</label>
           <span>' . "{$account->boincuser_id}_{$account->boincuser_weak_auth}" . '</span>
         </div>',
     );
     $form['account']['cpid'] = array(
-      '#value' => '
+        '#value' => '
         <div class="form-item">
           <label>' . bts('Cross-project ID') . '</label>
           <span>' . $account->boincuser_cpid . '</span>
@@ -810,30 +810,30 @@ 
     );
     
     $form['account']['separator_bottom'] = array(
-      '#value' => '<div class="separator buttons"></div>'
+        '#value' => '<div class="separator buttons"></div>'
     );
     
     // Wrap action buttons for styling consistency
     $form['form control tabs prefix'] = array(
-      '#value' => '<ul class="form-control tab-list">',
-      '#weight' => 1001,
+        '#value' => '<ul class="form-control tab-list">',
+        '#weight' => 1001,
     );
     $form['submit']['#prefix'] = '<li class="first tab">';
     $form['submit']['#value'] = bts('Save changes');
     $form['submit']['#suffix'] = '</li>';
     $form['submit']['#weight'] = 1002;
     if (isset($form['delete']) AND is_array($form['delete'])) {
-      $form['delete']['#prefix'] = '<li class="tab">';
-      $form['delete']['#suffix'] = '</li>';
-      $form['delete']['#weight'] = 1003;
+        $form['delete']['#prefix'] = '<li class="tab">';
+        $form['delete']['#suffix'] = '</li>';
+        $form['delete']['#weight'] = 1003;
     }
     $form['cancel'] = array(
-      '#value' => '<li class="tab">' . l(bts('Cancel'), $_GET['q']) . '</li>',
-      '#weight' => 1004,
+        '#value' => '<li class="tab">' . l(bts('Cancel'), $_GET['q']) . '</li>',
+        '#weight' => 1004,
     );
     $form['form control tabs suffix'] = array(
-      '#value' => '</ul>',
-      '#weight' => 1010,
+        '#value' => '</ul>',
+        '#weight' => 1010,
     );
     
     // Rearrange form elements
@@ -864,41 +864,41 @@ 
     
     // Internal fields to indicate where these user changes are taking place
     array_unshift($form, array(
-      'validation_source' => array(
+        'validation_source' => array(
         '#type' => 'hidden',
         '#value' => 'user_account'
-      ),
-      'update_source' => array(
+        ),
+        'update_source' => array(
         '#type' => 'hidden',
         '#value' => 'user_account'
-      )
+        )
     ));
     
     break;
     
-  case 'profile_node_form':
+    case 'profile_node_form':
     
     // Use the display name as the title, not the username
     $account = user_load($form['uid']['#value']);
     drupal_set_title($account->boincuser_name);
     
     $form['separator_bottom'] = array(
-      '#value' => '<div class="separator buttons"></div>',
-      '#weight' => 999,
+        '#value' => '<div class="separator buttons"></div>',
+        '#weight' => 999,
     );
     
     if (module_exists('captcha')) {
-      // Add an optional captcha
-      $form['profile_captcha'] = array(
+        // Add an optional captcha
+        $form['profile_captcha'] = array(
         '#type' => 'captcha',
         '#weight' => 1000,
-      );
+        );
     }
     
     // Wrap action buttons for styling consistency
     $form['buttons']['form control tabs prefix'] = array(
-      '#value' => '<ul class="form-control tab-list">',
-      '#weight' => 1001,
+        '#value' => '<ul class="form-control tab-list">',
+        '#weight' => 1001,
     );
     $form['buttons']['submit']['#prefix'] = '<li class="first tab">';
     $form['buttons']['submit']['#value'] = bts('Save changes');
@@ -911,15 +911,15 @@ 
     $form['buttons']['preview_changes']['#suffix'] = '</li>';
     $form['buttons']['preview_changes']['#weight'] = 1004;
     $form['buttons']['cancel'] = array(
-      '#value' => '<li class="tab">' . l(bts('Cancel'), $_GET['q']) . '</li>',
-      '#weight' => 1005,
+        '#value' => '<li class="tab">' . l(bts('Cancel'), $_GET['q']) . '</li>',
+        '#weight' => 1005,
     );
     $form['buttons']['delete']['#prefix'] = '<li class="tab">';
     $form['buttons']['delete']['#suffix'] = '</li>';
     $form['buttons']['delete']['#weight'] = 1006;
     $form['buttons']['form control tabs suffix'] = array(
-      '#value' => '</ul>',
-      '#weight' => 1010,
+        '#value' => '</ul>',
+        '#weight' => 1010,
     );
     
     $form['#after_build'][] = 'boincuser_profile_node_form_after_build';
@@ -936,79 +936,79 @@ 
     
     // Internal fields to indicate where these user changes are taking place
     array_unshift($form, array(
-      'validation_source' => array(
+        'validation_source' => array(
         '#type' => 'hidden',
         '#value' => 'user_profile'
-      ),
-      'update_source' => array(
+        ),
+        'update_source' => array(
         '#type' => 'hidden',
         '#value' => 'user_profile'
-      )
+        )
     ));
     break;
   
-  // Registration form
-  case 'user_register':
+    // Registration form
+    case 'user_register':
     array_unshift($form, array(
-      'boincuser_name' => array(
+        'boincuser_name' => array(
         '#type' => 'textfield', 
         '#title' => bts('Name'), 
         '#default_value' => $edit['boincuser_name'], 
         '#maxlength' => USERNAME_MAX_LENGTH, 
         '#description' => bts('Spaces are allowed; punctuation is not allowed except for periods, hyphens, and underscores.'), 
         '#required' => TRUE
-      ),
-      'validation_source' => array(
+        ),
+        'validation_source' => array(
         '#type' => 'hidden',
         '#value' => 'user_register'
-      )
+        )
     ));
     // Set name temporarily to dummy value to beat validation
     $form['name'] = array(
-      '#type' => 'hidden',
-      '#value' => rand() . '.' . time()
+        '#type' => 'hidden',
+        '#value' => rand() . '.' . time()
     );
     
     if (module_exists('captcha')) {
-      // Add an optional captcha
-      $form['register_captcha'] = array(
+        // Add an optional captcha
+        $form['register_captcha'] = array(
         '#type' => 'captcha',
         '#weight' => 1000,
-      );
+        );
     }
     
     $form['submit']['#weight'] = 1001;
     break;
     
-  // Request new password form
-  case 'user_pass':
+    // Request new password form
+    case 'user_pass':
     drupal_set_title(bts('Forgot password'));
     // Replace name/email text box with email only; retain "name" label
     // for compatibility with standard Drupal submit function
     unset($form['name']);
     array_unshift($form, array(
-      'name' => array(
+        'name' => array(
         '#type' => 'textfield',
         '#title' => bts('Email address'),
         '#size' => 60,
         '#maxlength' => EMAIL_MAX_LENGTH,
         '#required' => TRUE,
         '#description' => bts(
-          'Enter your email address to receive instructions for resetting your password (or use the !authenticator_login).',
-          array(
+            'Enter your email address to receive instructions for resetting your password (or use the !authenticator_login).',
+            array(
             '!authenticator_login' => l(
-              bts('authenticator-based login'),
-              'user/login/auth'
+                bts('authenticator-based login'),
+                'user/login/auth'
+            )
             )
-          )
         ),
-      ),
+        ),
     ));
     
     // Wrap action buttons for styling consistency
     $form['buttons']['form control tabs prefix'] = array(
-      '#value' => '<ul class="form-control tab-list">',
-      '#weight' => 1001,
+        '#value' => '<ul class="form-control tab-list">',
+        '#weight' => 1001,
     );
     $form['buttons']['submit'] = $form['submit'];
     $form['buttons']['submit']['#prefix'] = '<li class="first tab">';
@@ -1016,26 +1016,26 @@ 
     $form['buttons']['submit']['#suffix'] = '</li>';
     $form['buttons']['submit']['#weight'] = 1002;
     $form['buttons']['cancel'] = array(
-      '#value' => '<li class="tab">' . l(bts('Cancel'), 'user/login') . '</li>',
-      '#weight' => 1005,
+        '#value' => '<li class="tab">' . l(bts('Cancel'), 'user/login') . '</li>',
+        '#weight' => 1005,
     );
     $form['buttons']['form control tabs suffix'] = array(
-      '#value' => '</ul>',
-      '#weight' => 1010,
+        '#value' => '</ul>',
+        '#weight' => 1010,
     );
     unset($form['submit']);
     
     // If the form is being submitted, use BOINC validation handler.
     if (isset($form_state['post']['name'])) {
-      // Prepend the BOINC validation function to local validation
-      array_unshift($form['#validate'], 'boincuser_request_pass_validate');
+        // Prepend the BOINC validation function to local validation
+        array_unshift($form['#validate'], 'boincuser_request_pass_validate');
     }
     break;
     
-  case 'views_exposed_form':
+    case 'views_exposed_form':
     $form['submit']['#value'] = bts('Search');
     break;
-  }
+    }
 }
 
 /**
@@ -1043,10 +1043,10 @@ 
  * built; this is called from boincuser_form_alter()
  */
 function boincuser_profile_node_form_after_build($form, &$form_state) {
-  // Move to community prefs form
-  $form_state['storage']['avatar'] = $form['field_image'];
-  unset($form['field_image']);
-  return $form;
+    // Move to community prefs form
+    $form_state['storage']['avatar'] = $form['field_image'];
+    unset($form['field_image']);
+    return $form;
 }
 
 /**
@@ -1054,8 +1054,8 @@ 
  * @see http://api.drupal.org/api/drupal/developer--hooks--core.php/function/hook_elements/6
  */
 function boincuser_elements() {
-  $type['password_confirm']['#process'][] = 'boincuser_process_password_confirm';
-  return $type;
+    $type['password_confirm']['#process'][] = 'boincuser_process_password_confirm';
+    return $type;
 }
 
 /**
@@ -1063,11 +1063,11 @@ 
  * profile form
  */
 function boincuser_process_password_confirm($element) {
-  // Check if parent element is "account".
-  if ($element['#array_parents'][0] == 'account') {
+    // Check if parent element is "account".
+    if ($element['#array_parents'][0] == 'account') {
     $element['pass1']['#title'] = bts('Change password');
-  }
-  return $element;
+    }
+    return $element;
 }
 
 /**
@@ -1075,40 +1075,40 @@ 
  * Register theme functions for use in this module.
  */
 function boincuser_theme($existing, $type, $theme, $path) {
-  return array(
+    return array(
     'boincuser_user_pass' => array(
-      'arguments' => array()
+        'arguments' => array()
     )
-  );
+    );
 }
 
 /**
  * Implementation of hook_token_values
  */
 function boincuser_token_values($type, $object = NULL, $options = array()) {
-  if ($type == 'user') {
+    if ($type == 'user') {
     $account = user_load($object->uid);
     $tokens['display-name'] = $account->boincuser_name;
     return $tokens;
-  }
+    }
 }
 
 /**
  * Implementation of hook_token_list
  */
 function boincuser_token_list($type = 'all') {
-  if ($type == 'user' || $type == 'all') {
+    if ($type == 'user' || $type == 'all') {
     $tokens['user']['display-name']      = t("The user's name that should be displayed");
     return $tokens;
-  }
+    }
 }
 
 /**
  * Implementation of hook_views_pre_execute()
  */
 function boincuser_views_pre_execute(&$view) {
-  $account_id = $view->args[0];
-  if ($view->name=="user_activity") {
+    $account_id = $view->args[0];
+    if ($view->name=="user_activity") {
     // Run the following custom query for the user_activity view
     $view->build_info['query']= "
       SELECT node_revisions.vid AS vid,
@@ -1140,7 +1140,7 @@ 
     
     // count_query determines the pager.  Do this so the right item count is returned.
     $view->build_info['count_query'] = $view->build_info['query'];
-  }
+    }
 }
 
 
@@ -1152,17 +1152,17 @@ 
  * Page callback shortcut to recent posts for the logged in user
  */
 function boincuser_goto_recent_posts() {
-  global $user;
-  drupal_goto("account/{$user->uid}/posts");
+    global $user;
+    drupal_goto("account/{$user->uid}/posts");
 }
 
 /**
  * Page callback shortcut to the team of the logged in user
  */
 function boincuser_goto_team() {
-  global $user;
-  $account = user_load($user->uid);
-  drupal_goto("community/teams/{$account->team}");
+    global $user;
+    $account = user_load($user->uid);
+    drupal_goto("community/teams/{$account->team}");
 }
 
 /**
@@ -1171,58 +1171,58 @@ 
  * user profile pages, so use a wrapper for display
  */
 function boincuser_view_profile($account = null) {
-  // Create the user profile page
-  if (!$account) {
+    // Create the user profile page
+    if (!$account) {
     global $user;
     $account = $user;
-  }
-  // For now, just call the user module profile view function
-  user_build_content($account);
-  return theme('user_profile', $account);
+    }
+    // For now, just call the user module profile view function
+    user_build_content($account);
+    return theme('user_profile', $account);
 }
 
 /**
  * Page callback for editing a user profile
  */
 function boincuser_edit_profile($account = null) {
-  // Create the user profile form
-  if (!$account) {
+    // Create the user profile form
+    if (!$account) {
     global $user;
     $account = $user;
-  }
-  // Render the form
-  module_load_include('pages.inc', 'node', 'node');
-  return content_profile_page_edit('profile', $account);
+    }
+    // Render the form
+    module_load_include('pages.inc', 'node', 'node');
+    return content_profile_page_edit('profile', $account);
 }
 
 /**
-  * Join page menu callback.
-  * Display instructions on joining for new or existing BOINC users
-  */
+ * Join page menu callback.
+ * Display instructions on joining for new or existing BOINC users
+ */
 function join_page($type = null) {
-  global $base_url;
-  $registration_enabled = variable_get('user_register', 0);
-  $output = '<div class="join">';
-  switch ($type) {
-  case 'boinc':
+    global $base_url;
+    $registration_enabled = variable_get('user_register', 0);
+    $output = '<div class="join">';
+    switch ($type) {
+    case 'boinc':
     $output .= '<p>' . bts("If you're already running BOINC, select <i>Add
       Project</i>.") . '</p>';
     if ($registration_enabled) {
-      $output .= '<p>' . bts("If you're running a command-line or pre-5.0 version
+        $output .= '<p>' . bts("If you're running a command-line or pre-5.0 version
         of BOINC, !create_an_account first.", array('!create_an_account' => 
-          l(bts('create an account'), 'user/registration'))) . '</p>';
+            l(bts('create an account'), 'user/registration'))) . '</p>';
     }
     else {
-      $output .= '<p>' . bts("If you're running a command-line version of BOINC,
+        $output .= '<p>' . bts("If you're running a command-line version of BOINC,
         first use the BOINC Manager software (elsewhere if necessary) to create an
         an account at this project, then use that account to connect with the
         command-line version.") . '</p>';
-      $output .= '<p>' . bts("If you're running a pre-5.0 version of BOINC, please
+        $output .= '<p>' . bts("If you're running a pre-5.0 version of BOINC, please
         upgrade to a more recent version to create an account
         at this project.") . '</p>';
     }
     break;
-  case 'new':
+    case 'new':
   default:
     $output .= '<ol>';
     $output .= '  <li>' . bts('Download the desktop software');
@@ -1237,71 +1237,71 @@ 
     $output .= '  </li>';
     $output .= '  <li>' . bts('Run the installer') . '</li>';
     $output .= '  <li>' . bts('When prompted enter @siteurl', array(
-      '@siteurl' => $base_url)) . '</li>';
+        '@siteurl' => $base_url)) . '</li>';
     $output .=  '</ol>';
-  }
-  $output .= '</div>';
-  return $output;
+    }
+    $output .= '</div>';
+    return $output;
 }
 
 /**
-  * Home page content for embedding in Panels page
-  */
+ * Home page content for embedding in Panels page
+ */
 function boincuser_home_page() {
-  global $user;
-  $site_name = variable_get('site_name', '');
-  // get the front page message from database; this is set in the admin interface under BOINC Other
-  $site_message = variable_get('boinc_other_frontpage','');
+    global $user;
+    $site_name = variable_get('site_name', '');
+    // get the front page message from database; this is set in the admin interface under BOINC Other
+    $site_message = variable_get('boinc_other_frontpage','');
 
-  // Determine the user of the day
-  $current_uotd = db_fetch_object(db_query("
+    // Determine the user of the day
+    $current_uotd = db_fetch_object(db_query("
     SELECT
       uid,
       uotd_time
     FROM {boincuser}
     ORDER BY uotd_time DESC
     LIMIT 1"
-  ));
-  if ($current_uotd->uotd_time < strtotime('today midnight')) {
+    ));
+    if ($current_uotd->uotd_time < strtotime('today midnight')) {
     $uotd = boincuser_select_user_of_the_day();
-  }
-  else {
+    }
+    else {
     $uotd = user_load($current_uotd->uid);
-  }
-  $uotd_image = boincuser_get_user_profile_image($uotd->uid, FALSE);
-  $output = '<h2 class="pane-title">';
-  $output .= ($user->uid) ? bts('Welcome back!') : ($site_name ? bts('What is @this_project?', array('@this_project' => $site_name)) : bts('Welcome!'));
-  $output .= '</h2>';
-  $output .= '<div class="boinc-overview balance-height-front">';
-  $output .= '  <div>' . bts($site_message, array(), NULL, "project:front page") . ' ' . l(bts('Learn more'), 'about') . '</div>';
-  if ($user->uid) {
+    }
+    $uotd_image = boincuser_get_user_profile_image($uotd->uid, FALSE);
+    $output = '<h2 class="pane-title">';
+    $output .= ($user->uid) ? bts('Welcome back!') : ($site_name ? bts('What is @this_project?', array('@this_project' => $site_name)) : bts('Welcome!'));
+    $output .= '</h2>';
+    $output .= '<div class="boinc-overview balance-height-front">';
+    $output .= '  <div>' . bts($site_message, array(), NULL, "project:front page") . ' ' . l(bts('Learn more'), 'about') . '</div>';
+    if ($user->uid) {
     $output .= '  <div>' . l(bts('View account'), 'dashboard', array('attributes' => array('class' => 'join button'))) . '</div>';
-  }
-  else {
+    }
+    else {
     $output .= '  <div>' . l(bts('Join now'), 'join', array('attributes' => array('class' => 'join button'))) . '</div>';
-  }
-  $output .= '</div>';
-  $output .= '<div class="boinc-overview-details">';
-  $output .= '  <div class="detail-container">';
-  $output .= '    <a class="user-of-the-day" href="account/' . $uotd->uid . '">';
-  $output .= '      <div class="picture">';
-  $output .= theme('imagefield_image', $uotd_image['image'], $uotd_image['alt'],
+    }
+    $output .= '</div>';
+    $output .= '<div class="boinc-overview-details">';
+    $output .= '  <div class="detail-container">';
+    $output .= '    <a class="user-of-the-day" href="account/' . $uotd->uid . '">';
+    $output .= '      <div class="picture">';
+    $output .= theme('imagefield_image', $uotd_image['image'], $uotd_image['alt'],
     $uotd_image['alt'], array(), FALSE);
-  $output .= '      </div>';
-  $output .= '      <div class="text">' . bts('User of the day') . '</div>';
-  $output .= '      <div class="detail">' . $uotd->boincuser_name . '</div>';
-  $output .= '    </a>';
-  $output .= '    <div class="volunteers">';
-  $output .= '      <div class="text">' . bts('Over 500,000 volunteers and counting.') . '</div>';
-  $output .= '      <div class="platforms">';
-  $output .= '        <div class="detail platform windows">' . bts('Windows') . '</div>';
-  $output .= '        <div class="detail platform mac">' .  bts('Mac') . '</div>';
-  $output .= '        <div class="detail platform linux">' . bts('Linux') . '</div>';
-  $output .= '      </div>';
-  $output .= '    </div>';
-  $output .= '  </div>';
-  $output .= '</div>';
-  return $output;
+    $output .= '      </div>';
+    $output .= '      <div class="text">' . bts('User of the day') . '</div>';
+    $output .= '      <div class="detail">' . $uotd->boincuser_name . '</div>';
+    $output .= '    </a>';
+    $output .= '    <div class="volunteers">';
+    $output .= '      <div class="text">' . bts('Over 500,000 volunteers and counting.') . '</div>';
+    $output .= '      <div class="platforms">';
+    $output .= '        <div class="detail platform windows">' . bts('Windows') . '</div>';
+    $output .= '        <div class="detail platform mac">' .  bts('Mac') . '</div>';
+    $output .= '        <div class="detail platform linux">' . bts('Linux') . '</div>';
+    $output .= '      </div>';
+    $output .= '    </div>';
+    $output .= '  </div>';
+    $output .= '</div>';
+    return $output;
 }
 
 /**
@@ -1309,106 +1309,106 @@ 
  * Create a new user account based on supplied parameters.
  */
 function boincuser_create_account() {
-  require_boinc('boinc_db');
-  require_boinc('user');
-  require_boinc('xml');
-  $params = array(
+    require_boinc('boinc_db');
+    require_boinc('user');
+    require_boinc('xml');
+    $params = array(
     'email_addr' => isset($_GET['email_addr']) ? $_GET['email_addr'] : '',
     'user_name' => isset($_GET['user_name']) ? $_GET['user_name'] : '',
     'passwd_hash' => isset($_GET['passwd_hash']) ? $_GET['passwd_hash'] : ''
-  );
+    );
   
-  // Begin output
-  xml_header();
+    // Begin output
+    xml_header();
   
-  // Account creation disabled
+    // Account creation disabled
   
-  // Invalid invite code
+    // Invalid invite code
   
-  // Validate input
-  if (user_validate_mail($params['email_addr']) or !is_valid_email_addr($params['email_addr'])) {
+    // Validate input
+    if (user_validate_mail($params['email_addr']) or !is_valid_email_addr($params['email_addr'])) {
     xml_error(-205);
-  }
-  if ($error = user_validate_name($params['user_name'])) {
+    }
+    if ($error = user_validate_name($params['user_name'])) {
     xml_error(-205, $error);
-  }
-  if (strlen($params['passwd_hash']) != 32) {
+    }
+    if (strlen($params['passwd_hash']) != 32) {
     xml_error(-1, 'password hash length not 32');
-  }
+    }
   
-  // Process input
-  $boinc_user = BoincUser::lookup_email_addr($params['email_addr']);
-  if ($boinc_user) {
+    // Process input
+    $boinc_user = BoincUser::lookup_email_addr($params['email_addr']);
+    if ($boinc_user) {
     // Return authenticator for existing users
     if ($params['passwd_hash'] == $boinc_user->passwd_hash) {
-      $output = array('authenticator' => $boinc_user->authenticator);
+        $output = array('authenticator' => $boinc_user->authenticator);
     } else {
-      xml_error(-137);
+        xml_error(-137);
     }
-  } else {
+    } else {
     // Verify that there isn't somehow a Drupal user already (not possible with proper function)
     if ($existing_user = user_load(array('mail' => $params['email_addr']))) {
-      xml_error(-137, 'account error');
+        xml_error(-137, 'account error');
     }
     // Create new account
     $unrestricted_role = array_search('community member', user_roles(true)); 
     $newUser = array(
-      'name' => $params['user_name'],
-      'pass' => null, // $params['passwd_hash'], // note: passing a hash here requires ALL passwords to be hashed via hook prior to interacting with the hash stored in the db
-      'mail' => $params['email_addr'],
-      'status' => 1,
-      'init' => $params['email_addr'],
-      'roles' => array($unrestricted_role => ''),
+        'name' => $params['user_name'],
+        'pass' => null, // $params['passwd_hash'], // note: passing a hash here requires ALL passwords to be hashed via hook prior to interacting with the hash stored in the db
+        'mail' => $params['email_addr'],
+        'status' => 1,
+        'init' => $params['email_addr'],
+        'roles' => array($unrestricted_role => ''),
     );
     $user = user_save(null, $newUser);
     $boinc_user = make_user($params['email_addr'], $params['user_name'], $params['passwd_hash'], 'International');
     if (!$boinc_user) {
-      xml_error(-137);
+        xml_error(-137);
     }
     // Cross reference Drupal account with BOINC
     $reference = db_query("INSERT INTO {boincuser} SET uid=%d, boinc_id=%d", $user->uid, $boinc_user->id);
     if (!$reference) {
-      xml_error(-1, 'error connecting BOINC account to Drupal');
+        xml_error(-1, 'error connecting BOINC account to Drupal');
     }
-  }
-  echo " <account_out>\n";
-  echo "   <authenticator>{$boinc_user->authenticator}</authenticator>\n";
-  echo "</account_out>\n";
+    }
+    echo " <account_out>\n";
+    echo "   <authenticator>{$boinc_user->authenticator}</authenticator>\n";
+    echo "</account_out>\n";
 }
 
 /**
  * Determine if the user has permission to control community access
  */
 function boincuser_moderate_community_access() {
-  if (user_access('assign community member role')
+    if (user_access('assign community member role')
   OR user_access('assign all roles')) {
     return TRUE;
-  }
-  return FALSE;
+    }
+    return FALSE;
 }
 
 /**
  * Get the count of items in the moderation queue
  */
 function boincuser_moderation_queue_count($caller = 'user') {
-  $allowed = FALSE;
-  switch ($caller) {
-  case 'cron':
+    $allowed = FALSE;
+    switch ($caller) {
+    case 'cron':
     $allowed = TRUE;
     break;
-  case 'user':
+    case 'user':
   default:
     $allowed = user_access('edit any profile content');
-  }
-  if ($allowed) {
+    }
+    if ($allowed) {
     return db_result(db_query("
       SELECT COUNT(*)
       FROM {node}
       WHERE type = 'profile'
       AND moderate = 1"
     ));
-  }
-  return NULL;
+    }
+    return NULL;
 }
 
 /**
@@ -1417,43 +1417,43 @@ 
  * than through the user account info form.
  */
 function boincuser_control($uid = NULL, $action = NULL) {
-  if (!$uid OR !$account = user_load($uid)) {
+    if (!$uid OR !$account = user_load($uid)) {
     // What are you even doing here...
     return FALSE;
-  }
-  switch ($action) {
-  case 'ban':
+    }
+    switch ($action) {
+    case 'ban':
     if (boincuser_moderate_community_access()) {
-      $penalty_period = variable_get('boinc_penalty_period', 7*24*60*60);
-      $boincuser_record = array(
+        $penalty_period = variable_get('boinc_penalty_period', 7*24*60*60);
+        $boincuser_record = array(
         'uid' => $uid,
         'penalty_expiration' => time() + $penalty_period,
-      );
-      drupal_write_record('boincuser', $boincuser_record, 'uid');
-      $community_role = array_search('community member', user_roles(true));
-      if (isset($account->roles[$community_role])) {
+        );
+        drupal_write_record('boincuser', $boincuser_record, 'uid');
+        $community_role = array_search('community member', user_roles(true));
+        if (isset($account->roles[$community_role])) {
         unset($account->roles[$community_role]);
         user_save($account, array('roles' => $account->roles));
-      }
+        }
     }
     break;
-  case 'lift-ban':
+    case 'lift-ban':
     if (boincuser_moderate_community_access()) {
-      $boincuser_record = array(
+        $boincuser_record = array(
         'uid' => $uid,
         'penalty_expiration' => 0,
-      );
-      drupal_write_record('boincuser', $boincuser_record, 'uid');
-      $community_role = array_search('community member', user_roles(true));
-      if (!isset($account->roles[$community_role])) {
+        );
+        drupal_write_record('boincuser', $boincuser_record, 'uid');
+        $community_role = array_search('community member', user_roles(true));
+        if (!isset($account->roles[$community_role])) {
         $account->roles[$community_role] = 'community member';
         user_save($account, array('roles' => $account->roles));
-      }
+        }
     }
     break;
-  default:
+    default:
   }
-  drupal_goto("account/{$account->uid}");
+    drupal_goto("account/{$account->uid}");
 }
 
 /**
@@ -1461,29 +1461,29 @@ 
  * moderation flag.
  */
 function boincuser_moderate_profile_approve($account) {
-  $node = new stdClass;
-  $node->type = 'profile';
-  $node->language = '';
-  $nid = content_profile_profile_exists($node, $account->uid);
-  $profile = node_load($nid);
-  $profile->moderate = 0;
-  $profile->status = 1;
-  node_save($profile);
-  drupal_set_message('This profile has been marked as approved.');
-  drupal_goto();
+    $node = new stdClass;
+    $node->type = 'profile';
+    $node->language = '';
+    $nid = content_profile_profile_exists($node, $account->uid);
+    $profile = node_load($nid);
+    $profile->moderate = 0;
+    $profile->status = 1;
+    node_save($profile);
+    drupal_set_message('This profile has been marked as approved.');
+    drupal_goto();
 }
 
 /**
  * Mark a user profile as rejected and notify the user of the reason.
  */
 function boincuser_moderate_profile_reject($uid, $reason = '') {
-  $account = user_load($uid);
-  $node = new stdClass;
-  $node->type = 'profile';
-  $node->language = '';
-  $nid = content_profile_profile_exists($node, $uid);
-  $profile = node_load($nid);
-  if ($profile->nid) {
+    $account = user_load($uid);
+    $node = new stdClass;
+    $node->type = 'profile';
+    $node->language = '';
+    $nid = content_profile_profile_exists($node, $uid);
+    $profile = node_load($nid);
+    if ($profile->nid) {
     global $user;
     global $base_url;
     global $base_path;
@@ -1494,64 +1494,64 @@ 
     $profile->status = 0;
     node_save($profile);
     $settings = array(
-      'from' => '',
-      'subject' => "Profile moderation at {$site_name}",
-      'message' => ''
-      . "{$account->boincuser_name},\n"
-      . "\n"
-      . "{$moderator->boincuser_name} has rejected your profile at"
-      . " {$site_name} for the following reason: \n"
-      . "\n"
-      . "{$reason}\n"
-      . "\n"
-      . "\n"
-      . "Since it has not been approved, your profile is not visible to other"
-      . " {$site_name} users. Please make the needed changes here:\n"
-      . "\n"
-      . "{$site_url}account/profile \n"
-      . "\n"
-      . "Thanks, \n"
-      . "\n"
-      . "{$site_name} support team",
+        'from' => '',
+        'subject' => "Profile moderation at {$site_name}",
+        'message' => ''
+        . "{$account->boincuser_name},\n"
+        . "\n"
+        . "{$moderator->boincuser_name} has rejected your profile at"
+        . " {$site_name} for the following reason: \n"
+        . "\n"
+        . "{$reason}\n"
+        . "\n"
+        . "\n"
+        . "Since it has not been approved, your profile is not visible to other"
+        . " {$site_name} users. Please make the needed changes here:\n"
+        . "\n"
+        . "{$site_url}account/profile \n"
+        . "\n"
+        . "Thanks, \n"
+        . "\n"
+        . "{$site_name} support team",
     );
     rules_action_mail_to_user($account, $settings);
     drupal_set_message('This profile has been marked as rejected.');
-  }
-  drupal_goto();
+    }
+    drupal_goto();
 }
 
 /**
  * Ban a user and send a notification of the reason.
  */
 function boincuser_moderate_user_ban($uid, $reason = '', $duration = '') {
-  if (user_access('assign community member role')
+    if (user_access('assign community member role')
   OR user_access('assign all roles')) {
     $account = user_load($uid);
     if ($account->uid) {
-      module_load_include('inc', 'rules', 'modules/system.rules');
-      if ($duration === '') {
+        module_load_include('inc', 'rules', 'modules/system.rules');
+        if ($duration === '') {
         $duration = variable_get('boinc_penalty_period', 7*24*60*60);
-      }
-      $penalty_expiration = ($duration > 0) ? time() + $duration : 4294967295;
-      $boincuser_record = array(
+        }
+        $penalty_expiration = ($duration > 0) ? time() + $duration : 4294967295;
+        $boincuser_record = array(
         'uid' => $uid,
         'penalty_expiration' => $penalty_expiration,
-      );
-      drupal_write_record('boincuser', $boincuser_record, 'uid');
+        );
+        drupal_write_record('boincuser', $boincuser_record, 'uid');
       
-      $community_role = array_search('community member', user_roles(true));
-      if (isset($account->roles[$community_role])) {
+        $community_role = array_search('community member', user_roles(true));
+        if (isset($account->roles[$community_role])) {
         unset($account->roles[$community_role]);
         user_save($account, array('roles' => $account->roles));
-      }
+        }
       
-      global $user;
-      global $base_url;
-      global $base_path;
-      $site_name = variable_get('site_name', bts('project'));
-      $site_url = $base_url . $base_path;
-      $moderator = user_load($user->uid);
-      $settings = array(
+        global $user;
+        global $base_url;
+        global $base_path;
+        $site_name = variable_get('site_name', bts('project'));
+        $site_url = $base_url . $base_path;
+        $moderator = user_load($user->uid);
+        $settings = array(
         'from' => '',
         'subject' => "User moderation at {$site_name}",
         'message' => ''
@@ -1572,12 +1572,12 @@ 
         . "Thanks, \n"
         . "\n"
         . "{$site_name} support team",
-      );
-      rules_action_mail_to_user($account, $settings);
-      drupal_set_message('This user has been banned.');
+        );
+        rules_action_mail_to_user($account, $settings);
+        drupal_set_message('This user has been banned.');
     }
-  }
-  drupal_goto();
+    }
+    drupal_goto();
 }
 
 
@@ -1589,15 +1589,15 @@ 
  *
  */
 function boincuser_get_weak_auth($boinc_id = null) {
-  if (!$boinc_id) {
+    if (!$boinc_id) {
     global $user;
     $account = user_load($user->uid);
     $boinc_id = $account->boincuser_id;
-  }
+    }
   
-  $boinc_user = boincuser_load($account->boincuser_id);
+    $boinc_user = boincuser_load($account->boincuser_id);
   
-  return weak_auth($boinc_user);
+    return weak_auth($boinc_user);
 }
 
 /**
@@ -1606,74 +1606,74 @@ 
  * Drupal User so must be inserted into comments, etc. (not so by default)
  */
 function boincuser_get_user_profile_image($uid, $avatar = TRUE) {
-  // Though the function name implies otherwise, get the avatar by default
-  $image_field = ($avatar) ? 'field_image_fid' : 'field_profile_image_fid';
-  $image_fid = db_result(db_query("
+    // Though the function name implies otherwise, get the avatar by default
+    $image_field = ($avatar) ? 'field_image_fid' : 'field_profile_image_fid';
+    $image_fid = db_result(db_query("
     SELECT ctp.%s
     FROM {content_type_profile} ctp
     INNER JOIN {node} n ON ctp.nid = n.nid
     WHERE n.uid = %d AND n.type = '%s'",
     $image_field, $uid, 'profile'));
-  $user_image['image'] = field_file_load($image_fid);
-  if (!$user_image['image']['filepath']) {
+    $user_image['image'] = field_file_load($image_fid);
+    if (!$user_image['image']['filepath']) {
     // Load the default image if one does not exist
     $account = user_load($uid);
     if ($avatar AND module_exists('gravatar') AND user_access('use gravatar', $account) AND $account->gravatar) {
-      // Use a Gravatar rather than the system default image
-      $options = array(
+        // Use a Gravatar rather than the system default image
+        $options = array(
         'size' => 100,
         'rating' => 'G',
-      );
-      // Get the Gravatar URL and see if the image exists
-      $url = gravatar_get_gravatar($account->mail, $options);
-      $headers = @get_headers($url);
-      if (preg_match("|200|", $headers[0])) {
+        );
+        // Get the Gravatar URL and see if the image exists
+        $url = gravatar_get_gravatar($account->mail, $options);
+        $headers = @get_headers($url);
+        if (preg_match("|200|", $headers[0])) {
         return $url;
-      }
+        }
     }
     // Get default image if nothing else works
     $content_node_widget_settings = db_result(db_query("SELECT widget_settings FROM {content_node_field_instance} WHERE field_name = '%s'", ($avatar ? 'field_image' : 'field_profile_image')));
     $content_node_widget_settings = unserialize($content_node_widget_settings);
     $user_image['image'] = $content_node_widget_settings['default_image'];
-  }
-  $user = user_load($uid);
-  $user_image['alt'] = $user->name;
-  return $user_image;
+    }
+    $user = user_load($uid);
+    $user_image['alt'] = $user->name;
+    return $user_image;
 }
 
 /**
  * Generate a table of a user's projects
  */
 function boincuser_get_projects_table($account = null) {
-  if ($account AND is_numeric($account)) {
+    if ($account AND is_numeric($account)) {
     $account = user_load($account);
-  }
-  $projects = boincuser_get_projects($account);
-  if (!$projects) return bts('no projects...');
+    }
+    $projects = boincuser_get_projects($account);
+    if (!$projects) return bts('no projects...');
   
-  $output = '';
-  $output .= '<table class="user-projects">' . "\n";
-  $output .= '<thead>' . "\n";
-  $output .= '  <tr>' . "\n";
-  $output .= '    <th>' . bts('Name') . '</th>' . "\n";
-  $output .= '    <th class="numeric">' . bts('Avg credit') . '</th>' . "\n";
-  $output .= '    <th class="numeric">' . bts('Total credit') . '</th>' . "\n";
-  $output .= '  </tr>' . "\n";
-  $output .= '</thead>' . "\n";
-  $output .= '<tbody>' . "\n";
-  foreach ($projects AS $project) {
+    $output = '';
+    $output .= '<table class="user-projects">' . "\n";
+    $output .= '<thead>' . "\n";
+    $output .= '  <tr>' . "\n";
+    $output .= '    <th>' . bts('Name') . '</th>' . "\n";
+    $output .= '    <th class="numeric">' . bts('Avg credit') . '</th>' . "\n";
+    $output .= '    <th class="numeric">' . bts('Total credit') . '</th>' . "\n";
+    $output .= '  </tr>' . "\n";
+    $output .= '</thead>' . "\n";
+    $output .= '<tbody>' . "\n";
+    foreach ($projects AS $project) {
     $url = rtrim($project->url, '/') . '/show_user.php?userid=' . $project->id;
     $output .= '  <tr>' . "\n";
     $output .= '    <td>' . l($project->name, $url) . '</td>' . "\n";
     $output .= '    <td class="numeric">' . boincwork_format_stats((float) $project->expavg_credit) . '</td>' . "\n"; 
     $output .= '    <td class="numeric">' . boincwork_format_stats((float) $project->total_credit) . '</td>' . "\n";
     $output .= '  </tr>' . "\n";
-  }
-  $output .= '</tbody>' . "\n";
-  $output .= '</table>' . "\n";
-  $more_link = ($account) ? "user/{$account->uid}/stats" : 'account/stats';
-  //$output .= "<div class=\"more-link\"><a href=\"{$more_link}\">More stats</a></div>" . "\n";
-  return $output;
+    }
+    $output .= '</tbody>' . "\n";
+    $output .= '</table>' . "\n";
+    $more_link = ($account) ? "user/{$account->uid}/stats" : 'account/stats';
+    //$output .= "<div class=\"more-link\"><a href=\"{$more_link}\">More stats</a></div>" . "\n";
+    return $output;
 }
 
 /**
@@ -1681,25 +1681,25 @@ 
  */
 function boincuser_get_stats_user_data($cpid = null) {
   
-  // [TODO] Set this stuff in site config!
-  $stats_server = 'stats.gridrepublic.org';
-  $stats_rpc = 'rpc/get_user.php';
+    // [TODO] Set this stuff in site config!
+    $stats_server = 'stats.gridrepublic.org';
+    $stats_rpc = 'rpc/get_user.php';
   
-  // Construct query string
-  $get = array(
+    // Construct query string
+    $get = array(
     'cpid' => $cpid
-  );
-  $args = array();
-  foreach ($get as $arg => $value) $args[] = "{$arg}=" . rawurlencode($value);
-  $query = '?' . implode('&', $args);
+    );
+    $args = array();
+    foreach ($get as $arg => $value) $args[] = "{$arg}=" . rawurlencode($value);
+    $query = '?' . implode('&', $args);
   
-  // Load XML from RPC
-  $target_url = "http://{$stats_server}/{$stats_rpc}{$query}";
-  $result = drupal_http_request($target_url);
-  if (in_array($result->code, array(200, 304))) {
+    // Load XML from RPC
+    $target_url = "http://{$stats_server}/{$stats_rpc}{$query}";
+    $result = drupal_http_request($target_url);
+    if (in_array($result->code, array(200, 304))) {
     return simplexml_load_string($result->data);
-  }
-  return NULL;
+    }
+    return NULL;
 }
 
 /**
@@ -1707,15 +1707,15 @@ 
  */
 function boincuser_get_projects($account = null) {
   
-  // Use the current user by default
-  if (!$account) {
+    // Use the current user by default
+    if (!$account) {
     global $user;
     $account = user_load($user->uid);
-  }
+    }
   
-  $account_stats = boincuser_get_stats_user_data($account->boincuser_cpid);
+    $account_stats = boincuser_get_stats_user_data($account->boincuser_cpid);
   
-  return ($account_stats AND isset($account_stats->project)) ? $account_stats->project : null;
+    return ($account_stats AND isset($account_stats->project)) ? $account_stats->project : null;
 }
 
 
@@ -1723,11 +1723,11 @@ 
  * Get the links to display under the user profile
  */
 function boincuser_get_profile_links($uid) {
-  global $user;
-  $account = user_load($uid);
-  $profile = content_profile_load('profile', $account->uid);
-  $output = '';
-  if ($profile) {
+    global $user;
+    $account = user_load($uid);
+    $profile = content_profile_load('profile', $account->uid);
+    $output = '';
+    if ($profile) {
     $profile_is_approved = ($profile->status AND !$profile->moderate);
     $user_is_moderator = user_access('edit any profile content');
     $is_own_profile = ($user->uid == $account->uid);
@@ -1735,15 +1735,15 @@ 
     $links = array();
     
     if ($profile->moderate AND $user_is_moderator) {
-      $links['approve_profile'] = array(
+        $links['approve_profile'] = array(
         'title' => bts('Approve profile'),
         'href' => "{$profile_moderation_path}/approve",
         'attributes' => array(
-          'title' => bts('Approve this profile content'),
-          'class' => 'first primary tab',
+            'title' => bts('Approve this profile content'),
+            'class' => 'first primary tab',
         )
-      );
-      /*$links['edit_profile'] = array(
+        );
+        /*$links['edit_profile'] = array(
         'title' => bts('Edit profile'),
         'href' => "{$profile_moderation_path}/edit",
         'attributes' => array(
@@ -1751,27 +1751,27 @@ 
           'class' => 'tab',
         )
       );*/
-      $links['reject_profile'] = array(
+        $links['reject_profile'] = array(
         'title' => bts('Reject profile'),
         'href' => "{$profile_moderation_path}/reject",
         'attributes' => array(
-          'title' => bts('Reject this profile content'),
-          'class' => 'tab',
+            'title' => bts('Reject this profile content'),
+            'class' => 'tab',
         )
-      );
+        );
     }
     $output .= '<ul class="tab-list">';
     $count = 0;
     foreach ($links as $key => $link) {
-      $output .= '<li class="' . (($count == 0) ? 'first primary ' : '') . 'tab">';
-      $output .= l($link['title'], $link['href'], array('query' => drupal_get_destination()));
-      $output .= '</li>';
-      $count++;
+        $output .= '<li class="' . (($count == 0) ? 'first primary ' : '') . 'tab">';
+        $output .= l($link['title'], $link['href'], array('query' => drupal_get_destination()));
+        $output .= '</li>';
+        $count++;
     }
     $output .= '<li class="' . (($count) ? '' : 'first ') . 'last tab">' . flag_create_link('abuse_user', $account->uid) . '</li>';
     $output .= '</ul>';
-  }
-  return $output;
+    }
+    return $output;
 /*
 <ul class="tab-list">
   <li class="primary first tab">
@@ -1808,35 +1808,35 @@ 
  */
 function boincuser_apachesolr_index_documents_alter(array &$documents, $entity, $entity_type, $env_id) {
 
-  foreach ($documents as $document) {
+    foreach ($documents as $document) {
     if ( $document->entity_type=='node' AND $document->bundle=='profile' ) {
-      // Node information.
-      $nid = $document->entity_id;
-      $node = node_load($nid);
-      $account = user_load($node->uid);
+        // Node information.
+        $nid = $document->entity_id;
+        $node = node_load($nid);
+        $account = user_load($node->uid);
 
-      // Use boincuser name and not drupal user name
-      $document->label = apachesolr_clean_text($account->boincuser_name);
-      // Author information
-      if ($node->uid == 0 || strlen($node->name) == 0) {
+        // Use boincuser name and not drupal user name
+        $document->label = apachesolr_clean_text($account->boincuser_name);
+        // Author information
+        if ($node->uid == 0 || strlen($node->name) == 0) {
         // @see user_validate_name(). !'0' === TRUE.
         $document->ss_name = '0';
-      }
-      else {
+        }
+        else {
         $document->ss_name = $account->boincuser_name;
         // We want the name to be searchable for keywords.
         $document->tos_name = $account->boincuser_name;
-      }
+        }
 
-      // Rename "Profle" to "User"
-      $document->bundle = "User";
-      $document->bundle_name = "User";
+        // Rename "Profle" to "User"
+        $document->bundle = "User";
+        $document->bundle_name = "User";
 
-      // Replace the Solr document's created field with the date the user 
-      // account was created. This replaces the node creation date typically 
-      // used for indexing nodes.
-      $document->ds_created = apachesolr_date_iso($account->created);
+        // Replace the Solr document's created field with the date the user 
+        // account was created. This replaces the node creation date typically 
+        // used for indexing nodes.
+        $document->ds_created = apachesolr_date_iso($account->created);
+    }
     }
-  }
 
 }

Spacing +74 added lines, -74 removed lines

@@ -216,7 +216,7 @@ 
 function boincuser_user($op, &$edit, &$account, $category = NULL) {
   // Handle BOINC integration for users with UID > 1 (skip anonymous and admin)
   if ($account->uid > 1) {
-    switch($op) {
+    switch ($op) {
     case 'load':
       // User loading; insert BOINC data into the user object
       $drupal_user = db_fetch_object(db_query("
@@ -244,11 +244,11 @@ 
       ));
       $account->boincuser_name = $boinc_user->name;
       $account->boincuser_account_key = $boinc_user->authenticator;
-      $account->boincuser_weak_auth = md5($boinc_user->authenticator . $boinc_user->passwd_hash);
+      $account->boincuser_weak_auth = md5($boinc_user->authenticator.$boinc_user->passwd_hash);
       $account->boincuser_total_credit = round($boinc_user->total_credit);
       $account->boincuser_expavg_credit = round($boinc_user->expavg_credit);
       $account->boincuser_expavg_time = round($boinc_user->expavg_time);
-      $account->boincuser_cpid = md5($boinc_user->cross_project_id . $account->mail);
+      $account->boincuser_cpid = md5($boinc_user->cross_project_id.$account->mail);
       $account->boincuser_default_pref_set = $boinc_user->venue;
       $account->boincteam_id = $boinc_user->teamid;
       db_set_active('default');
@@ -445,7 +445,7 @@ 
   // In Drupal 7, these operation cases will all exist as their own hooks,
   // so let's approximate that here so that this function can simply be removed
   // upon migration to 7
-  switch($op) {
+  switch ($op) {
   case 'update':
     boincuser_node_update($node);
   }
@@ -456,7 +456,7 @@ 
  * is updated (forward compatible to Drupal 7)
  */
 function boincuser_node_update($node) {
-  switch($node->type) {
+  switch ($node->type) {
   case 'profile':
     // Update the BOINC database directly
     $account = user_load($node->uid);
@@ -576,7 +576,7 @@ 
     }
     
     $form['cancel'] = array(
-      '#value' => '<li class="tab">' . l(bts('Cancel'), $cancel_dest) . '</li>',
+      '#value' => '<li class="tab">'.l(bts('Cancel'), $cancel_dest).'</li>',
       '#weight' => 1004,
     );
     $form['form control tabs suffix'] = array(
@@ -609,7 +609,7 @@ 
     $form['buttons']['preview_changes']['#suffix'] = '</li>';
     $form['buttons']['preview_changes']['#weight'] = 1004;
     $form['buttons']['cancel'] = array(
-      '#value' => '<li class="tab">' . l(bts('Cancel'), "node/{$form['nid']['#value']}") . '</li>',
+      '#value' => '<li class="tab">'.l(bts('Cancel'), "node/{$form['nid']['#value']}").'</li>',
       '#weight' => 1005,
     );
     $form['buttons']['delete']['#prefix'] = '<li class="tab">';
@@ -641,7 +641,7 @@ 
     $form['actions']['submit']['#suffix'] = '</li>';
     $form['actions']['submit']['#weight'] = 1002;
     $form['actions']['cancel'] = array(
-      '#value' => '<li class="tab">' . l(bts('Cancel'), "node/{$form['nid']['#value']}") . '</li>',
+      '#value' => '<li class="tab">'.l(bts('Cancel'), "node/{$form['nid']['#value']}").'</li>',
       '#weight' => 1005,
     );
     $form['actions']['form control tabs suffix'] = array(
@@ -668,7 +668,7 @@ 
     $form['privatemsg']['preview']['#suffix'] = '</li>';
     $form['privatemsg']['preview']['#weight'] = 1003;
     $form['privatemsg']['cancel'] = array(
-      '#value' => '<li class="tab">' . l(bts('Cancel'), $_GET['q']) . '</li>',
+      '#value' => '<li class="tab">'.l(bts('Cancel'), $_GET['q']).'</li>',
       '#weight' => 1004,
     );
     $form['privatemsg']['form control tabs suffix'] = array(
@@ -776,36 +776,36 @@ 
     $form['account']['boincuser_id'] = array(
       '#value' => '
         <div class="form-item">
-          <label>' . bts('BOINC user ID') . '</label>
-          <span>' . $account->boincuser_id . '</span>
+          <label>' . bts('BOINC user ID').'</label>
+          <span>' . $account->boincuser_id.'</span>
         </div>',
     );
     $form['account']['user_id'] = array(
       '#value' => '
         <div class="form-item">
-          <label>' . bts('Drupal user ID') . '</label>
-          <span>' . $account->uid . '</span>
+          <label>' . bts('Drupal user ID').'</label>
+          <span>' . $account->uid.'</span>
         </div>',
     );
     $form['account']['account_key'] = array(
       '#value' => '
         <div class="form-item">
-          <label>' . bts('Account key') . '</label>
-          <span>' . $account->boincuser_account_key . '</span>
+          <label>' . bts('Account key').'</label>
+          <span>' . $account->boincuser_account_key.'</span>
         </div>',
     );
     $form['account']['weak_account_key'] = array(
       '#value' => '
         <div class="form-item">
-          <label>' . bts('Weak account key') . '</label>
-          <span>' . "{$account->boincuser_id}_{$account->boincuser_weak_auth}" . '</span>
+          <label>' . bts('Weak account key').'</label>
+          <span>' . "{$account->boincuser_id}_{$account->boincuser_weak_auth}".'</span>
         </div>',
     );
     $form['account']['cpid'] = array(
       '#value' => '
         <div class="form-item">
-          <label>' . bts('Cross-project ID') . '</label>
-          <span>' . $account->boincuser_cpid . '</span>
+          <label>' . bts('Cross-project ID').'</label>
+          <span>' . $account->boincuser_cpid.'</span>
         </div>',
     );
     
@@ -828,7 +828,7 @@ 
       $form['delete']['#weight'] = 1003;
     }
     $form['cancel'] = array(
-      '#value' => '<li class="tab">' . l(bts('Cancel'), $_GET['q']) . '</li>',
+      '#value' => '<li class="tab">'.l(bts('Cancel'), $_GET['q']).'</li>',
       '#weight' => 1004,
     );
     $form['form control tabs suffix'] = array(
@@ -911,7 +911,7 @@ 
     $form['buttons']['preview_changes']['#suffix'] = '</li>';
     $form['buttons']['preview_changes']['#weight'] = 1004;
     $form['buttons']['cancel'] = array(
-      '#value' => '<li class="tab">' . l(bts('Cancel'), $_GET['q']) . '</li>',
+      '#value' => '<li class="tab">'.l(bts('Cancel'), $_GET['q']).'</li>',
       '#weight' => 1005,
     );
     $form['buttons']['delete']['#prefix'] = '<li class="tab">';
@@ -966,7 +966,7 @@ 
     // Set name temporarily to dummy value to beat validation
     $form['name'] = array(
       '#type' => 'hidden',
-      '#value' => rand() . '.' . time()
+      '#value' => rand().'.'.time()
     );
     
     if (module_exists('captcha')) {
@@ -1016,7 +1016,7 @@ 
     $form['buttons']['submit']['#suffix'] = '</li>';
     $form['buttons']['submit']['#weight'] = 1002;
     $form['buttons']['cancel'] = array(
-      '#value' => '<li class="tab">' . l(bts('Cancel'), 'user/login') . '</li>',
+      '#value' => '<li class="tab">'.l(bts('Cancel'), 'user/login').'</li>',
       '#weight' => 1005,
     );
     $form['buttons']['form control tabs suffix'] = array(
@@ -1098,7 +1098,7 @@ 
  */
 function boincuser_token_list($type = 'all') {
   if ($type == 'user' || $type == 'all') {
-    $tokens['user']['display-name']      = t("The user's name that should be displayed");
+    $tokens['user']['display-name'] = t("The user's name that should be displayed");
     return $tokens;
   }
 }
@@ -1108,9 +1108,9 @@ 
  */
 function boincuser_views_pre_execute(&$view) {
   $account_id = $view->args[0];
-  if ($view->name=="user_activity") {
+  if ($view->name == "user_activity") {
     // Run the following custom query for the user_activity view
-    $view->build_info['query']= "
+    $view->build_info['query'] = "
       SELECT node_revisions.vid AS vid,
         node.nid AS node_nid,
         node.uid AS users_node_uid,
@@ -1205,27 +1205,27 @@ 
   $output = '<div class="join">';
   switch ($type) {
   case 'boinc':
-    $output .= '<p>' . bts("If you're already running BOINC, select <i>Add
-      Project</i>.") . '</p>';
+    $output .= '<p>'.bts("If you're already running BOINC, select <i>Add
+      Project</i>.").'</p>';
     if ($registration_enabled) {
-      $output .= '<p>' . bts("If you're running a command-line or pre-5.0 version
+      $output .= '<p>'.bts("If you're running a command-line or pre-5.0 version
         of BOINC, !create_an_account first.", array('!create_an_account' => 
-          l(bts('create an account'), 'user/registration'))) . '</p>';
+          l(bts('create an account'), 'user/registration'))).'</p>';
     }
     else {
-      $output .= '<p>' . bts("If you're running a command-line version of BOINC,
+      $output .= '<p>'.bts("If you're running a command-line version of BOINC,
         first use the BOINC Manager software (elsewhere if necessary) to create an
         an account at this project, then use that account to connect with the
-        command-line version.") . '</p>';
-      $output .= '<p>' . bts("If you're running a pre-5.0 version of BOINC, please
+        command-line version.").'</p>';
+      $output .= '<p>'.bts("If you're running a pre-5.0 version of BOINC, please
         upgrade to a more recent version to create an account
-        at this project.") . '</p>';
+        at this project.").'</p>';
     }
     break;
   case 'new':
   default:
     $output .= '<ol>';
-    $output .= '  <li>' . bts('Download the desktop software');
+    $output .= '  <li>'.bts('Download the desktop software');
     $output .= '    <p>';
     $output .= '      <a class="button" href="http://boinc.berkeley.edu/download.php">Download</a>';
     $output .= '    </p>';
@@ -1235,10 +1235,10 @@ 
     //$output .= '      <a href="files/boinc_i686-pc-linux-gnu.sh" class="platform linux">Linux</a>';
     //$output .= '    </p>';
     $output .= '  </li>';
-    $output .= '  <li>' . bts('Run the installer') . '</li>';
-    $output .= '  <li>' . bts('When prompted enter @siteurl', array(
-      '@siteurl' => $base_url)) . '</li>';
-    $output .=  '</ol>';
+    $output .= '  <li>'.bts('Run the installer').'</li>';
+    $output .= '  <li>'.bts('When prompted enter @siteurl', array(
+      '@siteurl' => $base_url)).'</li>';
+    $output .= '</ol>';
   }
   $output .= '</div>';
   return $output;
@@ -1251,7 +1251,7 @@ 
   global $user;
   $site_name = variable_get('site_name', '');
   // get the front page message from database; this is set in the admin interface under BOINC Other
-  $site_message = variable_get('boinc_other_frontpage','');
+  $site_message = variable_get('boinc_other_frontpage', '');
 
   // Determine the user of the day
   $current_uotd = db_fetch_object(db_query("
@@ -1273,30 +1273,30 @@ 
   $output .= ($user->uid) ? bts('Welcome back!') : ($site_name ? bts('What is @this_project?', array('@this_project' => $site_name)) : bts('Welcome!'));
   $output .= '</h2>';
   $output .= '<div class="boinc-overview balance-height-front">';
-  $output .= '  <div>' . bts($site_message, array(), NULL, "project:front page") . ' ' . l(bts('Learn more'), 'about') . '</div>';
+  $output .= '  <div>'.bts($site_message, array(), NULL, "project:front page").' '.l(bts('Learn more'), 'about').'</div>';
   if ($user->uid) {
-    $output .= '  <div>' . l(bts('View account'), 'dashboard', array('attributes' => array('class' => 'join button'))) . '</div>';
+    $output .= '  <div>'.l(bts('View account'), 'dashboard', array('attributes' => array('class' => 'join button'))).'</div>';
   }
   else {
-    $output .= '  <div>' . l(bts('Join now'), 'join', array('attributes' => array('class' => 'join button'))) . '</div>';
+    $output .= '  <div>'.l(bts('Join now'), 'join', array('attributes' => array('class' => 'join button'))).'</div>';
   }
   $output .= '</div>';
   $output .= '<div class="boinc-overview-details">';
   $output .= '  <div class="detail-container">';
-  $output .= '    <a class="user-of-the-day" href="account/' . $uotd->uid . '">';
+  $output .= '    <a class="user-of-the-day" href="account/'.$uotd->uid.'">';
   $output .= '      <div class="picture">';
   $output .= theme('imagefield_image', $uotd_image['image'], $uotd_image['alt'],
     $uotd_image['alt'], array(), FALSE);
   $output .= '      </div>';
-  $output .= '      <div class="text">' . bts('User of the day') . '</div>';
-  $output .= '      <div class="detail">' . $uotd->boincuser_name . '</div>';
+  $output .= '      <div class="text">'.bts('User of the day').'</div>';
+  $output .= '      <div class="detail">'.$uotd->boincuser_name.'</div>';
   $output .= '    </a>';
   $output .= '    <div class="volunteers">';
-  $output .= '      <div class="text">' . bts('Over 500,000 volunteers and counting.') . '</div>';
+  $output .= '      <div class="text">'.bts('Over 500,000 volunteers and counting.').'</div>';
   $output .= '      <div class="platforms">';
-  $output .= '        <div class="detail platform windows">' . bts('Windows') . '</div>';
-  $output .= '        <div class="detail platform mac">' .  bts('Mac') . '</div>';
-  $output .= '        <div class="detail platform linux">' . bts('Linux') . '</div>';
+  $output .= '        <div class="detail platform windows">'.bts('Windows').'</div>';
+  $output .= '        <div class="detail platform mac">'.bts('Mac').'</div>';
+  $output .= '        <div class="detail platform linux">'.bts('Linux').'</div>';
   $output .= '      </div>';
   $output .= '    </div>';
   $output .= '  </div>';
@@ -1488,7 +1488,7 @@ 
     global $base_url;
     global $base_path;
     $site_name = variable_get('site_name', bts('project'));
-    $site_url = $base_url . $base_path;
+    $site_url = $base_url.$base_path;
     $moderator = user_load($user->uid);
     $profile->moderate = 0;
     $profile->status = 0;
@@ -1549,7 +1549,7 @@ 
       global $base_url;
       global $base_path;
       $site_name = variable_get('site_name', bts('project'));
-      $site_url = $base_url . $base_path;
+      $site_url = $base_url.$base_path;
       $moderator = user_load($user->uid);
       $settings = array(
         'from' => '',
@@ -1652,25 +1652,25 @@ 
   if (!$projects) return bts('no projects...');
   
   $output = '';
-  $output .= '<table class="user-projects">' . "\n";
-  $output .= '<thead>' . "\n";
-  $output .= '  <tr>' . "\n";
-  $output .= '    <th>' . bts('Name') . '</th>' . "\n";
-  $output .= '    <th class="numeric">' . bts('Avg credit') . '</th>' . "\n";
-  $output .= '    <th class="numeric">' . bts('Total credit') . '</th>' . "\n";
-  $output .= '  </tr>' . "\n";
-  $output .= '</thead>' . "\n";
-  $output .= '<tbody>' . "\n";
+  $output .= '<table class="user-projects">'."\n";
+  $output .= '<thead>'."\n";
+  $output .= '  <tr>'."\n";
+  $output .= '    <th>'.bts('Name').'</th>'."\n";
+  $output .= '    <th class="numeric">'.bts('Avg credit').'</th>'."\n";
+  $output .= '    <th class="numeric">'.bts('Total credit').'</th>'."\n";
+  $output .= '  </tr>'."\n";
+  $output .= '</thead>'."\n";
+  $output .= '<tbody>'."\n";
   foreach ($projects AS $project) {
-    $url = rtrim($project->url, '/') . '/show_user.php?userid=' . $project->id;
-    $output .= '  <tr>' . "\n";
-    $output .= '    <td>' . l($project->name, $url) . '</td>' . "\n";
-    $output .= '    <td class="numeric">' . boincwork_format_stats((float) $project->expavg_credit) . '</td>' . "\n"; 
-    $output .= '    <td class="numeric">' . boincwork_format_stats((float) $project->total_credit) . '</td>' . "\n";
-    $output .= '  </tr>' . "\n";
+    $url = rtrim($project->url, '/').'/show_user.php?userid='.$project->id;
+    $output .= '  <tr>'."\n";
+    $output .= '    <td>'.l($project->name, $url).'</td>'."\n";
+    $output .= '    <td class="numeric">'.boincwork_format_stats((float)$project->expavg_credit).'</td>'."\n"; 
+    $output .= '    <td class="numeric">'.boincwork_format_stats((float)$project->total_credit).'</td>'."\n";
+    $output .= '  </tr>'."\n";
   }
-  $output .= '</tbody>' . "\n";
-  $output .= '</table>' . "\n";
+  $output .= '</tbody>'."\n";
+  $output .= '</table>'."\n";
   $more_link = ($account) ? "user/{$account->uid}/stats" : 'account/stats';
   //$output .= "<div class=\"more-link\"><a href=\"{$more_link}\">More stats</a></div>" . "\n";
   return $output;
@@ -1690,8 +1690,8 @@ 
     'cpid' => $cpid
   );
   $args = array();
-  foreach ($get as $arg => $value) $args[] = "{$arg}=" . rawurlencode($value);
-  $query = '?' . implode('&', $args);
+  foreach ($get as $arg => $value) $args[] = "{$arg}=".rawurlencode($value);
+  $query = '?'.implode('&', $args);
   
   // Load XML from RPC
   $target_url = "http://{$stats_server}/{$stats_rpc}{$query}";
@@ -1763,12 +1763,12 @@ 
     $output .= '<ul class="tab-list">';
     $count = 0;
     foreach ($links as $key => $link) {
-      $output .= '<li class="' . (($count == 0) ? 'first primary ' : '') . 'tab">';
+      $output .= '<li class="'.(($count == 0) ? 'first primary ' : '').'tab">';
       $output .= l($link['title'], $link['href'], array('query' => drupal_get_destination()));
       $output .= '</li>';
       $count++;
     }
-    $output .= '<li class="' . (($count) ? '' : 'first ') . 'last tab">' . flag_create_link('abuse_user', $account->uid) . '</li>';
+    $output .= '<li class="'.(($count) ? '' : 'first ').'last tab">'.flag_create_link('abuse_user', $account->uid).'</li>';
     $output .= '</ul>';
   }
   return $output;
@@ -1809,7 +1809,7 @@ 
 function boincuser_apachesolr_index_documents_alter(array &$documents, $entity, $entity_type, $env_id) {
 
   foreach ($documents as $document) {
-    if ( $document->entity_type=='node' AND $document->bundle=='profile' ) {
+    if ($document->entity_type == 'node' AND $document->bundle == 'profile') {
       // Node information.
       $nid = $document->entity_id;
       $node = node_load($nid);

Braces +9 added lines, -13 removed lines

@@ -303,8 +303,7 @@ 
           if (!boincuser_register_validate($edit)) {
             // BOINC user validation failed for registration; set an error accordingly
             form_set_error('mail', bts('An account already exists for @email. Log in or request password assistance to access your @project account.', array('@email' => $edit['mail'], '@project' => PROJECT)));
-          }
-          else {
+          } else {
             // Save profile information for use during Insert
             $_SESSION['profileInfo'] = array(
               'country' => $edit['field_country'][0]['value'],
@@ -557,8 +556,7 @@ 
       case FLAG_FRIEND_APPROVAL:
         if ($action == 'flag') {
           $form['flag_friend_submit']['#value'] = bts('Approve request');
-        }
-        elseif ($action == 'unflag') {
+        } elseif ($action == 'unflag') {
           unset($form['actions']);
           $form['flag_friend_submit']['#value'] = bts('Deny request');
         }
@@ -1211,8 +1209,7 @@ 
       $output .= '<p>' . bts("If you're running a command-line or pre-5.0 version
         of BOINC, !create_an_account first.", array('!create_an_account' => 
           l(bts('create an account'), 'user/registration'))) . '</p>';
-    }
-    else {
+    } else {
       $output .= '<p>' . bts("If you're running a command-line version of BOINC,
         first use the BOINC Manager software (elsewhere if necessary) to create an
         an account at this project, then use that account to connect with the
@@ -1264,8 +1261,7 @@ 
   ));
   if ($current_uotd->uotd_time < strtotime('today midnight')) {
     $uotd = boincuser_select_user_of_the_day();
-  }
-  else {
+  } else {
     $uotd = user_load($current_uotd->uid);
   }
   $uotd_image = boincuser_get_user_profile_image($uotd->uid, FALSE);
@@ -1276,8 +1272,7 @@ 
   $output .= '  <div>' . bts($site_message, array(), NULL, "project:front page") . ' ' . l(bts('Learn more'), 'about') . '</div>';
   if ($user->uid) {
     $output .= '  <div>' . l(bts('View account'), 'dashboard', array('attributes' => array('class' => 'join button'))) . '</div>';
-  }
-  else {
+  } else {
     $output .= '  <div>' . l(bts('Join now'), 'join', array('attributes' => array('class' => 'join button'))) . '</div>';
   }
   $output .= '</div>';
@@ -1690,7 +1685,9 @@ 
     'cpid' => $cpid
   );
   $args = array();
-  foreach ($get as $arg => $value) $args[] = "{$arg}=" . rawurlencode($value);
+  foreach ($get as $arg => $value) {
+      $args[] = "{$arg}=" . rawurlencode($value);
+  }
   $query = '?' . implode('&', $args);
   
   // Load XML from RPC
@@ -1821,8 +1818,7 @@ 
       if ($node->uid == 0 || strlen($node->name) == 0) {
         // @see user_validate_name(). !'0' === TRUE.
         $document->ss_name = '0';
-      }
-      else {
+      } else {
         $document->ss_name = $account->boincuser_name;
         // We want the name to be searchable for keywords.
         $document->tos_name = $account->boincuser_name;

Upper-Lower-Casing +35 added lines, -35 removed lines

@@ -177,7 +177,7 @@ 
     'title' => 'Create Account RPC',
     'description' => 'RPC for creating user accounts.',
     'page callback' => 'boincuser_create_account',
-    'access callback' => TRUE,
+    'access callback' => true,
     'type' => MENU_CALLBACK
   );
   return $items;
@@ -213,7 +213,7 @@ 
  * Implementation of hook_user(); add custom actions to standard
  * Drupal user operations
  */
-function boincuser_user($op, &$edit, &$account, $category = NULL) {
+function boincuser_user($op, &$edit, &$account, $category = null) {
   // Handle BOINC integration for users with UID > 1 (skip anonymous and admin)
   if ($account->uid > 1) {
     switch($op) {
@@ -253,7 +253,7 @@ 
       $account->boincteam_id = $boinc_user->teamid;
       db_set_active('default');
       // Set Drupal team ID
-      $account->team = NULL;
+      $account->team = null;
       if ($account->boincteam_id) {
         $account->team = db_result(db_query("
           SELECT nid FROM {boincteam} WHERE team_id = %d",
@@ -401,9 +401,9 @@ 
         case 'user_account':
           // Ensure that BOINC data is altered
           $boinc_user = BoincUser::lookup_id($account->boincuser_id);
-          $changing_email = ($edit['mail'] AND $edit['mail'] != $boinc_user->email_addr) ? true : false;
+          $changing_email = ($edit['mail'] and $edit['mail'] != $boinc_user->email_addr) ? true : false;
           $changing_pass = ($edit['pass']) ? true : false;
-          if ($changing_email OR $changing_pass) {
+          if ($changing_email or $changing_pass) {
             // Set password hash appropriately
             $passwd = ($edit['pass']) ? $edit['pass'] : $edit['current_pass'];
             $passwd_hash = md5($passwd.$edit['mail']);
@@ -692,7 +692,7 @@ 
         '#title' => bts('Email address'),
         '#size' => ($form_id == 'user_login_block') ? 15 : 60,
         '#maxlength' => EMAIL_MAX_LENGTH,
-        '#required' => TRUE,
+        '#required' => true,
         '#attributes' => array('tabindex' => '1'),
         '#description' => bts('Enter your @s email address.', array('@s' => variable_get('site_name', 'BOINC')))
       ),
@@ -726,7 +726,7 @@ 
     if (isset($form_state['post']['email']) and isset($form_state['post']['pass'])) {
       // Find the local validation function's entry so we can replace it.
       $array_key = array_search('user_login_authenticate_validate', $form['#validate']);
-      if ($array_key === FALSE) {
+      if ($array_key === false) {
         // Could not find it. Some other module must have run form_alter().
         // We will simply add our validation just before the final validator.
         $final_validator = array_pop($form['#validate']);
@@ -748,7 +748,7 @@ 
     
     // A bit hackish... but don't require the user to enter his password if
     // coming from the password reset function
-    $reset_pass = (strpos($_SERVER['HTTP_REFERER'], "/user/reset/{$form['#uid']}") === FALSE) ? 0 : 1;
+    $reset_pass = (strpos($_SERVER['HTTP_REFERER'], "/user/reset/{$form['#uid']}") === false) ? 0 : 1;
     if ($reset_pass) {
       $_SESSION['reset_pass'] = 1;
     }
@@ -758,7 +758,7 @@ 
     $form['account']['mail']['#size'] = 40;
     $form['account']['pass']['#size'] = 17;
     
-    if (!$reset_pass AND ($user->uid == $account->uid OR !user_access('administer users'))) {
+    if (!$reset_pass and ($user->uid == $account->uid or !user_access('administer users'))) {
       // Add a password authenticator, required to change email or pw
       $form['account']['current_pass'] = array(
         '#type' => 'password',
@@ -822,7 +822,7 @@ 
     $form['submit']['#value'] = bts('Save changes');
     $form['submit']['#suffix'] = '</li>';
     $form['submit']['#weight'] = 1002;
-    if (isset($form['delete']) AND is_array($form['delete'])) {
+    if (isset($form['delete']) and is_array($form['delete'])) {
       $form['delete']['#prefix'] = '<li class="tab">';
       $form['delete']['#suffix'] = '</li>';
       $form['delete']['#weight'] = 1003;
@@ -956,7 +956,7 @@ 
         '#default_value' => $edit['boincuser_name'], 
         '#maxlength' => USERNAME_MAX_LENGTH, 
         '#description' => bts('Spaces are allowed; punctuation is not allowed except for periods, hyphens, and underscores.'), 
-        '#required' => TRUE
+        '#required' => true
       ),
       'validation_source' => array(
         '#type' => 'hidden',
@@ -992,7 +992,7 @@ 
         '#title' => bts('Email address'),
         '#size' => 60,
         '#maxlength' => EMAIL_MAX_LENGTH,
-        '#required' => TRUE,
+        '#required' => true,
         '#description' => bts(
           'Enter your email address to receive instructions for resetting your password (or use the !authenticator_login).',
           array(
@@ -1085,7 +1085,7 @@ 
 /**
  * Implementation of hook_token_values
  */
-function boincuser_token_values($type, $object = NULL, $options = array()) {
+function boincuser_token_values($type, $object = null, $options = array()) {
   if ($type == 'user') {
     $account = user_load($object->uid);
     $tokens['display-name'] = $account->boincuser_name;
@@ -1268,12 +1268,12 @@ 
   else {
     $uotd = user_load($current_uotd->uid);
   }
-  $uotd_image = boincuser_get_user_profile_image($uotd->uid, FALSE);
+  $uotd_image = boincuser_get_user_profile_image($uotd->uid, false);
   $output = '<h2 class="pane-title">';
   $output .= ($user->uid) ? bts('Welcome back!') : ($site_name ? bts('What is @this_project?', array('@this_project' => $site_name)) : bts('Welcome!'));
   $output .= '</h2>';
   $output .= '<div class="boinc-overview balance-height-front">';
-  $output .= '  <div>' . bts($site_message, array(), NULL, "project:front page") . ' ' . l(bts('Learn more'), 'about') . '</div>';
+  $output .= '  <div>' . bts($site_message, array(), null, "project:front page") . ' ' . l(bts('Learn more'), 'about') . '</div>';
   if ($user->uid) {
     $output .= '  <div>' . l(bts('View account'), 'dashboard', array('attributes' => array('class' => 'join button'))) . '</div>';
   }
@@ -1286,7 +1286,7 @@ 
   $output .= '    <a class="user-of-the-day" href="account/' . $uotd->uid . '">';
   $output .= '      <div class="picture">';
   $output .= theme('imagefield_image', $uotd_image['image'], $uotd_image['alt'],
-    $uotd_image['alt'], array(), FALSE);
+    $uotd_image['alt'], array(), false);
   $output .= '      </div>';
   $output .= '      <div class="text">' . bts('User of the day') . '</div>';
   $output .= '      <div class="detail">' . $uotd->boincuser_name . '</div>';
@@ -1381,20 +1381,20 @@ 
  */
 function boincuser_moderate_community_access() {
   if (user_access('assign community member role')
-  OR user_access('assign all roles')) {
-    return TRUE;
+  or user_access('assign all roles')) {
+    return true;
   }
-  return FALSE;
+  return false;
 }
 
 /**
  * Get the count of items in the moderation queue
  */
 function boincuser_moderation_queue_count($caller = 'user') {
-  $allowed = FALSE;
+  $allowed = false;
   switch ($caller) {
   case 'cron':
-    $allowed = TRUE;
+    $allowed = true;
     break;
   case 'user':
   default:
@@ -1408,7 +1408,7 @@ 
       AND moderate = 1"
     ));
   }
-  return NULL;
+  return null;
 }
 
 /**
@@ -1416,10 +1416,10 @@ 
  * Allow community membership status to be set for users by direct link rather
  * than through the user account info form.
  */
-function boincuser_control($uid = NULL, $action = NULL) {
-  if (!$uid OR !$account = user_load($uid)) {
+function boincuser_control($uid = null, $action = null) {
+  if (!$uid or !$account = user_load($uid)) {
     // What are you even doing here...
-    return FALSE;
+    return false;
   }
   switch ($action) {
   case 'ban':
@@ -1525,7 +1525,7 @@ 
  */
 function boincuser_moderate_user_ban($uid, $reason = '', $duration = '') {
   if (user_access('assign community member role')
-  OR user_access('assign all roles')) {
+  or user_access('assign all roles')) {
     $account = user_load($uid);
     if ($account->uid) {
       module_load_include('inc', 'rules', 'modules/system.rules');
@@ -1605,7 +1605,7 @@ 
  * User profile image is managed by the content_profile module rather than core
  * Drupal User so must be inserted into comments, etc. (not so by default)
  */
-function boincuser_get_user_profile_image($uid, $avatar = TRUE) {
+function boincuser_get_user_profile_image($uid, $avatar = true) {
   // Though the function name implies otherwise, get the avatar by default
   $image_field = ($avatar) ? 'field_image_fid' : 'field_profile_image_fid';
   $image_fid = db_result(db_query("
@@ -1618,7 +1618,7 @@ 
   if (!$user_image['image']['filepath']) {
     // Load the default image if one does not exist
     $account = user_load($uid);
-    if ($avatar AND module_exists('gravatar') AND user_access('use gravatar', $account) AND $account->gravatar) {
+    if ($avatar and module_exists('gravatar') and user_access('use gravatar', $account) and $account->gravatar) {
       // Use a Gravatar rather than the system default image
       $options = array(
         'size' => 100,
@@ -1645,7 +1645,7 @@ 
  * Generate a table of a user's projects
  */
 function boincuser_get_projects_table($account = null) {
-  if ($account AND is_numeric($account)) {
+  if ($account and is_numeric($account)) {
     $account = user_load($account);
   }
   $projects = boincuser_get_projects($account);
@@ -1661,7 +1661,7 @@ 
   $output .= '  </tr>' . "\n";
   $output .= '</thead>' . "\n";
   $output .= '<tbody>' . "\n";
-  foreach ($projects AS $project) {
+  foreach ($projects as $project) {
     $url = rtrim($project->url, '/') . '/show_user.php?userid=' . $project->id;
     $output .= '  <tr>' . "\n";
     $output .= '    <td>' . l($project->name, $url) . '</td>' . "\n";
@@ -1699,7 +1699,7 @@ 
   if (in_array($result->code, array(200, 304))) {
     return simplexml_load_string($result->data);
   }
-  return NULL;
+  return null;
 }
 
 /**
@@ -1715,7 +1715,7 @@ 
   
   $account_stats = boincuser_get_stats_user_data($account->boincuser_cpid);
   
-  return ($account_stats AND isset($account_stats->project)) ? $account_stats->project : null;
+  return ($account_stats and isset($account_stats->project)) ? $account_stats->project : null;
 }
 
 
@@ -1728,13 +1728,13 @@ 
   $profile = content_profile_load('profile', $account->uid);
   $output = '';
   if ($profile) {
-    $profile_is_approved = ($profile->status AND !$profile->moderate);
+    $profile_is_approved = ($profile->status and !$profile->moderate);
     $user_is_moderator = user_access('edit any profile content');
     $is_own_profile = ($user->uid == $account->uid);
     $profile_moderation_path = "moderate/profile/{$account->uid}";
     $links = array();
     
-    if ($profile->moderate AND $user_is_moderator) {
+    if ($profile->moderate and $user_is_moderator) {
       $links['approve_profile'] = array(
         'title' => bts('Approve profile'),
         'href' => "{$profile_moderation_path}/approve",
@@ -1809,7 +1809,7 @@ 
 function boincuser_apachesolr_index_documents_alter(array &$documents, $entity, $entity_type, $env_id) {
 
   foreach ($documents as $document) {
-    if ( $document->entity_type=='node' AND $document->bundle=='profile' ) {
+    if ( $document->entity_type=='node' and $document->bundle=='profile' ) {
       // Node information.
       $nid = $document->entity_id;
       $node = node_load($nid);