BOINC / boinc

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

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

@@ -83,7 +83,7 @@ 
     'weight' => 5
   );
   $items['moderate/profile/%/reject'] = array(
-    'title' => bts('Reject profile', array(), NULL, 'boinc:moderate-user'),
+    'title' => bts('Reject profile', array(), null, 'boinc:moderate-user'),
     'description' => 'Reject profile content',
     'page callback' => 'drupal_get_form',
     'page arguments' => array('boincuser_moderate_profile_reject_form', 2),
@@ -92,7 +92,7 @@ 
     'weight' => 5
   );
   $items['moderate/user/%/ban'] = array(
-    'title' => bts('Ban user', array(), NULL, 'boinc:moderate-ban-user'),
+    'title' => bts('Ban user', array(), null, 'boinc:moderate-ban-user'),
     'description' => 'Ban a user from using community features',
     'page callback' => 'drupal_get_form',
     'page arguments' => array('boincuser_moderate_user_ban_form', 2),
@@ -123,7 +123,7 @@ 
     'weight' => 5
   );
   $items['user/login/auth'] = array(
-    'title' => bts('Authenticator login', array(), NULL, 'boinc:authenticator-login-page'),
+    'title' => bts('Authenticator login', array(), null, 'boinc:authenticator-login-page'),
     'description' => 'Log in using a user authenticator',
     'page callback' => 'drupal_get_form',
     'page arguments' => array('boincuser_authloginform'),
@@ -131,7 +131,7 @@ 
     'type' => MENU_CALLBACK,
   );
   $items['user/termsofuse'] = array(
-    'title' => bts('Terms of Use', array(), NULL, 'boinc:termsofuse-form'),
+    'title' => bts('Terms of Use', array(), null, 'boinc:termsofuse-form'),
     'description' => 'A site\'s term of use.',
     'page callback' => 'drupal_get_form',
     'page arguments' => array('boincuser_termsofuse_form'),
@@ -185,19 +185,19 @@ 
     'title' => 'Create Account RPC',
     'description' => 'RPC for creating user accounts.',
     'page callback' => 'boincuser_create_account',
-    'access callback' => TRUE,
+    'access callback' => true,
     'type' => MENU_CALLBACK
   );
   $items['account_finish.php'] = array(
     'title' => 'Welcome to ' . variable_get('site_name', 'Drupal-BOINC'),
     'description' => 'RPC for after a user has created an account.',
     'page callback' => 'boincuser_account_finish',
-    'access callback' => TRUE,
+    'access callback' => true,
     'type' => MENU_CALLBACK,
   );
   $items['boincuser/autocomplete'] = array(
     'page callback' => '_boincuser_user_name_autocomplete',
-    'access callback' => TRUE,
+    'access callback' => true,
     'type' => MENU_CALLBACK,
   );
   $items['user/%user/recoveremail/%'] = array(
@@ -212,7 +212,7 @@ 
     'title' => t('Recover previous email'),
     'description' => t('redirect'),
     'page callback' => '_boincuser_redirect_recover_email',
-    'access callback' => TRUE,
+    'access callback' => true,
     'type' => MENU_CALLBACK,
   );
   return $items;
@@ -247,7 +247,7 @@ 
   // Check if user has agreed to the terms of use. If not, send the
   // user to the terms-of-use form. This is only makes sense if the
   // termsofuse is enabled, by having text in the termsofuse variable.
-  $existinguser_tou = variable_get('boinc_weboptions_existinguser_tou', FALSE);
+  $existinguser_tou = variable_get('boinc_weboptions_existinguser_tou', false);
   $termsofuse = variable_get('boinc_weboptions_termsofuse', '');
   if ( (!empty($termsofuse)) and ($user->uid) ) {
     if ( !boincuser_check_termsofuse($user) and ($existinguser_tou) ) {
@@ -296,7 +296,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) {
   require_boinc('boinc_db');
   require_boinc('user');
   require_boinc('xml');
@@ -345,7 +345,7 @@ 
       $account->boincuser_email_addr_change_time = $boinc_user->email_addr_change_time;
       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",
@@ -453,7 +453,7 @@ 
         $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.', array(), NULL, 'boinc:add-new-user'));
+          form_set_error('email', bts('Error creating BOINC account.', array(), null, 'boinc:add-new-user'));
           return;
         }
 
@@ -499,9 +499,9 @@ 
         case 'user_account':
           // Ensure that BOINC data is altered
 
-          $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 email address to lower-case
             $lower_email_addr = strtolower($edit['mail']);
 
@@ -583,13 +583,13 @@ 
  *(forward compatible to Drupal 7).
  */
 function boincuser_user_login(&$edit, $account) {
-  $existinguser_tou = variable_get('boinc_weboptions_existinguser_tou', FALSE);
+  $existinguser_tou = variable_get('boinc_weboptions_existinguser_tou', false);
   $termsofuse = variable_get('boinc_weboptions_termsofuse', '');
 
   // Use the same code as boincuser_form_alter(), for case
   // 'user_profile_form', if the refering page is the user password
   // reset form, then do not check for terms of use.
-  $reset_pass = (strpos($_SERVER['HTTP_REFERER'], "/user/reset/$account->uid") === FALSE) ? 0 : 1;
+  $reset_pass = (strpos($_SERVER['HTTP_REFERER'], "/user/reset/$account->uid") === false) ? 0 : 1;
   if ($reset_pass) {
     return;
   }
@@ -737,7 +737,7 @@ 
 
       // General friend form overrides
       $form['flag_friend_submit']['#prefix'] = '<li class="first tab">';
-      $form['flag_friend_submit']['#value'] = bts('Send request', array(), NULL, 'boinc:friends-page');
+      $form['flag_friend_submit']['#value'] = bts('Send request', array(), null, 'boinc:friends-page');
       $form['flag_friend_submit']['#type'] = 'submit';
       $form['flag_friend_submit']['#suffix'] = '</li>';
       $form['flag_friend_submit']['#weight'] = 1002;
@@ -746,7 +746,7 @@ 
       case FLAG_FRIEND_BOTH:
       case FLAG_FRIEND_FLAGGED:
         unset($form['actions']);
-        $form['flag_friend_submit']['#value'] = bts('Remove friend', array(), NULL, 'boinc:friends-remove');
+        $form['flag_friend_submit']['#value'] = bts('Remove friend', array(), null, 'boinc:friends-remove');
         // Add in our hack handler to fix the flag_friend module
         $final_handler = array_pop($form['#submit']);
         $form['#submit'][] = 'boincuser_fix_unfriend_form_submit';
@@ -754,21 +754,21 @@ 
         break;
       case FLAG_FRIEND_PENDING:
         unset($form['actions']);
-        $form['flag_friend_submit']['#value'] = bts('Remove request', array(), NULL, 'boinc:friends-page');
+        $form['flag_friend_submit']['#value'] = bts('Remove request', array(), null, 'boinc:friends-page');
         break;
       case FLAG_FRIEND_APPROVAL:
         if ($action == 'flag') {
-          $form['flag_friend_submit']['#value'] = bts('Approve request', array(), NULL, 'boinc:friends-page');
+          $form['flag_friend_submit']['#value'] = bts('Approve request', array(), null, 'boinc:friends-page');
         }
         elseif ($action == 'unflag') {
           unset($form['actions']);
-          $form['flag_friend_submit']['#value'] = bts('Deny request', array(), NULL, 'boinc:friends-page');
+          $form['flag_friend_submit']['#value'] = bts('Deny request', array(), null, 'boinc:friends-page');
         }
         break;
       case FLAG_FRIEND_UNFLAGGED:
       default:
         $user_links[] = array(
-          'title' => bts('Add as friend', array(), NULL, 'boinc:friends-add'),
+          'title' => bts('Add as friend', array(), null, 'boinc:friends-add'),
           'href' => "flag/confirm/flag/friend/{$account->uid}"
         );
       }
@@ -778,7 +778,7 @@ 
     }
 
     $form['cancel'] = array(
-      '#value' => '<li class="tab">' . l(bts('Cancel', array(), NULL, 'boinc:form-cancel'), $cancel_dest) . '</li>',
+      '#value' => '<li class="tab">' . l(bts('Cancel', array(), null, 'boinc:form-cancel'), $cancel_dest) . '</li>',
       '#weight' => 1004,
     );
     $form['form control tabs suffix'] = array(
@@ -801,7 +801,7 @@ 
       '#weight' => 1001,
     );
     $form['buttons']['submit']['#prefix'] = '<li class="first tab">';
-    $form['buttons']['submit']['#value'] = bts('Save changes', array(), NULL, 'boinc:form-save');
+    $form['buttons']['submit']['#value'] = bts('Save changes', array(), null, 'boinc:form-save');
     $form['buttons']['submit']['#suffix'] = '</li>';
     $form['buttons']['submit']['#weight'] = 1002;
     $form['buttons']['preview']['#prefix'] = '<li class="tab">';
@@ -811,7 +811,7 @@ 
     $form['buttons']['preview_changes']['#suffix'] = '</li>';
     $form['buttons']['preview_changes']['#weight'] = 1004;
     $form['buttons']['cancel'] = array(
-      '#value' => '<li class="tab">' . l(bts('Cancel', array(), NULL, 'boinc:form-cancel'), "node/{$form['nid']['#value']}") . '</li>',
+      '#value' => '<li class="tab">' . l(bts('Cancel', array(), null, 'boinc:form-cancel'), "node/{$form['nid']['#value']}") . '</li>',
       '#weight' => 1005,
     );
     $form['buttons']['delete']['#prefix'] = '<li class="tab">';
@@ -839,11 +839,11 @@ 
       '#weight' => 1001,
     );
     $form['actions']['submit']['#prefix'] = '<li class="first tab">';
-    $form['actions']['submit']['#value'] = bts('Delete', array(), NULL, 'boinc:form-delete');
+    $form['actions']['submit']['#value'] = bts('Delete', array(), null, 'boinc:form-delete');
     $form['actions']['submit']['#suffix'] = '</li>';
     $form['actions']['submit']['#weight'] = 1002;
     $form['actions']['cancel'] = array(
-        '#value' => '<li class="tab">' . l(bts('Cancel', array(), NULL, 'boinc:form-cancel'), "node/{$form['nid']['#value']}") . '</li>',
+        '#value' => '<li class="tab">' . l(bts('Cancel', array(), null, 'boinc:form-cancel'), "node/{$form['nid']['#value']}") . '</li>',
       '#weight' => 1005,
     );
     $form['actions']['form control tabs suffix'] = array(
@@ -863,14 +863,14 @@ 
       '#weight' => 1001,
     );
     $form['privatemsg']['submit']['#prefix'] = '<li class="first tab">';
-    $form['privatemsg']['submit']['#value'] = bts('Send message', array(), NULL, 'boinc:private-message');
+    $form['privatemsg']['submit']['#value'] = bts('Send message', array(), null, 'boinc:private-message');
     $form['privatemsg']['submit']['#suffix'] = '</li>';
     $form['privatemsg']['submit']['#weight'] = 1002;
     $form['privatemsg']['preview']['#prefix'] = '<li class="tab">';
     $form['privatemsg']['preview']['#suffix'] = '</li>';
     $form['privatemsg']['preview']['#weight'] = 1003;
     $form['privatemsg']['cancel'] = array(
-      '#value' => '<li class="tab">' . l(bts('Cancel', array(), NULL, 'boinc:form-cancel'), $_GET['q']) . '</li>',
+      '#value' => '<li class="tab">' . l(bts('Cancel', array(), null, 'boinc:form-cancel'), $_GET['q']) . '</li>',
       '#weight' => 1004,
     );
     $form['privatemsg']['form control tabs suffix'] = array(
@@ -885,18 +885,18 @@ 
   // Login form
   case 'user_login':
   case 'user_login_block':
-      drupal_set_title(bts('Login', array(), NULL, 'boinc:menu-link'));
+      drupal_set_title(bts('Login', array(), null, 'boinc:menu-link'));
     // Replace name with email in login form
     unset($form['name']);
     array_unshift($form, array(
       'email' => array(
         '#type' => 'textfield',
-        '#title' => bts('Email address', array(), NULL, 'boinc:email-address-to-login'),
+        '#title' => bts('Email address', array(), null, 'boinc:email-address-to-login'),
         '#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', 'Drupal-BOINC')), NULL, 'boinc:standard-login-page')
+        '#description' => bts('Enter your @s email address.', array('@s' => variable_get('site_name', 'Drupal-BOINC')), null, 'boinc:standard-login-page')
       ),
       'validation_source' => array(
         '#type' => 'hidden',
@@ -912,7 +912,7 @@ 
     );
     $form['buttons']['submit'] = $form['submit'];
     $form['buttons']['submit']['#prefix'] = '<li class="first tab">';
-    $form['buttons']['submit']['#value'] = bts('Log in', array(), NULL, 'boinc:standard-login-page');
+    $form['buttons']['submit']['#value'] = bts('Log in', array(), null, 'boinc:standard-login-page');
     $form['buttons']['submit']['#suffix'] = '</li>';
     $form['buttons']['submit']['#weight'] = 1002;
     $form['buttons']['form control tabs suffix'] = array(
@@ -928,7 +928,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']);
@@ -956,26 +956,26 @@ 
     }
 
     // Set special message if user has not agreed to TOU
-    $existinguser_tou = variable_get('boinc_weboptions_existinguser_tou', FALSE);
+    $existinguser_tou = variable_get('boinc_weboptions_existinguser_tou', false);
     $termsofuse = variable_get('boinc_weboptions_termsofuse', '');
     if ( (!boincuser_check_termsofuse($account)) and ($existinguser_tou) and (!empty($termsofuse)) and (!user_access('administer users')) ) {
       drupal_set_message(
         bts('INFO: You have not agreed to the terms of use for @project. You may use this form to change your email address and/or password. Please note: you may not delete your account within seven (7) days of changing your email address.',
         array(
           '@project' => variable_get('site_name','Drupal-BOINC'),
-        ), NULL, 'boinc:account-credentials-change')
+        ), null, 'boinc:account-credentials-change')
       , 'info');
     }
 
     // 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;
     }
 
     // Adjust form elements already present
-    $form['account']['name']['#title'] = bts('Drupal Username', array(), NULL, 'boinc:drupal-username');
+    $form['account']['name']['#title'] = bts('Drupal Username', array(), null, 'boinc:drupal-username');
     $form['account']['name']['#size'] = 40;
     $form['account']['name']['#attributes']['autocomplete'] = 'off';
     $form['account']['name']['#description'] .= '<p>This is the drupal (internal) username. The title above shows the BOINC username, which is the display name used publicly that the user can change in Community Preferences.';
@@ -986,11 +986,11 @@ 
       // Add BOINC username (aka displayname)
       $form['account']['boincuser_name'] = array(
         '#type' => 'textfield',
-        '#title' => bts('BOINC Username', array(), NULL, 'boinc:user-or-team-name'),
+        '#title' => bts('BOINC Username', array(), null, 'boinc:user-or-team-name'),
         '#default_value' => $account->boincuser_name,
         '#maxlength' => USERNAME_MAX_LENGTH,
-        '#required' => TRUE,
-        '#description' => bts('This is the BOINC (external) username. This is the same setting as found in Account -> Preferences -> Community.', array(), NULL, 'boinc:username-change'),
+        '#required' => true,
+        '#description' => bts('This is the BOINC (external) username. This is the same setting as found in Account -> Preferences -> Community.', array(), null, 'boinc:username-change'),
         '#size' => 40,
       );
     }
@@ -999,23 +999,23 @@ 
     // ago, it cannot be changed again.
     $duration = TOKEN_DURATION_ONE_WEEK;
     if (($account->boincuser_email_addr_change_time + $duration) > time() and (!user_access('administer users'))) {
-      $form['account']['mail']['#required'] = FALSE;
+      $form['account']['mail']['#required'] = false;
       $form['account']['mailhelp'] = array(
         '#value' => bts("You email address was changed within the past seven (7) days. Please look for an email to !prev_email if you need to revert this change. You may change your email address on !time.",
           array(
             '!prev_email' => $account->boincuser_previous_email_addr,
             '!time' => date('F j, Y \a\t G:i T', $account->boincuser_email_addr_change_time + $duration),
-          ), NULL, 'boinc:account-credentials-change'),
+          ), null, 'boinc:account-credentials-change'),
       );
     }
 
-    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',
-        '#title' => bts('Enter your password to save changes', array(), NULL, 'boinc:account-credentials-change'),
+        '#title' => bts('Enter your password to save changes', array(), null, 'boinc:account-credentials-change'),
         '#description' => bts('Enter your current password if changing your email
-          address or password.', array(), NULL, 'boinc:account-credentials-change'),
+          address or password.', array(), null, 'boinc:account-credentials-change'),
         '#size' => 17,
         '#attributes' => array(
           'autocomplete' => 'off',
@@ -1027,35 +1027,35 @@ 
     $form['account']['boincuser_id'] = array(
       '#value' => '
         <div class="form-item">
-          <label>' . bts('BOINC user ID', array(), NULL, 'boinc:account-credentials-change') . '</label>
+          <label>' . bts('BOINC user ID', array(), null, 'boinc:account-credentials-change') . '</label>
           <span>' . $account->boincuser_id . '</span>
         </div>',
     );
     $form['account']['user_id'] = array(
       '#value' => '
         <div class="form-item">
-          <label>' . bts('Drupal user ID', array(), NULL, 'boinc:account-credentials-change') . '</label>
+          <label>' . bts('Drupal user ID', array(), null, 'boinc:account-credentials-change') . '</label>
           <span>' . $account->uid . '</span>
         </div>',
     );
     $form['account']['account_key'] = array(
       '#value' => '
         <div class="form-item">
-          <label>' . bts('Account key', array(), NULL, 'boinc:account-credentials-change') . '</label>
+          <label>' . bts('Account key', array(), null, 'boinc:account-credentials-change') . '</label>
           <span>' . $account->boincuser_account_key . '</span>
         </div>',
     );
     $form['account']['weak_account_key'] = array(
       '#value' => '
         <div class="form-item">
-          <label>' . bts('Weak account key', array(), NULL, 'boinc:account-credentials-change') . '</label>
+          <label>' . bts('Weak account key', array(), null, 'boinc:account-credentials-change') . '</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', array(), NULL, 'boinc:account-credentials-change') . '</label>
+          <label>' . bts('Cross-project ID', array(), null, 'boinc:account-credentials-change') . '</label>
           <span>' . $account->boincuser_cpid . '</span>
         </div>',
     );
@@ -1070,14 +1070,14 @@ 
       '#weight' => 1001,
     );
     $form['submit']['#prefix'] = '<li class="first tab">';
-    $form['submit']['#value'] = bts('Save changes', array(), NULL, 'boinc:form-save');
+    $form['submit']['#value'] = bts('Save changes', array(), null, 'boinc:form-save');
     $form['submit']['#suffix'] = '</li>';
     $form['submit']['#weight'] = 1002;
     $form['cancel'] = array(
-      '#value' => '<li class="tab">' . l(bts('Cancel', array(), NULL, 'boinc:form-cancel'), $_GET['q']) . '</li>',
+      '#value' => '<li class="tab">' . l(bts('Cancel', array(), null, 'boinc:form-cancel'), $_GET['q']) . '</li>',
       '#weight' => 1003,
     );
-    if (isset($form['delete']) AND is_array($form['delete'])) {
+    if (isset($form['delete']) and is_array($form['delete'])) {
       $form['delete']['#prefix'] = '<li class="first alt tab">';
       $form['delete']['#suffix'] = '</li>';
       $form['delete']['#weight'] = 1004;
@@ -1139,7 +1139,7 @@ 
 
     // Set form['title'], but hide it
     $form['title']['#default_value'] = $account->boincuser_name;
-    $form['title']['#access'] = FALSE;
+    $form['title']['#access'] = false;
 
     $form['separator_bottom'] = array(
       '#value' => '<div class="separator buttons"></div>',
@@ -1160,7 +1160,7 @@ 
       '#weight' => 1001,
     );
     $form['buttons']['submit']['#prefix'] = '<li class="first tab">';
-    $form['buttons']['submit']['#value'] = bts('Save changes', array(), NULL, 'boinc:form-save');
+    $form['buttons']['submit']['#value'] = bts('Save changes', array(), null, 'boinc:form-save');
     $form['buttons']['submit']['#suffix'] = '</li>';
     $form['buttons']['submit']['#weight'] = 1002;
     $form['buttons']['preview']['#prefix'] = '<li class="tab">';
@@ -1170,12 +1170,12 @@ 
     $form['buttons']['preview_changes']['#suffix'] = '</li>';
     $form['buttons']['preview_changes']['#weight'] = 1004;
     $form['buttons']['cancel'] = array(
-      '#value' => '<li class="tab">' . l(bts('Cancel', array(), NULL, 'boinc:form-cancel'), $_GET['q']) . '</li>',
+      '#value' => '<li class="tab">' . l(bts('Cancel', array(), null, 'boinc:form-cancel'), $_GET['q']) . '</li>',
       '#weight' => 1005,
     );
     $form['buttons']['delete']['#prefix'] = '<li class="tab">';
     $form['buttons']['delete']['#suffix'] = '</li>';
-    $form['buttons']['delete']['#value'] = bts('Clear User Profile', array(), NULL, 'boinc:form-save');
+    $form['buttons']['delete']['#value'] = bts('Clear User Profile', array(), null, 'boinc:form-save');
     $form['buttons']['delete']['#weight'] = 1006;
     $form['buttons']['delete']['#submit'] = array('_boincuser_node_profile_delete_submit');
     $form['buttons']['form control tabs suffix'] = array(
@@ -1209,11 +1209,11 @@ 
     array_unshift($form, array(
       'boincuser_name' => array(
         '#type' => 'textfield',
-        '#title' => bts('Name', array(), NULL, 'boinc:user-or-team-name'),
+        '#title' => bts('Name', array(), null, 'boinc:user-or-team-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.', array(), NULL, 'boinc:user-register'),
-        '#required' => TRUE
+        '#description' => bts('Spaces are allowed; punctuation is not allowed except for periods, hyphens, and underscores.', array(), null, 'boinc:user-register'),
+        '#required' => true
       ),
     ));
     // Set name temporarily to dummy value to beat validation
@@ -1233,27 +1233,27 @@ 
         '#type'   => 'fieldset',
         '#prefix' => '<div id="termsofuse-wrapper">', // This is our wrapper div.
         '#suffix' => '</div>',
-        '#tree'   => TRUE,
+        '#tree'   => true,
         '#weight' => -15,
       );
 
       $form['termsofuse']['title1'] = array(
         '#weight' => -12,
-        '#value' => '<h2>' . bts(variable_get('boinc_weboptions_registrationtitle', 'Please read and acknowledge our terms of use'), array(), NULL, 'project:user-register' ) . '</h2>',
+        '#value' => '<h2>' . bts(variable_get('boinc_weboptions_registrationtitle', 'Please read and acknowledge our terms of use'), array(), null, 'project:user-register' ) . '</h2>',
         '#prefix' => '<div id="register-title1">',
         '#suffix' => '</div>',
       );
 
       $form['termsofuse']['body'] = array(
         '#weight' => -10,
-        '#value' => bts($termsofuse, array(), NULL, 'project:user-register'),
+        '#value' => bts($termsofuse, array(), null, 'project:user-register'),
         '#prefix' => '<div id="register-termsofuse">',
         '#suffix' => '</div>',
       );
 
       $form['termsofuse']['agreeTOU'] = array(
         '#type' => 'checkbox',
-        '#title' => bts(variable_get('boinc_weboptions_agreequestion', 'Do you agree with the above terms of use?'), array(), NULL, 'project:user-register'),
+        '#title' => bts(variable_get('boinc_weboptions_agreequestion', 'Do you agree with the above terms of use?'), array(), null, 'project:user-register'),
         '#weight' => -8,
         '#prefix' => '<div id="register-checkbox">',
         '#suffix' => '</div>',
@@ -1262,7 +1262,7 @@ 
 
     $form['title2'] = array(
       '#weight' => -6,
-      '#value' => '<h2>' . bts(variable_get('boinc_weboptions_registrationtitle2', 'Fill in your name, email, and choose a secure passphrase.'), array(), NULL, 'project:user-register') . '</h2>',
+      '#value' => '<h2>' . bts(variable_get('boinc_weboptions_registrationtitle2', 'Fill in your name, email, and choose a secure passphrase.'), array(), null, 'project:user-register') . '</h2>',
       '#prefix' => '<div id="register-title2">',
       '#suffix' => '</div>',
     );
@@ -1282,26 +1282,26 @@ 
 
   // Request new password form
   case 'user_pass':
-    drupal_set_title(bts('Forgot password', array(), NULL, 'boinc:forgot-password'));
+    drupal_set_title(bts('Forgot password', array(), null, 'boinc: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(
         '#type' => 'textfield',
-        '#title' => bts('Email address', array(), NULL, 'boinc:email-address-to-login'),
+        '#title' => bts('Email address', array(), null, 'boinc:email-address-to-login'),
         '#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(
             '!authenticator_login' => l(
-              bts('authenticator-based login', array(), NULL, 'boinc:forgot-password'),
+              bts('authenticator-based login', array(), null, 'boinc:forgot-password'),
               'user/login/auth'
             )
           )
-        , NULL, 'boinc:forgot-password'),
+        , null, 'boinc:forgot-password'),
       ),
     ));
 
@@ -1322,11 +1322,11 @@ 
     );
     $form['buttons']['submit'] = $form['submit'];
     $form['buttons']['submit']['#prefix'] = '<li class="first tab">';
-    $form['buttons']['submit']['#value'] = bts('Send', array(), NULL, 'boinc:form-send');
+    $form['buttons']['submit']['#value'] = bts('Send', array(), null, 'boinc:form-send');
     $form['buttons']['submit']['#suffix'] = '</li>';
     $form['buttons']['submit']['#weight'] = 1002;
     $form['buttons']['cancel'] = array(
-      '#value' => '<li class="tab">' . l(bts('Cancel', array(), NULL, 'boinc:form-cancel'), 'user/login') . '</li>',
+      '#value' => '<li class="tab">' . l(bts('Cancel', array(), null, 'boinc:form-cancel'), 'user/login') . '</li>',
       '#weight' => 1005,
     );
     $form['buttons']['form control tabs suffix'] = array(
@@ -1343,7 +1343,7 @@ 
     break;
 
   case 'views_exposed_form':
-      $form['submit']['#value'] = bts('Search', array(), NULL, 'boinc:search-user');
+      $form['submit']['#value'] = bts('Search', array(), null, 'boinc:search-user');
     break;
   }
 }
@@ -1375,7 +1375,7 @@ 
 function boincuser_process_password_confirm($element) {
   // Check if parent element is "account".
   if ($element['#array_parents'][0] == 'account') {
-    $element['pass1']['#title'] = bts('Change password', array(), NULL, 'boinc:forgot-password');
+    $element['pass1']['#title'] = bts('Change password', array(), null, 'boinc:forgot-password');
   }
   return $element;
 }
@@ -1395,7 +1395,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;
@@ -1463,13 +1463,13 @@ 
   require_boinc('boinc_db');
   $num_deleted = BoincUserDeleted::delete_expired();
   if ($num_deleted>0) {
-    watchdog('boincuser', "Deleted ${num_deleted} users from user_deleted table", WATCHDOG_NOTICE);
+    watchdog('boincuser', "deleted ${num_deleted} users from user_deleted table", WATCHDOG_NOTICE);
   }
 
   // Delete expired tokens from token table
   $tokens_deleted = BoincToken::delete_expired();
   if ($tokens_deleted>0) {
-    watchdog('boincuser', "Deleted ${tokens_deleted} tokens from token table", WATCHDOG_NOTICE);
+    watchdog('boincuser', "deleted ${tokens_deleted} tokens from token table", WATCHDOG_NOTICE);
   }
 }
 
@@ -1511,8 +1511,8 @@ 
   if (!isset($account->roles[$verified_contributor])) {
     drupal_set_message(bts(
         'You may only create or modify your user profile after earning @count credits.',
-        array('@count' => $min_credit_to_post), NULL, 'boinc:view-profile'
-    ), 'warning', FALSE);
+        array('@count' => $min_credit_to_post), null, 'boinc:view-profile'
+    ), 'warning', false);
   }
 
   // For now, just call the user module profile view function
@@ -1555,35 +1555,35 @@ 
     if ($registration_enabled) {
       $output .= '<li>' . bts('First !create_an_account here at @sitename.',
       array(
-        '!create_an_account' => l(bts('create an account', array(), NULL, 'boinc:join-page'), 'user/registration'),
+        '!create_an_account' => l(bts('create an account', array(), null, 'boinc:join-page'), 'user/registration'),
         '@sitename' => $site_name,
-      ), NULL, 'boinc:join-page') . '</li>';
+      ), null, 'boinc:join-page') . '</li>';
     }
-    $output .= '  <li>' . bts("Install BOINC on this device if not already present.", array(), NULL, 'boinc:join-page') . '</li>';
+    $output .= '  <li>' . bts("Install BOINC on this device if not already present.", array(), null, 'boinc:join-page') . '</li>';
     $output .= '  <li>' . bts("Select <i>Tools / Add Project</i>. Choose @sitename from the list, or enter @siteurl.",
     array(
       '@sitename' => $site_name,
       '@siteurl' => $base_url,
-    ), NULL, 'boinc:join-page') . '</li>';
+    ), null, 'boinc:join-page') . '</li>';
     if ($registration_enabled) {
       $output .= '<li>' . bts("If you're running a command-line or pre-5.0 version of BOINC, use <b>!boinccmd</b> to add the project.",
         array(
           '!boinccmd' => l('boinccmd --project_attach', 'http://boinc.berkeley.edu/wiki/Boinccmd_tool'),
-        ), NULL, 'boinc:join-page') . '</li>';
+        ), null, 'boinc:join-page') . '</li>';
     }
     else {
       $output .= '<li>' . bts("If you're running a command-line version of BOINC,
         please follow the <b>!instructionslink</b> to first <i>create an account</i>, and then <i>attach</i> to this project. Use the same project URL as above.",
         array(
           '!instructionslink' => l('instructions', 'http://boinc.berkeley.edu/wiki/Boinccmd_tool'),
-        ), NULL, 'boinc:join-page') . '</li>';
+        ), null, 'boinc:join-page') . '</li>';
     }
     $output .= '<li>' . bts("If you're running a pre-5.0 version of BOINC, please
       upgrade to a more recent version of BOINC to create an account
       at @this_project.",
       array(
         '@this_project' => $site_name,
-      ), NULL, 'boinc:join-page') . '</li>';
+      ), null, 'boinc:join-page') . '</li>';
     $output .=  '</ol>';
     break;
   case 'new':
@@ -1603,29 +1603,29 @@ 
     if ($registration_enabled) {
       $output .= '<li>' . bts('First !create_an_account here at @sitename.',
       array(
-        '!create_an_account' => l(bts('create an account', array(), NULL, 'boinc:join-page'), 'user/registration'),
+        '!create_an_account' => l(bts('create an account', array(), null, 'boinc:join-page'), 'user/registration'),
         '@sitename' => $site_name,
-      ), NULL, 'boinc:join-page') . '</li>';
+      ), null, 'boinc:join-page') . '</li>';
     }
     else if ( menu_valid_path(array('link_path' => $ruleslink)) ) {
       $output .= '  <li>' . bts("Read our !rules_and_policies.", array(
-        '!rules_and_policies' => l(bts('Rules and Policies', array(), NULL, 'boinc:join-page'), $ruleslink),
-      ), NULL, 'boinc:join-page') . '</li>';
+        '!rules_and_policies' => l(bts('Rules and Policies', array(), null, 'boinc:join-page'), $ruleslink),
+      ), null, 'boinc:join-page') . '</li>';
     }
-    $output .= '  <li>' . bts('Download the BOINC desktop software.', array(), NULL, 'boinc:join-page');
+    $output .= '  <li>' . bts('Download the BOINC desktop software.', array(), null, 'boinc:join-page');
     $output .= '    <p>';
     $output .= '      <a class="button" href="http://boinc.berkeley.edu/download.php">Download</a>';
     $output .= '    </p>';
     $output .= '    ' . bts("For Android devices, visit !fdroid_link or download !apk_link directly.", array(
       '!fdroid_link' => '<a href="https://f-droid.org/en/packages/edu.berkeley.boinc">F-Droid</a>',
       '!apk_link' => '<a href="https://boinc.berkeley.edu/download_all.php">BOINC for Android</a>',
-    ), NULL, 'boinc:join-page');
+    ), null, 'boinc:join-page');
     $output .= '  </li>';
-    $output .= '  <li>' . bts('Run the installer.', array(), NULL, 'boinc:join-page') . '</li>';
+    $output .= '  <li>' . bts('Run the installer.', array(), null, 'boinc:join-page') . '</li>';
     $output .= '  <li>' . bts('Choose @sitename from the list, or enter @siteurl.', array(
       '@sitename' => $site_name,
       '@siteurl' => $base_url,
-    ), NULL, 'boinc:join-page') . '</li>';
+    ), null, 'boinc:join-page') . '</li>';
     $output .=  '</ol>';
   }
   $output .= '</div>';
@@ -1656,17 +1656,17 @@ 
   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!', array(), NULL, 'boinc:front-page') : ($site_name ? bts('What is @this_project?', array('@this_project' => $site_name)) : bts('Welcome!', array(), NULL, 'boinc:front-page'));
+  $output .= ($user->uid) ? bts('Welcome back!', array(), null, 'boinc:front-page') : ($site_name ? bts('What is @this_project?', array('@this_project' => $site_name)) : bts('Welcome!', array(), null, 'boinc:front-page'));
   $output .= '</h2>';
   $output .= '<div class="boinc-overview balance-height-front">';
-  $output .= '  <div>' . bts($site_message, array(), NULL, "project:front page") . ' ' . l(bts('Learn more', array(), NULL, 'boinc:front-page'), 'about') . '</div>';
+  $output .= '  <div>' . bts($site_message, array(), null, "project:front page") . ' ' . l(bts('Learn more', array(), null, 'boinc:front-page'), 'about') . '</div>';
   if ($user->uid) {
-    $output .= '  <div>' . l(bts('View account', array(), NULL, 'boinc:front-page'), 'dashboard', array('attributes' => array('class' => 'join button'))) . '</div>';
+    $output .= '  <div>' . l(bts('View account', array(), null, 'boinc:front-page'), 'dashboard', array('attributes' => array('class' => 'join button'))) . '</div>';
   }
   else {
-    $output .= '  <div>' . l(bts('Join now', array(), NULL, 'boinc:front-page'), 'join', array('attributes' => array('class' => 'join button'))) . '</div>';
+    $output .= '  <div>' . l(bts('Join now', array(), null, 'boinc:front-page'), 'join', array('attributes' => array('class' => 'join button'))) . '</div>';
   }
   $output .= '</div>';
   $output .= '<div class="boinc-overview-details">';
@@ -1674,17 +1674,17 @@ 
   $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', array(), NULL, 'boinc:front-page') . '</div>';
+  $output .= '      <div class="text">' . bts('User of the day', array(), null, 'boinc:front-page') . '</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.', array(), NULL, 'boinc:front-page') . '</div>';
+  $output .= '      <div class="text">' . bts('Over 500,000 volunteers and counting.', array(), null, 'boinc:front-page') . '</div>';
   $output .= '      <div class="platforms">';
-  $output .= '        <div class="detail platform windows">' . bts('Windows', array(), NULL, 'boinc:front-page') . '</div>';
-  $output .= '        <div class="detail platform mac">' .  bts('Mac', array(), NULL, 'boinc:front-page') . '</div>';
-  $output .= '        <div class="detail platform linux">' . bts('Linux', array(), NULL, 'boinc:front-page') . '</div>';
+  $output .= '        <div class="detail platform windows">' . bts('Windows', array(), null, 'boinc:front-page') . '</div>';
+  $output .= '        <div class="detail platform mac">' .  bts('Mac', array(), null, 'boinc:front-page') . '</div>';
+  $output .= '        <div class="detail platform linux">' . bts('Linux', array(), null, 'boinc:front-page') . '</div>';
   $output .= '      </div>';
   $output .= '    </div>';
   $output .= '  </div>';
@@ -1712,12 +1712,12 @@ 
   xml_header();
 
   // Account creation disabled
-  $enablethisRPC = variable_get('boinc_weboptions_enableaccountcreateRPC', TRUE);
+  $enablethisRPC = variable_get('boinc_weboptions_enableaccountcreateRPC', true);
   if (!$enablethisRPC) {
     $mess = bts('Account creation is done through our Web site. Please register at @url', array(
       '@url' => $base_url . '/user/registration',
     ),
-    NULL, 'boinc:create_account');
+    null, 'boinc:create_account');
     xml_error(-208, $mess);
   }
   // Invalid invite code
@@ -1771,7 +1771,7 @@ 
       'init' => $params['email_addr'],
       'roles' => array($unrestricted_role => ''),
       'boincuser_name' => $params['user_name'],
-      'boinchash_flag' => TRUE,
+      'boinchash_flag' => true,
     );
 
     // Create the drupal user. If the drupal user cannot be created,
@@ -1808,20 +1808,20 @@ 
   }
 
   if (strlen($authtoken) != 32) {
-    drupal_set_message(bts('ERROR: There is no account with that authenticator.', array(), NULL, 'boinc:account-finish'), 'error');
+    drupal_set_message(bts('ERROR: There is no account with that authenticator.', array(), null, 'boinc:account-finish'), 'error');
     drupal_goto();
   }
 
   require_boinc('boinc_db');
   $boinc_user = BoincUser::lookup("authenticator='".addslashes($authtoken)."'");
   if (!$boinc_user) {
-    drupal_set_message(bts('ERROR: There is no account with that authenticator.', array(), NULL, 'boinc:account-finish'), 'error');
+    drupal_set_message(bts('ERROR: There is no account with that authenticator.', array(), null, 'boinc:account-finish'), 'error');
     drupal_goto();
   }
   $user = user_load(get_drupal_id($boinc_user->id));
 
   if (!$user) {
-    drupal_set_message(bts('ERROR: There was a problem loading your account. Try logging in with your user name and password.', array(), NULL, 'boinc:account-finish'), 'error');
+    drupal_set_message(bts('ERROR: There was a problem loading your account. Try logging in with your user name and password.', array(), null, 'boinc:account-finish'), 'error');
     drupal_goto();
   }
 
@@ -1842,9 +1842,9 @@ 
   // Check moderation page exists
   $moderationpath = drupal_lookup_path('source', variable_get('boinc_weboptions_moderationpage', '') );
   if ( menu_valid_path(array('link_path' => $moderationpath)) ) {
-    $modsentence = bts('Please note: user profiles are subject to !moderation.', array('!moderation' => l(bts('moderation', array(), NULL, 'boinc:account-finish'), $moderationpath, $options)), NULL, 'boinc:account-finish');
+    $modsentence = bts('Please note: user profiles are subject to !moderation.', array('!moderation' => l(bts('moderation', array(), null, 'boinc:account-finish'), $moderationpath, $options)), null, 'boinc:account-finish');
   } else {
-    $modsentence = bts('Please note: user profiles are subject to moderation.', array(), NULL, 'boinc:account-finish');
+    $modsentence = bts('Please note: user profiles are subject to moderation.', array(), null, 'boinc:account-finish');
   }
 
   $username = $user->boincuser_name;
@@ -1853,73 +1853,73 @@ 
   array(
     '@user_name' => $username,
     '@site_name' => $site_name,
-  ), NULL, 'boinc:account-finish') . "</p>";
+  ), null, 'boinc:account-finish') . "</p>";
 
   $links = array(
     array(
       'data' => bts('Change your username at !community_preferences.', array(
-        '!community_preferences' => l(bts('Community Preferences', array(), NULL, 'boinc:account-fininsh'), 'account/prefs/community', $options),
-      ), NULL, 'boinc:account-finish'),
+        '!community_preferences' => l(bts('Community Preferences', array(), null, 'boinc:account-fininsh'), 'account/prefs/community', $options),
+      ), null, 'boinc:account-finish'),
       'children' => array(
-        bts('Your username is used to identify yourself to other volunteers on this Web site.', array(), NULL, 'boinc:account-finish'),
-        bts('In addition, you may set your account\'s default language and adjust notification settings.', array(), NULL, 'boinc:account-finish'),
+        bts('Your username is used to identify yourself to other volunteers on this Web site.', array(), null, 'boinc:account-finish'),
+        bts('In addition, you may set your account\'s default language and adjust notification settings.', array(), null, 'boinc:account-finish'),
       ),
     ),
     array(
       'data' => bts('Change your !computing_preferences.', array(
-        '!computing_preferences' => l(bts('Computing Preferences', array(), NULL, 'boinc:account-finish'), 'account/prefs', $options),
-      ), NULL, 'boinc:account-finish'),
+        '!computing_preferences' => l(bts('Computing Preferences', array(), null, 'boinc:account-finish'), 'account/prefs', $options),
+      ), null, 'boinc:account-finish'),
       'children' => array(
-        bts('You may adjust how much CPU, RAM, and Disk space the BOINC client is allowed to use for tasks on your computer.', array(), NULL, 'boinc:account-finish'),
+        bts('You may adjust how much CPU, RAM, and Disk space the BOINC client is allowed to use for tasks on your computer.', array(), null, 'boinc:account-finish'),
         bts('By default, you will run @site_name tasks without any additional configuration.', array(
           '@site_name' => $site_name,
-        ), NULL, 'boinc:account-finish'),
+        ), null, 'boinc:account-finish'),
         bts('It is recommended new volunteers leave the default settings until they gain experience running some tasks. Ask questions in the !forums to get advice before making changes to a setting you don\'t understand.', array(
-          '!forums' => l(bts('forums', array(), NULL, 'boinc:account-finish'), 'community/forum', $options),
-        ), NULL, 'boinc:account-finish'),
+          '!forums' => l(bts('forums', array(), null, 'boinc:account-finish'), 'community/forum', $options),
+        ), null, 'boinc:account-finish'),
       ),
     ),
     array(
         'data' => bts('Create a !user_profile.', array(
-          '!user_profile' => l(bts('User Profile', array(), NULL, 'boinc:account-finish'), '/account/profile/edit', $options),
-        ), NULL, 'boinc:account-finish'),
+          '!user_profile' => l(bts('User Profile', array(), null, 'boinc:account-finish'), '/account/profile/edit', $options),
+        ), null, 'boinc:account-finish'),
       'children' => array(
         bts('A user profile will inform other volunteers who you are and why you joined @site_name.', array(
           '@site_name' => $site_name,
-        ), NULL, 'boinc:account-finish'),
+        ), null, 'boinc:account-finish'),
         $modsentence,
       ),
     ),
     array(
       'data' => bts('Join a !team.', array(
-        '!team' => l(bts('Team', array(), NULL, 'boinc:account-finish'), '/community/teams', $options),
-      ), NULL, 'boinc:account-finish'),
+        '!team' => l(bts('Team', array(), null, 'boinc:account-finish'), '/community/teams', $options),
+      ), null, 'boinc:account-finish'),
       'children' => array(
-        bts('You may join a team, made up of other volunteers.', array(), NULL, 'boinc:account-finish'),
+        bts('You may join a team, made up of other volunteers.', array(), null, 'boinc:account-finish'),
       ),
     ),
     array(
       'data' => bts('Go to your !account_dashboard.', array(
-        '!account_dashboard'=> l(bts('Account Dashboard', array(), NULL, 'boinc:account-finish'), 'account/dashboard', $options),
-      ), NULL, 'boinc:account-finish'),
+        '!account_dashboard'=> l(bts('Account Dashboard', array(), null, 'boinc:account-finish'), 'account/dashboard', $options),
+      ), null, 'boinc:account-finish'),
       'children' => array(
-        bts('Your account dashboard has information and links about your computer(s) and task(s) assigned.', array(), NULL, 'boinc:account-finish'),
+        bts('Your account dashboard has information and links about your computer(s) and task(s) assigned.', array(), null, 'boinc:account-finish'),
       ),
     ),
     array(
       'data' => bts('Visit our !help pages.', array(
-        '!help' => l(bts('Help', array(), NULL, 'boinc:account-finish'), '/help', $options)
-      ), NULL, 'boinc:account-finish'),
+        '!help' => l(bts('Help', array(), null, 'boinc:account-finish'), '/help', $options)
+      ), null, 'boinc:account-finish'),
       'children' => array(
         bts('Ask for help in our community\'s !forums.', array(
-          '!forums' => l(bts('forums', array(), NULL, 'boinc:account-finish'), 'community/forum', $options)
-        ), NULL, 'boinc:account-finish'),
+          '!forums' => l(bts('forums', array(), null, 'boinc:account-finish'), 'community/forum', $options)
+        ), null, 'boinc:account-finish'),
       ),
     ),
   );
 
   //List of links
-  $output .= theme_item_list($links, $title = NULL, $type='ul');
+  $output .= theme_item_list($links, $title = null, $type='ul');
 
   return $output;
 }
@@ -1929,20 +1929,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:
@@ -1956,7 +1956,7 @@ 
       AND moderate = 1"
     ));
   }
-  return NULL;
+  return null;
 }
 
 /**
@@ -1964,10 +1964,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':
@@ -2073,7 +2073,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');
@@ -2153,7 +2153,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("
@@ -2166,7 +2166,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,
@@ -2194,23 +2194,23 @@ 
  * 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...', array(), NULL, 'boinc:account-dashboard');
+  if (!$projects) return bts('no projects...', array(), null, 'boinc:account-dashboard');
 
   $output = '';
   $output .= '<table class="user-projects">' . "\n";
   $output .= '<thead>' . "\n";
   $output .= '  <tr>' . "\n";
-  $output .= '    <th>' . bts('Name', array(), NULL, 'boinc:project-name:-1:ignoreoverwrite') . '</th>' . "\n";
-  $output .= '    <th class="numeric">' . bts('Avg credit', array(), NULL, 'boinc:account-dashboard') . '</th>' . "\n";
-  $output .= '    <th class="numeric">' . bts('Total credit', array(), NULL, 'boinc:user-or-team-total-credits') . '</th>' . "\n";
+  $output .= '    <th>' . bts('Name', array(), null, 'boinc:project-name:-1:ignoreoverwrite') . '</th>' . "\n";
+  $output .= '    <th class="numeric">' . bts('Avg credit', array(), null, 'boinc:account-dashboard') . '</th>' . "\n";
+  $output .= '    <th class="numeric">' . bts('Total credit', array(), null, 'boinc:user-or-team-total-credits') . '</th>' . "\n";
   $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";
@@ -2248,7 +2248,7 @@ 
   if (in_array($result->code, array(200, 304)) || in_array($result->redirect_code, array(200, 304))) {
     return simplexml_load_string($result->data);
   }
-  return NULL;
+  return null;
 }
 
 /**
@@ -2264,7 +2264,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;
 }
 
 
@@ -2277,18 +2277,18 @@ 
   $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', array(), NULL, 'boinc:moderate-user'),
+        'title' => bts('Approve profile', array(), null, 'boinc:moderate-user'),
         'href' => "{$profile_moderation_path}/approve",
         'attributes' => array(
-          'title' => bts('Approve this profile content', array(), NULL, 'boinc:moderate-user'),
+          'title' => bts('Approve this profile content', array(), null, 'boinc:moderate-user'),
           'class' => 'first primary tab',
         )
       );
@@ -2301,10 +2301,10 @@ 
         )
       );*/
       $links['reject_profile'] = array(
-        'title' => bts('Reject profile', array(), NULL, 'boinc:moderate-user'),
+        'title' => bts('Reject profile', array(), null, 'boinc:moderate-user'),
         'href' => "{$profile_moderation_path}/reject",
         'attributes' => array(
-          'title' => bts('Reject this profile content', array(), NULL, 'boinc:moderate-user'),
+          'title' => bts('Reject this profile content', array(), null, 'boinc:moderate-user'),
           'class' => 'tab',
         )
       );
@@ -2358,7 +2358,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);
@@ -2412,7 +2412,7 @@ 
     if ($recipient = user_load(array('uid' => $drupalid))) {
       // Double-check that the loaded user matches both boincuser_id
       // and boincuser_name.
-      if ( ($boincid == $recipient->boincuser_id) AND ($boincname == $recipient->boincuser_name) ) {
+      if ( ($boincid == $recipient->boincuser_id) and ($boincname == $recipient->boincuser_name) ) {
         return $recipient;
       }
     }