BOINC / boinc

drupal/sites/default/boinc/modules/contrib/privatemsg/pm_block_user/pm_block_user.module

Doc Comments +3 added lines, -3 removed lines

@@ -374,7 +374,7 @@ 
  *   messages from.
  *
  * @return
- *   TRUE if the user is allowed to block $account, or FALSE if not.
+ boolean   TRUE if the user is allowed to block $account, or FALSE if not.
  */
 function _pm_block_user_access($account) {
   global $user;
@@ -399,11 +399,11 @@ 
  *   Author user object to check.
  * @param $recipient
  *   Receiver user object to check.
- * @param $action
+ * @param integer $action
  *   The action to be taken, defaults to PM_BLOCK_USER_DISALLOW_BLOCKING.
  *
  * @return
- *   TRUE if a rule exists for the combination of author recipient and action.
+ boolean   TRUE if a rule exists for the combination of author recipient and action.
  */
 function _pm_block_user_rule_exists($author, $recipient, $action = PM_BLOCK_USER_DISALLOW_BLOCKING) {
   $block_actions = variable_get('pm_block_user_actions', array());

Indentation +222 added lines, -222 removed lines

@@ -19,17 +19,17 @@ 
  * Implements hook_help().
  */
 function pm_block_user_help($path) {
-  switch ($path) {
+    switch ($path) {
     case 'admin/settings/messages/block':
       return '<p>' . t('This area is used to define user blocking rules for the Privatemsg module. Rules allow control of who may block messages from whom. By default all users are allowed to block messages from anyone else. However, a site may have groups of users that need to contact or get information to others, for example: the site may have administrative staff or be a forum with moderators. Groups of users are defined by roles, which can be managed on the <a href="@roles">roles configuration page</a>.', array('@roles' => url('admin/user/roles'))) . '</p>';
-  }
+    }
 }
 
 /**
  * Implements hook_menu().
  */
 function pm_block_user_menu() {
-  $items['messages/block/%user'] = array(
+    $items['messages/block/%user'] = array(
     'title'            => 'Block user messages',
     'page callback'    => 'drupal_get_form',
     'page arguments'   => array('pm_block_user_form', 2),
@@ -37,61 +37,61 @@ 
     'access arguments' => array(2),
     'type'             => MENU_CALLBACK,
     'weight'           => -10,
-  );
+    );
 
-  $items['admin/settings/messages/block'] = array(
+    $items['admin/settings/messages/block'] = array(
     'title'            => 'User blocking rules',
     'description'      => 'Configure rules for which users may block each other.',
     'page callback'    => 'drupal_get_form',
     'page arguments'   => array('pm_block_user_settings'),
     'access arguments' => array('administer privatemsg settings'),
     'type'             => MENU_LOCAL_TASK,
-  );
+    );
 
-  $items['messages/block/js'] = array(
+    $items['messages/block/js'] = array(
     'title' => 'Javascript block actions form',
     'page callback' => 'pm_block_user_js',
     'access arguments' => array('administer privatemsg settings'),
     'type' => MENU_CALLBACK,
-  );
+    );
 
-  return $items;
+    return $items;
 }
 
 /**
  * Menu callback for blocked user settings.
  */
 function pm_block_user_settings(&$form_state) {
-  drupal_add_css(drupal_get_path('module', 'pm_block_user') .'/pm_block_user.css');
-  // Need to cache form for AHAH, so it can be rebuilt from cache later.
-  $form = array(
+    drupal_add_css(drupal_get_path('module', 'pm_block_user') .'/pm_block_user.css');
+    // Need to cache form for AHAH, so it can be rebuilt from cache later.
+    $form = array(
     '#cache' => TRUE,
-  );
+    );
 
-  // Container for just the actions, used for AHAH.
-  $form['block_actions'] = array(
+    // Container for just the actions, used for AHAH.
+    $form['block_actions'] = array(
     '#tree' => TRUE,
     '#prefix' => '<div id="block-actions">',
     '#suffix' => '</div>',
     '#theme' => 'pm_block_user_actions',
-  );
+    );
 
-  // Should we populate the form with data from $form_state or the database?
-  if (!isset($form_state['pm_block_user']['block_actions'])) {
+    // Should we populate the form with data from $form_state or the database?
+    if (!isset($form_state['pm_block_user']['block_actions'])) {
     $block_actions = variable_get('pm_block_user_actions', array());
-  }
-  else {
+    }
+    else {
     $block_actions = $form_state['pm_block_user']['block_actions'];
-  }
-  // Work through each rule, adding it as a new element in
-  // $form['block_actions'] ready to be themed later.
-  foreach ($block_actions as $delta => $details) {
+    }
+    // Work through each rule, adding it as a new element in
+    // $form['block_actions'] ready to be themed later.
+    foreach ($block_actions as $delta => $details) {
     $details['delta'] = $delta;
     $form['block_actions'][$delta] = _pm_block_user_actions_form($details);
-  }
+    }
 
-  // The magic AHAH callback button that adds more rows.
-  $form['pm_block_actions_more'] = array(
+    // The magic AHAH callback button that adds more rows.
+    $form['pm_block_actions_more'] = array(
     '#type' => 'submit',
     '#value' => t('More'),
     '#weight' => 1,
@@ -99,20 +99,20 @@ 
     '#suffix' => '<label for="edit-pm-block-actions-more">' . t('Add new rule') . '</label></div>',
     '#submit' => array('pm_block_user_more_submit'),
     '#ahah' => array(
-      'path' => 'messages/block/js',
-      'wrapper' => 'block-actions',
-      'method' => 'replace',
-      'effect' => 'fade',
+        'path' => 'messages/block/js',
+        'wrapper' => 'block-actions',
+        'method' => 'replace',
+        'effect' => 'fade',
     ),
-  );
+    );
 
-  $form['submit_form'] = array(
+    $form['submit_form'] = array(
     '#type' => 'submit',
     '#weight' => 10,
     '#value' => t('Save configuration'),
-  );
+    );
 
-  return $form;
+    return $form;
 }
 
 /**
@@ -132,50 +132,50 @@ 
  *   Part of a form with controls for sending, receiving and actions.
  */
 function _pm_block_user_actions_form($details, $blacklist = TRUE) {
-  $form = array(
+    $form = array(
     '#tree' => TRUE,
-  );
-  $delta = $details['delta'];
-  // FALSE by default, or if the user has checked the 'Enabled' check box for
-  // this row.
-  $row_disabled = (isset($details['enabled']) ? !$details['enabled'] : FALSE);
+    );
+    $delta = $details['delta'];
+    // FALSE by default, or if the user has checked the 'Enabled' check box for
+    // this row.
+    $row_disabled = (isset($details['enabled']) ? !$details['enabled'] : FALSE);
 
-  $form['author'] = array(
+    $form['author'] = array(
     '#type' => 'select',
     '#options' => user_roles(TRUE),
     '#default_value' => (isset($details['author']) ? $details['author'] : DRUPAL_AUTHENTICATED_RID),
     '#disabled' => $row_disabled,
-  );
-  $form['recipient'] = array(
+    );
+    $form['recipient'] = array(
     '#type' => 'select',
     '#options' => user_roles(TRUE),
     '#default_value' => (isset($details['recipient']) ? $details['recipient'] : DRUPAL_AUTHENTICATED_RID),
     '#disabled' => $row_disabled,
-  );
+    );
 
-  // Provide different action radios if we're using a whitelist or a blacklist.
-  if ($blacklist) {
+    // Provide different action radios if we're using a whitelist or a blacklist.
+    if ($blacklist) {
     $options = array(
-      PM_BLOCK_USER_DISALLOW_BLOCKING => t('Disallow blocking author'),
-      PM_BLOCK_USER_DISALLOW_SENDING => t('Disallow sending message'),
+        PM_BLOCK_USER_DISALLOW_BLOCKING => t('Disallow blocking author'),
+        PM_BLOCK_USER_DISALLOW_SENDING => t('Disallow sending message'),
     );
     $default_value = (isset($details['action']) ? $details['action'] : PM_BLOCK_USER_DISALLOW_BLOCKING);
-  }
-  else {
+    }
+    else {
     // @todo: add whitelist options/default_value here.
-  }
-  $form['action']  = array(
+    }
+    $form['action']  = array(
     '#type' => 'radios',
     '#options' => $options,
     '#disabled' => $row_disabled,
     '#default_value' => $default_value,
-  );
+    );
 
-  $form['enabled'] = array(
+    $form['enabled'] = array(
     '#type' => 'checkbox',
     '#default_value' => (isset($details['enabled']) ? $details['enabled'] : TRUE),
-  );
-  $form['remove'] = array(
+    );
+    $form['remove'] = array(
     '#type' => 'submit',
     '#submit' => array('pm_block_user_remove_submit'),
     '#value' => t('Remove'),
@@ -183,14 +183,14 @@ 
     '#prefix' => '<div id="remove-rule-button">',
     '#suffix' => '<label for="edit-remove">' . t('Remove rule') . '</label></div>',
     '#ahah' => array(
-      'path' => 'messages/block/js',
-      'wrapper' => 'block-actions',
-      'method' => 'replace',
-      'effect' => 'fade',
+        'path' => 'messages/block/js',
+        'wrapper' => 'block-actions',
+        'method' => 'replace',
+        'effect' => 'fade',
     ),
-  );
+    );
 
-  return $form;
+    return $form;
 }
 
 /**
@@ -199,17 +199,17 @@ 
  * @see pm_block_user_remove_submit()
  */
 function pm_block_user_more_submit($form, &$form_state) {
-  unset($form_state['submit_handlers']);
-  form_execute_handlers('submit', $form, $form_state);
-  // Get the submitted actions, then put them into a special area of
-  // the $form_state.
-  $submitted_values = $form_state['values'];
-  // Add an empty action.
-  $submitted_values['block_actions'][] = array();
-  $form_state['pm_block_user'] = $submitted_values;
-  // Rebuild the form by passing our $form_state through the
-  // pm_block_user_settings() builder function.
-  $form_state['rebuild'] = TRUE;
+    unset($form_state['submit_handlers']);
+    form_execute_handlers('submit', $form, $form_state);
+    // Get the submitted actions, then put them into a special area of
+    // the $form_state.
+    $submitted_values = $form_state['values'];
+    // Add an empty action.
+    $submitted_values['block_actions'][] = array();
+    $form_state['pm_block_user'] = $submitted_values;
+    // Rebuild the form by passing our $form_state through the
+    // pm_block_user_settings() builder function.
+    $form_state['rebuild'] = TRUE;
 }
 
 /**
@@ -218,33 +218,33 @@ 
  * @see pm_block_user_more_submit()
  */
 function pm_block_user_remove_submit($form, &$form_state) {
-  unset($form_state['submit_handlers']);
-  form_execute_handlers('submit', $form, $form_state);
-  $submitted_values = $form_state['values'];
-  // Remove the requested action.
-  $delta = $form_state['clicked_button']['#parents'][1];
-  unset($submitted_values['block_actions'][$delta]);
-  $form_state['pm_block_user'] = $submitted_values;
-  $form_state['rebuild'] = TRUE;
+    unset($form_state['submit_handlers']);
+    form_execute_handlers('submit', $form, $form_state);
+    $submitted_values = $form_state['values'];
+    // Remove the requested action.
+    $delta = $form_state['clicked_button']['#parents'][1];
+    unset($submitted_values['block_actions'][$delta]);
+    $form_state['pm_block_user'] = $submitted_values;
+    $form_state['rebuild'] = TRUE;
 }
 
 /**
  * Submit handler for admin form.
  */
 function pm_block_user_settings_submit($form, &$form_state) {
-  // We don't want it to submit when we're adding/removing actions.
-  if ($form_state['clicked_button']['#id'] == 'edit-submit-form') {
+    // We don't want it to submit when we're adding/removing actions.
+    if ($form_state['clicked_button']['#id'] == 'edit-submit-form') {
     // If the form's 'block_actions' aren't set, the user has deleted all the
     // rows in the table, so we save an empty array to stop errors in the form
     // builder.
     if (isset($form_state['values']['block_actions'])) {
-      variable_set('pm_block_user_actions', _pm_block_user_settings_filter($form_state['values']['block_actions']));
+        variable_set('pm_block_user_actions', _pm_block_user_settings_filter($form_state['values']['block_actions']));
     }
     else {
-      variable_set('pm_block_user_actions', array());
+        variable_set('pm_block_user_actions', array());
     }
     drupal_set_message(t('The configuration options have been saved.'));
-  }
+    }
 }
 
 /**
@@ -259,60 +259,60 @@ 
  * @see pm_block_user_settings_submit()
  */
 function _pm_block_user_settings_filter($settings) {
-  // Add-in the names of any settings to be saved into the array below.
-  $save_keys = array('author', 'recipient', 'action', 'enabled');
-  $matching = array();
-  // Run through each of the keys we want to save, creating a new array.
-  // It's not possible to simply check for unwanted values and unset() them as
-  // the array is multi-dimensional.
-  foreach ($save_keys as $save_key) {
+    // Add-in the names of any settings to be saved into the array below.
+    $save_keys = array('author', 'recipient', 'action', 'enabled');
+    $matching = array();
+    // Run through each of the keys we want to save, creating a new array.
+    // It's not possible to simply check for unwanted values and unset() them as
+    // the array is multi-dimensional.
+    foreach ($save_keys as $save_key) {
     if (isset($settings[$save_key])) {
-      $matching[$save_key] = $settings[$save_key];
+        $matching[$save_key] = $settings[$save_key];
+    }
     }
-  }
-  if (count($matching) > 0) {
+    if (count($matching) > 0) {
     return $matching;
-  }
-  else {
+    }
+    else {
     return array_map('_pm_block_user_settings_filter', $settings);
-  }
+    }
 }
 
 /**
  * Menu callback for AHAH handling.
  */
 function pm_block_user_js() {
-  // See: http://drupal.org/node/331941 for the philosophy of Drupal AHAH.
-  $form_state = array('storage' => NULL, 'submitted' => FALSE);
-  $form_build_id = $_POST['form_build_id'];
-  $form = form_get_cache($form_build_id, $form_state);
-  $args = $form['#parameters'];
-  $form_id = array_shift($args);
-  $form['#post'] = $_POST;
-  $form['#redirect'] = FALSE;
-  $form['#programmed'] = FALSE;
-  $form_state['post'] = $_POST;
-  drupal_process_form($form_id, $form, $form_state);
-  $form = drupal_rebuild_form($form_id, $form_state, $args, $form_build_id);
-  $output_form = $form['block_actions'];
-  unset($output_form['#prefix'], $output_form['#suffix']); // Prevent duplicate wrappers.
-  $output = theme('status_messages') . drupal_render($output_form);
-
-  // Hack to make behaviours attach to new HTML controls (delete buttons in
-  // this case).
-  $javascript = drupal_add_js(NULL, NULL, 'header');
-  drupal_json(array('status' => TRUE, 'data' => $output));
+    // See: http://drupal.org/node/331941 for the philosophy of Drupal AHAH.
+    $form_state = array('storage' => NULL, 'submitted' => FALSE);
+    $form_build_id = $_POST['form_build_id'];
+    $form = form_get_cache($form_build_id, $form_state);
+    $args = $form['#parameters'];
+    $form_id = array_shift($args);
+    $form['#post'] = $_POST;
+    $form['#redirect'] = FALSE;
+    $form['#programmed'] = FALSE;
+    $form_state['post'] = $_POST;
+    drupal_process_form($form_id, $form, $form_state);
+    $form = drupal_rebuild_form($form_id, $form_state, $args, $form_build_id);
+    $output_form = $form['block_actions'];
+    unset($output_form['#prefix'], $output_form['#suffix']); // Prevent duplicate wrappers.
+    $output = theme('status_messages') . drupal_render($output_form);
+
+    // Hack to make behaviours attach to new HTML controls (delete buttons in
+    // this case).
+    $javascript = drupal_add_js(NULL, NULL, 'header');
+    drupal_json(array('status' => TRUE, 'data' => $output));
 }
 
 /**
  * Implements hook_theme().
  */
 function pm_block_user_theme() {
-  return array(
+    return array(
     'pm_block_user_actions' => array(
-      'arguments' => array('form' => NULL),
+        'arguments' => array('form' => NULL),
     ),
-  );
+    );
 }
 
 /**
@@ -321,49 +321,49 @@ 
  * @ingroup themeable
  */
 function theme_pm_block_user_actions($form) {
-  $rows = array();
-  $headers = array(
+    $rows = array();
+    $headers = array(
     t('If the author has the role'),
     t('And the recipient has the role'),
     t('Action'),
     t('Enabled'),
     '',
-  );
-  $form_data = element_children($form);
+    );
+    $form_data = element_children($form);
 
-  foreach ($form_data as $key) {
+    foreach ($form_data as $key) {
     // Build the table row.
     $row = array(
-      'data' => array(
+        'data' => array(
         array('data' => drupal_render($form[$key]['author'])),
         array('data' => drupal_render($form[$key]['recipient'])),
         array('data' => drupal_render($form[$key]['action'])),
         array('data' => drupal_render($form[$key]['enabled'])),
         array('data' => drupal_render($form[$key]['remove'])),
-      ),
+        ),
     );
 
     // Add additional attributes to the row, such as a class for this row.
     if (isset($form[$key]['#attributes'])) {
-      $row = array_merge($row, $form[$key]['#attributes']);
+        $row = array_merge($row, $form[$key]['#attributes']);
     }
     $rows[] = $row;
-  }
+    }
 
-  // If there are no rows, output some instructions for the user.
-  if (empty($form_data)) {
+    // If there are no rows, output some instructions for the user.
+    if (empty($form_data)) {
     $rows[] = array(
-      array(
+        array(
         'data' => t("No rules have been added. All users may block private messages from each other. To limit which users may be blocked, click 'Add new rule'."),
         'colspan' => '5',
-      ),
+        ),
     );
-  }
+    }
 
-  $output = theme('table', $headers, $rows);
-  $output .= drupal_render($form);
+    $output = theme('table', $headers, $rows);
+    $output .= drupal_render($form);
 
-  return $output;
+    return $output;
 }
 
 /**
@@ -377,14 +377,14 @@ 
  *   TRUE if the user is allowed to block $account, or FALSE if not.
  */
 function _pm_block_user_access($account) {
-  global $user;
-  if (!privatemsg_user_access('read privatemsg', $user)) {
+    global $user;
+    if (!privatemsg_user_access('read privatemsg', $user)) {
     return FALSE;
-  }
-  if (_pm_block_user_rule_exists($account, $user, PM_BLOCK_USER_DISALLOW_BLOCKING)) {
+    }
+    if (_pm_block_user_rule_exists($account, $user, PM_BLOCK_USER_DISALLOW_BLOCKING)) {
     return FALSE;
-  }
-  return TRUE;
+    }
+    return TRUE;
 }
 
 /**
@@ -406,179 +406,179 @@ 
  *   TRUE if a rule exists for the combination of author recipient and action.
  */
 function _pm_block_user_rule_exists($author, $recipient, $action = PM_BLOCK_USER_DISALLOW_BLOCKING) {
-  $block_actions = variable_get('pm_block_user_actions', array());
-  foreach ($block_actions as $delta => $details) {
+    $block_actions = variable_get('pm_block_user_actions', array());
+    foreach ($block_actions as $delta => $details) {
     // If this rule doesn't relate to $action, or it's disabled
     // ignore it and go to next loop iteration.
     if ($details['action'] != $action || !$details['enabled']) {
-      continue;
+        continue;
     }
     // There are no rules governing user one, but user one may have roles that
     // affect other users, so these exceptions are narrow in scope.
     // Disallow sending affects private message authors.
     if ($author->uid == 1 && $action == PM_BLOCK_USER_DISALLOW_SENDING) {
-      continue;
+        continue;
     }
     // Disallow blocking affects private message recipients.
     if ($recipient->uid == 1 && $action == PM_BLOCK_USER_DISALLOW_BLOCKING) {
-      continue;
+        continue;
     }
     // The author has a role matching the rule and so does the recipient.
     if (isset($author->roles[$details['author']]) && isset($recipient->roles[$details['recipient']])) {
-      return TRUE;
+        return TRUE;
+    }
     }
-  }
-  return FALSE;
+    return FALSE;
 }
 
 function pm_block_user_form($form_state, $author) {
-  global $user;
+    global $user;
 
-  $form['author'] = array(
+    $form['author'] = array(
     '#type' => 'value',
     '#value' => $author->uid,
-  );
+    );
     $form['recipient'] = array(
     '#type' => 'value',
     '#value' => $user->uid,
-  );
-  $form['author_name'] = array(
+    );
+    $form['author_name'] = array(
     '#type' => 'value',
     '#value' => $author->name,
-  );
-  $form['destination'] = array(
+    );
+    $form['destination'] = array(
     '#type' => 'value',
     '#value' => isset($_GET['destination']) ? $_GET['destination'] : 'messages/',
-  );
+    );
 
-  if (db_result(db_query('SELECT COUNT(recipient) FROM {pm_block_user} WHERE author = %d AND recipient = %d', $author->uid, $user->uid))) {
+    if (db_result(db_query('SELECT COUNT(recipient) FROM {pm_block_user} WHERE author = %d AND recipient = %d', $author->uid, $user->uid))) {
     $form['block_action'] = array(
-      '#type' => 'value',
-      '#value' => 'unblock_user',
+        '#type' => 'value',
+        '#value' => 'unblock_user',
     );
     return confirm_form($form,
-      t('You have previously blocked "@author" from sending you any more messages. Are you sure you want to unblock this user?', array('@author' => $author->name)),
-      isset($_GET['destination']) ? $_GET['destination'] : 'messages/',
-      t('This action cannot be undone.'),
-      t('Unblock @author', array('@author' => $author->name)),
-      t('Cancel')
+        t('You have previously blocked "@author" from sending you any more messages. Are you sure you want to unblock this user?', array('@author' => $author->name)),
+        isset($_GET['destination']) ? $_GET['destination'] : 'messages/',
+        t('This action cannot be undone.'),
+        t('Unblock @author', array('@author' => $author->name)),
+        t('Cancel')
     );
-  }
-  else {
+    }
+    else {
     $form['block_action'] = array(
-      '#type' => 'value',
-      '#value' => 'block_user',
+        '#type' => 'value',
+        '#value' => 'block_user',
     );
     return confirm_form($form,
-      t('Are you sure you want to block "@author" from sending you any more messages?', array('@author' => $author->name)),
-      isset($_GET['destination']) ? $_GET['destination'] : 'messages/',
-      '',
-      t('Block @author', array('@author' => $author->name)),
-      t('Cancel')
+        t('Are you sure you want to block "@author" from sending you any more messages?', array('@author' => $author->name)),
+        isset($_GET['destination']) ? $_GET['destination'] : 'messages/',
+        '',
+        t('Block @author', array('@author' => $author->name)),
+        t('Cancel')
     );
-  }
+    }
 }
 
 /**
  * Implements hook_form_submit().
  */
 function pm_block_user_form_submit($form, &$form_state) {
-  if ($form_state['values']['confirm']) {
+    if ($form_state['values']['confirm']) {
     switch ($form_state['values']['block_action']) {
-      case 'block_user':
+        case 'block_user':
         db_query('INSERT INTO {pm_block_user} (author, recipient) VALUES (%d, %d)', $form_state['values']['author'], $form_state['values']['recipient']);
         drupal_set_message(t('@author has been blocked from sending you any further messages.', array('@author' => $form_state['values']['author_name'])));
-      break;
-      case 'unblock_user':
+        break;
+        case 'unblock_user':
         db_query('DELETE FROM {pm_block_user} WHERE author = %d AND recipient = %d', $form_state['values']['author'], $form_state['values']['recipient']);
         drupal_set_message(t('@author is now allowed to send you new messages.', array('@author' => $form_state['values']['author_name'])));
-      break;
+        break;
+    }
     }
-  }
-  $form_state['redirect'] = $form_state['values']['destination'];
+    $form_state['redirect'] = $form_state['values']['destination'];
 }
 
 /**
  * Implements hook_privatemsg_block_message().
  */
 function pm_block_user_privatemsg_block_message($author, $recipients) {
-  $blocked = array();
-  // Loop through each recipient and ensure there is no rule blocking this
-  // author from sending them private messages. Use a reference, so when
-  // user_load() is needed here the array is updated, negating the need for
-  // further calls to user_load() later in the code.
-  foreach (array_keys($recipients) as $uid) {
+    $blocked = array();
+    // Loop through each recipient and ensure there is no rule blocking this
+    // author from sending them private messages. Use a reference, so when
+    // user_load() is needed here the array is updated, negating the need for
+    // further calls to user_load() later in the code.
+    foreach (array_keys($recipients) as $uid) {
 
     // Ensure we have a recipient user object which includes roles.
     if (!isset($recipients[$uid]->roles)) {
-      $recipients[$uid] = user_load($uid);
+        $recipients[$uid] = user_load($uid);
     }
     // Note: this is checks whether the author may send the message (see third
     // parameter). Further below is a check whether the recipient may block it.
     if (_pm_block_user_rule_exists($author, $recipients[$uid], PM_BLOCK_USER_DISALLOW_SENDING)) {
-      $blocked[] = array(
+        $blocked[] = array(
         'uid' => $uid,
         'message' => t('Sorry, private messaging rules forbid sending messages to !name.', array('!name' => $recipients[$uid]->name)),
-      );
+        );
+    }
     }
-  }
 
-  $args = array_merge(array($author->uid), array_keys($recipients));
-  $result = db_query('SELECT recipient FROM {pm_block_user} WHERE author = %d AND recipient IN ('. db_placeholders($recipients) .') GROUP BY recipient', $args);
-  while ($row = db_fetch_array($result)) {
+    $args = array_merge(array($author->uid), array_keys($recipients));
+    $result = db_query('SELECT recipient FROM {pm_block_user} WHERE author = %d AND recipient IN ('. db_placeholders($recipients) .') GROUP BY recipient', $args);
+    while ($row = db_fetch_array($result)) {
     $recipient = $recipients[$row['recipient']];
     // If there's a rule disallowing blocking of this message, send it anyway.
     if (_pm_block_user_rule_exists($author, $recipient, PM_BLOCK_USER_DISALLOW_BLOCKING)) {
-      continue;
+        continue;
     }
     $blocked[] = array(
-      'uid' => $row['recipient'],
-      'message' => t('%name has chosen to not recieve any more messages from you.', array('%name' => $recipients[$row['recipient']]->name))
+        'uid' => $row['recipient'],
+        'message' => t('%name has chosen to not recieve any more messages from you.', array('%name' => $recipients[$row['recipient']]->name))
     );
-  }
-  return $blocked;
+    }
+    return $blocked;
 }
 
 function pm_block_user_privatemsg_sql_load_alter(&$fragments, $pmid, $uid) {
-  $fragments['select'][] = 'pmbu.recipient AS is_blocked';
+    $fragments['select'][] = 'pmbu.recipient AS is_blocked';
 
-  $fragments['inner_join'][] = 'LEFT JOIN {pm_block_user} pmbu ON (pm.author = pmbu.author AND pmi.uid = pmbu.recipient)';
+    $fragments['inner_join'][] = 'LEFT JOIN {pm_block_user} pmbu ON (pm.author = pmbu.author AND pmi.uid = pmbu.recipient)';
 }
 
 /**
  * Implements hook_privatemsg_message_view_alter().
  */
 function pm_block_user_privatemsg_message_view_alter(&$vars) {
-  global $user;
+    global $user;
 
-  $author = $vars['message']['author'];
-  if (_pm_block_user_rule_exists($author, $user, PM_BLOCK_USER_DISALLOW_BLOCKING)) {
+    $author = $vars['message']['author'];
+    if (_pm_block_user_rule_exists($author, $user, PM_BLOCK_USER_DISALLOW_BLOCKING)) {
     return;
-  }
-  if (!isset($vars['message']['thread_id'])) {
+    }
+    if (!isset($vars['message']['thread_id'])) {
     // No thread id, this is probably only a preview
     return;
-  }
-  $thread_id = $vars['message']['thread_id'];
+    }
+    $thread_id = $vars['message']['thread_id'];
 
-  if ($user->uid <> $author->uid) {
+    if ($user->uid <> $author->uid) {
     if ($vars['message']['is_blocked']) {
-      $vars['message_actions']['unblock_author'] = array('title' => t('Unblock author'), 'href' => 'messages/block/'. $author->uid, 'query' => 'destination=messages/view/' . $thread_id);
+        $vars['message_actions']['unblock_author'] = array('title' => t('Unblock author'), 'href' => 'messages/block/'. $author->uid, 'query' => 'destination=messages/view/' . $thread_id);
     }
     else {
-      $vars['message_actions']['block_author'] = array('title' => t('Block author'), 'href' => 'messages/block/'. $author->uid, 'query' => 'destination=messages/view/' . $thread_id);
+        $vars['message_actions']['block_author'] = array('title' => t('Block author'), 'href' => 'messages/block/'. $author->uid, 'query' => 'destination=messages/view/' . $thread_id);
+    }
     }
-  }
 }
 
 /**
  * Implement hook_user().
  */
 function pm_block_user_user($op, &$edit, &$account, $category = NULL) {
-  switch ($op) {
-      case 'delete':
+    switch ($op) {
+        case 'delete':
       // Delete blocking rules which involve this user.
       db_query("DELETE FROM {pm_block_user} WHERE author = %d OR recipient = %d", $account->uid, $account->uid);
-      break;
-  }
+        break;
+    }
 }

Switch Indentation +14 added lines, -14 removed lines

@@ -20,8 +20,8 @@ 
  */
 function pm_block_user_help($path) {
   switch ($path) {
-    case 'admin/settings/messages/block':
-      return '<p>' . t('This area is used to define user blocking rules for the Privatemsg module. Rules allow control of who may block messages from whom. By default all users are allowed to block messages from anyone else. However, a site may have groups of users that need to contact or get information to others, for example: the site may have administrative staff or be a forum with moderators. Groups of users are defined by roles, which can be managed on the <a href="@roles">roles configuration page</a>.', array('@roles' => url('admin/user/roles'))) . '</p>';
+  case 'admin/settings/messages/block':
+    return '<p>' . t('This area is used to define user blocking rules for the Privatemsg module. Rules allow control of who may block messages from whom. By default all users are allowed to block messages from anyone else. However, a site may have groups of users that need to contact or get information to others, for example: the site may have administrative staff or be a forum with moderators. Groups of users are defined by roles, which can be managed on the <a href="@roles">roles configuration page</a>.', array('@roles' => url('admin/user/roles'))) . '</p>';
   }
 }
 
@@ -485,14 +485,14 @@ 
 function pm_block_user_form_submit($form, &$form_state) {
   if ($form_state['values']['confirm']) {
     switch ($form_state['values']['block_action']) {
-      case 'block_user':
-        db_query('INSERT INTO {pm_block_user} (author, recipient) VALUES (%d, %d)', $form_state['values']['author'], $form_state['values']['recipient']);
-        drupal_set_message(t('@author has been blocked from sending you any further messages.', array('@author' => $form_state['values']['author_name'])));
-      break;
-      case 'unblock_user':
-        db_query('DELETE FROM {pm_block_user} WHERE author = %d AND recipient = %d', $form_state['values']['author'], $form_state['values']['recipient']);
-        drupal_set_message(t('@author is now allowed to send you new messages.', array('@author' => $form_state['values']['author_name'])));
-      break;
+    case 'block_user':
+      db_query('INSERT INTO {pm_block_user} (author, recipient) VALUES (%d, %d)', $form_state['values']['author'], $form_state['values']['recipient']);
+      drupal_set_message(t('@author has been blocked from sending you any further messages.', array('@author' => $form_state['values']['author_name'])));
+    break;
+    case 'unblock_user':
+      db_query('DELETE FROM {pm_block_user} WHERE author = %d AND recipient = %d', $form_state['values']['author'], $form_state['values']['recipient']);
+      drupal_set_message(t('@author is now allowed to send you new messages.', array('@author' => $form_state['values']['author_name'])));
+    break;
     }
   }
   $form_state['redirect'] = $form_state['values']['destination'];
@@ -576,9 +576,9 @@ 
  */
 function pm_block_user_user($op, &$edit, &$account, $category = NULL) {
   switch ($op) {
-      case 'delete':
-      // Delete blocking rules which involve this user.
-      db_query("DELETE FROM {pm_block_user} WHERE author = %d OR recipient = %d", $account->uid, $account->uid);
-      break;
+  case 'delete':
+  // Delete blocking rules which involve this user.
+  db_query("DELETE FROM {pm_block_user} WHERE author = %d OR recipient = %d", $account->uid, $account->uid);
+  break;
   }
 }

Spacing +9 added lines, -9 removed lines

@@ -21,7 +21,7 @@ 
 function pm_block_user_help($path) {
   switch ($path) {
     case 'admin/settings/messages/block':
-      return '<p>' . t('This area is used to define user blocking rules for the Privatemsg module. Rules allow control of who may block messages from whom. By default all users are allowed to block messages from anyone else. However, a site may have groups of users that need to contact or get information to others, for example: the site may have administrative staff or be a forum with moderators. Groups of users are defined by roles, which can be managed on the <a href="@roles">roles configuration page</a>.', array('@roles' => url('admin/user/roles'))) . '</p>';
+      return '<p>'.t('This area is used to define user blocking rules for the Privatemsg module. Rules allow control of who may block messages from whom. By default all users are allowed to block messages from anyone else. However, a site may have groups of users that need to contact or get information to others, for example: the site may have administrative staff or be a forum with moderators. Groups of users are defined by roles, which can be managed on the <a href="@roles">roles configuration page</a>.', array('@roles' => url('admin/user/roles'))).'</p>';
   }
 }
 
@@ -62,7 +62,7 @@ 
  * Menu callback for blocked user settings.
  */
 function pm_block_user_settings(&$form_state) {
-  drupal_add_css(drupal_get_path('module', 'pm_block_user') .'/pm_block_user.css');
+  drupal_add_css(drupal_get_path('module', 'pm_block_user').'/pm_block_user.css');
   // Need to cache form for AHAH, so it can be rebuilt from cache later.
   $form = array(
     '#cache' => TRUE,
@@ -96,7 +96,7 @@ 
     '#value' => t('More'),
     '#weight' => 1,
     '#prefix' => '<div id="add-rule-button">',
-    '#suffix' => '<label for="edit-pm-block-actions-more">' . t('Add new rule') . '</label></div>',
+    '#suffix' => '<label for="edit-pm-block-actions-more">'.t('Add new rule').'</label></div>',
     '#submit' => array('pm_block_user_more_submit'),
     '#ahah' => array(
       'path' => 'messages/block/js',
@@ -164,7 +164,7 @@ 
   else {
     // @todo: add whitelist options/default_value here.
   }
-  $form['action']  = array(
+  $form['action'] = array(
     '#type' => 'radios',
     '#options' => $options,
     '#disabled' => $row_disabled,
@@ -181,7 +181,7 @@ 
     '#value' => t('Remove'),
     '#attributes' => array('class' => 'remove-action'),
     '#prefix' => '<div id="remove-rule-button">',
-    '#suffix' => '<label for="edit-remove">' . t('Remove rule') . '</label></div>',
+    '#suffix' => '<label for="edit-remove">'.t('Remove rule').'</label></div>',
     '#ahah' => array(
       'path' => 'messages/block/js',
       'wrapper' => 'block-actions',
@@ -296,7 +296,7 @@ 
   $form = drupal_rebuild_form($form_id, $form_state, $args, $form_build_id);
   $output_form = $form['block_actions'];
   unset($output_form['#prefix'], $output_form['#suffix']); // Prevent duplicate wrappers.
-  $output = theme('status_messages') . drupal_render($output_form);
+  $output = theme('status_messages').drupal_render($output_form);
 
   // Hack to make behaviours attach to new HTML controls (delete buttons in
   // this case).
@@ -524,7 +524,7 @@ 
   }
 
   $args = array_merge(array($author->uid), array_keys($recipients));
-  $result = db_query('SELECT recipient FROM {pm_block_user} WHERE author = %d AND recipient IN ('. db_placeholders($recipients) .') GROUP BY recipient', $args);
+  $result = db_query('SELECT recipient FROM {pm_block_user} WHERE author = %d AND recipient IN ('.db_placeholders($recipients).') GROUP BY recipient', $args);
   while ($row = db_fetch_array($result)) {
     $recipient = $recipients[$row['recipient']];
     // If there's a rule disallowing blocking of this message, send it anyway.
@@ -563,10 +563,10 @@ 
 
   if ($user->uid <> $author->uid) {
     if ($vars['message']['is_blocked']) {
-      $vars['message_actions']['unblock_author'] = array('title' => t('Unblock author'), 'href' => 'messages/block/'. $author->uid, 'query' => 'destination=messages/view/' . $thread_id);
+      $vars['message_actions']['unblock_author'] = array('title' => t('Unblock author'), 'href' => 'messages/block/'.$author->uid, 'query' => 'destination=messages/view/'.$thread_id);
     }
     else {
-      $vars['message_actions']['block_author'] = array('title' => t('Block author'), 'href' => 'messages/block/'. $author->uid, 'query' => 'destination=messages/view/' . $thread_id);
+      $vars['message_actions']['block_author'] = array('title' => t('Block author'), 'href' => 'messages/block/'.$author->uid, 'query' => 'destination=messages/view/'.$thread_id);
     }
   }
 }

Braces +6 added lines, -12 removed lines

@@ -79,8 +79,7 @@ 
   // Should we populate the form with data from $form_state or the database?
   if (!isset($form_state['pm_block_user']['block_actions'])) {
     $block_actions = variable_get('pm_block_user_actions', array());
-  }
-  else {
+  } else {
     $block_actions = $form_state['pm_block_user']['block_actions'];
   }
   // Work through each rule, adding it as a new element in
@@ -160,8 +159,7 @@ 
       PM_BLOCK_USER_DISALLOW_SENDING => t('Disallow sending message'),
     );
     $default_value = (isset($details['action']) ? $details['action'] : PM_BLOCK_USER_DISALLOW_BLOCKING);
-  }
-  else {
+  } else {
     // @todo: add whitelist options/default_value here.
   }
   $form['action']  = array(
@@ -239,8 +237,7 @@ 
     // builder.
     if (isset($form_state['values']['block_actions'])) {
       variable_set('pm_block_user_actions', _pm_block_user_settings_filter($form_state['values']['block_actions']));
-    }
-    else {
+    } else {
       variable_set('pm_block_user_actions', array());
     }
     drupal_set_message(t('The configuration options have been saved.'));
@@ -272,8 +269,7 @@ 
   }
   if (count($matching) > 0) {
     return $matching;
-  }
-  else {
+  } else {
     return array_map('_pm_block_user_settings_filter', $settings);
   }
 }
@@ -463,8 +459,7 @@ 
       t('Unblock @author', array('@author' => $author->name)),
       t('Cancel')
     );
-  }
-  else {
+  } else {
     $form['block_action'] = array(
       '#type' => 'value',
       '#value' => 'block_user',
@@ -564,8 +559,7 @@ 
   if ($user->uid <> $author->uid) {
     if ($vars['message']['is_blocked']) {
       $vars['message_actions']['unblock_author'] = array('title' => t('Unblock author'), 'href' => 'messages/block/'. $author->uid, 'query' => 'destination=messages/view/' . $thread_id);
-    }
-    else {
+    } else {
       $vars['message_actions']['block_author'] = array('title' => t('Block author'), 'href' => 'messages/block/'. $author->uid, 'query' => 'destination=messages/view/' . $thread_id);
     }
   }

drupal/sites/default/boinc/modules/contrib/privatemsg/pm_email_notify/pm_email_notify.module

Doc Comments +2 added lines

@@ -148,6 +148,8 @@
  * Implements hook_user().
  *
  * Display settings form and store its information.
+ * @param string $op
+ * @param string $category
  */
 function pm_email_notify_user($op, &$edit, &$account, $category = NULL) {
   switch ($op) {

Indentation +59 added lines, -59 removed lines

@@ -9,7 +9,7 @@ 
  * Implements hook_menu().
  */
 function pm_email_notify_menu() {
-  $items['admin/settings/messages/notify'] = array(
+    $items['admin/settings/messages/notify'] = array(
     'title'            => 'E-mail notify',
     'description'      => 'E-mail notification settings',
     'page callback'    => 'drupal_get_form',
@@ -17,47 +17,47 @@ 
     'access arguments' => array('administer privatemsg settings'),
     'type'             => MENU_LOCAL_TASK,
     'weight'           => 10,
-  );
-  return $items;
+    );
+    return $items;
 }
 
 /**
  * Menu callback for administration settings.
  */
 function pm_email_notify_admin_settings_form() {
-  $form['pm_email'] = array(
+    $form['pm_email'] = array(
     '#type' => 'fieldset',
     '#title' => t('Privatemsg e-mail notification'),
     '#collapsible' => FALSE,
     '#collapsed' => FALSE,
-  );
-  $form['pm_email']['pm_email_notify_default'] = array(
+    );
+    $form['pm_email']['pm_email_notify_default'] = array(
     '#type' => 'checkbox',
     '#title' => t('Notify users of new private messages by default'),
     '#default_value' => variable_get('pm_email_notify_default', TRUE),
     '#weight' => 0,
-  );
+    );
 
-  $form['pm_email']['pm_email_notify_desc'] = array(
+    $form['pm_email']['pm_email_notify_desc'] = array(
     '#type' => 'item',
     '#value' => t('Customize the email messages sent to users upon receipt of a new private message. <br />Available variables are: !author, !author_uid, !pm_subject, !pm_body, !thread, !site, !login_url, !uri, !uri_brief, !message (URL) and !settings (URL).'),
     '#weight' => 1,
-  );
+    );
 
-  $form['pm_email']['pm_email_notify_subject'] = array(
+    $form['pm_email']['pm_email_notify_subject'] = array(
     '#type' => 'textfield',
     '#title' => t('Subject of notification messages'),
     '#default_value' => variable_get('pm_email_notify_subject', 'New private message at !site.'),
     '#weight' => 2,
-  );
+    );
 
-  $form['pm_email']['pm_email_notify_body'] = array(
+    $form['pm_email']['pm_email_notify_body'] = array(
     '#type' => 'textarea',
     '#title' => t('Body of notification messages'),
     '#default_value' => variable_get('pm_email_notify_body', _pm_email_notify_default_body()),
     '#weight' => 3,
-  );
-  return system_settings_form($form);
+    );
+    return system_settings_form($form);
 }
 
 /**
@@ -70,47 +70,47 @@ 
  *   User uid
  */
 function _pm_email_notify_is_enabled($uid) {
-  static $notifications = array();
-  // Cache the result set in case this method is executed in batched operation which will perform many unnecessary repeated selects for the same user
-  if ( !isset($notifications[$uid]) ) {
+    static $notifications = array();
+    // Cache the result set in case this method is executed in batched operation which will perform many unnecessary repeated selects for the same user
+    if ( !isset($notifications[$uid]) ) {
     $mail_notification = db_result(db_query('SELECT email_notify_is_enabled FROM {pm_email_notify} WHERE user_id = %d', $uid));
     if ($mail_notification === FALSE) {
-      // db_result() returns FALSE if result was not found.
-      $mail_notification = variable_get('pm_email_notify_default', TRUE);
+        // db_result() returns FALSE if result was not found.
+        $mail_notification = variable_get('pm_email_notify_default', TRUE);
     }
     $notifications[$uid] = $mail_notification;
-  }
-  return $notifications[$uid];
+    }
+    return $notifications[$uid];
 }
 
 /**
  * Implements hook_privatemsg_message_insert().
  */
 function pm_email_notify_privatemsg_message_insert($message) {
-  foreach ($message['recipients'] as $recipient) {
+    foreach ($message['recipients'] as $recipient) {
     // check if recipient enabled email notifications
     if (_pm_email_notify_is_enabled($recipient->uid)) {
-      // send them a new pm notification email if they did
-      $params['recipient'] = $recipient;
-      $params['message'] = $message;
-      drupal_mail('pm_email_notify', 'notice', $recipient->mail, user_preferred_language($recipient), $params);
+        // send them a new pm notification email if they did
+        $params['recipient'] = $recipient;
+        $params['message'] = $message;
+        drupal_mail('pm_email_notify', 'notice', $recipient->mail, user_preferred_language($recipient), $params);
+    }
     }
-  }
 }
 
 /**
  * Implements hook_mail().
  */
 function pm_email_notify_mail($key, &$message, $params) {
-  $language = $message['language'];
-  $variables = user_mail_tokens($params['recipient'], $language);
-  $variables = array_merge($variables, _pm_email_notify_token($params['recipient'], $params['message'], $language));
-  switch ($key) {
+    $language = $message['language'];
+    $variables = user_mail_tokens($params['recipient'], $language);
+    $variables = array_merge($variables, _pm_email_notify_token($params['recipient'], $params['message'], $language));
+    switch ($key) {
     case 'notice':
       $message['subject'] = t(variable_get('pm_email_notify_subject', 'New private message at !site.'), $variables, $language->language);
-      $message['body'] = t(variable_get('pm_email_notify_body', _pm_email_notify_default_body()), $variables, $language->language);
-      break;
-  }
+        $message['body'] = t(variable_get('pm_email_notify_body', _pm_email_notify_default_body()), $variables, $language->language);
+        break;
+    }
 }
 
 /**
@@ -123,7 +123,7 @@ 
  *   Array of mappings from token names to values (for use with strtr()).
  */
 function _pm_email_notify_token($recipient, $message, $language) {
-  $tokens = array(
+    $tokens = array(
     '!author_uid' => $message['author']->uid,
     '!author' => $message['author']->name,
     '!pm_subject' => trim(drupal_html_to_text(check_plain($message['subject']))),
@@ -132,16 +132,16 @@ 
     '!user_uid' => $recipient->uid,
     '!message' => url('messages/view/' . $message['thread_id'], array('absolute' => TRUE, 'language' => $language)),
     '!settings' => url('user/' . $recipient->uid . '/edit', array('absolute' => TRUE, 'language' => $language)),
-  );
+    );
 
-  return $tokens;
+    return $tokens;
 }
 
 /**
  * Returns default email notification body.
  */
 function _pm_email_notify_default_body() {
-  return "Hi !username,\n\nThis is an automatic reminder from the site !site. You have received a new private message from !author.\n\nTo read your message, follow this link:\n!message\n\nIf you don't want to receive these emails again, change your preferences here:\n!settings";
+    return "Hi !username,\n\nThis is an automatic reminder from the site !site. You have received a new private message from !author.\n\nTo read your message, follow this link:\n!message\n\nIf you don't want to receive these emails again, change your preferences here:\n!settings";
 }
 
 /**
@@ -150,43 +150,43 @@ 
  * Display settings form and store its information.
  */
 function pm_email_notify_user($op, &$edit, &$account, $category = NULL) {
-  switch ($op) {
+    switch ($op) {
     case 'form':
       if ($category == 'account' && privatemsg_user_access('read privatemsg', $account)) {
         $form['enable_pm_mail'] = array(
-          '#type' => 'fieldset',
-          '#title' => t('Privatemsg e-mail notification'),
-          '#collapsible' => TRUE,
-          '#collapsed' => FALSE,
-          '#weight' => 10,
+            '#type' => 'fieldset',
+            '#title' => t('Privatemsg e-mail notification'),
+            '#collapsible' => TRUE,
+            '#collapsed' => FALSE,
+            '#weight' => 10,
         );
         $form['enable_pm_mail']['pm_send_notifications'] = array(
-          '#type' => 'checkbox',
-          '#title' => t('Receive email notification for incoming private messages'),
-          '#default_value' => _pm_email_notify_is_enabled($account->uid),
+            '#type' => 'checkbox',
+            '#title' => t('Receive email notification for incoming private messages'),
+            '#default_value' => _pm_email_notify_is_enabled($account->uid),
         );
-      }
-      return $form;
+        }
+        return $form;
 
-     case 'submit':
+        case 'submit':
       if (isset($edit['pm_send_notifications']) && privatemsg_user_access('read privatemsg', $account)) {
         $pm_email_enabled = $edit['pm_send_notifications'];
         unset($edit['pm_send_notifications']);
         // Update database entry with user preference.
         $exists = db_result(db_query("SELECT 1 FROM {pm_email_notify} WHERE user_id = %d", $account->uid));
         if ($exists) {
-          // If there is an existing entry, update.
-          db_query("UPDATE {pm_email_notify} SET email_notify_is_enabled = %d WHERE user_id = %d", $pm_email_enabled, $account->uid);
+            // If there is an existing entry, update.
+            db_query("UPDATE {pm_email_notify} SET email_notify_is_enabled = %d WHERE user_id = %d", $pm_email_enabled, $account->uid);
         }
         else {
-          // If not, create a new one.
-          db_query("INSERT INTO {pm_email_notify} (email_notify_is_enabled, user_id) VALUES (%d, %d)", $pm_email_enabled, $account->uid);
+            // If not, create a new one.
+            db_query("INSERT INTO {pm_email_notify} (email_notify_is_enabled, user_id) VALUES (%d, %d)", $pm_email_enabled, $account->uid);
+        }
         }
-      }
-      break;
+        break;
 
-     case 'delete':
+        case 'delete':
       db_query("DELETE FROM {pm_email_notify} WHERE user_id = %d", $account->uid);
-      break;
-  }
+        break;
+    }
 }

Switch Indentation +39 added lines, -39 removed lines

@@ -106,10 +106,10 @@ 
   $variables = user_mail_tokens($params['recipient'], $language);
   $variables = array_merge($variables, _pm_email_notify_token($params['recipient'], $params['message'], $language));
   switch ($key) {
-    case 'notice':
-      $message['subject'] = t(variable_get('pm_email_notify_subject', 'New private message at !site.'), $variables, $language->language);
-      $message['body'] = t(variable_get('pm_email_notify_body', _pm_email_notify_default_body()), $variables, $language->language);
-      break;
+  case 'notice':
+    $message['subject'] = t(variable_get('pm_email_notify_subject', 'New private message at !site.'), $variables, $language->language);
+    $message['body'] = t(variable_get('pm_email_notify_body', _pm_email_notify_default_body()), $variables, $language->language);
+    break;
   }
 }
 
@@ -151,42 +151,42 @@ 
  */
 function pm_email_notify_user($op, &$edit, &$account, $category = NULL) {
   switch ($op) {
-    case 'form':
-      if ($category == 'account' && privatemsg_user_access('read privatemsg', $account)) {
-        $form['enable_pm_mail'] = array(
-          '#type' => 'fieldset',
-          '#title' => t('Privatemsg e-mail notification'),
-          '#collapsible' => TRUE,
-          '#collapsed' => FALSE,
-          '#weight' => 10,
-        );
-        $form['enable_pm_mail']['pm_send_notifications'] = array(
-          '#type' => 'checkbox',
-          '#title' => t('Receive email notification for incoming private messages'),
-          '#default_value' => _pm_email_notify_is_enabled($account->uid),
-        );
-      }
-      return $form;
+  case 'form':
+    if ($category == 'account' && privatemsg_user_access('read privatemsg', $account)) {
+      $form['enable_pm_mail'] = array(
+        '#type' => 'fieldset',
+        '#title' => t('Privatemsg e-mail notification'),
+        '#collapsible' => TRUE,
+        '#collapsed' => FALSE,
+        '#weight' => 10,
+      );
+      $form['enable_pm_mail']['pm_send_notifications'] = array(
+        '#type' => 'checkbox',
+        '#title' => t('Receive email notification for incoming private messages'),
+        '#default_value' => _pm_email_notify_is_enabled($account->uid),
+      );
+    }
+    return $form;
 
-     case 'submit':
-      if (isset($edit['pm_send_notifications']) && privatemsg_user_access('read privatemsg', $account)) {
-        $pm_email_enabled = $edit['pm_send_notifications'];
-        unset($edit['pm_send_notifications']);
-        // Update database entry with user preference.
-        $exists = db_result(db_query("SELECT 1 FROM {pm_email_notify} WHERE user_id = %d", $account->uid));
-        if ($exists) {
-          // If there is an existing entry, update.
-          db_query("UPDATE {pm_email_notify} SET email_notify_is_enabled = %d WHERE user_id = %d", $pm_email_enabled, $account->uid);
-        }
-        else {
-          // If not, create a new one.
-          db_query("INSERT INTO {pm_email_notify} (email_notify_is_enabled, user_id) VALUES (%d, %d)", $pm_email_enabled, $account->uid);
-        }
-      }
-      break;
+  case 'submit':
+   if (isset($edit['pm_send_notifications']) && privatemsg_user_access('read privatemsg', $account)) {
+     $pm_email_enabled = $edit['pm_send_notifications'];
+     unset($edit['pm_send_notifications']);
+     // Update database entry with user preference.
+     $exists = db_result(db_query("SELECT 1 FROM {pm_email_notify} WHERE user_id = %d", $account->uid));
+     if ($exists) {
+       // If there is an existing entry, update.
+       db_query("UPDATE {pm_email_notify} SET email_notify_is_enabled = %d WHERE user_id = %d", $pm_email_enabled, $account->uid);
+     }
+     else {
+       // If not, create a new one.
+       db_query("INSERT INTO {pm_email_notify} (email_notify_is_enabled, user_id) VALUES (%d, %d)", $pm_email_enabled, $account->uid);
+     }
+   }
+   break;
 
-     case 'delete':
-      db_query("DELETE FROM {pm_email_notify} WHERE user_id = %d", $account->uid);
-      break;
+  case 'delete':
+   db_query("DELETE FROM {pm_email_notify} WHERE user_id = %d", $account->uid);
+   break;
   }
 }

Spacing +3 added lines, -3 removed lines

@@ -72,7 +72,7 @@ 
 function _pm_email_notify_is_enabled($uid) {
   static $notifications = array();
   // Cache the result set in case this method is executed in batched operation which will perform many unnecessary repeated selects for the same user
-  if ( !isset($notifications[$uid]) ) {
+  if (!isset($notifications[$uid])) {
     $mail_notification = db_result(db_query('SELECT email_notify_is_enabled FROM {pm_email_notify} WHERE user_id = %d', $uid));
     if ($mail_notification === FALSE) {
       // db_result() returns FALSE if result was not found.
@@ -130,8 +130,8 @@ 
     '!pm_body' => trim(drupal_html_to_text(check_markup($message['body'], $message['format'], FALSE))),
     '!thread' => $message['thread_id'],
     '!user_uid' => $recipient->uid,
-    '!message' => url('messages/view/' . $message['thread_id'], array('absolute' => TRUE, 'language' => $language)),
-    '!settings' => url('user/' . $recipient->uid . '/edit', array('absolute' => TRUE, 'language' => $language)),
+    '!message' => url('messages/view/'.$message['thread_id'], array('absolute' => TRUE, 'language' => $language)),
+    '!settings' => url('user/'.$recipient->uid.'/edit', array('absolute' => TRUE, 'language' => $language)),
   );
 
   return $tokens;

Braces +1 added lines, -2 removed lines

@@ -177,8 +177,7 @@
         if ($exists) {
           // If there is an existing entry, update.
           db_query("UPDATE {pm_email_notify} SET email_notify_is_enabled = %d WHERE user_id = %d", $pm_email_enabled, $account->uid);
-        }
-        else {
+        } else {
           // If not, create a new one.
           db_query("INSERT INTO {pm_email_notify} (email_notify_is_enabled, user_id) VALUES (%d, %d)", $pm_email_enabled, $account->uid);
         }

drupal/sites/default/boinc/modules/contrib/privatemsg/privatemsg.module

Doc Comments +7 added lines, -5 removed lines

@@ -37,6 +37,7 @@ 
  *
  * @param $userstring
  *   A string with user id, for example 1,2,4. Returned by the list query
+ * @param integer $slice
  *
  * @return
  *   Array with user objects.
@@ -72,7 +73,7 @@ 
  *   Array with user objects, for example the one returnd by
  *   _privatemsg_generate_user_array.
  *
- * @param $limit
+ * @param integer $limit
  *   Limit the number of user objects which should be displayed.
  * @param $no_text
  *   When TRUE, don't display the Participants/From text.
@@ -215,7 +216,7 @@ 
  *   Permission string, defaults to read privatemsg
  *
  * @return
- *   TRUE if user has access, FALSE if not
+ boolean   TRUE if user has access, FALSE if not
  *
  * @ingroup api
  */
@@ -730,7 +731,7 @@ 
  *
  * @param $pmid
  *   Message id
- * @param $status
+ * @param integer $status
  *   Either PRIVATEMSG_READ or PRIVATEMSG_UNREAD
  * @param $account
  *   User object, defaults to the current user
@@ -1949,9 +1950,10 @@ 
  *   Recipient of the message
  * @param $account
  *   Sender of the message, defaults to the current user
+ * @param string $subject
  *
  * @return
- *   Either FALSE or a URL string
+ false|string   Either FALSE or a URL string
  *
  * @ingroup api
  */
@@ -2273,7 +2275,7 @@ 
  *
  * Uses @link theming theme patterns @endlink to theme single headers.
  *
- * @param $has_posts
+ * @param boolean $has_posts
  *   TRUE when there is at least one row. Decides if the select all checkbox
  *   should be displayed.
  * @param $keys

Indentation +1076 added lines, -1077 removed lines

@@ -22,13 +22,13 @@ 
  * Implements hook_perm().
  */
 function privatemsg_perm() {
-  return array(
+    return array(
     'read privatemsg',
     'read all private messages',
     'administer privatemsg settings',
     'write privatemsg',
     'delete privatemsg',
-  );
+    );
 }
 
 /**
@@ -42,27 +42,27 @@ 
  *   Array with user objects.
  */
 function _privatemsg_generate_user_array($userstring, $slice = NULL) {
-  static $user_cache = array();
-
-  // Convert user uid list (uid1,uid2,uid3) into an array. If $slice is not NULL
-  // pass that as argument to array_slice(). For example, -4 will only load the
-  // last four users.
-  // This is done to avoid loading user objects that are not displayed, for
-  // obvious performance reasons.
-  $users = explode(',', $userstring);
-  if (!is_null($slice)) {
+    static $user_cache = array();
+
+    // Convert user uid list (uid1,uid2,uid3) into an array. If $slice is not NULL
+    // pass that as argument to array_slice(). For example, -4 will only load the
+    // last four users.
+    // This is done to avoid loading user objects that are not displayed, for
+    // obvious performance reasons.
+    $users = explode(',', $userstring);
+    if (!is_null($slice)) {
     $users = array_slice($users, $slice);
-  }
-  $participants = array();
-  foreach ($users as $uid) {
+    }
+    $participants = array();
+    foreach ($users as $uid) {
     if (!array_key_exists($uid, $user_cache)) {
-      $user_cache[$uid] = user_load($uid);
+        $user_cache[$uid] = user_load($uid);
     }
     if (is_object($user_cache[$uid])) {
-      $participants[$uid] = $user_cache[$uid];
+        $participants[$uid] = $user_cache[$uid];
+    }
     }
-  }
-  return $participants;
+    return $participants;
 }
 
 /**
@@ -80,60 +80,60 @@ 
  *   String with formated user objects, like user1, user2.
  */
 function _privatemsg_format_participants($part_array, $limit = NULL, $no_text = FALSE) {
-  if (count($part_array) > 0) {
+    if (count($part_array) > 0) {
     $to = array();
     $limited = FALSE;
     foreach ($part_array as $account) {
-      if (is_int($limit) && count($to) >= $limit) {
+        if (is_int($limit) && count($to) >= $limit) {
         $limited = TRUE;
         break;
-      }
-      $to[] = theme('username', $account);
+        }
+        $to[] = theme('username', $account);
     }
 
     $limit_string = '';
     if ($limited) {
-      $limit_string = t(' and others');
+        $limit_string = t(' and others');
     }
 
 
     if ($no_text) {
-      return implode(', ', $to) . $limit_string;
+        return implode(', ', $to) . $limit_string;
     }
 
     $last = array_pop($to);
     if (count($to) == 0) { // Only one participant
-      return t("From !last", array('!last' => $last));
+        return t("From !last", array('!last' => $last));
     }
     else { // Multipe participants..
-      $participants = implode(', ', $to);
-      return t('Participants: !participants and !last', array('!participants' => $participants, '!last' => $last));
+        $participants = implode(', ', $to);
+        return t('Participants: !participants and !last', array('!participants' => $participants, '!last' => $last));
+    }
     }
-  }
-  return '';
+    return '';
 }
 
 /**
  * Implements hook_menu().
  */
 function privatemsg_menu() {
-  $items['messages'] = array(
+    $items['messages'] = array(
     'title'            => 'Messages',
     'title callback'  => 'privatemsg_title_callback',
     'page callback'    => 'drupal_get_form',
     'page arguments'   => array('privatemsg_list', 'list'),
     'access callback'  => 'privatemsg_user_access',
     'type'             => MENU_NORMAL_ITEM,
-  );
-  $items['messages/list'] = array(
+    );
+    $items['messages/list'] = array(
     'title'            => 'Messages',
     'page callback'    => 'drupal_get_form',
     'page arguments'   => array('privatemsg_list', 'list'),
     'access callback'  => 'privatemsg_user_access',
     'type'             => MENU_DEFAULT_LOCAL_TASK,
     'weight'           => -10,
-  );
-  $items['messages/view/%privatemsg_thread'] = array(
+    );
+    $items['messages/view/%privatemsg_thread'] = array(
     'title'            => 'Read message',
     // Set the third argument to TRUE so that we can show access denied instead
     // of not found.
@@ -144,16 +144,16 @@ 
     'access arguments' => array(2),
     'type'             => MENU_LOCAL_TASK,
     'weight'           => -5,
-  );
-  $items['messages/delete/%privatemsg_thread/%privatemsg_message'] = array(
+    );
+    $items['messages/delete/%privatemsg_thread/%privatemsg_message'] = array(
     'title'            => 'Delete message',
     'page callback'    => 'drupal_get_form',
     'page arguments'   => array('privatemsg_delete', 2, 3),
     'access callback'  => 'privatemsg_user_access',
     'access arguments' => array('delete privatemsg'),
     'type'             => MENU_CALLBACK,
-  );
-  $items['messages/new'] = array(
+    );
+    $items['messages/new'] = array(
     'title'            => 'Write new message',
     'page callback'    => 'drupal_get_form',
     'page arguments'   => array('privatemsg_new', 2, 3, NULL),
@@ -161,24 +161,24 @@ 
     'access arguments' => array('write privatemsg'),
     'type'             => MENU_LOCAL_TASK,
     'weight'           => -3,
-  );
-  // Auto-completes available user names & removes duplicates.
-  $items['messages/user-name-autocomplete'] = array(
+    );
+    // Auto-completes available user names & removes duplicates.
+    $items['messages/user-name-autocomplete'] = array(
     'page callback'    => 'privatemsg_user_name_autocomplete',
     'access callback'  => 'privatemsg_user_access',
     'access arguments' => array('write privatemsg'),
     'type'             => MENU_CALLBACK,
     'weight'           => -10,
-  );
-  $items['admin/settings/messages'] = array(
+    );
+    $items['admin/settings/messages'] = array(
     'title'            => 'Private messages',
     'description'      => 'Configure private messaging settings.',
     'page callback'    => 'drupal_get_form',
     'page arguments'   => array('private_message_settings'),
     'access arguments' => array('administer privatemsg settings'),
     'type'             => MENU_NORMAL_ITEM,
-  );
-  $items['admin/settings/messages/default'] = array(
+    );
+    $items['admin/settings/messages/default'] = array(
     'title'            => 'Private messages',
     'description'      => 'Configure private messaging settings.',
     'page callback'    => 'drupal_get_form',
@@ -186,24 +186,24 @@ 
     'access arguments' => array('administer privatemsg settings'),
     'type'             => MENU_DEFAULT_LOCAL_TASK,
     'weight'           => -10,
-  );
-  $items['messages/undo/action'] = array(
+    );
+    $items['messages/undo/action'] = array(
     'title'            => 'Private messages',
     'description'      => 'Undo last thread action',
     'page callback'    => 'privatemsg_undo_action',
     'access arguments' => array('read privatemsg'),
     'type'             => MENU_CALLBACK,
-  );
-  $items['user/%/messages'] = array(
+    );
+    $items['user/%/messages'] = array(
     'title' => 'Messages',
     'page callback'    => 'drupal_get_form',
     'page arguments'   => array('privatemsg_list', 'list', 1),
     'access callback'  => 'privatemsg_user_access',
     'access arguments' => array('read all private messages'),
     'type' => MENU_LOCAL_TASK,
-  );
+    );
 
-  return $items;
+    return $items;
 }
 
 /**
@@ -220,17 +220,17 @@ 
  * @ingroup api
  */
 function privatemsg_user_access($permission = 'read privatemsg', $account = NULL) {
-  if ( $account === NULL ) {
+    if ( $account === NULL ) {
     global $user;
     $account = $user;
-  }
-  if (!$account->uid) { // Disallow anonymous access, regardless of permissions
+    }
+    if (!$account->uid) { // Disallow anonymous access, regardless of permissions
     return FALSE;
-  }
-  if (!user_access($permission, $account)) {
+    }
+    if (!user_access($permission, $account)) {
     return FALSE;
-  }
-  return TRUE;
+    }
+    return TRUE;
 }
 
 /**
@@ -247,15 +247,15 @@ 
  * @ingroup api
  */
 function privatemsg_view_access($thread = array()) {
-  // Do not allow access to threads without messages.
-  if (empty($thread['messages'])) {
+    // Do not allow access to threads without messages.
+    if (empty($thread['messages'])) {
     // Count all messages, if there
     return FALSE;
-  }
-  if (privatemsg_user_access('read privatemsg') && arg(1) == 'view') {
+    }
+    if (privatemsg_user_access('read privatemsg') && arg(1) == 'view') {
     return TRUE;
-  }
-  return FALSE;
+    }
+    return FALSE;
 }
 
 /**
@@ -284,142 +284,141 @@ 
  *
  *   If no messages are found, or the thread_id is invalid, the function returns
  *   FALSE.
-
  * @ingroup api
  */
 function privatemsg_thread_load($thread_id, $account = NULL, $start = NULL, $useAccessDenied = FALSE) {
-  static $threads = array();
-  if ((int)$thread_id > 0) {
+    static $threads = array();
+    if ((int)$thread_id > 0) {
     $thread = array('thread_id' => $thread_id);
 
     if (is_null($account)) {
-      global $user;
-      $account = drupal_clone($user);
+        global $user;
+        $account = drupal_clone($user);
     }
 
     if (!isset($threads[$account->uid])) {
-      $threads[$account->uid] = array();
+        $threads[$account->uid] = array();
     }
 
     if (!array_key_exists($thread_id, $threads[$account->uid])) {
-      // Load the list of participants.
-      $query = _privatemsg_assemble_query('participants', $thread_id);
-      $participants = db_query($query['query']);
-      $thread['participants'] = array();
-      while ($participant = db_fetch_object($participants)) {
+        // Load the list of participants.
+        $query = _privatemsg_assemble_query('participants', $thread_id);
+        $participants = db_query($query['query']);
+        $thread['participants'] = array();
+        while ($participant = db_fetch_object($participants)) {
         $thread['participants'][$participant->uid] = $participant;
-      }
-      $thread['read_all'] = FALSE;
-      if (!array_key_exists($account->uid, $thread['participants']) && privatemsg_user_access('read all private messages', $account)) {
+        }
+        $thread['read_all'] = FALSE;
+        if (!array_key_exists($account->uid, $thread['participants']) && privatemsg_user_access('read all private messages', $account)) {
         $thread['read_all'] = TRUE;
-      }
+        }
 
-      // Load messages returned by the messages query with privatemsg_message_load_multiple().
-      $query = _privatemsg_assemble_query('messages', array($thread_id), $thread['read_all'] ? NULL : $account);
-      $thread['message_count'] = $thread['to'] = db_result(db_query($query['count']));
-      $thread['from'] = 1;
-      // Check if we need to limit the messages.
-      $max_amount = variable_get('privatemsg_view_max_amount', 20);
+        // Load messages returned by the messages query with privatemsg_message_load_multiple().
+        $query = _privatemsg_assemble_query('messages', array($thread_id), $thread['read_all'] ? NULL : $account);
+        $thread['message_count'] = $thread['to'] = db_result(db_query($query['count']));
+        $thread['from'] = 1;
+        // Check if we need to limit the messages.
+        $max_amount = variable_get('privatemsg_view_max_amount', 20);
 
-      // If there is no start value, select based on get params.
-      if (is_null($start)) {
+        // If there is no start value, select based on get params.
+        if (is_null($start)) {
         if (isset($_GET['start']) && $_GET['start'] < $thread['message_count']) {
-          $start = $_GET['start'];
+            $start = $_GET['start'];
         }
         elseif (!variable_get('privatemsg_view_use_max_as_default', FALSE) && $max_amount == PRIVATEMSG_UNLIMITED) {
-          $start = PRIVATEMSG_UNLIMITED;
+            $start = PRIVATEMSG_UNLIMITED;
         }
         else {
-          $start = $thread['message_count'] - (variable_get('privatemsg_view_use_max_as_default', FALSE) ? variable_get('privatemsg_view_default_amount', 10) : $max_amount);
+            $start = $thread['message_count'] - (variable_get('privatemsg_view_use_max_as_default', FALSE) ? variable_get('privatemsg_view_default_amount', 10) : $max_amount);
+        }
         }
-      }
 
-      if ($start != PRIVATEMSG_UNLIMITED) {
+        if ($start != PRIVATEMSG_UNLIMITED) {
         if ($max_amount == PRIVATEMSG_UNLIMITED) {
-          $last_page = 0;
-          $max_amount = $thread['message_count'];
+            $last_page = 0;
+            $max_amount = $thread['message_count'];
         }
         else {
-          // Calculate the number of messages on the "last" page to avoid
-          // message overlap.
-          // Note - the last page lists the earliest messages, not the latest.
-          $paging_count = variable_get('privatemsg_view_use_max_as_default', FALSE) ? $thread['message_count'] - variable_get('privatemsg_view_default_amount', 10) : $thread['message_count'];
-          $last_page = $paging_count % $max_amount;
+            // Calculate the number of messages on the "last" page to avoid
+            // message overlap.
+            // Note - the last page lists the earliest messages, not the latest.
+            $paging_count = variable_get('privatemsg_view_use_max_as_default', FALSE) ? $thread['message_count'] - variable_get('privatemsg_view_default_amount', 10) : $thread['message_count'];
+            $last_page = $paging_count % $max_amount;
         }
 
         // Sanity check - we cannot start from a negative number.
         if ($start < 0) {
-          $start = 0;
+            $start = 0;
         }
         $thread['start'] = $start;
 
         //If there are newer messages on the page, show pager link allowing to go to the newer messages.
         if (($start + $max_amount + 1) < $thread['message_count']) {
-          $thread['to'] = $start + $max_amount;
-          $thread['newer_start'] = $start + $max_amount;
+            $thread['to'] = $start + $max_amount;
+            $thread['newer_start'] = $start + $max_amount;
         }
         if ($start - $max_amount >= 0) {
-          $thread['older_start'] = $start - $max_amount;
+            $thread['older_start'] = $start - $max_amount;
         }
         elseif ($start > 0) {
-          $thread['older_start'] = 0;
+            $thread['older_start'] = 0;
         }
 
         // Do not show messages on the last page that would show on the page
         // before. This will only work when using the visual pager.
         if ($start < $last_page && $max_amount != PRIVATEMSG_UNLIMITED && $max_amount < $thread['message_count']) {
-          unset($thread['older_start']);
-          $thread['to'] = $thread['newer_start'] = $max_amount = $last_page;
-          // Start from the first message - this is a specific hack to make sure
-          // the message display has sane paging on the last page.
-          $start = 0;
+            unset($thread['older_start']);
+            $thread['to'] = $thread['newer_start'] = $max_amount = $last_page;
+            // Start from the first message - this is a specific hack to make sure
+            // the message display has sane paging on the last page.
+            $start = 0;
         }
         // Visual counts start from 1 instead of zero, so plus one.
         $thread['from'] = $start + 1;
         $conversation = db_query_range($query['query'], $start, $max_amount);
-      }
-      else {
+        }
+        else {
         $conversation = db_query($query['query']);
-      }
-      $mids = array();
-      while ($result = db_fetch_array($conversation)) {
+        }
+        $mids = array();
+        while ($result = db_fetch_array($conversation)) {
         $mids[] = $result['mid'];
-      }
-      // Load messages returned by the messages query.
-      $thread['messages'] = privatemsg_message_load_multiple($mids, $thread['read_all'] ? NULL : $account);
+        }
+        // Load messages returned by the messages query.
+        $thread['messages'] = privatemsg_message_load_multiple($mids, $thread['read_all'] ? NULL : $account);
 
-      // If there are no messages, don't allow access to the thread.
-      if (empty($thread['messages'])) {
+        // If there are no messages, don't allow access to the thread.
+        if (empty($thread['messages'])) {
         if ($useAccessDenied) {
-          // Generate new query with read all to see if the thread does exist.
-          $query = _privatemsg_assemble_query('messages', array($thread_id), NULL);
-          $exists = db_result(db_query($query['count']));
-          if (!$exists) {
+            // Generate new query with read all to see if the thread does exist.
+            $query = _privatemsg_assemble_query('messages', array($thread_id), NULL);
+            $exists = db_result(db_query($query['count']));
+            if (!$exists) {
             // Thread does not exist, display 404.
             $thread = FALSE;
-          }
+            }
         }
         else {
-          $thread = FALSE;
+            $thread = FALSE;
+        }
         }
-      }
-      else {
+        else {
         // General data, assume subject is the same for all messages of that thread.
         $thread['user'] = $account;
         $message = current($thread['messages']);
         $thread['subject'] = $message['subject'];
-      }
-      $threads[$account->uid][$thread_id] = $thread;
+        }
+        $threads[$account->uid][$thread_id] = $thread;
     }
 
     return $threads[$account->uid][$thread_id];
-  }
-  return FALSE;
+    }
+    return FALSE;
 }
 
 function private_message_view_options() {
-  $options = module_invoke_all('privatemsg_view_template');
-  return $options;
+    $options = module_invoke_all('privatemsg_view_template');
+    return $options;
 }
 
 /**
@@ -435,108 +434,108 @@ 
  * };
  */
 function privatemsg_privatemsg_view_template() {
-  return array(
+    return array(
     'privatemsg-view' => 'Default view',
-  );
+    );
 }
 
 function private_message_settings() {
-  $form = array();
+    $form = array();
 
-  $form['theming_settings'] = array(
+    $form['theming_settings'] = array(
     '#type'        => 'fieldset',
     '#collapsible' => TRUE,
     '#collapsed'   => TRUE,
     '#title'       => t('Theming settings'),
-  );
-  $form['theming_settings']['private_message_view_template'] = array(
+    );
+    $form['theming_settings']['private_message_view_template'] = array(
     '#type'          => 'radios',
     '#title'         => t('Private message display template'),
     '#default_value' => variable_get('private_message_view_template', 'privatemsg-view'),
     '#options'       => private_message_view_options(),
-  );
-  $form['privatemsg_display_loginmessage'] = array(
+    );
+    $form['privatemsg_display_loginmessage'] = array(
     '#type' => 'checkbox',
     '#title' => t('Inform the user about new messages on login'),
     '#default_value' => variable_get('privatemsg_display_loginmessage', TRUE),
     '#description' => t('This option can safely be disabled if the "New message indication" block is used instead.'),
-  );
+    );
 
-  $form['flush_deleted'] = array(
+    $form['flush_deleted'] = array(
     '#type'        => 'fieldset',
     '#collapsible' => TRUE,
     '#collapsed'   => TRUE,
     '#title'       => t('Flush deleted messages'),
     '#description' => t('By default, deleted messages are only hidden from the user but still stored in the database. These settings control if and when messages should be removed.'),
-  );
+    );
 
-  $form['flush_deleted']['privatemsg_flush_enabled'] = array(
+    $form['flush_deleted']['privatemsg_flush_enabled'] = array(
     '#type'          => 'checkbox',
     '#title'         => t('Flush deleted messages'),
     '#default_value' => variable_get('privatemsg_flush_enabled', FALSE),
     '#description'   => t('Enable the flushing of deleted messages. Requires that cron is enabled'),
-  );
+    );
 
-  $form['flush_deleted']['privatemsg_flush_days'] = array(
+    $form['flush_deleted']['privatemsg_flush_days'] = array(
     '#type' => 'select',
     '#title' => t('Flush messages after they have been deleted for more days than'),
     '#default_value' => variable_get('privatemsg_flush_days', 30),
     '#options' => drupal_map_assoc(array(0, 1, 2, 5, 10, 30, 100)),
-  );
+    );
 
-  $form['flush_deleted']['privatemsg_flush_max'] = array(
+    $form['flush_deleted']['privatemsg_flush_max'] = array(
     '#type' => 'select',
     '#title' => t('Maximum number of messages to flush per cron run'),
     '#default_value' => variable_get('privatemsg_flush_max', 200),
     '#options' => drupal_map_assoc(array(50, 100, 200, 500, 1000)),
-  );
+    );
 
-  $form['privatemsg_listing'] = array(
+    $form['privatemsg_listing'] = array(
     '#type' => 'fieldset',
     '#title' => t('Configure listings'),
     '#collapsible' => TRUE,
     '#collapsed' => FALSE,
-  );
+    );
 
-  $form['privatemsg_listing']['privatemsg_per_page'] = array(
+    $form['privatemsg_listing']['privatemsg_per_page'] = array(
     '#type' => 'select',
     '#title' => t('Threads per page'),
     '#default_value' => variable_get('privatemsg_per_page', 25),
     '#options' => drupal_map_assoc(array(10, 25, 50, 75, 100)),
     '#description' => t('Choose the number of conversations that should be listed per page.'),
-  );
+    );
 
-  $form['privatemsg_listing']['privatemsg_display_fields'] = array(
+    $form['privatemsg_listing']['privatemsg_display_fields'] = array(
     '#type' => 'checkboxes',
     '#title' => t('Configure fields'),
     '#description' => t('Select which columns/fields should be displayed in the message listings. Subject and Last updated cannot be disabled.'),
     '#options' => array(
-      'participants'    => t('Participants'),
-      'thread_started'  => t('Started'),
-      'count'           => t('Messages'),
+        'participants'    => t('Participants'),
+        'thread_started'  => t('Started'),
+        'count'           => t('Messages'),
     ),
     '#default_value' => variable_get('privatemsg_display_fields', array('participants')),
-  );
+    );
 
-  $amounts = drupal_map_assoc(array(5, 10, 20, 30, 50, 70, 90, 150, 200, 250, 300));
-  $form['privatemsg_listing']['privatemsg_view_max_amount'] = array(
+    $amounts = drupal_map_assoc(array(5, 10, 20, 30, 50, 70, 90, 150, 200, 250, 300));
+    $form['privatemsg_listing']['privatemsg_view_max_amount'] = array(
     '#type'          => 'select',
     '#title'         => t('Number of messages on thread pages'),
     '#options'       => $amounts + array(PRIVATEMSG_UNLIMITED => t('Unlimited')),
     '#default_value' => variable_get('privatemsg_view_max_amount', 20),
     '#description'   => t('Threads will not show more than this number of messages on a single page.'),
     '#weight'        => 10,
-  );
+    );
 
-  $form['privatemsg_listing']['privatemsg_view_use_max_as_default'] = array(
+    $form['privatemsg_listing']['privatemsg_view_use_max_as_default'] = array(
     '#type'          => 'checkbox',
     '#title'         => t('Display different amount of messages on first thread page'),
     '#default_value' => variable_get('privatemsg_view_use_max_as_default', FALSE),
     '#description'   => t('By default, the first thread page shows the maximally allowed amount of messages. Enable this checkbox to set a different value.'),
     '#weight'        => 15,
-  );
+    );
 
-  $form['privatemsg_listing']['privatemsg_view_default_amount'] = array(
+    $form['privatemsg_listing']['privatemsg_view_default_amount'] = array(
     '#prefix'        => '<div id="privatemsg-view-default-button">',
     '#suffix'        => '</div>',
     '#type'          => 'select',
@@ -545,15 +544,15 @@ 
     '#description'   => t('The number of messages to be displayed on first thread page. Displays the newest messages.'),
     '#options'       => $amounts,
     '#weight'        => 20,
-  );
-  drupal_add_js(drupal_get_path('module', 'privatemsg') .'/privatemsg-admin.js');
+    );
+    drupal_add_js(drupal_get_path('module', 'privatemsg') .'/privatemsg-admin.js');
 
-  $form['#submit'][] = 'private_message_settings_submit';
-  return system_settings_form($form);
+    $form['#submit'][] = 'private_message_settings_submit';
+    return system_settings_form($form);
 }
 
 function private_message_settings_submit() {
-  drupal_rebuild_theme_registry();
+    drupal_rebuild_theme_registry();
 }
 
 /**
@@ -563,107 +562,107 @@ 
  * old enough are flushed.
  */
 function privatemsg_cron() {
-  if (variable_get('privatemsg_flush_enabled', FALSE)) {
+    if (variable_get('privatemsg_flush_enabled', FALSE)) {
     $query = _privatemsg_assemble_query('deleted', variable_get('privatemsg_flush_days', 30));
     $result = db_query($query['query']);
 
     $flushed = 0;
     while (($row = db_fetch_array($result)) && ($flushed < variable_get('privatemsg_flush_max', 200))) {
-      $message = privatemsg_message_load($row['mid']);
-      module_invoke_all('privatemsg_message_flush', $message);
+        $message = privatemsg_message_load($row['mid']);
+        module_invoke_all('privatemsg_message_flush', $message);
 
-      // Delete recipients of the message.
-      db_query('DELETE FROM {pm_index} WHERE mid = %d', $row['mid']);
-      // Delete message itself.
-      db_query('DELETE FROM {pm_message} WHERE mid = %d', $row['mid']);
-      $flushed++;
+        // Delete recipients of the message.
+        db_query('DELETE FROM {pm_index} WHERE mid = %d', $row['mid']);
+        // Delete message itself.
+        db_query('DELETE FROM {pm_message} WHERE mid = %d', $row['mid']);
+        $flushed++;
+    }
     }
-  }
 }
 
 function privatemsg_theme() {
-  return array(
+    return array(
     'privatemsg_view'    => array(
-      'arguments'        => array('message' => NULL),
-      'template'         => variable_get('private_message_view_template', 'privatemsg-view'), // 'privatemsg',
+        'arguments'        => array('message' => NULL),
+        'template'         => variable_get('private_message_view_template', 'privatemsg-view'), // 'privatemsg',
     ),
     'privatemsg_from'    => array(
-      'arguments'        => array('author' => NULL),
-      'template'         => 'privatemsg-from',
+        'arguments'        => array('author' => NULL),
+        'template'         => 'privatemsg-from',
     ),
     'privatemsg_recipients' => array(
-      'arguments'        => array('message' => NULL),
-      'template'         => 'privatemsg-recipients',
+        'arguments'        => array('message' => NULL),
+        'template'         => 'privatemsg-recipients',
     ),
     'privatemsg_between' => array(
-      'arguments'        => array('recipients' => NULL),
-      'template'         => 'privatemsg-between',
+        'arguments'        => array('recipients' => NULL),
+        'template'         => 'privatemsg-between',
     ),
     'privatemsg_list'    => array(
-      'file'                  => 'privatemsg.theme.inc',
-      'path'                  => drupal_get_path('module', 'privatemsg'),
-      'arguments'        => array('form'),
+        'file'                  => 'privatemsg.theme.inc',
+        'path'                  => drupal_get_path('module', 'privatemsg'),
+        'arguments'        => array('form'),
     ),
     // Define pattern for header/field templates. The theme system will register all
     // theme functions that start with the defined pattern.
     'privatemsg_list_header'  => array(
-      'file'                  => 'privatemsg.theme.inc',
-      'path'                  => drupal_get_path('module', 'privatemsg'),
-      'pattern'               => 'privatemsg_list_header__',
-      'arguments'             => array(),
+        'file'                  => 'privatemsg.theme.inc',
+        'path'                  => drupal_get_path('module', 'privatemsg'),
+        'pattern'               => 'privatemsg_list_header__',
+        'arguments'             => array(),
     ),
     'privatemsg_list_field'   => array(
-      'file'                  => 'privatemsg.theme.inc',
-      'path'                  => drupal_get_path('module', 'privatemsg'),
-      'pattern'               => 'privatemsg_list_field__',
-      'arguments'             => array('thread'),
+        'file'                  => 'privatemsg.theme.inc',
+        'path'                  => drupal_get_path('module', 'privatemsg'),
+        'pattern'               => 'privatemsg_list_field__',
+        'arguments'             => array('thread'),
     ),
     'privatemsg_new_block'  => array(
-      'file'                  => 'privatemsg.theme.inc',
-      'path'                  => drupal_get_path('module', 'privatemsg'),
-      'arguments'             => array('count'),
+        'file'                  => 'privatemsg.theme.inc',
+        'path'                  => drupal_get_path('module', 'privatemsg'),
+        'arguments'             => array('count'),
     ),
-  );
+    );
 }
 
 function template_preprocess_privatemsg_view(&$vars) {
 //  drupal_set_message('<pre>'. print_r($vars,1 ) . '</pre>');
 
-  $message = $vars['message'];
-  $vars['mid'] = isset($message['mid']) ? $message['mid'] : NULL;
-  $vars['thread_id'] = isset($message['thread_id']) ? $message['thread_id'] : NULL;
-  $vars['author_picture'] = theme('user_picture', $message['author']);
-  $vars['author_name_link'] = theme('username', $message['author']);
-  /**
-   * @todo perhaps make this timestamp configurable via admin UI?
-   */
-  $vars['message_timestamp'] = format_date($message['timestamp'], 'small');
-  $vars['message_body'] = check_markup($message['body'], $message['format'], FALSE);
-  if (isset($vars['mid']) && isset($vars['thread_id']) && privatemsg_user_access('delete privatemsg')) {
+    $message = $vars['message'];
+    $vars['mid'] = isset($message['mid']) ? $message['mid'] : NULL;
+    $vars['thread_id'] = isset($message['thread_id']) ? $message['thread_id'] : NULL;
+    $vars['author_picture'] = theme('user_picture', $message['author']);
+    $vars['author_name_link'] = theme('username', $message['author']);
+    /**
+     * @todo perhaps make this timestamp configurable via admin UI?
+     */
+    $vars['message_timestamp'] = format_date($message['timestamp'], 'small');
+    $vars['message_body'] = check_markup($message['body'], $message['format'], FALSE);
+    if (isset($vars['mid']) && isset($vars['thread_id']) && privatemsg_user_access('delete privatemsg')) {
     $vars['message_actions'][] = array('title' => t('Delete message'), 'href' => 'messages/delete/' . $vars['thread_id'] . '/' . $vars['mid']);
-  }
-  $vars['message_anchors'][] = 'privatemsg-mid-' . $vars['mid'];
-  if (!empty($message['is_new'])) {
+    }
+    $vars['message_anchors'][] = 'privatemsg-mid-' . $vars['mid'];
+    if (!empty($message['is_new'])) {
     $vars['message_anchors'][] = 'new';
     $vars['new'] = drupal_ucfirst(t('new'));
-  }
+    }
 
-  // call hook_privatemsg_message_view_alter
-  drupal_alter('privatemsg_message_view', $vars);
+    // call hook_privatemsg_message_view_alter
+    drupal_alter('privatemsg_message_view', $vars);
 
-  $vars['message_actions'] = !empty($vars['message_actions']) ? theme('links', $vars['message_actions'], array('class' => 'message-actions')) : '';
+    $vars['message_actions'] = !empty($vars['message_actions']) ? theme('links', $vars['message_actions'], array('class' => 'message-actions')) : '';
 
-  $vars['anchors'] = '';
-  foreach ($vars['message_anchors'] as $anchor) {
+    $vars['anchors'] = '';
+    foreach ($vars['message_anchors'] as $anchor) {
     $vars['anchors'] .= '<a name="' . $anchor . '"></a>';
-  }
+    }
 }
 
 function template_preprocess_privatemsg_recipients(&$vars) {
-  $vars['participants'] = ''; // assign a default empty value
-  if (isset($vars['message']['participants'])) {
+    $vars['participants'] = ''; // assign a default empty value
+    if (isset($vars['message']['participants'])) {
     $vars['participants'] = _privatemsg_format_participants($vars['message']['participants']);
-  }
+    }
 }
 
 /**
@@ -680,49 +679,49 @@ 
  *   Form array
  */
 function privatemsg_list(&$form_state, $argument = 'list', $uid = NULL) {
-  global $user;
+    global $user;
 
-  // Setting default behavior...
-  $account = $user;
-  // Because uid is submitted by the menu system, it's a string not a integer.
-  if ((int)$uid > 0 && $uid != $user->uid) {
+    // Setting default behavior...
+    $account = $user;
+    // Because uid is submitted by the menu system, it's a string not a integer.
+    if ((int)$uid > 0 && $uid != $user->uid) {
     // Trying to view someone else's messages...
     if (!privatemsg_user_access('read all private messages')) {
-      drupal_set_message(t("You do not have sufficient rights to view someone else's messages"), 'warning');
+        drupal_set_message(t("You do not have sufficient rights to view someone else's messages"), 'warning');
     }
     elseif ($account_check = user_load(array('uid' => $uid))) {
-      // Has rights and user_load return an array so user does exist
-      $account = $account_check;
+        // Has rights and user_load return an array so user does exist
+        $account = $account_check;
+    }
     }
-  }
-  // By this point we have figured out for which user we are listing messages and now it is safe to use $account->uid in the listing query.
+    // By this point we have figured out for which user we are listing messages and now it is safe to use $account->uid in the listing query.
 
-  $query = _privatemsg_assemble_query('list', $account, $argument);
-  $result = pager_query($query['query'], variable_get('privatemsg_per_page', 25), 0, $query['count']);
+    $query = _privatemsg_assemble_query('list', $account, $argument);
+    $result = pager_query($query['query'], variable_get('privatemsg_per_page', 25), 0, $query['count']);
 
-  $threads = array();
-  $form['#data'] = array();
-  while ($row = db_fetch_array($result)) {
+    $threads = array();
+    $form['#data'] = array();
+    while ($row = db_fetch_array($result)) {
     // Store the raw row data.
     $form['#data'][$row['thread_id']] = $row;
     // store thread id for the checkboxes array
     $threads[$row['thread_id']] = '';
-  }
-  if (!empty($form['#data'])) {
+    }
+    if (!empty($form['#data'])) {
     $form['actions'] = _privatemsg_action_form();
-  }
+    }
 
-  // Save the currently active account, used for actions.
-  $form['account'] = array('#type' => 'value', '#value' => $account);
+    // Save the currently active account, used for actions.
+    $form['account'] = array('#type' => 'value', '#value' => $account);
 
-  // Define checkboxes, pager and theme
-  $form['threads'] = array('#type' => 'checkboxes', '#options' => $threads);
-  $form['pager'] = array('#value' => theme('pager'), '#weight' => 20);
-  $form['#theme'] = 'privatemsg_list';
+    // Define checkboxes, pager and theme
+    $form['threads'] = array('#type' => 'checkboxes', '#options' => $threads);
+    $form['pager'] = array('#value' => theme('pager'), '#weight' => 20);
+    $form['#theme'] = 'privatemsg_list';
 
-  // Store the account for which the threads are displayed.
-  $form['#account'] = $account;
-  return $form;
+    // Store the account for which the threads are displayed.
+    $form['#account'] = $account;
+    return $form;
 }
 
 /**
@@ -736,12 +735,12 @@ 
  *   User object, defaults to the current user
  */
 function privatemsg_message_change_status($pmid, $status, $account = NULL) {
-  if (!$account) {
+    if (!$account) {
     global $user;
     $account = $user;
-  }
-  $query = "UPDATE {pm_index} SET is_new = %d WHERE mid = %d AND uid = %d";
-  db_query($query, $status, $pmid, $account->uid);
+    }
+    $query = "UPDATE {pm_index} SET is_new = %d WHERE mid = %d AND uid = %d";
+    db_query($query, $status, $pmid, $account->uid);
 }
 
 /**
@@ -753,16 +752,16 @@ 
  * @ingroup api
  */
 function privatemsg_unread_count($account = NULL) {
-  static $counts = array();
-  if (!$account || $account->uid == 0) {
+    static $counts = array();
+    if (!$account || $account->uid == 0) {
     global $user;
     $account = $user;
-  }
-  if ( !isset($counts[$account->uid])) {
+    }
+    if ( !isset($counts[$account->uid])) {
     $query = _privatemsg_assemble_query('unread_count', $account);
     $counts[$account->uid] = db_result(db_query($query['query']));
-  }
-  return $counts[$account->uid];
+    }
+    return $counts[$account->uid];
 }
 
 /**
@@ -776,282 +775,282 @@ 
  * @see privatemsg_thread_load()
  */
 function privatemsg_view($thread) {
-  drupal_set_title(check_plain($thread['subject']));
+    drupal_set_title(check_plain($thread['subject']));
 
-  // Generate paging links.
-  $older = '';
-  if (isset($thread['older_start'])) {
+    // Generate paging links.
+    $older = '';
+    if (isset($thread['older_start'])) {
     $options = array(
-      'query' => array('start' => $thread['older_start']),
-      'title' => t('Display older messages'),
+        'query' => array('start' => $thread['older_start']),
+        'title' => t('Display older messages'),
     );
-     $older = l(t('<<'), 'messages/view/' . $thread['thread_id'], $options);
-  }
-  $newer = '';
-  if (isset($thread['newer_start'])) {
+        $older = l(t('<<'), 'messages/view/' . $thread['thread_id'], $options);
+    }
+    $newer = '';
+    if (isset($thread['newer_start'])) {
     $options = array(
-      'query' => array('start' => $thread['newer_start']),
-      'title' => t('Display newer messages'),
+        'query' => array('start' => $thread['newer_start']),
+        'title' => t('Display newer messages'),
     );
     $newer = l(t('>>'), 'messages/view/' . $thread['thread_id'], $options);
-  }
-  $substitutions = array('@from' => $thread['from'], '@to' => $thread['to'], '@total' => $thread['message_count'], '!previous_link' => $older, '!newer_link' => $newer);
-  $title = t('!previous_link Displaying messages @from - @to of @total !newer_link', $substitutions);
-  $content['pager_top'] = array(
+    }
+    $substitutions = array('@from' => $thread['from'], '@to' => $thread['to'], '@total' => $thread['message_count'], '!previous_link' => $older, '!newer_link' => $newer);
+    $title = t('!previous_link Displaying messages @from - @to of @total !newer_link', $substitutions);
+    $content['pager_top'] = array(
     '#value'  => trim($title),
     '#prefix' => '<div class="privatemsg-view-pager">',
     '#suffix' => '</div>',
     '#weight' => -10,
-  );
+    );
 
-  // Display a copy at the end.
-  $content['pager_bottom'] = $content['pager_top'];
-  $content['pager_bottom']['#weight'] = 3;
+    // Display a copy at the end.
+    $content['pager_bottom'] = $content['pager_top'];
+    $content['pager_bottom']['#weight'] = 3;
 
-  // Render the participants.
-  $content['participants']['#value'] = theme('privatemsg_recipients', $thread);
-  $content['participants']['#weight'] = -5;
+    // Render the participants.
+    $content['participants']['#value'] = theme('privatemsg_recipients', $thread);
+    $content['participants']['#weight'] = -5;
 
-  // Render the messages.
-  $output = '';
-  foreach ($thread['messages'] as $pmid => $message) {
+    // Render the messages.
+    $output = '';
+    foreach ($thread['messages'] as $pmid => $message) {
     // Set message as read and theme it.
     if (!empty($message['is_new'])) {
-      privatemsg_message_change_status($pmid, PRIVATEMSG_READ, $thread['user']);
+        privatemsg_message_change_status($pmid, PRIVATEMSG_READ, $thread['user']);
     }
     $output .= theme('privatemsg_view', $message);
-  }
-  $content['messages']['#value'] = $output;
-  $content['messages']['#weight'] = 0;
+    }
+    $content['messages']['#value'] = $output;
+    $content['messages']['#weight'] = 0;
 
-  // Display the reply form if user is allowed to use it.
-  if (privatemsg_user_access('write privatemsg')) {
+    // Display the reply form if user is allowed to use it.
+    if (privatemsg_user_access('write privatemsg')) {
     $content['reply']['#value'] = drupal_get_form('privatemsg_new', $thread['participants'], $thread['subject'], $thread['thread_id'], $thread['read_all']);
     $content['reply']['#weight'] = 5;
-  }
+    }
 
-  // Check after calling the privatemsg_new form so that this message is only
-  // displayed when we are not sending a message.
-  if ($thread['read_all']) {
+    // Check after calling the privatemsg_new form so that this message is only
+    // displayed when we are not sending a message.
+    if ($thread['read_all']) {
     // User has permission to read all messages AND is not a participant of the current thread.
     drupal_set_message(t('This conversation is being viewed with escalated priviledges and may not be the same as shown to normal users.'), 'warning');
-  }
+    }
 
-  // Allow other modules to hook into the $content array and alter it.
-  drupal_alter('privatemsg_view_messages', $content, $thread);
-  return drupal_render($content);
+    // Allow other modules to hook into the $content array and alter it.
+    drupal_alter('privatemsg_view_messages', $content, $thread);
+    return drupal_render($content);
 }
 
 
 function privatemsg_new(&$form_state, $recipients = array(), $subject = '', $thread_id = NULL, $read_all = FALSE) {
-  global $user;
+    global $user;
 
-  $recipients_string = '';
-  $body      = '';
+    $recipients_string = '';
+    $body      = '';
 
-  // convert recipients to array of user objects
-  if (!empty($recipients) && is_string($recipients) || is_int($recipients)) {
+    // convert recipients to array of user objects
+    if (!empty($recipients) && is_string($recipients) || is_int($recipients)) {
     $recipients = _privatemsg_generate_user_array($recipients);
-  }
-  elseif (is_object($recipients)) {
+    }
+    elseif (is_object($recipients)) {
     $recipients = array($recipients);
-  }
-  elseif (empty($recipients) && is_string($recipients)) {
+    }
+    elseif (empty($recipients) && is_string($recipients)) {
     $recipients = array();
-  }
+    }
 
-  $usercount = 0;
-  $to = array();
-  $to_themed = array();
-  $blocked = FALSE;
-  foreach ($recipients as $recipient) {
+    $usercount = 0;
+    $to = array();
+    $to_themed = array();
+    $blocked = FALSE;
+    foreach ($recipients as $recipient) {
     if (in_array($recipient->name, $to)) {
-      // We already added the recipient to the list, skip him.
-      continue;
+        // We already added the recipient to the list, skip him.
+        continue;
     }
     // Check if another module is blocking the sending of messages to the recipient by current user.
     $user_blocked = module_invoke_all('privatemsg_block_message', $user, array($recipient->uid => $recipient));
     if (!count($user_blocked) <> 0 && $recipient->uid) {
-      if ($recipient->uid == $user->uid) {
+        if ($recipient->uid == $user->uid) {
         $usercount++;
         // Skip putting author in the recipients list for now.
         continue;
-      }
-      $to[] = $recipient->name;
-      $to_themed[$recipient->uid] = theme('username', $recipient);
+        }
+        $to[] = $recipient->name;
+        $to_themed[$recipient->uid] = theme('username', $recipient);
     }
     else {
-      // Recipient list contains blocked users.
-      $blocked = TRUE;
+        // Recipient list contains blocked users.
+        $blocked = TRUE;
+    }
     }
-  }
 
-  if (empty($to) && $usercount >= 1 && !$blocked) {
+    if (empty($to) && $usercount >= 1 && !$blocked) {
     // Assume the user sent message to own account as if the usercount is one or less, then the user sent a message but not to self.
     $to[] = $user->name;
     $to_themed[$user->uid] = theme('username', $user);
-  }
+    }
 
-  if (!empty($to)) {
+    if (!empty($to)) {
     $recipients_string = implode(', ', $to);
-  }
-  if (isset($form_state['values'])) {
+    }
+    if (isset($form_state['values'])) {
     if (isset($form_state['values']['recipient'])) {
-      $recipients_string = $form_state['values']['recipient'];
+        $recipients_string = $form_state['values']['recipient'];
 
     }
     $subject   = $form_state['values']['subject'];
     $body      = $form_state['values']['body'];
-  }
-  if (!$thread_id && !empty($recipients_string)) {
+    }
+    if (!$thread_id && !empty($recipients_string)) {
     drupal_set_title(t('Write new message to %recipient', array('%recipient' => $recipients_string)));
-  }
-  elseif (!$thread_id) {
+    }
+    elseif (!$thread_id) {
     drupal_set_title(t('Write new message'));
-  }
+    }
 
-  $form = array();
-  if (isset($form_state['privatemsg_preview'])) {
+    $form = array();
+    if (isset($form_state['privatemsg_preview'])) {
     $form['message_header'] = array(
-      '#type' => 'fieldset',
-      '#attributes' => array('class' => 'preview'),
+        '#type' => 'fieldset',
+        '#attributes' => array('class' => 'preview'),
     );
     $form['message_header']['message_preview'] = array(
-      '#value'  => $form_state['privatemsg_preview'],
+        '#value'  => $form_state['privatemsg_preview'],
     );
-  }
-  $form['privatemsg'] = array(
+    }
+    $form['privatemsg'] = array(
     '#type'               => 'fieldset',
     '#access'             => privatemsg_user_access('write privatemsg'),
-  );
-  $form['privatemsg']['author'] = array(
+    );
+    $form['privatemsg']['author'] = array(
     '#type' => 'value',
     '#value' => $user,
-  );
-  if (is_null($thread_id)) {
+    );
+    if (is_null($thread_id)) {
     $form['privatemsg']['recipient'] = array(
-      '#type'               => 'textfield',
-      '#title'              => t('To'),
-      '#description'        => t('Separate multiple names with commas.'),
-      '#default_value'      => $recipients_string,
-      '#required'           => TRUE,
-      '#weight'             => -10,
-      '#size'               => 50,
-      '#autocomplete_path'  => 'messages/user-name-autocomplete',
-      // Do not hardcode #maxlength, make it configurable by number of recipients, not their name length.
+        '#type'               => 'textfield',
+        '#title'              => t('To'),
+        '#description'        => t('Separate multiple names with commas.'),
+        '#default_value'      => $recipients_string,
+        '#required'           => TRUE,
+        '#weight'             => -10,
+        '#size'               => 50,
+        '#autocomplete_path'  => 'messages/user-name-autocomplete',
+        // Do not hardcode #maxlength, make it configurable by number of recipients, not their name length.
     );
-  }
-  $form['privatemsg']['subject'] = array(
+    }
+    $form['privatemsg']['subject'] = array(
     '#type'               => 'textfield',
     '#title'              => t('Subject'),
     '#size'               => 50,
     '#maxlength'          => 255,
     '#default_value'      => $subject,
     '#weight'             => -5,
-  );
-  $form['privatemsg']['body'] = array(
+    );
+    $form['privatemsg']['body'] = array(
     '#type'               => 'textarea',
     '#title'              => t('Message'),
     '#rows'               => 6,
     '#weight'             => 0,
     '#default_value'      => $body,
     '#resizable'          => TRUE,
-  );
-  $format = FILTER_FORMAT_DEFAULT;
-  // The input filter widget looses the format during preview, specify it
-  // explicitly.
-  if (isset($form_state['values']) && array_key_exists('format', $form_state['values'])) {
+    );
+    $format = FILTER_FORMAT_DEFAULT;
+    // The input filter widget looses the format during preview, specify it
+    // explicitly.
+    if (isset($form_state['values']) && array_key_exists('format', $form_state['values'])) {
     $format = $form_state['values']['format'];
-  }
-  $form['privatemsg']['format'] = filter_form($format);
-  $form['privatemsg']['preview'] = array(
+    }
+    $form['privatemsg']['format'] = filter_form($format);
+    $form['privatemsg']['preview'] = array(
     '#type'               => 'submit',
     '#value'              => t('Preview message'),
     '#submit'             => array('pm_preview'),
     '#validate'           => array('pm_send_validate'),
     '#weight'             => 10,
-  );
-  $form['privatemsg']['submit'] = array(
+    );
+    $form['privatemsg']['submit'] = array(
     '#type'               => 'submit',
     '#value'              => t('Send message'),
     '#submit'             => array('pm_send'),
     '#validate'           => array('pm_send_validate'),
     '#weight'             => 15,
-  );
-  $url = 'messages';
-  $title = t('Cancel');
-  if (isset($_REQUEST['destination'])) {
+    );
+    $url = 'messages';
+    $title = t('Cancel');
+    if (isset($_REQUEST['destination'])) {
     $url = $_REQUEST['destination'];
-  }
-  elseif (!is_null($thread_id)) {
+    }
+    elseif (!is_null($thread_id)) {
     $url = $_GET['q'];
     $title = t('Clear');
-  }
+    }
 
-  $form['privatemsg']['cancel'] = array(
+    $form['privatemsg']['cancel'] = array(
     '#value'              => l($title, $url, array('attributes' => array('id' => 'edit-cancel'))),
     '#weight'             => 20,
-  );
+    );
 
-  if (!is_null($thread_id)) {
+    if (!is_null($thread_id)) {
     $form['privatemsg']['thread_id'] = array(
-      '#type' => 'value',
-      '#value' => $thread_id,
+        '#type' => 'value',
+        '#value' => $thread_id,
     );
     $form['privatemsg']['subject'] = array(
-          '#type' => 'value',
-          '#default_value' => $subject,
+            '#type' => 'value',
+            '#default_value' => $subject,
     );
     $recipients_string_themed = implode(', ', $to_themed);
     $form['privatemsg']['recipient_display'] = array(
-      '#value' =>  '<p>'. t('<strong>Reply to thread</strong>:<br /> Recipients: !to', array('!to' => $recipients_string_themed)) .'</p>',
-      '#weight' => -10,
+        '#value' =>  '<p>'. t('<strong>Reply to thread</strong>:<br /> Recipients: !to', array('!to' => $recipients_string_themed)) .'</p>',
+        '#weight' => -10,
     );
     if (empty($recipients_string)) {
-      // If there are no valid recipients, unset the message reply form.
-      $form['privatemsg']['#access'] = FALSE;
+        // If there are no valid recipients, unset the message reply form.
+        $form['privatemsg']['#access'] = FALSE;
+    }
     }
-  }
-  $form['privatemsg']['read_all'] = array(
+    $form['privatemsg']['read_all'] = array(
     '#type'  => 'value',
     '#value' => $read_all,
-  );
-  return $form;
+    );
+    return $form;
 }
 
 function pm_send_validate($form, &$form_state) {
-  // The actual message that is being sent, we create this during validation and pass to submit to send out.
-  $message = $form_state['values'];
-  $message['timestamp'] = time();
-  // Avoid subjects which only consist of a space as these can not be clicked.
-  $message['subject'] = trim($message['subject']);
+    // The actual message that is being sent, we create this during validation and pass to submit to send out.
+    $message = $form_state['values'];
+    $message['timestamp'] = time();
+    // Avoid subjects which only consist of a space as these can not be clicked.
+    $message['subject'] = trim($message['subject']);
 
 
-  $trimed_body = trim(truncate_utf8(strip_tags($message['body']), 50, TRUE, TRUE));
-  if (empty($message['subject']) && !empty($trimed_body)) {
+    $trimed_body = trim(truncate_utf8(strip_tags($message['body']), 50, TRUE, TRUE));
+    if (empty($message['subject']) && !empty($trimed_body)) {
     $message['subject'] = $trimed_body;
-  }
-  // Only parse the user string for a new thread.
-  if (!isset($message['thread_id'])) {
+    }
+    // Only parse the user string for a new thread.
+    if (!isset($message['thread_id'])) {
     list($message['recipients'], $invalid) = _privatemsg_parse_userstring($message['recipient']);
-  }
-  else {
+    }
+    else {
     // Load participants.
     $message['recipients'] = _privatemsg_load_thread_participants($message['thread_id']);
     // Remove author.
     if (isset($message['recipients'][$message['author']->uid]) && count($message['recipients']) > 1) {
-      unset($message['recipients'][$message['author']->uid]);
+        unset($message['recipients'][$message['author']->uid]);
+    }
     }
-  }
 
-  $validated = _privatemsg_validate_message($message, TRUE);
-  foreach ($validated['messages'] as $type => $text) {
+    $validated = _privatemsg_validate_message($message, TRUE);
+    foreach ($validated['messages'] as $type => $text) {
     drupal_set_message($text, $type);
-  }
-  $form_state['validate_built_message'] = $message;
-  if (!empty($invalid)) {
+    }
+    $form_state['validate_built_message'] = $message;
+    if (!empty($invalid)) {
     drupal_set_message(t('The following users will not receive this private message: @invalid', array('@invalid' => implode(", ", $invalid))), 'error');
-  }
+    }
 }
 
 /**
@@ -1061,15 +1060,15 @@ 
  *   Thread ID for wich the participants should be loaded.
  */
 function _privatemsg_load_thread_participants($thread_id) {
-  $query = _privatemsg_assemble_query('participants', $thread_id);
-  $result = db_query($query['query']);
-  $participants = array();
-  while ($uid = db_fetch_object($result)) {
+    $query = _privatemsg_assemble_query('participants', $thread_id);
+    $result = db_query($query['query']);
+    $participants = array();
+    while ($uid = db_fetch_object($result)) {
     if (($recipient = user_load($uid->uid))) {
-      $participants[$recipient->uid] = $recipient;
+        $participants[$recipient->uid] = $recipient;
+    }
     }
-  }
-  return $participants;
+    return $participants;
 }
 
 /**
@@ -1085,68 +1084,68 @@ 
  *   with invalid names.
  */
 function _privatemsg_parse_userstring($input) {
-  if (is_string($input)) {
+    if (is_string($input)) {
     $input = explode(',', $input);
-  }
+    }
 
-  // Start working through the input array.
-  $invalid = array();
-  $recipients = array();
-  foreach ($input as $string) {
+    // Start working through the input array.
+    $invalid = array();
+    $recipients = array();
+    foreach ($input as $string) {
     $string = trim($string);
     if (!empty($string)) { // We don't care about white space names.
 
-      // First, check if another module is able to resolve the string into an
-      // user object.
-      foreach (module_implements('privatemsg_name_lookup') as $module) {
+        // First, check if another module is able to resolve the string into an
+        // user object.
+        foreach (module_implements('privatemsg_name_lookup') as $module) {
         $function = $module . '_privatemsg_name_lookup';
         if (($recipient = $function($string)) && is_object($recipient)) {
-          // If there is a match, continue with the next input string.
-          $recipients[$recipient->uid] = $recipient;
-          continue 2;
+            // If there is a match, continue with the next input string.
+            $recipients[$recipient->uid] = $recipient;
+            continue 2;
         }
-      }
-      // Fall back to the default username lookup.
-      if (!$error = module_invoke('user', 'validate_name', $string)) {
+        }
+        // Fall back to the default username lookup.
+        if (!$error = module_invoke('user', 'validate_name', $string)) {
         // String is a valid username, look it up.
         if ($recipient = user_load(array('name' => $string))) {
-          $recipients[$recipient->uid] = $recipient;
-          continue;
+            $recipients[$recipient->uid] = $recipient;
+            continue;
+        }
         }
-      }
-      $invalid[$string] = $string;
+        $invalid[$string] = $string;
+    }
     }
-  }
 
-  return array($recipients, $invalid);
+    return array($recipients, $invalid);
 }
 
 /**
  * Submit callback for the privatemsg_new form.
  */
 function pm_send($form, &$form_state) {
-  $status = _privatemsg_send($form_state['validate_built_message']);
-  // Load usernames to which the message was sent to.
-  $recipient_names = array();
-  foreach ($form_state['validate_built_message']['recipients'] as $recipient) {
+    $status = _privatemsg_send($form_state['validate_built_message']);
+    // Load usernames to which the message was sent to.
+    $recipient_names = array();
+    foreach ($form_state['validate_built_message']['recipients'] as $recipient) {
     $recipient_names[] = theme('username', $recipient);
-  }
-  if ($status !== FALSE )  {
+    }
+    if ($status !== FALSE )  {
     drupal_set_message(t('A message has been sent to !recipients.', array('!recipients' => implode(', ', $recipient_names))));
-  }
-  else {
+    }
+    else {
     drupal_set_message(t('An attempt to send a message <em>may have failed</em> when sending to !recipients.', array('!recipients' => implode(', ', $recipient_names))), 'error');
-  }
+    }
 }
 
 function pm_preview($form, &$form_state) {
 
     drupal_validate_form($form['form_id']['#value'], $form, $form_state);
     if (!form_get_errors()) {
-      $form_state['privatemsg_preview'] = theme('privatemsg_view', $form_state['validate_built_message']);
+        $form_state['privatemsg_preview'] = theme('privatemsg_view', $form_state['validate_built_message']);
     }
 
-  $form_state['rebuild'] = TRUE; // this forces our form to be rebuilt instead of being submitted.
+    $form_state['rebuild'] = TRUE; // this forces our form to be rebuilt instead of being submitted.
 }
 
 /**
@@ -1167,65 +1166,65 @@ 
  */
 
 function privatemsg_sql_list(&$fragments, $account, $argument = 'list') {
-  $fragments['primary_table'] = '{pm_message} pm';
-
-  // Load enabled columns.
-  $fields = array_filter(variable_get('privatemsg_display_fields', array('participants')));
-
-  // Required columns.
-  $fragments['select'][]      = 'pmi.thread_id';
-  // We have to use MIN as the subject might not be the same in some threads.
-  // MIN() does not have a useful meaning except that it helps to correctly
-  // aggregate the thread on PostgreSQL.
-  $fragments['select'][]      = 'MIN(pm.subject) as subject';
-  $fragments['select'][]      = 'MAX(pm.timestamp) as last_updated';
-  // We use SUM so that we can count the number of unread messages.
-  $fragments['select'][]      = 'SUM(pmi.is_new) as is_new';
-
-  // Select number of messages in the thread if the count is
-  // set to be displayed.
-  if (in_array('count', $fields)) {
+    $fragments['primary_table'] = '{pm_message} pm';
+
+    // Load enabled columns.
+    $fields = array_filter(variable_get('privatemsg_display_fields', array('participants')));
+
+    // Required columns.
+    $fragments['select'][]      = 'pmi.thread_id';
+    // We have to use MIN as the subject might not be the same in some threads.
+    // MIN() does not have a useful meaning except that it helps to correctly
+    // aggregate the thread on PostgreSQL.
+    $fragments['select'][]      = 'MIN(pm.subject) as subject';
+    $fragments['select'][]      = 'MAX(pm.timestamp) as last_updated';
+    // We use SUM so that we can count the number of unread messages.
+    $fragments['select'][]      = 'SUM(pmi.is_new) as is_new';
+
+    // Select number of messages in the thread if the count is
+    // set to be displayed.
+    if (in_array('count', $fields)) {
     $fragments['select'][]      = 'COUNT(distinct pmi.mid) as count';
-  }
-  if (in_array('participants', $fields)) {
+    }
+    if (in_array('participants', $fields)) {
     // Query for a string with uid's, for example "1,6,7".
     // @todo: Replace this with a single query similiar to the tag list.
     if ($GLOBALS['db_type'] == 'pgsql') {
-      // PostgreSQL does not know GROUP_CONCAT, so a subquery is required.
-      $fragments['select'][]      = "array_to_string(array(SELECT DISTINCT textin(int4out(pmia.uid))
+        // PostgreSQL does not know GROUP_CONCAT, so a subquery is required.
+        $fragments['select'][]      = "array_to_string(array(SELECT DISTINCT textin(int4out(pmia.uid))
                                                             FROM {pm_index} pmia
                                                             WHERE pmia.thread_id = pmi.thread_id), ',') AS participants";
     }
     else {
-      $fragments['select'][]      = '(SELECT GROUP_CONCAT(DISTINCT pmia.uid SEPARATOR ",")
+        $fragments['select'][]      = '(SELECT GROUP_CONCAT(DISTINCT pmia.uid SEPARATOR ",")
                                                             FROM {pm_index} pmia
                                                             WHERE pmia.thread_id = pmi.thread_id) AS participants';
     }
-  }
-  if (in_array('thread_started', $fields)) {
+    }
+    if (in_array('thread_started', $fields)) {
     $fragments['select'][]      = 'MIN(pm.timestamp) as thread_started';
-  }
+    }
 
-  $fragments['inner_join'][]  = 'INNER JOIN {pm_index} pmi ON pm.mid = pmi.mid';
+    $fragments['inner_join'][]  = 'INNER JOIN {pm_index} pmi ON pm.mid = pmi.mid';
 
-  // Only load undeleted messages of the current user and group by thread.
-  $fragments['where'][]       = 'pmi.uid = %d';
-  $fragments['query_args']['where'][]  = $account->uid;
-  $fragments['where'][]       = 'pmi.deleted = 0';
-  $fragments['group_by'][]    = 'pmi.thread_id';
+    // Only load undeleted messages of the current user and group by thread.
+    $fragments['where'][]       = 'pmi.uid = %d';
+    $fragments['query_args']['where'][]  = $account->uid;
+    $fragments['where'][]       = 'pmi.deleted = 0';
+    $fragments['group_by'][]    = 'pmi.thread_id';
 
-  $order_by_first = 'MAX(pmi.is_new) DESC, ';
-  // MySQL 4.1 does not allow to order by aggregate functions. MAX() is used
-  // to avoid a ordering bug with multiple new messages.
-  if ($GLOBALS['db_type'] != 'pgsql' && version_compare(db_version(), '5.0.0') < 0) {
+    $order_by_first = 'MAX(pmi.is_new) DESC, ';
+    // MySQL 4.1 does not allow to order by aggregate functions. MAX() is used
+    // to avoid a ordering bug with multiple new messages.
+    if ($GLOBALS['db_type'] != 'pgsql' && version_compare(db_version(), '5.0.0') < 0) {
     $order_by_first = 'is_new DESC, ';
-  }
+    }
 
-  // tablesort_sql() generates a ORDER BY string. However, the "ORDER BY " part 
-  // is not needed and added by the query builder. Discard the first 9
-  // characters of the string.
-  $order_by = drupal_substr(tablesort_sql(_privatemsg_list_headers( FALSE, array_merge(array('subject', 'last_updated'), $fields)), $order_by_first), 9);
-  $fragments['order_by'][]  = $order_by;
+    // tablesort_sql() generates a ORDER BY string. However, the "ORDER BY " part 
+    // is not needed and added by the query builder. Discard the first 9
+    // characters of the string.
+    $order_by = drupal_substr(tablesort_sql(_privatemsg_list_headers( FALSE, array_merge(array('subject', 'last_updated'), $fields)), $order_by_first), 9);
+    $fragments['order_by'][]  = $order_by;
 }
 
 /**
@@ -1239,27 +1238,27 @@ 
  *   Account for which the messages should be loaded.
  */
 function privatemsg_sql_load(&$fragments, $pmids, $account = NULL) {
-  $fragments['primary_table'] = '{pm_message} pm';
-
-  $fragments['select'][]      = "pm.mid";
-  $fragments['select'][]      = "pm.author";
-  $fragments['select'][]      = "pm.subject";
-  $fragments['select'][]      = "pm.body";
-  $fragments['select'][]      = "pm.timestamp";
-  $fragments['select'][]      = "pm.format";
-  $fragments['select'][]      = "pmi.is_new";
-  $fragments['select'][]      = "pmi.thread_id";
-
-  $fragments['inner_join'][]  = 'INNER JOIN {pm_index} pmi ON pm.mid = pmi.mid';
-  // Use IN() to load multiple messages at the same time.
-  $fragments['where'][]       = 'pmi.mid IN (' . db_placeholders($pmids) . ')';
-  $fragments['query_args']['where']  += $pmids;
-  if ($account) {
+    $fragments['primary_table'] = '{pm_message} pm';
+
+    $fragments['select'][]      = "pm.mid";
+    $fragments['select'][]      = "pm.author";
+    $fragments['select'][]      = "pm.subject";
+    $fragments['select'][]      = "pm.body";
+    $fragments['select'][]      = "pm.timestamp";
+    $fragments['select'][]      = "pm.format";
+    $fragments['select'][]      = "pmi.is_new";
+    $fragments['select'][]      = "pmi.thread_id";
+
+    $fragments['inner_join'][]  = 'INNER JOIN {pm_index} pmi ON pm.mid = pmi.mid';
+    // Use IN() to load multiple messages at the same time.
+    $fragments['where'][]       = 'pmi.mid IN (' . db_placeholders($pmids) . ')';
+    $fragments['query_args']['where']  += $pmids;
+    if ($account) {
     $fragments['where'][]       = 'pmi.uid = %d';
     $fragments['query_args']['where'][]  = $account->uid;
-  }
-  $fragments['order_by'][] = 'pm.timestamp ASC';
-  $fragments['order_by'][] = 'pm.mid ASC';
+    }
+    $fragments['order_by'][] = 'pm.timestamp ASC';
+    $fragments['order_by'][] = 'pm.mid ASC';
 }
 
 /**
@@ -1275,30 +1274,30 @@ 
  *   Deleted messages are only loaded if this is set to TRUE.
  */
 function privatemsg_sql_messages(&$fragments, $threads, $account = NULL, $load_all = FALSE) {
-  $fragments['primary_table'] = '{pm_index} pmi';
+    $fragments['primary_table'] = '{pm_index} pmi';
 
-  $fragments['select'][]      = 'pmi.mid';
-  $fragments['where'][]       = 'pmi.thread_id IN ('. db_placeholders($threads) .')';
-  $fragments['query_args']['where']   += $threads;
-  $fragments['inner_join'][]  = 'INNER JOIN {pm_message} pm ON (pm.mid = pmi.mid)';
-  if ($account) {
+    $fragments['select'][]      = 'pmi.mid';
+    $fragments['where'][]       = 'pmi.thread_id IN ('. db_placeholders($threads) .')';
+    $fragments['query_args']['where']   += $threads;
+    $fragments['inner_join'][]  = 'INNER JOIN {pm_message} pm ON (pm.mid = pmi.mid)';
+    if ($account) {
     // Only load the user's messages.
     $fragments['where'][]     = 'pmi.uid = %d';
     $fragments['query_args']['where'][]  = $account->uid;
-  }
-  if (!$load_all) {
+    }
+    if (!$load_all) {
     // Also load deleted messages when requested.
     $fragments['where'][]     = 'pmi.deleted = 0';
-  }
-  // Only load each mid once.
-  $fragments['group_by'][]    = 'pmi.mid';
-  $fragments['group_by'][]    = 'pm.timestamp';
+    }
+    // Only load each mid once.
+    $fragments['group_by'][]    = 'pmi.mid';
+    $fragments['group_by'][]    = 'pm.timestamp';
 
-  // Order by timestamp first.
-  $fragments['order_by'][]    = 'pm.timestamp ASC';
-  // If there are multiple inserts during the same second (tests, for example)
-  // sort by mid second to have them in the same order as they were saved.
-  $fragments['order_by'][]    = 'pmi.mid ASC';
+    // Order by timestamp first.
+    $fragments['order_by'][]    = 'pm.timestamp ASC';
+    // If there are multiple inserts during the same second (tests, for example)
+    // sort by mid second to have them in the same order as they were saved.
+    $fragments['order_by'][]    = 'pmi.mid ASC';
 
 }
 
@@ -1311,16 +1310,16 @@ 
  *   Thread id from which the participants should be loaded.
  */
 function privatemsg_sql_participants(&$fragments, $thread_id) {
-  $fragments['primary_table'] = '{pm_index} pmi';
+    $fragments['primary_table'] = '{pm_index} pmi';
 
-  // Only load each participant once since they are listed as recipient for
-  // every message of that thread.
-  $fragments['select'][]      = 'DISTINCT(pmi.uid) AS uid';
-  $fragments['select'][]      = 'u.name AS name';
+    // Only load each participant once since they are listed as recipient for
+    // every message of that thread.
+    $fragments['select'][]      = 'DISTINCT(pmi.uid) AS uid';
+    $fragments['select'][]      = 'u.name AS name';
 
-  $fragments['inner_join'][]  = 'INNER JOIN {users} u ON (u.uid = pmi.uid)';
-  $fragments['where'][]       = 'pmi.thread_id = %d';
-  $fragments['query_args']['where'][]  = $thread_id;
+    $fragments['inner_join'][]  = 'INNER JOIN {users} u ON (u.uid = pmi.uid)';
+    $fragments['where'][]       = 'pmi.thread_id = %d';
+    $fragments['query_args']['where'][]  = $thread_id;
 }
 
 /**
@@ -1332,15 +1331,15 @@ 
  *   User object for which the messages are being counted.
  */
 function privatemsg_sql_unread_count(&$fragments, $account) {
-  $fragments['primary_table'] = '{pm_index} pmi';
+    $fragments['primary_table'] = '{pm_index} pmi';
 
-  $fragments['select'][]      = 'COUNT(DISTINCT thread_id) as unread_count';
+    $fragments['select'][]      = 'COUNT(DISTINCT thread_id) as unread_count';
 
-  // Only count new messages that have not been deleted.
-  $fragments['where'][]       = 'pmi.deleted = 0';
-  $fragments['where'][]       = 'pmi.is_new = 1';
-  $fragments['where'][]       = 'pmi.uid = %d';
-  $fragments['query_args']['where'][]  = $account->uid;
+    // Only count new messages that have not been deleted.
+    $fragments['where'][]       = 'pmi.deleted = 0';
+    $fragments['where'][]       = 'pmi.is_new = 1';
+    $fragments['where'][]       = 'pmi.uid = %d';
+    $fragments['query_args']['where'][]  = $account->uid;
 }
 
 /**
@@ -1354,19 +1353,19 @@ 
  *   Array of names not to be used as suggestions.
  */
 function privatemsg_sql_autocomplete(&$fragments, $search, $names) {
-  $fragments['primary_table'] = '{users} u';
-  $fragments['select'][] = 'u.name';
-  // Escape the % to get it through the placeholder replacement.
-  $fragments['where'][] = "u.name LIKE '%s'";
-  $fragments['query_args']['where'][] = $search .'%%';
-  if (!empty($names)) {
+    $fragments['primary_table'] = '{users} u';
+    $fragments['select'][] = 'u.name';
+    // Escape the % to get it through the placeholder replacement.
+    $fragments['where'][] = "u.name LIKE '%s'";
+    $fragments['query_args']['where'][] = $search .'%%';
+    if (!empty($names)) {
     // If there are already names selected, exclude them from the suggestions.
     $fragments['where'][] = "u.name NOT IN (". db_placeholders($names, 'text') .")";
     $fragments['query_args']['where'] += $names;
-  }
-  // Only load active users and sort them by name.
-  $fragments['where'][] = 'u.status <> 0';
-  $fragments['order_by'][] = 'u.name ASC';
+    }
+    // Only load active users and sort them by name.
+    $fragments['where'][] = 'u.status <> 0';
+    $fragments['order_by'][] = 'u.name ASC';
 }
 
 /**
@@ -1378,24 +1377,24 @@ 
  *   Select messages older than x days.
  */
 function privatemsg_sql_deleted(&$fragments, $days) {
-  $fragments['primary_table'] = '{pm_message} pm';
+    $fragments['primary_table'] = '{pm_message} pm';
 
-  $fragments['select'][] = 'pm.mid';
-  // The lowest value is higher than 0 if all recipients have deleted a message.
-  $fragments['select'][] = 'MIN(pmi.deleted) as is_deleted';
-  // The time the most recent deletion happened.
-  $fragments['select'][] = 'MAX(pmi.deleted) as last_deleted';
+    $fragments['select'][] = 'pm.mid';
+    // The lowest value is higher than 0 if all recipients have deleted a message.
+    $fragments['select'][] = 'MIN(pmi.deleted) as is_deleted';
+    // The time the most recent deletion happened.
+    $fragments['select'][] = 'MAX(pmi.deleted) as last_deleted';
 
-  $fragments['inner_join'][] = 'INNER JOIN {pm_index} pmi ON (pmi.mid = pm.mid)';
+    $fragments['inner_join'][] = 'INNER JOIN {pm_index} pmi ON (pmi.mid = pm.mid)';
 
-  $fragments['group_by'][] = 'pm.mid';
+    $fragments['group_by'][] = 'pm.mid';
 
-  // Ignore messages that have not been deleted by all users.
-  $fragments['having'][] = 'MIN(pmi.deleted) > 0';
+    // Ignore messages that have not been deleted by all users.
+    $fragments['having'][] = 'MIN(pmi.deleted) > 0';
 
-  // Only select messages that have been deleted more than n days ago.
-  $fragments['having'][] = 'MAX(pmi.deleted) < %d';
-  $fragments['query_args']['having'][] = time() - $days * 86400;
+    // Only select messages that have been deleted more than n days ago.
+    $fragments['having'][] = 'MAX(pmi.deleted) < %d';
+    $fragments['query_args']['having'][] = time() - $days * 86400;
 }
 
 /**
@@ -1408,192 +1407,192 @@ 
  * Prevents usernames from being used and/or suggested twice.
  */
 function privatemsg_user_name_autocomplete($string) {
-  $names = array();
-  // 1: Parse $string and build list of valid user names.
-  $fragments = explode(',', $string);
-  foreach ($fragments as $index => $name) {
+    $names = array();
+    // 1: Parse $string and build list of valid user names.
+    $fragments = explode(',', $string);
+    foreach ($fragments as $index => $name) {
     if ($name = trim($name)) {
-      $names[$name] = $name;
-    }
-  }
-  // By using user_validate_user we can ensure that names included in $names are at least logisticaly possible.
-  // 2: Find the next user name suggestion.
-  $fragment = array_pop($names);
-  $matches = array();
-  if (!empty($fragment)) {
+        $names[$name] = $name;
+    }
+    }
+    // By using user_validate_user we can ensure that names included in $names are at least logisticaly possible.
+    // 2: Find the next user name suggestion.
+    $fragment = array_pop($names);
+    $matches = array();
+    if (!empty($fragment)) {
     $query = _privatemsg_assemble_query('autocomplete', $fragment, $names);
     $result = db_query_range($query['query'], $fragment, 0, 10);
     $prefix = count($names) ? implode(", ", $names) .", " : '';
     // 3: Build proper suggestions and print.
     while ($user = db_fetch_object($result)) {
-      $matches[$prefix . $user->name .", "] = $user->name;
+        $matches[$prefix . $user->name .", "] = $user->name;
     }
-  }
-  // convert to object to prevent drupal bug, see http://drupal.org/node/175361
-  drupal_json((object)$matches);
+    }
+    // convert to object to prevent drupal bug, see http://drupal.org/node/175361
+    drupal_json((object)$matches);
 }
 
 function privatemsg_user($op, &$edit, &$account, $category = NULL) {
-  global $user;
+    global $user;
 
-  switch ($op) {
+    switch ($op) {
     case 'view':
       if ($url = privatemsg_get_link(array($account))) {
         $account->content['privatemsg_send_new_message'] = array(
-          '#type'   => 'markup',
-          '#value'  => l(t('Send this user a message'), $url, array('query' => drupal_get_destination())),
-          '#weight' => 10,
+            '#type'   => 'markup',
+            '#value'  => l(t('Send this user a message'), $url, array('query' => drupal_get_destination())),
+            '#weight' => 10,
         );
-      }
-      break;
+        }
+        break;
     case 'login':
       if (variable_get('privatemsg_display_loginmessage', TRUE) && privatemsg_user_access()) {
         $count = privatemsg_unread_count();
         if ($count) {
-          drupal_set_message(format_plural($count, 'You have <a href="@messages">1 unread message</a>.', 'You have <a href="@messages">@count unread messages</a>', array('@messages' => url('messages'))));
+            drupal_set_message(format_plural($count, 'You have <a href="@messages">1 unread message</a>.', 'You have <a href="@messages">@count unread messages</a>', array('@messages' => url('messages'))));
         }
-      }
-      break;
+        }
+        break;
     case 'delete':
 
       // Load all mids of the messages the user wrote.
       $result = db_query("SELECT mid FROM {pm_message} WHERE author = %d", $account->uid);
-      $mids = array();
-      while ($row = db_fetch_array($result)) {
+        $mids = array();
+        while ($row = db_fetch_array($result)) {
         $mids[] = $row['mid'];
-      }
+        }
 
-      // Delete messages the user wrote.
-      db_query('DELETE FROM {pm_message} WHERE author = %d', $account->uid);
+        // Delete messages the user wrote.
+        db_query('DELETE FROM {pm_message} WHERE author = %d', $account->uid);
 
-      if (!empty($mids)) {
+        if (!empty($mids)) {
         // Delete recipient entries in {pm_index} of the messages the user wrote.
         db_query('DELETE FROM {pm_index} WHERE mid IN (' . db_placeholders($mids) . ')', $mids);
-      }
+        }
 
-      // Delete recipient entries of that user.
-      db_query('DELETE FROM {pm_index} WHERE uid = %d', $account->uid);
-      break;
-  }
+        // Delete recipient entries of that user.
+        db_query('DELETE FROM {pm_index} WHERE uid = %d', $account->uid);
+        break;
+    }
 }
 
 function privatemsg_block($op = 'list', $delta = 0, $edit = array()) {
-  if ('list' == $op) {
+    if ('list' == $op) {
     $blocks = array();
     $blocks['privatemsg-menu'] = array(
-      'info' => t('Privatemsg links'),
-      'cache' => BLOCK_NO_CACHE,
+        'info' => t('Privatemsg links'),
+        'cache' => BLOCK_NO_CACHE,
     );
     $blocks['privatemsg-new'] = array(
-      'info' => t('New message indication'),
-      'cache' => BLOCK_NO_CACHE,
+        'info' => t('New message indication'),
+        'cache' => BLOCK_NO_CACHE,
     );
 
     return $blocks;
-  }
-  elseif ('view' == $op) {
+    }
+    elseif ('view' == $op) {
     $block = array();
     switch ($delta) {
-      case 'privatemsg-menu':
+        case 'privatemsg-menu':
         $block = _privatemsg_block_menu();
         break;
-      case 'privatemsg-new':
+        case 'privatemsg-new':
         $block = _privatemsg_block_new();
         break;
     }
     return $block;
-  }
+    }
 }
 
 function privatemsg_title_callback($title = NULL) {
-  $count = privatemsg_unread_count();
+    $count = privatemsg_unread_count();
 
-  if ($count > 0) {
+    if ($count > 0) {
     return format_plural($count, 'Messages (1 new)', 'Messages (@count new)');
-  }
-  return t('Messages');
+    }
+    return t('Messages');
 }
 
 
 function _privatemsg_block_new() {
-  $block = array();
+    $block = array();
 
-  if (!privatemsg_user_access()) {
+    if (!privatemsg_user_access()) {
     return $block;
-  }
+    }
 
-  $count = privatemsg_unread_count();
-  if ($count) {
+    $count = privatemsg_unread_count();
+    if ($count) {
     $block = array(
-      'subject' => format_plural($count, 'New message', 'New messages'),
-      'content' => theme('privatemsg_new_block', $count),
+        'subject' => format_plural($count, 'New message', 'New messages'),
+        'content' => theme('privatemsg_new_block', $count),
     );
     return $block;
-  }
-  return array();
+    }
+    return array();
 }
 
 function _privatemsg_block_menu() {
-  $block = array();
+    $block = array();
 
-  $links = array();
-  if (privatemsg_user_access('write privatemsg')) {
+    $links = array();
+    if (privatemsg_user_access('write privatemsg')) {
     $links[] = l(t('Write new message'), 'messages/new');
-  }
-  if (privatemsg_user_access('read privatemsg') || privatemsg_user_access('read all private messages') ) {
+    }
+    if (privatemsg_user_access('read privatemsg') || privatemsg_user_access('read all private messages') ) {
     $links[] = l(privatemsg_title_callback(), 'messages');
-  }
-  if ( count( $links ) ) {
+    }
+    if ( count( $links ) ) {
     $block = array(
-      'subject' => t('Private messages'),
-      'content' => theme('item_list', $links),
+        'subject' => t('Private messages'),
+        'content' => theme('item_list', $links),
     );
-  }
-  return $block;
+    }
+    return $block;
 }
 
 function privatemsg_delete($form_state, $thread, $message) {
-  $form['pmid'] = array(
+    $form['pmid'] = array(
     '#type' => 'value',
     '#value' => $message['mid'],
-  );
-  $form['delete_destination'] = array(
+    );
+    $form['delete_destination'] = array(
     '#type' => 'value',
     '#value' => count($thread['messages']) > 1 ? 'messages/view/' . $message['thread_id'] : 'messages',
-  );
+    );
 
-  if (privatemsg_user_access('read all private messages')) {
+    if (privatemsg_user_access('read all private messages')) {
     $form['delete_options'] = array(
-      '#type' => 'checkbox',
-      '#title' => t('Delete this message for all users?'),
-      '#description' => t('Tick the box to delete the message for all users.'),
-      '#default_value' => FALSE,
+        '#type' => 'checkbox',
+        '#title' => t('Delete this message for all users?'),
+        '#description' => t('Tick the box to delete the message for all users.'),
+        '#default_value' => FALSE,
     );
-  }
-  return confirm_form($form,
+    }
+    return confirm_form($form,
     t('Are you sure you want to delete this message?'),
     isset($_GET['destination']) ? $_GET['destination'] : 'messages/view/'. $message['thread_id'],
     t('This action cannot be undone.'),
     t('Delete'),
     t('Cancel')
-  );
+    );
 }
 
 function privatemsg_delete_submit($form, &$form_state) {
-  global $user;
-  $account = drupal_clone($user);
+    global $user;
+    $account = drupal_clone($user);
 
-  if ($form_state['values']['confirm']) {
+    if ($form_state['values']['confirm']) {
     if (isset($form_state['values']['delete_options']) && $form_state['values']['delete_options']) {
-      privatemsg_message_change_delete($form_state['values']['pmid'], 1);
-      drupal_set_message(t('Message has been deleted for all users.'));
+        privatemsg_message_change_delete($form_state['values']['pmid'], 1);
+        drupal_set_message(t('Message has been deleted for all users.'));
     }
     else {
-      privatemsg_message_change_delete($form_state['values']['pmid'], 1, $account);
-      drupal_set_message(t('Message has been deleted.'));
+        privatemsg_message_change_delete($form_state['values']['pmid'], 1, $account);
+        drupal_set_message(t('Message has been deleted.'));
     }
-  }
-  $form_state['redirect'] = $form_state['values']['delete_destination'];
+    }
+    $form_state['redirect'] = $form_state['values']['delete_destination'];
 }
 
 /**
@@ -1610,18 +1609,18 @@ 
  * @ingroup api
  */
 function privatemsg_message_change_delete($pmid, $delete, $account = NULL) {
-  $delete_value = 0;
-  if ($delete == TRUE) {
+    $delete_value = 0;
+    if ($delete == TRUE) {
     $delete_value = time();
-  }
+    }
 
-  if ($account) {
+    if ($account) {
     db_query('UPDATE {pm_index} SET deleted = %d WHERE mid = %d AND uid = %d', $delete_value, $pmid, $account->uid);
-  }
-  else {
+    }
+    else {
     // Mark deleted for all users.
     db_query('UPDATE {pm_index} SET deleted = %d WHERE mid = %d', $delete_value, $pmid);
-  }
+    }
 }
 
 /**
@@ -1664,35 +1663,35 @@ 
  * @ingroup api
  */
 function privatemsg_new_thread($recipients, $subject, $body = NULL, $options = array()) {
-  global $user;
-  $author = drupal_clone($user);
-
-  $message = array();
-  $message['subject'] = $subject;
-  $message['body'] = $body;
-  // Make sure that recipients are keyed by user id and are not added
-  // multiple times.
-  foreach ($recipients as $recipient) {
+    global $user;
+    $author = drupal_clone($user);
+
+    $message = array();
+    $message['subject'] = $subject;
+    $message['body'] = $body;
+    // Make sure that recipients are keyed by user id and are not added
+    // multiple times.
+    foreach ($recipients as $recipient) {
     $message['recipients'][$recipient->uid] = $recipient;
-  }
+    }
 
-  // Set custom options, if any.
-  if (!empty($options)) {
+    // Set custom options, if any.
+    if (!empty($options)) {
     $message += $options;
-  }
-  // Apply defaults - this will not overwrite existing keys.
-  $message += array(
+    }
+    // Apply defaults - this will not overwrite existing keys.
+    $message += array(
     'author' => $author,
     'timestamp' => time(),
     'format' => filter_resolve_format(FILTER_FORMAT_DEFAULT),
-  );
+    );
 
-  $validated = _privatemsg_validate_message($message);
-  if ($validated['success']) {
+    $validated = _privatemsg_validate_message($message);
+    if ($validated['success']) {
     $validated['message'] = _privatemsg_send($message);
-  }
+    }
 
-  return $validated;
+    return $validated;
 }
 
 /**
@@ -1729,144 +1728,144 @@ 
  * @ingroup api
  */
 function privatemsg_reply($thread_id, $body, $options = array()) {
-  global $user;
-  $author = drupal_clone($user);
+    global $user;
+    $author = drupal_clone($user);
 
-  $message = array();
-  $message['body'] = $body;
+    $message = array();
+    $message['body'] = $body;
 
-  // set custom options, if any
-  if (!empty($options)) {
+    // set custom options, if any
+    if (!empty($options)) {
     $message += $options;
-  }
-  // apply defaults
-  $message += array(
+    }
+    // apply defaults
+    $message += array(
     'author' => $author,
     'timestamp' => time(),
     'format' => filter_resolve_format(FILTER_FORMAT_DEFAULT),
-  );
+    );
 
-  // We don't know the subject and the recipients, so we need to load them..
-  // thread_id == mid on the first message of the thread
-  $first_message = privatemsg_message_load($thread_id, $message['author']);
-  if (!$first_message) {
+    // We don't know the subject and the recipients, so we need to load them..
+    // thread_id == mid on the first message of the thread
+    $first_message = privatemsg_message_load($thread_id, $message['author']);
+    if (!$first_message) {
     return array(t('Thread %thread_id not found, unable to answer', array('%thread_id' => $thread_id)));
-  }
+    }
 
-  $message['thread_id'] = $thread_id;
-  // Load participants.
-  $message['recipients'] = _privatemsg_load_thread_participants($thread_id);
-  // Remove author.
-  if (isset($message['recipients'][$message['author']->uid]) && count($message['recipients']) > 1) {
+    $message['thread_id'] = $thread_id;
+    // Load participants.
+    $message['recipients'] = _privatemsg_load_thread_participants($thread_id);
+    // Remove author.
+    if (isset($message['recipients'][$message['author']->uid]) && count($message['recipients']) > 1) {
     unset($message['recipients'][$message['author']->uid]);
-  }
-  $message['subject'] = $first_message['subject'];
+    }
+    $message['subject'] = $first_message['subject'];
 
-  $validated = _privatemsg_validate_message($message);
-  if ($validated['success']) {
+    $validated = _privatemsg_validate_message($message);
+    if ($validated['success']) {
     $validated['message'] = _privatemsg_send($message);
-  }
-  return $validated;
+    }
+    return $validated;
 }
 
 function _privatemsg_validate_message(&$message, $form = FALSE) {
-  $messages = array('error' => array(), 'warning' => array());
-  if (!privatemsg_user_access('write privatemsg', $message['author'])) {
+    $messages = array('error' => array(), 'warning' => array());
+    if (!privatemsg_user_access('write privatemsg', $message['author'])) {
     // no need to do further checks in this case...
     if ($form) {
-      form_set_error('author', t('User @user is not allowed to write messages', array('@user' => $message['author']->name)));
-      return array(
+        form_set_error('author', t('User @user is not allowed to write messages', array('@user' => $message['author']->name)));
+        return array(
         'success'  => FALSE,
         'messages'   => $messages,
-      );
+        );
     }
     else {
-      $messages['error'][] = t('User @user is not allowed to write messages', array('@user' => $message['author']->name));
-      return array(
-         'success'  => FALSE,
-         'messages'   => $messages,
-      );
+        $messages['error'][] = t('User @user is not allowed to write messages', array('@user' => $message['author']->name));
+        return array(
+            'success'  => FALSE,
+            'messages'   => $messages,
+        );
+    }
     }
-  }
 
-  // Prevent subjects which only consist of a space as these can not be clicked.
-  $message['subject'] = trim($message['subject']);
-  if (empty($message['subject'])) {
+    // Prevent subjects which only consist of a space as these can not be clicked.
+    $message['subject'] = trim($message['subject']);
+    if (empty($message['subject'])) {
     if ($form) {
-      form_set_error('subject', t('Disallowed to send a message without subject'));
+        form_set_error('subject', t('Disallowed to send a message without subject'));
     }
     else {
-      $messages['error'][] = t('Disallowed to send a message without subject');
+        $messages['error'][] = t('Disallowed to send a message without subject');
+    }
     }
-  }
 
-  // Don't allow replies without a body.
-  if (!empty($message['thread_id']) && ($message['body'] === NULL || $message['body'] === '') ) {
+    // Don't allow replies without a body.
+    if (!empty($message['thread_id']) && ($message['body'] === NULL || $message['body'] === '') ) {
     if ($form) {
-      form_set_error('body', t('Disallowed to send reply without a message.'));
+        form_set_error('body', t('Disallowed to send reply without a message.'));
     }
     else {
-      $messages['error'][] = t('Disallowed to send reply without a message.');
+        $messages['error'][] = t('Disallowed to send reply without a message.');
+    }
     }
-  }
-  // Check if an allowed format is used. global $user needs to be changed since
-  // it is not possible to do the check for a specific user.
-  global $user;
-  $original_user = drupal_clone($user);
-  session_save_session(FALSE);
-  $user = $message['author'];
+    // Check if an allowed format is used. global $user needs to be changed since
+    // it is not possible to do the check for a specific user.
+    global $user;
+    $original_user = drupal_clone($user);
+    session_save_session(FALSE);
+    $user = $message['author'];
 
-  if (!filter_access($message['format'])) {
+    if (!filter_access($message['format'])) {
     if ($form) {
-      form_set_error('format', t('You are not allowed to use the specified input format.'));
+        form_set_error('format', t('You are not allowed to use the specified input format.'));
     }
     else {
-      $messages['error'][] = t('User @user is not allowed to use the specified input format.', array('@user' => $message['author']->name));
+        $messages['error'][] = t('User @user is not allowed to use the specified input format.', array('@user' => $message['author']->name));
+    }
     }
-  }
 
-  $user = $original_user;
-  session_save_session(TRUE);
+    $user = $original_user;
+    session_save_session(TRUE);
 
-  if (empty($message['recipients']) || !is_array($message['recipients'])) {
+    if (empty($message['recipients']) || !is_array($message['recipients'])) {
     if ($form) {
-      form_set_error('to', t('Disallowed to send a message without at least one valid recipient'));
+        form_set_error('to', t('Disallowed to send a message without at least one valid recipient'));
     }
     else {
-      $messages['error'][] = t('Disallowed to send a message without at least one valid recipient');
+        $messages['error'][] = t('Disallowed to send a message without at least one valid recipient');
+    }
     }
-  }
 
-  if (!empty($message['recipients']) && is_array($message['recipients'])) {
+    if (!empty($message['recipients']) && is_array($message['recipients'])) {
     foreach (module_invoke_all('privatemsg_block_message', $message['author'], $message['recipients']) as $blocked) {
-      unset($message['recipients'][$blocked['uid']]);
-      if ($form) {
+        unset($message['recipients'][$blocked['uid']]);
+        if ($form) {
         drupal_set_message($blocked['message'], 'warning');
-      }
-      else {
+        }
+        else {
         $messages['warning'][] = $blocked['message'];
-      }
+        }
+    }
     }
-  }
 
-  // Check again, give another error message if all recipients are blocked
-  if (empty($message['recipients'])) {
+    // Check again, give another error message if all recipients are blocked
+    if (empty($message['recipients'])) {
     if ($form) {
-      form_set_error('to', t('Disallowed to send message because all recipients are blocked'));
+        form_set_error('to', t('Disallowed to send message because all recipients are blocked'));
     }
     else {
-      $messages['error'][] = t('Disallowed to send message because all recipients are blocked');
+        $messages['error'][] = t('Disallowed to send message because all recipients are blocked');
+    }
     }
-  }
 
-  $messages = array_merge_recursive(module_invoke_all('privatemsg_message_validate', $message, $form), $messages);
+    $messages = array_merge_recursive(module_invoke_all('privatemsg_message_validate', $message, $form), $messages);
 
-  // Check if there are errors in $messages or if $form is TRUE, there are form errors.
-  $success = empty($messages['error']) || ($form && count((array)form_get_errors()) > 0);
-  return array(
+    // Check if there are errors in $messages or if $form is TRUE, there are form errors.
+    $success = empty($messages['error']) || ($form && count((array)form_get_errors()) > 0);
+    return array(
     'success'  => $success,
     'messages'   => $messages,
-  );
+    );
 }
 
 /**
@@ -1882,61 +1881,61 @@ 
  */
 function _privatemsg_send($message) {
 
-  drupal_alter('privatemsg_message_presave', $message);
+    drupal_alter('privatemsg_message_presave', $message);
 
-  $index_sql = "INSERT INTO {pm_index} (mid, thread_id, uid, is_new, deleted) VALUES (%d, %d, %d, %d, 0)";
-  if (isset($message['read_all']) && $message['read_all']) {
+    $index_sql = "INSERT INTO {pm_index} (mid, thread_id, uid, is_new, deleted) VALUES (%d, %d, %d, %d, 0)";
+    if (isset($message['read_all']) && $message['read_all']) {
     // The message was sent in read all mode, add the author as recipient to all
     // existing messages.
     $query_messages = _privatemsg_assemble_query('messages', array($message['thread_id']), NULL);
     $conversation = db_query($query_messages['query']);
     while ($result = db_fetch_array($conversation)) {
-      if (!db_query($index_sql, $result['mid'], $message['thread_id'], $message['author']->uid, 0)) {
+        if (!db_query($index_sql, $result['mid'], $message['thread_id'], $message['author']->uid, 0)) {
         return FALSE;
-      }
-    }
-  }
-
-  // 1) Save the message body first.
-  $args = array();
-  $args[] = $message['subject'];
-  $args[] = $message['author']->uid;
-  $args[] = $message['body'];
-  $args[] = $message['format'];
-  $args[] = $message['timestamp'];
-  $message_sql = "INSERT INTO {pm_message} (subject, author, body, format, timestamp) VALUES ('%s', %d, '%s', %d, %d)";
-  db_query($message_sql, $args);
-  $mid = db_last_insert_id('pm_message', 'mid');
-  $message['mid'] = $mid;
-
-  // Thread ID is the same as the mid if it's the first message in the thread.
-  if (!isset($message['thread_id'])) {
+        }
+    }
+    }
+
+    // 1) Save the message body first.
+    $args = array();
+    $args[] = $message['subject'];
+    $args[] = $message['author']->uid;
+    $args[] = $message['body'];
+    $args[] = $message['format'];
+    $args[] = $message['timestamp'];
+    $message_sql = "INSERT INTO {pm_message} (subject, author, body, format, timestamp) VALUES ('%s', %d, '%s', %d, %d)";
+    db_query($message_sql, $args);
+    $mid = db_last_insert_id('pm_message', 'mid');
+    $message['mid'] = $mid;
+
+    // Thread ID is the same as the mid if it's the first message in the thread.
+    if (!isset($message['thread_id'])) {
     $message['thread_id'] = $mid;
-  }
+    }
 
-  // 2) Save message to recipients.
-  // Each recipient gets a record in the pm_index table.
-  foreach ($message['recipients'] as $recipient) {
+    // 2) Save message to recipients.
+    // Each recipient gets a record in the pm_index table.
+    foreach ($message['recipients'] as $recipient) {
     if (!db_query($index_sql, $mid, $message['thread_id'], $recipient->uid, 1) ) {
-      // We assume if one insert failed then the rest may fail too against the
-      // same table.
-      return FALSE;
+        // We assume if one insert failed then the rest may fail too against the
+        // same table.
+        return FALSE;
+    }
     }
-  }
 
-  // When author is also the recipient, we want to set message to UNREAD.
-  // All other times the message is set to READ.
-  $is_new = isset($message['recipients'][$message['author']->uid]) ? 1 : 0;
+    // When author is also the recipient, we want to set message to UNREAD.
+    // All other times the message is set to READ.
+    $is_new = isset($message['recipients'][$message['author']->uid]) ? 1 : 0;
 
-  // Also add a record for the author to the pm_index table.
-  if (!db_query($index_sql, $mid, $message['thread_id'], $message['author']->uid, $is_new)) {
+    // Also add a record for the author to the pm_index table.
+    if (!db_query($index_sql, $mid, $message['thread_id'], $message['author']->uid, $is_new)) {
     return FALSE;
-  }
+    }
 
-  module_invoke_all('privatemsg_message_insert', $message);
+    module_invoke_all('privatemsg_message_insert', $message);
 
-  // If we reached here that means we were successful at writing all messages to db.
-  return $message;
+    // If we reached here that means we were successful at writing all messages to db.
+    return $message;
 }
 
 /**
@@ -1956,37 +1955,37 @@ 
  * @ingroup api
  */
 function privatemsg_get_link($recipients, $account = array(), $subject = NULL) {
-  if ($account == NULL) {
+    if ($account == NULL) {
     global $user;
     $account = $user;
-  }
+    }
 
-  if (!is_array($recipients)) {
+    if (!is_array($recipients)) {
     $recipients = array($recipients);
-  }
+    }
 
-  if (!privatemsg_user_access('write privatemsg', $account) || $account->uid == 0) {
+    if (!privatemsg_user_access('write privatemsg', $account) || $account->uid == 0) {
     return FALSE;
-  }
+    }
 
-  $validated = array();
-  foreach ($recipients as $recipient) {
+    $validated = array();
+    foreach ($recipients as $recipient) {
     if (!privatemsg_user_access('read privatemsg', $recipient)) {
-      continue;
+        continue;
     }
     if (count(module_invoke_all('privatemsg_block_message', $account, array($recipient->uid => $recipient))) > 0) {
-      continue;
+        continue;
     }
     $validated[] = $recipient->uid;
-  }
-  if (empty($validated)) {
+    }
+    if (empty($validated)) {
     return FALSE;
-  }
-  $url = 'messages/new/'. implode(',', $validated);
-  if (!is_null($subject)) {
+    }
+    $url = 'messages/new/'. implode(',', $validated);
+    if (!is_null($subject)) {
     $url .= '/'. $subject;
-  }
-  return $url;
+    }
+    return $url;
 }
 
 /**
@@ -2001,8 +2000,8 @@ 
  * @ingroup api
  */
 function privatemsg_message_load($pmid, $account = NULL) {
-  $messages = privatemsg_message_load_multiple(array($pmid), $account);
-  return current($messages);
+    $messages = privatemsg_message_load_multiple(array($pmid), $account);
+    return current($messages);
 }
 
 /**
@@ -2017,27 +2016,27 @@ 
  * @ingroup api
  */
 function privatemsg_message_load_multiple($pmids, $account = NULL) {
-  // Avoid SQL error that would happen with an empty pm.mid IN () clause.
-  if (empty($pmids)) {
+    // Avoid SQL error that would happen with an empty pm.mid IN () clause.
+    if (empty($pmids)) {
     return array();
-  }
+    }
 
-  $query = _privatemsg_assemble_query('load', $pmids, $account);
-  $result = db_query($query['query']);
-  $messages = array();
-  while ($message = db_fetch_array($result)) {
+    $query = _privatemsg_assemble_query('load', $pmids, $account);
+    $result = db_query($query['query']);
+    $messages = array();
+    while ($message = db_fetch_array($result)) {
     // Load author of message.
     if (!($message['author'] = user_load($message['author']))) {
-      // If user does not exist, load anonymous user.
-      $message['author'] = user_load(array('uid' => 0));
+        // If user does not exist, load anonymous user.
+        $message['author'] = user_load(array('uid' => 0));
     }
     $returned = module_invoke_all('privatemsg_message_load', $message);
     if (!empty($returned)) {
-      $message = array_merge_recursive($returned, $message);
+        $message = array_merge_recursive($returned, $message);
     }
     $messages[$message['mid']] = $message;
-  }
-  return $messages;
+    }
+    return $messages;
 }
 
 /**
@@ -2056,26 +2055,26 @@ 
  */
 function _privatemsg_assemble_query($query) {
 
-  // Modules will be allowed to choose the prefix for the querybuilder, but if there is not one supplied, 'privatemsg' will be taken by default.
-  if (is_array($query)) {
+    // Modules will be allowed to choose the prefix for the querybuilder, but if there is not one supplied, 'privatemsg' will be taken by default.
+    if (is_array($query)) {
     $query_id = $query[0];
     $query_group = $query[1];
-  }
-  else {
+    }
+    else {
     $query_id = $query;
     $query_group = 'privatemsg';
-  }
-
-  $SELECT = array();
-  $INNER_JOIN = array();
-  $WHERE = array();
-  $GROUP_BY = array();
-  $HAVING   = array();
-  $ORDER_BY = array();
-  $QUERY_ARGS = array('select' => array(), 'where' => array(), 'join' => array(), 'having' => array());
-  $primary_table = '';
-
-  $fragments = array(
+    }
+
+    $SELECT = array();
+    $INNER_JOIN = array();
+    $WHERE = array();
+    $GROUP_BY = array();
+    $HAVING   = array();
+    $ORDER_BY = array();
+    $QUERY_ARGS = array('select' => array(), 'where' => array(), 'join' => array(), 'having' => array());
+    $primary_table = '';
+
+    $fragments = array(
     'select'      => $SELECT,
     'inner_join'  => $INNER_JOIN,
     'where'       => $WHERE,
@@ -2084,52 +2083,52 @@ 
     'order_by'    => $ORDER_BY,
     'query_args'  => $QUERY_ARGS,
     'primary_table'  => $primary_table,
-  );
-
-  /**
-   * Begin: dynamic arguments
-   */
-  $args = func_get_args();
-  unset($args[0]);
-  // we do the merge because we call call_user_func_array and not drupal_alter
-  // this is necessary because otherwise we would not be able to use $args correctly (otherwise it doesnt unfold)
-  $alterargs = array(&$fragments);
-  $query_function = $query_group .'_sql_'. $query_id;
-  if (!empty($args)) {
+    );
+
+    /**
+     * Begin: dynamic arguments
+     */
+    $args = func_get_args();
+    unset($args[0]);
+    // we do the merge because we call call_user_func_array and not drupal_alter
+    // this is necessary because otherwise we would not be able to use $args correctly (otherwise it doesnt unfold)
+    $alterargs = array(&$fragments);
+    $query_function = $query_group .'_sql_'. $query_id;
+    if (!empty($args)) {
     $alterargs = array_merge($alterargs, $args);
-  }
-  /**
-   * END: Dynamic arguments
-   */
-  if (!function_exists($query_function)) {
+    }
+    /**
+     * END: Dynamic arguments
+     */
+    if (!function_exists($query_function)) {
     drupal_set_message(t('Query function %function does not exist', array('%function' => $query_function)), 'error');
     return FALSE;
-  }
-  call_user_func_array($query_function, $alterargs);
-
-  array_unshift($alterargs, $query_function);
-  call_user_func_array('drupal_alter', $alterargs);
-
-  $SELECT = $fragments['select'];
-  $INNER_JOIN = $fragments['inner_join'];
-  $WHERE = $fragments['where'];
-  $GROUP_BY = $fragments['group_by'];
-  $HAVING   = $fragments['having'];
-  $ORDER_BY = $fragments['order_by'];
-  $QUERY_ARGS = $fragments['query_args'];
-  $primary_table = $fragments['primary_table'];
-
-  // pgsql has a case sensitive LIKE - replace it with ILIKE. see http://drupal.org/node/462982
-  if ($GLOBALS['db_type'] == 'pgsql') {
+    }
+    call_user_func_array($query_function, $alterargs);
+
+    array_unshift($alterargs, $query_function);
+    call_user_func_array('drupal_alter', $alterargs);
+
+    $SELECT = $fragments['select'];
+    $INNER_JOIN = $fragments['inner_join'];
+    $WHERE = $fragments['where'];
+    $GROUP_BY = $fragments['group_by'];
+    $HAVING   = $fragments['having'];
+    $ORDER_BY = $fragments['order_by'];
+    $QUERY_ARGS = $fragments['query_args'];
+    $primary_table = $fragments['primary_table'];
+
+    // pgsql has a case sensitive LIKE - replace it with ILIKE. see http://drupal.org/node/462982
+    if ($GLOBALS['db_type'] == 'pgsql') {
     $WHERE = str_replace('LIKE', 'ILIKE', $WHERE);
-  }
+    }
 
-  if (empty($primary_table)) {
+    if (empty($primary_table)) {
     $primary_table = '{privatemsg} pm';
-  }
+    }
 
-  // Perform the whole query assembly only if we have something to select.
-  if (!empty($SELECT)) {
+    // Perform the whole query assembly only if we have something to select.
+    if (!empty($SELECT)) {
     $str_select = implode(", ", $SELECT);
     $query = "SELECT {$str_select} FROM ". $primary_table;
 
@@ -2137,51 +2136,51 @@ 
     // In most cases,  "COUNT(*)" is enough to get the count query, but in queries involving a GROUP BY, we want a count of the number of groups we have, not the count of elements inside each group.
     // So we test if there is GROUP BY and if there is, count the number of distinct groups. If not, we go the normal wal and do a plain COUNT(*).
     if (!empty($GROUP_BY)) {
-      // PostgreSQL does not support COUNT(sometextfield, someintfield), so I'm only using the first one
-      // Works fine for thread_id/list but may generate an error when a more complex GROUP BY is used.
-      $str_group_by_count = current($GROUP_BY);
-      $count = "SELECT COUNT(DISTINCT {$str_group_by_count}) FROM ". $primary_table;
+        // PostgreSQL does not support COUNT(sometextfield, someintfield), so I'm only using the first one
+        // Works fine for thread_id/list but may generate an error when a more complex GROUP BY is used.
+        $str_group_by_count = current($GROUP_BY);
+        $count = "SELECT COUNT(DISTINCT {$str_group_by_count}) FROM ". $primary_table;
     }
     else {
-      $count = "SELECT COUNT(*) FROM ". $primary_table;
+        $count = "SELECT COUNT(*) FROM ". $primary_table;
     }
 
     if (!empty($INNER_JOIN)) {
-      $str_inner_join = implode(' ', $INNER_JOIN);
-      $query .= " {$str_inner_join}";
-      $count .= " {$str_inner_join}";
+        $str_inner_join = implode(' ', $INNER_JOIN);
+        $query .= " {$str_inner_join}";
+        $count .= " {$str_inner_join}";
     }
     if (!empty($WHERE)) {
-      $str_where = '('. implode(') AND (', $WHERE) .')';
-      $query .= " WHERE {$str_where}";
-      $count .= " WHERE {$str_where}";
+        $str_where = '('. implode(') AND (', $WHERE) .')';
+        $query .= " WHERE {$str_where}";
+        $count .= " WHERE {$str_where}";
     }
     if (!empty($GROUP_BY)) {
-      $str_group_by = ' GROUP BY '. implode(", ", $GROUP_BY) ;
-      $query .= " {$str_group_by}";
+        $str_group_by = ' GROUP BY '. implode(", ", $GROUP_BY) ;
+        $query .= " {$str_group_by}";
     }
     if (!empty($HAVING)) {
-      $str_having = '('. implode(') AND (', $HAVING) .')';
-      $query .= " HAVING {$str_having}";
-      // queries containing a HAVING break the count query on pgsql.
-      // In this case, use the subquery method as outlined in http://drupal.org/node/303087#comment-1370752 .
-      // The subquery method will work for all COUNT queries, but it is thought to be much slower, so we are only using it where other cross database approaches fail.
-      $count = 'SELECT COUNT(*) FROM ('. $query .') as count';
+        $str_having = '('. implode(') AND (', $HAVING) .')';
+        $query .= " HAVING {$str_having}";
+        // queries containing a HAVING break the count query on pgsql.
+        // In this case, use the subquery method as outlined in http://drupal.org/node/303087#comment-1370752 .
+        // The subquery method will work for all COUNT queries, but it is thought to be much slower, so we are only using it where other cross database approaches fail.
+        $count = 'SELECT COUNT(*) FROM ('. $query .') as count';
     }
     if (!empty($ORDER_BY)) {
-      $str_order_by = ' ORDER BY '. implode(", ", $ORDER_BY) ;
-      $query .= " {$str_order_by}";
+        $str_order_by = ' ORDER BY '. implode(", ", $ORDER_BY) ;
+        $query .= " {$str_order_by}";
     }
     $QUERY_ARGS = array_merge($QUERY_ARGS['select'], $QUERY_ARGS['join'], $QUERY_ARGS['where'], $QUERY_ARGS['having']);
     if (!empty($QUERY_ARGS)) {
-      _db_query_callback($QUERY_ARGS, TRUE);
-      $query = preg_replace_callback(DB_QUERY_REGEXP, '_db_query_callback', $query);
-      _db_query_callback($QUERY_ARGS, TRUE);
-      $count = preg_replace_callback(DB_QUERY_REGEXP, '_db_query_callback', $count);
+        _db_query_callback($QUERY_ARGS, TRUE);
+        $query = preg_replace_callback(DB_QUERY_REGEXP, '_db_query_callback', $query);
+        _db_query_callback($QUERY_ARGS, TRUE);
+        $count = preg_replace_callback(DB_QUERY_REGEXP, '_db_query_callback', $count);
     }
     return array('query' => $query, 'count' => $count);
-  }
-  return FALSE;
+    }
+    return FALSE;
 }
 
 /**
@@ -2198,44 +2197,44 @@ 
  *   The FAPI definitions for the thread action form.
  */
 function _privatemsg_action_form() {
-  $form = array(
-      '#type'        => 'fieldset',
-      '#title'       => t('Actions'),
-      '#prefix'      => '<div class="container-inline">',
-      '#suffix'      => '</div>',
-      '#collapsible' => TRUE,
-      '#collapsed'   => FALSE,
-      '#weight'      => 15,
-  );
-  if (privatemsg_user_access('delete privatemsg')) {
+    $form = array(
+        '#type'        => 'fieldset',
+        '#title'       => t('Actions'),
+        '#prefix'      => '<div class="container-inline">',
+        '#suffix'      => '</div>',
+        '#collapsible' => TRUE,
+        '#collapsed'   => FALSE,
+        '#weight'      => 15,
+    );
+    if (privatemsg_user_access('delete privatemsg')) {
     $form['delete'] = array(
         '#type'   => 'submit',
         '#value'  => t('Delete'),
     );
-  }
-  // Display all operations which have a label.
-  $options = array(0 => t('More actions...'));
-  foreach (module_invoke_all('privatemsg_thread_operations') as $operation => $array) {
+    }
+    // Display all operations which have a label.
+    $options = array(0 => t('More actions...'));
+    foreach (module_invoke_all('privatemsg_thread_operations') as $operation => $array) {
     if (isset($array['label'])) {
-      $options[$operation] = $array['label'];
-    }
-  }
-  $form['operation'] = array(
-      '#type'          => 'select',
-      '#options'       => $options,
-      '#default_value' => 0,
-  );
-  $form['submit'] = array(
-      '#prefix'     => '<div class="privatemsg-op-button">',
-      '#suffix'    => '</div>',
-      '#type'       => 'submit',
-      '#value'      => t('Execute'),
-      '#submit'     => array('privatemsg_list_submit'),
-      '#attributes' => array('class' => 'privatemsg-action-button'),
-  );
-  // JS for hiding the execute button.
-  drupal_add_js(drupal_get_path('module', 'privatemsg') .'/privatemsg-list.js');
-  return $form;
+        $options[$operation] = $array['label'];
+    }
+    }
+    $form['operation'] = array(
+        '#type'          => 'select',
+        '#options'       => $options,
+        '#default_value' => 0,
+    );
+    $form['submit'] = array(
+        '#prefix'     => '<div class="privatemsg-op-button">',
+        '#suffix'    => '</div>',
+        '#type'       => 'submit',
+        '#value'      => t('Execute'),
+        '#submit'     => array('privatemsg_list_submit'),
+        '#attributes' => array('class' => 'privatemsg-action-button'),
+    );
+    // JS for hiding the execute button.
+    drupal_add_js(drupal_get_path('module', 'privatemsg') .'/privatemsg-list.js');
+    return $form;
 }
 
 /**
@@ -2250,23 +2249,23 @@ 
  *   current user.
  */
 function privatemsg_thread_change_status($threads, $status, $account = NULL) {
-  if (!is_array($threads)) {
+    if (!is_array($threads)) {
     $threads = array($threads);
-  }
-  if (empty($account)) {
+    }
+    if (empty($account)) {
     global $user;
     $account = drupal_clone($user);
-  }
-  // Merge status and uid with the threads list. array_merge() will not overwrite/ignore thread_id 1.
-  $params = array_merge(array($status, $account->uid), $threads);
-  db_query('UPDATE {pm_index} SET is_new = %d WHERE uid = %d AND thread_id IN ('. db_placeholders($threads) .')', $params);
+    }
+    // Merge status and uid with the threads list. array_merge() will not overwrite/ignore thread_id 1.
+    $params = array_merge(array($status, $account->uid), $threads);
+    db_query('UPDATE {pm_index} SET is_new = %d WHERE uid = %d AND thread_id IN ('. db_placeholders($threads) .')', $params);
 
-  if ($status == PRIVATEMSG_UNREAD) {
+    if ($status == PRIVATEMSG_UNREAD) {
     drupal_set_message(format_plural(count($threads), 'Marked 1 thread as unread.', 'Marked @count threads as unread.'));
-  }
-  else {
+    }
+    else {
     drupal_set_message(format_plural(count($threads), 'Marked 1 thread as read.', 'Marked @count threads as read.'));
-  }
+    }
 }
 /**
  * Returns a table header definition based on the submitted keys.
@@ -2282,25 +2281,25 @@ 
  *   Array with header defintions for tablesort_sql and theme('table').
  */
 function _privatemsg_list_headers($has_posts, $keys) {
-  $select_header = $has_posts ? theme('table_select_header_cell') : '';
-  $select_header['#weight'] = -50;
+    $select_header = $has_posts ? theme('table_select_header_cell') : '';
+    $select_header['#weight'] = -50;
 
-  // theme() doesn't include the theme file for patterns, we need to do it manually.
-  include_once drupal_get_path('module', 'privatemsg') .'/privatemsg.theme.inc';
+    // theme() doesn't include the theme file for patterns, we need to do it manually.
+    include_once drupal_get_path('module', 'privatemsg') .'/privatemsg.theme.inc';
 
-  $header = array($select_header);
-  foreach ($keys as $key) {
+    $header = array($select_header);
+    foreach ($keys as $key) {
     // First, try to load a specific theme for that header, if not present, use the default.
     if ($return = theme(array('privatemsg_list_header__'. $key, 'privatemsg_list_header'))) {
-      // The default theme returns nothing, only store the value if we have something.
-      $header[$key] = $return;
+        // The default theme returns nothing, only store the value if we have something.
+        $header[$key] = $return;
+    }
     }
-  }
-  if (count($header) == 1) {
+    if (count($header) == 1) {
     // No header definition returned, fallback to the default.
     $header += _privatemsg_list_headers_fallback($keys);
-  }
-  return $header;
+    }
+    return $header;
 }
 
 /**
@@ -2310,15 +2309,15 @@ 
  *   Array with the correct headers.
  */
 function _privatemsg_list_headers_fallback($keys) {
-  $header = array();
-  foreach ($keys as $key) {
+    $header = array();
+    foreach ($keys as $key) {
     $theme_function = 'phptemplate_privatemsg_list_header__' . $key;
     if (function_exists($theme_function)) {
-      $header[$key] = $theme_function();
+        $header[$key] = $theme_function();
+    }
     }
-  }
 
-  return $header;
+    return $header;
 }
 
 /**
@@ -2332,23 +2331,23 @@ 
  *   Row definition for use with theme('table')
  */
 function _privatemsg_list_thread($thread) {
-  $row = array('data' => array());
+    $row = array('data' => array());
 
-  if (!empty($thread['is_new'])) {
+    if (!empty($thread['is_new'])) {
     // Set the css class in the tr tag.
     $row['class'] = 'privatemsg-unread';
-  }
-  foreach ($thread as $key => $data) {
+    }
+    foreach ($thread as $key => $data) {
     // First, try to load a specific theme for that field, if not present, use the default.
     if ($return = theme(array('privatemsg_list_field__'. $key, 'privatemsg_list_field'), $thread)) {
-      // The default theme returns nothing, only store the value if we have something.
-      $row['data'][$key] = $return;
+        // The default theme returns nothing, only store the value if we have something.
+        $row['data'][$key] = $return;
+    }
     }
-  }
-  if (empty($row['data'])) {
+    if (empty($row['data'])) {
     $row['data'] = _privatemsg_list_thread_fallback($thread);
-  }
-  return $row;
+    }
+    return $row;
 }
 
 /**
@@ -2358,15 +2357,15 @@ 
  *   Array with row data.
  */
 function _privatemsg_list_thread_fallback($thread) {
-  $row_data = array();
-  foreach ($thread as $key => $data) {
+    $row_data = array();
+    foreach ($thread as $key => $data) {
     $theme_function = 'phptemplate_privatemsg_list_field__' . $key;
     if (function_exists($theme_function)) {
-      $row_data[$key] = $theme_function($thread);
+        $row_data[$key] = $theme_function($thread);
+    }
     }
-  }
 
-  return $row_data;
+    return $row_data;
 }
 
 /**
@@ -2376,20 +2375,20 @@ 
  * execute it.
  */
 function privatemsg_undo_action() {
-  // Check if a undo callback for that user exists.
-  if (isset($_SESSION['privatemsg']['undo callback']) && is_array($_SESSION['privatemsg']['undo callback'])) {
+    // Check if a undo callback for that user exists.
+    if (isset($_SESSION['privatemsg']['undo callback']) && is_array($_SESSION['privatemsg']['undo callback'])) {
     $undo = $_SESSION['privatemsg']['undo callback'];
     // If the defined undo callback exists, execute it
     if (isset($undo['function']) && isset($undo['args'])) {
-      // Load the user object.
-      if (isset($undo['args']['account']) && $undo['args']['account'] > 0) {
+        // Load the user object.
+        if (isset($undo['args']['account']) && $undo['args']['account'] > 0) {
         $undo['args']['account'] = user_load((int)$undo['args']['account']);
-      }
-      call_user_func_array($undo['function'], $undo['args']);
+        }
+        call_user_func_array($undo['function'], $undo['args']);
     }
     // Return back to the site defined by the destination GET param.
     drupal_goto();
-  }
+    }
 }
 
 /**
@@ -2399,29 +2398,29 @@ 
  * based on node_admin_nodes_submit().
  */
 function privatemsg_list_submit($form, &$form_state) {
-  // Load all available operation definitions.
-  $operations = module_invoke_all('privatemsg_thread_operations');
+    // Load all available operation definitions.
+    $operations = module_invoke_all('privatemsg_thread_operations');
 
-  // Default "default" operation, which won't do anything.
-  $operation = array('callback' => 0);
+    // Default "default" operation, which won't do anything.
+    $operation = array('callback' => 0);
 
-  // Check if a valid operation has been submitted.
-  if (isset($form_state['values']['operation']) && isset($operations[$form_state['values']['operation']])) {
+    // Check if a valid operation has been submitted.
+    if (isset($form_state['values']['operation']) && isset($operations[$form_state['values']['operation']])) {
     $operation = $operations[$form_state['values']['operation']];
-  }
+    }
 
-  // Load all keys where the value is the current op.
-  $keys = array_keys($form_state['values'], $form_state['values']['op']);
+    // Load all keys where the value is the current op.
+    $keys = array_keys($form_state['values'], $form_state['values']['op']);
 
-  // The first one is op itself, we need to use the second.
-  if (isset($keys[1]) && isset($operations[$keys[1]])) {
+    // The first one is op itself, we need to use the second.
+    if (isset($keys[1]) && isset($operations[$keys[1]])) {
     $operation = $operations[$keys[1]];
-  }
+    }
 
-  // Only execute something if we have a valid callback and at least one checked thread.
-  if (!empty($operation['callback'])) {
+    // Only execute something if we have a valid callback and at least one checked thread.
+    if (!empty($operation['callback'])) {
     privatemsg_operation_execute($operation, $form_state['values']['threads'], $form_state['values']['account']);
-  }
+    }
 }
 
 /**
@@ -2435,42 +2434,42 @@ 
  *   array can be directly passed to it.
  */
 function privatemsg_operation_execute($operation, $threads, $account = null) {
-  // Filter out unchecked threads, this gives us an array of "checked" threads.
-  $threads = array_filter($threads);
+    // Filter out unchecked threads, this gives us an array of "checked" threads.
+    $threads = array_filter($threads);
 
-  if (empty($threads)) {
+    if (empty($threads)) {
     // Do not execute anything if there are no checked threads.
     return;
-  }
-  // Add in callback arguments if present.
-  if (isset($operation['callback arguments'])) {
+    }
+    // Add in callback arguments if present.
+    if (isset($operation['callback arguments'])) {
     $args = array_merge(array($threads), $operation['callback arguments']);
-  }
-  else {
+    }
+    else {
     $args = array($threads);
-  }
+    }
 
-  // Add the user object to the arguments.
-  if ($account) {
+    // Add the user object to the arguments.
+    if ($account) {
     $args[] = $account;
-  }
+    }
 
-  // Execute the chosen action and pass the defined arguments.
-  call_user_func_array($operation['callback'], $args);
+    // Execute the chosen action and pass the defined arguments.
+    call_user_func_array($operation['callback'], $args);
 
-  // Check if that operation has defined an undo callback.
-  if (isset($operation['undo callback']) && $undo_function = $operation['undo callback']) {
+    // Check if that operation has defined an undo callback.
+    if (isset($operation['undo callback']) && $undo_function = $operation['undo callback']) {
     // Add in callback arguments if present.
     if (isset($operation['undo callback arguments'])) {
-      $undo_args = array_merge(array($threads), $operation['undo callback arguments']);
+        $undo_args = array_merge(array($threads), $operation['undo callback arguments']);
     }
     else {
-      $undo_args = array($threads);
+        $undo_args = array($threads);
     }
 
     // Avoid saving the complete user object in the session.
     if ($account) {
-      $undo_args['account'] = $account->uid;
+        $undo_args['account'] = $account->uid;
     }
     // Store the undo callback in the session and display a "Undo" link.
     // @todo: Provide a more flexible solution for such an undo action, operation defined string for example.
@@ -2478,7 +2477,7 @@ 
     $undo = url('messages/undo/action', array('query' => drupal_get_destination()));
 
     drupal_set_message(t('The previous action can be <a href="!undo">undone</a>.', array('!undo' => $undo)));
-  }
+    }
 }
 
 /**
@@ -2494,71 +2493,71 @@ 
  *   defaults to the current user.
  */
 function privatemsg_thread_change_delete($threads, $delete, $account = NULL) {
-  if (!is_array($threads)) {
+    if (!is_array($threads)) {
     $threads = array($threads);
-  }
-  if (empty($account)) {
+    }
+    if (empty($account)) {
     global $user;
     $account = drupal_clone($user);
-  }
+    }
 
-  // Merge status and uid with the threads list. array_merge() will not overwrite/ignore thread_id 1.
-  $params = array_merge(array($delete, $account->uid), $threads);
+    // Merge status and uid with the threads list. array_merge() will not overwrite/ignore thread_id 1.
+    $params = array_merge(array($delete, $account->uid), $threads);
 
-  // Load all messages of those threads including the deleted.
-  $query = _privatemsg_assemble_query('messages', $threads, $account, TRUE);
-  $result = db_query($query['query']);
+    // Load all messages of those threads including the deleted.
+    $query = _privatemsg_assemble_query('messages', $threads, $account, TRUE);
+    $result = db_query($query['query']);
 
-  // Delete each message. We need to do that to trigger the delete hook.
-  while ($row = db_fetch_array($result)) {
+    // Delete each message. We need to do that to trigger the delete hook.
+    while ($row = db_fetch_array($result)) {
     privatemsg_message_change_delete($row['mid'], $delete, $account);
-  }
+    }
 
-  if ($delete) {
+    if ($delete) {
     drupal_set_message(format_plural(count($threads), 'Deleted 1 thread.', 'Deleted @count threads.'));
-  }
-  else {
+    }
+    else {
     drupal_set_message(format_plural(count($threads), 'Restored 1 thread.', 'Restored @count threads.'));
-  }
+    }
 }
 
 /**
  * Implements hook_privatemsg_thread_operations().
  */
 function privatemsg_privatemsg_thread_operations() {
-  $operations = array(
+    $operations = array(
     'mark as read' => array(
-      'label' => t('Mark as read'),
-      'callback' => 'privatemsg_thread_change_status',
-      'callback arguments' => array('status' => PRIVATEMSG_READ),
-      'undo callback' => 'privatemsg_thread_change_status',
-      'undo callback arguments' => array('status' => PRIVATEMSG_UNREAD),
+        'label' => t('Mark as read'),
+        'callback' => 'privatemsg_thread_change_status',
+        'callback arguments' => array('status' => PRIVATEMSG_READ),
+        'undo callback' => 'privatemsg_thread_change_status',
+        'undo callback arguments' => array('status' => PRIVATEMSG_UNREAD),
     ),
     'mark as unread' => array(
-      'label' => t('Mark as unread'),
-      'callback' => 'privatemsg_thread_change_status',
-      'callback arguments' => array('status' => PRIVATEMSG_UNREAD),
-      'undo callback' => 'privatemsg_thread_change_status',
-      'undo callback arguments' => array('status' => PRIVATEMSG_READ),
+        'label' => t('Mark as unread'),
+        'callback' => 'privatemsg_thread_change_status',
+        'callback arguments' => array('status' => PRIVATEMSG_UNREAD),
+        'undo callback' => 'privatemsg_thread_change_status',
+        'undo callback arguments' => array('status' => PRIVATEMSG_READ),
     ),
-  );
-  if (privatemsg_user_access('delete privatemsg')) {
+    );
+    if (privatemsg_user_access('delete privatemsg')) {
     $operations['delete'] = array(
-      'callback' => 'privatemsg_thread_change_delete',
-      'callback arguments' => array('delete' => 1),
-      'undo callback' => 'privatemsg_thread_change_delete',
-      'undo callback arguments' => array('delete' => 0),
+        'callback' => 'privatemsg_thread_change_delete',
+        'callback arguments' => array('delete' => 1),
+        'undo callback' => 'privatemsg_thread_change_delete',
+        'undo callback arguments' => array('delete' => 0),
     );
-  }
-  return $operations;
+    }
+    return $operations;
 }
 
 /**
  * Implementation of hook_views_api().
  */
 function privatemsg_views_api() {
-  return array(
+    return array(
     'api' => 2,
     'path' => drupal_get_path('module', 'privatemsg') . '/views',
-  );
+    );
 }

Switch Indentation +38 added lines, -38 removed lines

@@ -1437,43 +1437,43 @@ 
   global $user;
 
   switch ($op) {
-    case 'view':
-      if ($url = privatemsg_get_link(array($account))) {
-        $account->content['privatemsg_send_new_message'] = array(
-          '#type'   => 'markup',
-          '#value'  => l(t('Send this user a message'), $url, array('query' => drupal_get_destination())),
-          '#weight' => 10,
-        );
-      }
-      break;
-    case 'login':
-      if (variable_get('privatemsg_display_loginmessage', TRUE) && privatemsg_user_access()) {
-        $count = privatemsg_unread_count();
-        if ($count) {
-          drupal_set_message(format_plural($count, 'You have <a href="@messages">1 unread message</a>.', 'You have <a href="@messages">@count unread messages</a>', array('@messages' => url('messages'))));
-        }
+  case 'view':
+    if ($url = privatemsg_get_link(array($account))) {
+      $account->content['privatemsg_send_new_message'] = array(
+        '#type'   => 'markup',
+        '#value'  => l(t('Send this user a message'), $url, array('query' => drupal_get_destination())),
+        '#weight' => 10,
+      );
+    }
+    break;
+  case 'login':
+    if (variable_get('privatemsg_display_loginmessage', TRUE) && privatemsg_user_access()) {
+      $count = privatemsg_unread_count();
+      if ($count) {
+        drupal_set_message(format_plural($count, 'You have <a href="@messages">1 unread message</a>.', 'You have <a href="@messages">@count unread messages</a>', array('@messages' => url('messages'))));
       }
-      break;
-    case 'delete':
+    }
+    break;
+  case 'delete':
 
-      // Load all mids of the messages the user wrote.
-      $result = db_query("SELECT mid FROM {pm_message} WHERE author = %d", $account->uid);
-      $mids = array();
-      while ($row = db_fetch_array($result)) {
-        $mids[] = $row['mid'];
-      }
+    // Load all mids of the messages the user wrote.
+    $result = db_query("SELECT mid FROM {pm_message} WHERE author = %d", $account->uid);
+    $mids = array();
+    while ($row = db_fetch_array($result)) {
+      $mids[] = $row['mid'];
+    }
 
-      // Delete messages the user wrote.
-      db_query('DELETE FROM {pm_message} WHERE author = %d', $account->uid);
+    // Delete messages the user wrote.
+    db_query('DELETE FROM {pm_message} WHERE author = %d', $account->uid);
 
-      if (!empty($mids)) {
-        // Delete recipient entries in {pm_index} of the messages the user wrote.
-        db_query('DELETE FROM {pm_index} WHERE mid IN (' . db_placeholders($mids) . ')', $mids);
-      }
+    if (!empty($mids)) {
+      // Delete recipient entries in {pm_index} of the messages the user wrote.
+      db_query('DELETE FROM {pm_index} WHERE mid IN (' . db_placeholders($mids) . ')', $mids);
+    }
 
-      // Delete recipient entries of that user.
-      db_query('DELETE FROM {pm_index} WHERE uid = %d', $account->uid);
-      break;
+    // Delete recipient entries of that user.
+    db_query('DELETE FROM {pm_index} WHERE uid = %d', $account->uid);
+    break;
   }
 }
 
@@ -1494,12 +1494,12 @@ 
   elseif ('view' == $op) {
     $block = array();
     switch ($delta) {
-      case 'privatemsg-menu':
-        $block = _privatemsg_block_menu();
-        break;
-      case 'privatemsg-new':
-        $block = _privatemsg_block_new();
-        break;
+    case 'privatemsg-menu':
+      $block = _privatemsg_block_menu();
+      break;
+    case 'privatemsg-new':
+      $block = _privatemsg_block_new();
+      break;
     }
     return $block;
   }

Spacing +62 added lines, -62 removed lines

@@ -98,7 +98,7 @@ 
 
 
     if ($no_text) {
-      return implode(', ', $to) . $limit_string;
+      return implode(', ', $to).$limit_string;
     }
 
     $last = array_pop($to);
@@ -220,7 +220,7 @@ 
  * @ingroup api
  */
 function privatemsg_user_access($permission = 'read privatemsg', $account = NULL) {
-  if ( $account === NULL ) {
+  if ($account === NULL) {
     global $user;
     $account = $user;
   }
@@ -344,7 +344,7 @@ 
           // message overlap.
           // Note - the last page lists the earliest messages, not the latest.
           $paging_count = variable_get('privatemsg_view_use_max_as_default', FALSE) ? $thread['message_count'] - variable_get('privatemsg_view_default_amount', 10) : $thread['message_count'];
-          $last_page = $paging_count % $max_amount;
+          $last_page = $paging_count%$max_amount;
         }
 
         // Sanity check - we cannot start from a negative number.
@@ -546,7 +546,7 @@ 
     '#options'       => $amounts,
     '#weight'        => 20,
   );
-  drupal_add_js(drupal_get_path('module', 'privatemsg') .'/privatemsg-admin.js');
+  drupal_add_js(drupal_get_path('module', 'privatemsg').'/privatemsg-admin.js');
 
   $form['#submit'][] = 'private_message_settings_submit';
   return system_settings_form($form);
@@ -640,9 +640,9 @@ 
   $vars['message_timestamp'] = format_date($message['timestamp'], 'small');
   $vars['message_body'] = check_markup($message['body'], $message['format'], FALSE);
   if (isset($vars['mid']) && isset($vars['thread_id']) && privatemsg_user_access('delete privatemsg')) {
-    $vars['message_actions'][] = array('title' => t('Delete message'), 'href' => 'messages/delete/' . $vars['thread_id'] . '/' . $vars['mid']);
+    $vars['message_actions'][] = array('title' => t('Delete message'), 'href' => 'messages/delete/'.$vars['thread_id'].'/'.$vars['mid']);
   }
-  $vars['message_anchors'][] = 'privatemsg-mid-' . $vars['mid'];
+  $vars['message_anchors'][] = 'privatemsg-mid-'.$vars['mid'];
   if (!empty($message['is_new'])) {
     $vars['message_anchors'][] = 'new';
     $vars['new'] = drupal_ucfirst(t('new'));
@@ -655,7 +655,7 @@ 
 
   $vars['anchors'] = '';
   foreach ($vars['message_anchors'] as $anchor) {
-    $vars['anchors'] .= '<a name="' . $anchor . '"></a>';
+    $vars['anchors'] .= '<a name="'.$anchor.'"></a>';
   }
 }
 
@@ -758,7 +758,7 @@ 
     global $user;
     $account = $user;
   }
-  if ( !isset($counts[$account->uid])) {
+  if (!isset($counts[$account->uid])) {
     $query = _privatemsg_assemble_query('unread_count', $account);
     $counts[$account->uid] = db_result(db_query($query['query']));
   }
@@ -785,7 +785,7 @@ 
       'query' => array('start' => $thread['older_start']),
       'title' => t('Display older messages'),
     );
-     $older = l(t('<<'), 'messages/view/' . $thread['thread_id'], $options);
+     $older = l(t('<<'), 'messages/view/'.$thread['thread_id'], $options);
   }
   $newer = '';
   if (isset($thread['newer_start'])) {
@@ -793,7 +793,7 @@ 
       'query' => array('start' => $thread['newer_start']),
       'title' => t('Display newer messages'),
     );
-    $newer = l(t('>>'), 'messages/view/' . $thread['thread_id'], $options);
+    $newer = l(t('>>'), 'messages/view/'.$thread['thread_id'], $options);
   }
   $substitutions = array('@from' => $thread['from'], '@to' => $thread['to'], '@total' => $thread['message_count'], '!previous_link' => $older, '!newer_link' => $newer);
   $title = t('!previous_link Displaying messages @from - @to of @total !newer_link', $substitutions);
@@ -847,7 +847,7 @@ 
   global $user;
 
   $recipients_string = '';
-  $body      = '';
+  $body = '';
 
   // convert recipients to array of user objects
   if (!empty($recipients) && is_string($recipients) || is_int($recipients)) {
@@ -1004,7 +1004,7 @@ 
     );
     $recipients_string_themed = implode(', ', $to_themed);
     $form['privatemsg']['recipient_display'] = array(
-      '#value' =>  '<p>'. t('<strong>Reply to thread</strong>:<br /> Recipients: !to', array('!to' => $recipients_string_themed)) .'</p>',
+      '#value' =>  '<p>'.t('<strong>Reply to thread</strong>:<br /> Recipients: !to', array('!to' => $recipients_string_themed)).'</p>',
       '#weight' => -10,
     );
     if (empty($recipients_string)) {
@@ -1099,7 +1099,7 @@ 
       // First, check if another module is able to resolve the string into an
       // user object.
       foreach (module_implements('privatemsg_name_lookup') as $module) {
-        $function = $module . '_privatemsg_name_lookup';
+        $function = $module.'_privatemsg_name_lookup';
         if (($recipient = $function($string)) && is_object($recipient)) {
           // If there is a match, continue with the next input string.
           $recipients[$recipient->uid] = $recipient;
@@ -1131,7 +1131,7 @@ 
   foreach ($form_state['validate_built_message']['recipients'] as $recipient) {
     $recipient_names[] = theme('username', $recipient);
   }
-  if ($status !== FALSE )  {
+  if ($status !== FALSE) {
     drupal_set_message(t('A message has been sent to !recipients.', array('!recipients' => implode(', ', $recipient_names))));
   }
   else {
@@ -1173,7 +1173,7 @@ 
   $fields = array_filter(variable_get('privatemsg_display_fields', array('participants')));
 
   // Required columns.
-  $fragments['select'][]      = 'pmi.thread_id';
+  $fragments['select'][] = 'pmi.thread_id';
   // We have to use MIN as the subject might not be the same in some threads.
   // MIN() does not have a useful meaning except that it helps to correctly
   // aggregate the thread on PostgreSQL.
@@ -1185,32 +1185,32 @@ 
   // Select number of messages in the thread if the count is
   // set to be displayed.
   if (in_array('count', $fields)) {
-    $fragments['select'][]      = 'COUNT(distinct pmi.mid) as count';
+    $fragments['select'][] = 'COUNT(distinct pmi.mid) as count';
   }
   if (in_array('participants', $fields)) {
     // Query for a string with uid's, for example "1,6,7".
     // @todo: Replace this with a single query similiar to the tag list.
     if ($GLOBALS['db_type'] == 'pgsql') {
       // PostgreSQL does not know GROUP_CONCAT, so a subquery is required.
-      $fragments['select'][]      = "array_to_string(array(SELECT DISTINCT textin(int4out(pmia.uid))
+      $fragments['select'][] = "array_to_string(array(SELECT DISTINCT textin(int4out(pmia.uid))
                                                             FROM {pm_index} pmia
                                                             WHERE pmia.thread_id = pmi.thread_id), ',') AS participants";
     }
     else {
-      $fragments['select'][]      = '(SELECT GROUP_CONCAT(DISTINCT pmia.uid SEPARATOR ",")
+      $fragments['select'][] = '(SELECT GROUP_CONCAT(DISTINCT pmia.uid SEPARATOR ",")
                                                             FROM {pm_index} pmia
                                                             WHERE pmia.thread_id = pmi.thread_id) AS participants';
     }
   }
   if (in_array('thread_started', $fields)) {
-    $fragments['select'][]      = 'MIN(pm.timestamp) as thread_started';
+    $fragments['select'][] = 'MIN(pm.timestamp) as thread_started';
   }
 
   $fragments['inner_join'][]  = 'INNER JOIN {pm_index} pmi ON pm.mid = pmi.mid';
 
   // Only load undeleted messages of the current user and group by thread.
   $fragments['where'][]       = 'pmi.uid = %d';
-  $fragments['query_args']['where'][]  = $account->uid;
+  $fragments['query_args']['where'][] = $account->uid;
   $fragments['where'][]       = 'pmi.deleted = 0';
   $fragments['group_by'][]    = 'pmi.thread_id';
 
@@ -1224,8 +1224,8 @@ 
   // tablesort_sql() generates a ORDER BY string. However, the "ORDER BY " part 
   // is not needed and added by the query builder. Discard the first 9
   // characters of the string.
-  $order_by = drupal_substr(tablesort_sql(_privatemsg_list_headers( FALSE, array_merge(array('subject', 'last_updated'), $fields)), $order_by_first), 9);
-  $fragments['order_by'][]  = $order_by;
+  $order_by = drupal_substr(tablesort_sql(_privatemsg_list_headers(FALSE, array_merge(array('subject', 'last_updated'), $fields)), $order_by_first), 9);
+  $fragments['order_by'][] = $order_by;
 }
 
 /**
@@ -1252,11 +1252,11 @@ 
 
   $fragments['inner_join'][]  = 'INNER JOIN {pm_index} pmi ON pm.mid = pmi.mid';
   // Use IN() to load multiple messages at the same time.
-  $fragments['where'][]       = 'pmi.mid IN (' . db_placeholders($pmids) . ')';
-  $fragments['query_args']['where']  += $pmids;
+  $fragments['where'][]       = 'pmi.mid IN ('.db_placeholders($pmids).')';
+  $fragments['query_args']['where'] += $pmids;
   if ($account) {
-    $fragments['where'][]       = 'pmi.uid = %d';
-    $fragments['query_args']['where'][]  = $account->uid;
+    $fragments['where'][] = 'pmi.uid = %d';
+    $fragments['query_args']['where'][] = $account->uid;
   }
   $fragments['order_by'][] = 'pm.timestamp ASC';
   $fragments['order_by'][] = 'pm.mid ASC';
@@ -1278,13 +1278,13 @@ 
   $fragments['primary_table'] = '{pm_index} pmi';
 
   $fragments['select'][]      = 'pmi.mid';
-  $fragments['where'][]       = 'pmi.thread_id IN ('. db_placeholders($threads) .')';
-  $fragments['query_args']['where']   += $threads;
+  $fragments['where'][]       = 'pmi.thread_id IN ('.db_placeholders($threads).')';
+  $fragments['query_args']['where'] += $threads;
   $fragments['inner_join'][]  = 'INNER JOIN {pm_message} pm ON (pm.mid = pmi.mid)';
   if ($account) {
     // Only load the user's messages.
     $fragments['where'][]     = 'pmi.uid = %d';
-    $fragments['query_args']['where'][]  = $account->uid;
+    $fragments['query_args']['where'][] = $account->uid;
   }
   if (!$load_all) {
     // Also load deleted messages when requested.
@@ -1320,7 +1320,7 @@ 
 
   $fragments['inner_join'][]  = 'INNER JOIN {users} u ON (u.uid = pmi.uid)';
   $fragments['where'][]       = 'pmi.thread_id = %d';
-  $fragments['query_args']['where'][]  = $thread_id;
+  $fragments['query_args']['where'][] = $thread_id;
 }
 
 /**
@@ -1340,7 +1340,7 @@ 
   $fragments['where'][]       = 'pmi.deleted = 0';
   $fragments['where'][]       = 'pmi.is_new = 1';
   $fragments['where'][]       = 'pmi.uid = %d';
-  $fragments['query_args']['where'][]  = $account->uid;
+  $fragments['query_args']['where'][] = $account->uid;
 }
 
 /**
@@ -1358,10 +1358,10 @@ 
   $fragments['select'][] = 'u.name';
   // Escape the % to get it through the placeholder replacement.
   $fragments['where'][] = "u.name LIKE '%s'";
-  $fragments['query_args']['where'][] = $search .'%%';
+  $fragments['query_args']['where'][] = $search.'%%';
   if (!empty($names)) {
     // If there are already names selected, exclude them from the suggestions.
-    $fragments['where'][] = "u.name NOT IN (". db_placeholders($names, 'text') .")";
+    $fragments['where'][] = "u.name NOT IN (".db_placeholders($names, 'text').")";
     $fragments['query_args']['where'] += $names;
   }
   // Only load active users and sort them by name.
@@ -1395,7 +1395,7 @@ 
 
   // Only select messages that have been deleted more than n days ago.
   $fragments['having'][] = 'MAX(pmi.deleted) < %d';
-  $fragments['query_args']['having'][] = time() - $days * 86400;
+  $fragments['query_args']['having'][] = time() - $days*86400;
 }
 
 /**
@@ -1423,10 +1423,10 @@ 
   if (!empty($fragment)) {
     $query = _privatemsg_assemble_query('autocomplete', $fragment, $names);
     $result = db_query_range($query['query'], $fragment, 0, 10);
-    $prefix = count($names) ? implode(", ", $names) .", " : '';
+    $prefix = count($names) ? implode(", ", $names).", " : '';
     // 3: Build proper suggestions and print.
     while ($user = db_fetch_object($result)) {
-      $matches[$prefix . $user->name .", "] = $user->name;
+      $matches[$prefix.$user->name.", "] = $user->name;
     }
   }
   // convert to object to prevent drupal bug, see http://drupal.org/node/175361
@@ -1468,7 +1468,7 @@ 
 
       if (!empty($mids)) {
         // Delete recipient entries in {pm_index} of the messages the user wrote.
-        db_query('DELETE FROM {pm_index} WHERE mid IN (' . db_placeholders($mids) . ')', $mids);
+        db_query('DELETE FROM {pm_index} WHERE mid IN ('.db_placeholders($mids).')', $mids);
       }
 
       // Delete recipient entries of that user.
@@ -1540,10 +1540,10 @@ 
   if (privatemsg_user_access('write privatemsg')) {
     $links[] = l(t('Write new message'), 'messages/new');
   }
-  if (privatemsg_user_access('read privatemsg') || privatemsg_user_access('read all private messages') ) {
+  if (privatemsg_user_access('read privatemsg') || privatemsg_user_access('read all private messages')) {
     $links[] = l(privatemsg_title_callback(), 'messages');
   }
-  if ( count( $links ) ) {
+  if (count($links)) {
     $block = array(
       'subject' => t('Private messages'),
       'content' => theme('item_list', $links),
@@ -1559,7 +1559,7 @@ 
   );
   $form['delete_destination'] = array(
     '#type' => 'value',
-    '#value' => count($thread['messages']) > 1 ? 'messages/view/' . $message['thread_id'] : 'messages',
+    '#value' => count($thread['messages']) > 1 ? 'messages/view/'.$message['thread_id'] : 'messages',
   );
 
   if (privatemsg_user_access('read all private messages')) {
@@ -1572,7 +1572,7 @@ 
   }
   return confirm_form($form,
     t('Are you sure you want to delete this message?'),
-    isset($_GET['destination']) ? $_GET['destination'] : 'messages/view/'. $message['thread_id'],
+    isset($_GET['destination']) ? $_GET['destination'] : 'messages/view/'.$message['thread_id'],
     t('This action cannot be undone.'),
     t('Delete'),
     t('Cancel')
@@ -1801,7 +1801,7 @@ 
   }
 
   // Don't allow replies without a body.
-  if (!empty($message['thread_id']) && ($message['body'] === NULL || $message['body'] === '') ) {
+  if (!empty($message['thread_id']) && ($message['body'] === NULL || $message['body'] === '')) {
     if ($form) {
       form_set_error('body', t('Disallowed to send reply without a message.'));
     }
@@ -1917,7 +1917,7 @@ 
   // 2) Save message to recipients.
   // Each recipient gets a record in the pm_index table.
   foreach ($message['recipients'] as $recipient) {
-    if (!db_query($index_sql, $mid, $message['thread_id'], $recipient->uid, 1) ) {
+    if (!db_query($index_sql, $mid, $message['thread_id'], $recipient->uid, 1)) {
       // We assume if one insert failed then the rest may fail too against the
       // same table.
       return FALSE;
@@ -1982,9 +1982,9 @@ 
   if (empty($validated)) {
     return FALSE;
   }
-  $url = 'messages/new/'. implode(',', $validated);
+  $url = 'messages/new/'.implode(',', $validated);
   if (!is_null($subject)) {
-    $url .= '/'. $subject;
+    $url .= '/'.$subject;
   }
   return $url;
 }
@@ -2094,7 +2094,7 @@ 
   // we do the merge because we call call_user_func_array and not drupal_alter
   // this is necessary because otherwise we would not be able to use $args correctly (otherwise it doesnt unfold)
   $alterargs = array(&$fragments);
-  $query_function = $query_group .'_sql_'. $query_id;
+  $query_function = $query_group.'_sql_'.$query_id;
   if (!empty($args)) {
     $alterargs = array_merge($alterargs, $args);
   }
@@ -2131,7 +2131,7 @@ 
   // Perform the whole query assembly only if we have something to select.
   if (!empty($SELECT)) {
     $str_select = implode(", ", $SELECT);
-    $query = "SELECT {$str_select} FROM ". $primary_table;
+    $query = "SELECT {$str_select} FROM ".$primary_table;
 
     // Also build a count query which can be passed to pager_query to get a "page count" as that does not play well with queries including "GROUP BY".
     // In most cases,  "COUNT(*)" is enough to get the count query, but in queries involving a GROUP BY, we want a count of the number of groups we have, not the count of elements inside each group.
@@ -2140,10 +2140,10 @@ 
       // PostgreSQL does not support COUNT(sometextfield, someintfield), so I'm only using the first one
       // Works fine for thread_id/list but may generate an error when a more complex GROUP BY is used.
       $str_group_by_count = current($GROUP_BY);
-      $count = "SELECT COUNT(DISTINCT {$str_group_by_count}) FROM ". $primary_table;
+      $count = "SELECT COUNT(DISTINCT {$str_group_by_count}) FROM ".$primary_table;
     }
     else {
-      $count = "SELECT COUNT(*) FROM ". $primary_table;
+      $count = "SELECT COUNT(*) FROM ".$primary_table;
     }
 
     if (!empty($INNER_JOIN)) {
@@ -2152,24 +2152,24 @@ 
       $count .= " {$str_inner_join}";
     }
     if (!empty($WHERE)) {
-      $str_where = '('. implode(') AND (', $WHERE) .')';
+      $str_where = '('.implode(') AND (', $WHERE).')';
       $query .= " WHERE {$str_where}";
       $count .= " WHERE {$str_where}";
     }
     if (!empty($GROUP_BY)) {
-      $str_group_by = ' GROUP BY '. implode(", ", $GROUP_BY) ;
+      $str_group_by = ' GROUP BY '.implode(", ", $GROUP_BY);
       $query .= " {$str_group_by}";
     }
     if (!empty($HAVING)) {
-      $str_having = '('. implode(') AND (', $HAVING) .')';
+      $str_having = '('.implode(') AND (', $HAVING).')';
       $query .= " HAVING {$str_having}";
       // queries containing a HAVING break the count query on pgsql.
       // In this case, use the subquery method as outlined in http://drupal.org/node/303087#comment-1370752 .
       // The subquery method will work for all COUNT queries, but it is thought to be much slower, so we are only using it where other cross database approaches fail.
-      $count = 'SELECT COUNT(*) FROM ('. $query .') as count';
+      $count = 'SELECT COUNT(*) FROM ('.$query.') as count';
     }
     if (!empty($ORDER_BY)) {
-      $str_order_by = ' ORDER BY '. implode(", ", $ORDER_BY) ;
+      $str_order_by = ' ORDER BY '.implode(", ", $ORDER_BY);
       $query .= " {$str_order_by}";
     }
     $QUERY_ARGS = array_merge($QUERY_ARGS['select'], $QUERY_ARGS['join'], $QUERY_ARGS['where'], $QUERY_ARGS['having']);
@@ -2234,7 +2234,7 @@ 
       '#attributes' => array('class' => 'privatemsg-action-button'),
   );
   // JS for hiding the execute button.
-  drupal_add_js(drupal_get_path('module', 'privatemsg') .'/privatemsg-list.js');
+  drupal_add_js(drupal_get_path('module', 'privatemsg').'/privatemsg-list.js');
   return $form;
 }
 
@@ -2259,7 +2259,7 @@ 
   }
   // Merge status and uid with the threads list. array_merge() will not overwrite/ignore thread_id 1.
   $params = array_merge(array($status, $account->uid), $threads);
-  db_query('UPDATE {pm_index} SET is_new = %d WHERE uid = %d AND thread_id IN ('. db_placeholders($threads) .')', $params);
+  db_query('UPDATE {pm_index} SET is_new = %d WHERE uid = %d AND thread_id IN ('.db_placeholders($threads).')', $params);
 
   if ($status == PRIVATEMSG_UNREAD) {
     drupal_set_message(format_plural(count($threads), 'Marked 1 thread as unread.', 'Marked @count threads as unread.'));
@@ -2286,12 +2286,12 @@ 
   $select_header['#weight'] = -50;
 
   // theme() doesn't include the theme file for patterns, we need to do it manually.
-  include_once drupal_get_path('module', 'privatemsg') .'/privatemsg.theme.inc';
+  include_once drupal_get_path('module', 'privatemsg').'/privatemsg.theme.inc';
 
   $header = array($select_header);
   foreach ($keys as $key) {
     // First, try to load a specific theme for that header, if not present, use the default.
-    if ($return = theme(array('privatemsg_list_header__'. $key, 'privatemsg_list_header'))) {
+    if ($return = theme(array('privatemsg_list_header__'.$key, 'privatemsg_list_header'))) {
       // The default theme returns nothing, only store the value if we have something.
       $header[$key] = $return;
     }
@@ -2312,7 +2312,7 @@ 
 function _privatemsg_list_headers_fallback($keys) {
   $header = array();
   foreach ($keys as $key) {
-    $theme_function = 'phptemplate_privatemsg_list_header__' . $key;
+    $theme_function = 'phptemplate_privatemsg_list_header__'.$key;
     if (function_exists($theme_function)) {
       $header[$key] = $theme_function();
     }
@@ -2340,7 +2340,7 @@ 
   }
   foreach ($thread as $key => $data) {
     // First, try to load a specific theme for that field, if not present, use the default.
-    if ($return = theme(array('privatemsg_list_field__'. $key, 'privatemsg_list_field'), $thread)) {
+    if ($return = theme(array('privatemsg_list_field__'.$key, 'privatemsg_list_field'), $thread)) {
       // The default theme returns nothing, only store the value if we have something.
       $row['data'][$key] = $return;
     }
@@ -2360,7 +2360,7 @@ 
 function _privatemsg_list_thread_fallback($thread) {
   $row_data = array();
   foreach ($thread as $key => $data) {
-    $theme_function = 'phptemplate_privatemsg_list_field__' . $key;
+    $theme_function = 'phptemplate_privatemsg_list_field__'.$key;
     if (function_exists($theme_function)) {
       $row_data[$key] = $theme_function($thread);
     }
@@ -2559,6 +2559,6 @@ 
 function privatemsg_views_api() {
   return array(
     'api' => 2,
-    'path' => drupal_get_path('module', 'privatemsg') . '/views',
+    'path' => drupal_get_path('module', 'privatemsg').'/views',
   );
 }

Braces +33 added lines, -66 removed lines

@@ -104,8 +104,7 @@ 
     $last = array_pop($to);
     if (count($to) == 0) { // Only one participant
       return t("From !last", array('!last' => $last));
-    }
-    else { // Multipe participants..
+    } else { // Multipe participants..
       $participants = implode(', ', $to);
       return t('Participants: !participants and !last', array('!participants' => $participants, '!last' => $last));
     }
@@ -325,11 +324,9 @@ 
       if (is_null($start)) {
         if (isset($_GET['start']) && $_GET['start'] < $thread['message_count']) {
           $start = $_GET['start'];
-        }
-        elseif (!variable_get('privatemsg_view_use_max_as_default', FALSE) && $max_amount == PRIVATEMSG_UNLIMITED) {
+        } elseif (!variable_get('privatemsg_view_use_max_as_default', FALSE) && $max_amount == PRIVATEMSG_UNLIMITED) {
           $start = PRIVATEMSG_UNLIMITED;
-        }
-        else {
+        } else {
           $start = $thread['message_count'] - (variable_get('privatemsg_view_use_max_as_default', FALSE) ? variable_get('privatemsg_view_default_amount', 10) : $max_amount);
         }
       }
@@ -338,8 +335,7 @@ 
         if ($max_amount == PRIVATEMSG_UNLIMITED) {
           $last_page = 0;
           $max_amount = $thread['message_count'];
-        }
-        else {
+        } else {
           // Calculate the number of messages on the "last" page to avoid
           // message overlap.
           // Note - the last page lists the earliest messages, not the latest.
@@ -360,8 +356,7 @@ 
         }
         if ($start - $max_amount >= 0) {
           $thread['older_start'] = $start - $max_amount;
-        }
-        elseif ($start > 0) {
+        } elseif ($start > 0) {
           $thread['older_start'] = 0;
         }
 
@@ -377,8 +372,7 @@ 
         // Visual counts start from 1 instead of zero, so plus one.
         $thread['from'] = $start + 1;
         $conversation = db_query_range($query['query'], $start, $max_amount);
-      }
-      else {
+      } else {
         $conversation = db_query($query['query']);
       }
       $mids = array();
@@ -398,12 +392,10 @@ 
             // Thread does not exist, display 404.
             $thread = FALSE;
           }
-        }
-        else {
+        } else {
           $thread = FALSE;
         }
-      }
-      else {
+      } else {
         // General data, assume subject is the same for all messages of that thread.
         $thread['user'] = $account;
         $message = current($thread['messages']);
@@ -689,8 +681,7 @@ 
     // Trying to view someone else's messages...
     if (!privatemsg_user_access('read all private messages')) {
       drupal_set_message(t("You do not have sufficient rights to view someone else's messages"), 'warning');
-    }
-    elseif ($account_check = user_load(array('uid' => $uid))) {
+    } elseif ($account_check = user_load(array('uid' => $uid))) {
       // Has rights and user_load return an array so user does exist
       $account = $account_check;
     }
@@ -852,11 +843,9 @@ 
   // convert recipients to array of user objects
   if (!empty($recipients) && is_string($recipients) || is_int($recipients)) {
     $recipients = _privatemsg_generate_user_array($recipients);
-  }
-  elseif (is_object($recipients)) {
+  } elseif (is_object($recipients)) {
     $recipients = array($recipients);
-  }
-  elseif (empty($recipients) && is_string($recipients)) {
+  } elseif (empty($recipients) && is_string($recipients)) {
     $recipients = array();
   }
 
@@ -879,8 +868,7 @@ 
       }
       $to[] = $recipient->name;
       $to_themed[$recipient->uid] = theme('username', $recipient);
-    }
-    else {
+    } else {
       // Recipient list contains blocked users.
       $blocked = TRUE;
     }
@@ -905,8 +893,7 @@ 
   }
   if (!$thread_id && !empty($recipients_string)) {
     drupal_set_title(t('Write new message to %recipient', array('%recipient' => $recipients_string)));
-  }
-  elseif (!$thread_id) {
+  } elseif (!$thread_id) {
     drupal_set_title(t('Write new message'));
   }
 
@@ -982,8 +969,7 @@ 
   $title = t('Cancel');
   if (isset($_REQUEST['destination'])) {
     $url = $_REQUEST['destination'];
-  }
-  elseif (!is_null($thread_id)) {
+  } elseif (!is_null($thread_id)) {
     $url = $_GET['q'];
     $title = t('Clear');
   }
@@ -1034,8 +1020,7 @@ 
   // Only parse the user string for a new thread.
   if (!isset($message['thread_id'])) {
     list($message['recipients'], $invalid) = _privatemsg_parse_userstring($message['recipient']);
-  }
-  else {
+  } else {
     // Load participants.
     $message['recipients'] = _privatemsg_load_thread_participants($message['thread_id']);
     // Remove author.
@@ -1133,8 +1118,7 @@ 
   }
   if ($status !== FALSE )  {
     drupal_set_message(t('A message has been sent to !recipients.', array('!recipients' => implode(', ', $recipient_names))));
-  }
-  else {
+  } else {
     drupal_set_message(t('An attempt to send a message <em>may have failed</em> when sending to !recipients.', array('!recipients' => implode(', ', $recipient_names))), 'error');
   }
 }
@@ -1195,8 +1179,7 @@ 
       $fragments['select'][]      = "array_to_string(array(SELECT DISTINCT textin(int4out(pmia.uid))
                                                             FROM {pm_index} pmia
                                                             WHERE pmia.thread_id = pmi.thread_id), ',') AS participants";
-    }
-    else {
+    } else {
       $fragments['select'][]      = '(SELECT GROUP_CONCAT(DISTINCT pmia.uid SEPARATOR ",")
                                                             FROM {pm_index} pmia
                                                             WHERE pmia.thread_id = pmi.thread_id) AS participants';
@@ -1490,8 +1473,7 @@ 
     );
 
     return $blocks;
-  }
-  elseif ('view' == $op) {
+  } elseif ('view' == $op) {
     $block = array();
     switch ($delta) {
       case 'privatemsg-menu':
@@ -1587,8 +1569,7 @@ 
     if (isset($form_state['values']['delete_options']) && $form_state['values']['delete_options']) {
       privatemsg_message_change_delete($form_state['values']['pmid'], 1);
       drupal_set_message(t('Message has been deleted for all users.'));
-    }
-    else {
+    } else {
       privatemsg_message_change_delete($form_state['values']['pmid'], 1, $account);
       drupal_set_message(t('Message has been deleted.'));
     }
@@ -1617,8 +1598,7 @@ 
 
   if ($account) {
     db_query('UPDATE {pm_index} SET deleted = %d WHERE mid = %d AND uid = %d', $delete_value, $pmid, $account->uid);
-  }
-  else {
+  } else {
     // Mark deleted for all users.
     db_query('UPDATE {pm_index} SET deleted = %d WHERE mid = %d', $delete_value, $pmid);
   }
@@ -1779,8 +1759,7 @@ 
         'success'  => FALSE,
         'messages'   => $messages,
       );
-    }
-    else {
+    } else {
       $messages['error'][] = t('User @user is not allowed to write messages', array('@user' => $message['author']->name));
       return array(
          'success'  => FALSE,
@@ -1794,8 +1773,7 @@ 
   if (empty($message['subject'])) {
     if ($form) {
       form_set_error('subject', t('Disallowed to send a message without subject'));
-    }
-    else {
+    } else {
       $messages['error'][] = t('Disallowed to send a message without subject');
     }
   }
@@ -1804,8 +1782,7 @@ 
   if (!empty($message['thread_id']) && ($message['body'] === NULL || $message['body'] === '') ) {
     if ($form) {
       form_set_error('body', t('Disallowed to send reply without a message.'));
-    }
-    else {
+    } else {
       $messages['error'][] = t('Disallowed to send reply without a message.');
     }
   }
@@ -1819,8 +1796,7 @@ 
   if (!filter_access($message['format'])) {
     if ($form) {
       form_set_error('format', t('You are not allowed to use the specified input format.'));
-    }
-    else {
+    } else {
       $messages['error'][] = t('User @user is not allowed to use the specified input format.', array('@user' => $message['author']->name));
     }
   }
@@ -1831,8 +1807,7 @@ 
   if (empty($message['recipients']) || !is_array($message['recipients'])) {
     if ($form) {
       form_set_error('to', t('Disallowed to send a message without at least one valid recipient'));
-    }
-    else {
+    } else {
       $messages['error'][] = t('Disallowed to send a message without at least one valid recipient');
     }
   }
@@ -1842,8 +1817,7 @@ 
       unset($message['recipients'][$blocked['uid']]);
       if ($form) {
         drupal_set_message($blocked['message'], 'warning');
-      }
-      else {
+      } else {
         $messages['warning'][] = $blocked['message'];
       }
     }
@@ -1853,8 +1827,7 @@ 
   if (empty($message['recipients'])) {
     if ($form) {
       form_set_error('to', t('Disallowed to send message because all recipients are blocked'));
-    }
-    else {
+    } else {
       $messages['error'][] = t('Disallowed to send message because all recipients are blocked');
     }
   }
@@ -2060,8 +2033,7 @@ 
   if (is_array($query)) {
     $query_id = $query[0];
     $query_group = $query[1];
-  }
-  else {
+  } else {
     $query_id = $query;
     $query_group = 'privatemsg';
   }
@@ -2141,8 +2113,7 @@ 
       // Works fine for thread_id/list but may generate an error when a more complex GROUP BY is used.
       $str_group_by_count = current($GROUP_BY);
       $count = "SELECT COUNT(DISTINCT {$str_group_by_count}) FROM ". $primary_table;
-    }
-    else {
+    } else {
       $count = "SELECT COUNT(*) FROM ". $primary_table;
     }
 
@@ -2263,8 +2234,7 @@ 
 
   if ($status == PRIVATEMSG_UNREAD) {
     drupal_set_message(format_plural(count($threads), 'Marked 1 thread as unread.', 'Marked @count threads as unread.'));
-  }
-  else {
+  } else {
     drupal_set_message(format_plural(count($threads), 'Marked 1 thread as read.', 'Marked @count threads as read.'));
   }
 }
@@ -2445,8 +2415,7 @@ 
   // Add in callback arguments if present.
   if (isset($operation['callback arguments'])) {
     $args = array_merge(array($threads), $operation['callback arguments']);
-  }
-  else {
+  } else {
     $args = array($threads);
   }
 
@@ -2463,8 +2432,7 @@ 
     // Add in callback arguments if present.
     if (isset($operation['undo callback arguments'])) {
       $undo_args = array_merge(array($threads), $operation['undo callback arguments']);
-    }
-    else {
+    } else {
       $undo_args = array($threads);
     }
 
@@ -2516,8 +2484,7 @@ 
 
   if ($delete) {
     drupal_set_message(format_plural(count($threads), 'Deleted 1 thread.', 'Deleted @count threads.'));
-  }
-  else {
+  } else {
     drupal_set_message(format_plural(count($threads), 'Restored 1 thread.', 'Restored @count threads.'));
   }
 }

drupal/sites/default/boinc/modules/contrib/wysiwyg/editors/ckeditor.inc

Doc Comments +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
  *   An array containing editor properties as returned from hook_editor().
  *
  * @return
- *   The installed editor version.
+ null|string   The installed editor version.
  */
 function wysiwyg_ckeditor_version($editor) {
   $library = $editor['library path'] . '/ckeditor.js';

Indentation +122 added lines, -122 removed lines

@@ -9,23 +9,23 @@ 
  * Plugin implementation of hook_editor().
  */
 function wysiwyg_ckeditor_editor() {
-  $editor['ckeditor'] = array(
+    $editor['ckeditor'] = array(
     'title' => 'CKEditor',
     'vendor url' => 'http://ckeditor.com',
     'download url' => 'http://ckeditor.com/download',
     'libraries' => array(
-      '' => array(
+        '' => array(
         'title' => 'Default',
         'files' => array(
-          'ckeditor.js' => array('preprocess' => FALSE),
+            'ckeditor.js' => array('preprocess' => FALSE),
         ),
-      ),
-      'src' => array(
+        ),
+        'src' => array(
         'title' => 'Source',
         'files' => array(
-          'ckeditor_source.js' => array('preprocess' => FALSE),
+            'ckeditor_source.js' => array('preprocess' => FALSE),
+        ),
         ),
-      ),
     ),
     'version callback' => 'wysiwyg_ckeditor_version',
     'themes callback' => 'wysiwyg_ckeditor_themes',
@@ -33,19 +33,19 @@ 
     'plugin callback' => 'wysiwyg_ckeditor_plugins',
     'plugin settings callback' => 'wysiwyg_ckeditor_plugin_settings',
     'proxy plugin' => array(
-      'drupal' => array(
+        'drupal' => array(
         'load' => TRUE,
         'proxy' => TRUE,
-      ),
+        ),
     ),
     'proxy plugin settings callback' => 'wysiwyg_ckeditor_proxy_plugin_settings',
     'versions' => array(
-      '3.0.0.3665' => array(
+        '3.0.0.3665' => array(
         'js files' => array('ckeditor-3.0.js'),
-      ),
+        ),
     ),
-  );
-  return $editor;
+    );
+    return $editor;
 }
 
 /**
@@ -58,25 +58,25 @@ 
  *   The installed editor version.
  */
 function wysiwyg_ckeditor_version($editor) {
-  $library = $editor['library path'] . '/ckeditor.js';
-  if (!file_exists($library)) {
+    $library = $editor['library path'] . '/ckeditor.js';
+    if (!file_exists($library)) {
     return;
-  }
-  $library = fopen($library, 'r');
-  $max_lines = 8;
-  while ($max_lines && $line = fgets($library, 500)) {
+    }
+    $library = fopen($library, 'r');
+    $max_lines = 8;
+    while ($max_lines && $line = fgets($library, 500)) {
     // version:'CKEditor 3.0 SVN',revision:'3665'
     // version:'3.0 RC',revision:'3753'
     // version:'3.0.1',revision:'4391'
     if (preg_match('@version:\'(?:CKEditor )?([\d\.]+)(?:.+revision:\'([\d]+))?@', $line, $version)) {
-      fclose($library);
-      // Version numbers need to have three parts since 3.0.1.
-      $version[1] = preg_replace('/^(\d+)\.(\d+)$/', '${1}.${2}.0', $version[1]);
-      return $version[1] . '.' . $version[2];
+        fclose($library);
+        // Version numbers need to have three parts since 3.0.1.
+        $version[1] = preg_replace('/^(\d+)\.(\d+)$/', '${1}.${2}.0', $version[1]);
+        return $version[1] . '.' . $version[2];
     }
     $max_lines--;
-  }
-  fclose($library);
+    }
+    fclose($library);
 }
 
 /**
@@ -92,23 +92,23 @@ 
  *   theme name.
  */
 function wysiwyg_ckeditor_themes($editor, $profile) {
-  // @todo Skins are not themes but this will do for now.
-  $path = $editor['library path'] . '/skins/';
-  if (file_exists($path) && ($dir_handle = opendir($path))) {
+    // @todo Skins are not themes but this will do for now.
+    $path = $editor['library path'] . '/skins/';
+    if (file_exists($path) && ($dir_handle = opendir($path))) {
     $themes = array();
     while ($file = readdir($dir_handle)) {
-      if (is_dir($path . $file) && substr($file, 0, 1) != '.' && $file != 'CVS') {
+        if (is_dir($path . $file) && substr($file, 0, 1) != '.' && $file != 'CVS') {
         $themes[] = $file;
-      }
+        }
     }
     closedir($dir_handle);
     natcasesort($themes);
     $themes = array_values($themes);
     return !empty($themes) ? $themes : array('default');
-  }
-  else {
+    }
+    else {
     return array('default');
-  }
+    }
 }
 
 /**
@@ -126,7 +126,7 @@ 
  *   Drupal.settings.wysiwyg.configs.{editor}
  */
 function wysiwyg_ckeditor_settings($editor, $config, $theme) {
-  $settings = array(
+    $settings = array(
     'baseHref' => $GLOBALS['base_url'] . '/',
     'width' => '100%',
     // For better compatibility with smaller textareas.
@@ -142,155 +142,155 @@ 
     // @todo Check whether completely disabling ProcessHTMLEntities is an option.
     'entities_latin' => FALSE,
     'entities_greek' => FALSE,
-  );
+    );
 
-  // Add HTML block format settings; common block formats are already predefined
-  // by CKEditor.
-  if (isset($config['block_formats'])) {
+    // Add HTML block format settings; common block formats are already predefined
+    // by CKEditor.
+    if (isset($config['block_formats'])) {
     $block_formats = explode(',', drupal_strtolower($config['block_formats']));
     $predefined_formats = array('h1', 'h2', 'h3', 'h4', 'h5', 'h6', 'p', 'pre', 'address', 'div');
     foreach (array_diff($block_formats, $predefined_formats) as $tag) {
-      $tag = trim($tag);
-      $settings["format_$tag"] = array('element' => $tag);
+        $tag = trim($tag);
+        $settings["format_$tag"] = array('element' => $tag);
     }
     $settings['format_tags'] = implode(';', $block_formats);
-  }
+    }
 
-  if (isset($config['apply_source_formatting'])) {
+    if (isset($config['apply_source_formatting'])) {
     $settings['apply_source_formatting'] = $config['apply_source_formatting'];
-  }
+    }
 
-  if (isset($config['css_setting'])) {
+    if (isset($config['css_setting'])) {
     // Versions below 3.0.1 could only handle one stylesheet.
     if (version_compare($editor['installed version'], '3.0.1.4391', '<')) {
-      if ($config['css_setting'] == 'theme') {
+        if ($config['css_setting'] == 'theme') {
         $settings['contentsCss'] = reset(wysiwyg_get_css());
-      }
-      elseif ($config['css_setting'] == 'self' && isset($config['css_path'])) {
+        }
+        elseif ($config['css_setting'] == 'self' && isset($config['css_path'])) {
         $settings['contentsCss'] = strtr($config['css_path'], array('%b' => base_path(), '%t' => path_to_theme()));
-      }
+        }
     }
     else {
-      if ($config['css_setting'] == 'theme') {
+        if ($config['css_setting'] == 'theme') {
         $settings['contentsCss'] = wysiwyg_get_css();
-      }
-      elseif ($config['css_setting'] == 'self' && isset($config['css_path'])) {
+        }
+        elseif ($config['css_setting'] == 'self' && isset($config['css_path'])) {
         $settings['contentsCss'] = explode(',', strtr($config['css_path'], array('%b' => base_path(), '%t' => path_to_theme())));
-      }
+        }
+    }
     }
-  }
 
-  if (isset($config['language'])) {
+    if (isset($config['language'])) {
     $settings['language'] = $config['language'];
-  }
-  if (isset($config['resizing'])) {
+    }
+    if (isset($config['resizing'])) {
     // CKEditor tests "!== false", so ensure it is a Boolean.
     $settings['resize_enabled'] = (bool) $config['resizing'];
-  }
-  if (isset($config['toolbar_loc'])) {
+    }
+    if (isset($config['toolbar_loc'])) {
     $settings['toolbarLocation'] = $config['toolbar_loc'];
-  }
+    }
 
-  $settings['toolbar'] = array();
-  if (!empty($config['buttons'])) {
+    $settings['toolbar'] = array();
+    if (!empty($config['buttons'])) {
     $extra_plugins = array();
     $plugins = wysiwyg_get_plugins($editor['name']);
     foreach ($config['buttons'] as $plugin => $buttons) {
-      foreach ($buttons as $button => $enabled) {
+        foreach ($buttons as $button => $enabled) {
         // Iterate separately over buttons and extensions properties.
         foreach (array('buttons', 'extensions') as $type) {
-          // Skip unavailable plugins.
-          if (!isset($plugins[$plugin][$type][$button])) {
+            // Skip unavailable plugins.
+            if (!isset($plugins[$plugin][$type][$button])) {
             continue;
-          }
-          // Add buttons.
-          if ($type == 'buttons') {
+            }
+            // Add buttons.
+            if ($type == 'buttons') {
             $settings['toolbar'][] = $button;
-          }
-          // Add external Drupal plugins to the list of extensions.
-          if ($type == 'buttons' && !empty($plugins[$plugin]['proxy'])) {
+            }
+            // Add external Drupal plugins to the list of extensions.
+            if ($type == 'buttons' && !empty($plugins[$plugin]['proxy'])) {
             $extra_plugins[] = $button;
-          }
-          // Add external plugins to the list of extensions.
-          elseif ($type == 'buttons' && empty($plugins[$plugin]['internal'])) {
+            }
+            // Add external plugins to the list of extensions.
+            elseif ($type == 'buttons' && empty($plugins[$plugin]['internal'])) {
             $extra_plugins[] = $plugin;
-          }
-          // Add internal buttons that also need to be loaded as extension.
-          elseif ($type == 'buttons' && !empty($plugins[$plugin]['load'])) {
+            }
+            // Add internal buttons that also need to be loaded as extension.
+            elseif ($type == 'buttons' && !empty($plugins[$plugin]['load'])) {
             $extra_plugins[] = $plugin;
-          }
-          // Add plain extensions.
-          elseif ($type == 'extensions' && !empty($plugins[$plugin]['load'])) {
+            }
+            // Add plain extensions.
+            elseif ($type == 'extensions' && !empty($plugins[$plugin]['load'])) {
             $extra_plugins[] = $plugin;
-          }
-          // Allow plugins to add or override global configuration settings.
-          if (!empty($plugins[$plugin]['options'])) {
+            }
+            // Allow plugins to add or override global configuration settings.
+            if (!empty($plugins[$plugin]['options'])) {
             $settings = array_merge($settings, $plugins[$plugin]['options']);
-          }
+            }
+        }
         }
-      }
     }
     if (!empty($extra_plugins)) {
-      $settings['extraPlugins'] = implode(',', $extra_plugins);
+        $settings['extraPlugins'] = implode(',', $extra_plugins);
     }
-  }
-  // For now, all buttons are placed into one row.
-  $settings['toolbar'] = array($settings['toolbar']);
+    }
+    // For now, all buttons are placed into one row.
+    $settings['toolbar'] = array($settings['toolbar']);
 
-  return $settings;
+    return $settings;
 }
 
 /**
  * Build a JS settings array of native external plugins that need to be loaded separately.
  */
 function wysiwyg_ckeditor_plugin_settings($editor, $profile, $plugins) {
-  $settings = array();
-  foreach ($plugins as $name => $plugin) {
+    $settings = array();
+    foreach ($plugins as $name => $plugin) {
     // Register all plugins that need to be loaded.
     if (!empty($plugin['load'])) {
-      $settings[$name] = array();
-      // Add path for native external plugins.
-      if (empty($plugin['internal']) && isset($plugin['path'])) {
+        $settings[$name] = array();
+        // Add path for native external plugins.
+        if (empty($plugin['internal']) && isset($plugin['path'])) {
         $settings[$name]['path'] = base_path() . $plugin['path'] . '/';
-      }
-      // Force native internal plugins to use the standard path.
-      else {
+        }
+        // Force native internal plugins to use the standard path.
+        else {
         $settings[$name]['path'] = base_path() . $editor['library path'] . '/plugins/' . $name . '/';
-      }
-      // CKEditor defaults to 'plugin.js' on its own when filename is not set.
-      if (!empty($plugin['filename'])) {
+        }
+        // CKEditor defaults to 'plugin.js' on its own when filename is not set.
+        if (!empty($plugin['filename'])) {
         $settings[$name]['fileName'] = $plugin['filename'];
-      }
+        }
     }
-  }
-  return $settings;
+    }
+    return $settings;
 }
 
 /**
  * Build a JS settings array for Drupal plugins loaded via the proxy plugin.
  */
 function wysiwyg_ckeditor_proxy_plugin_settings($editor, $profile, $plugins) {
-  $settings = array();
-  foreach ($plugins as $name => $plugin) {
+    $settings = array();
+    foreach ($plugins as $name => $plugin) {
     // Populate required plugin settings.
     $settings[$name] = $plugin['dialog settings'] + array(
-      'title' => $plugin['title'],
-      'icon' => base_path() . $plugin['icon path'] . '/' . $plugin['icon file'],
-      'iconTitle' => $plugin['icon title'],
-      // @todo These should only be set if the plugin defined them.
-      'css' => base_path() . $plugin['css path'] . '/' . $plugin['css file'],
+        'title' => $plugin['title'],
+        'icon' => base_path() . $plugin['icon path'] . '/' . $plugin['icon file'],
+        'iconTitle' => $plugin['icon title'],
+        // @todo These should only be set if the plugin defined them.
+        'css' => base_path() . $plugin['css path'] . '/' . $plugin['css file'],
     );
-  }
-  return $settings;
+    }
+    return $settings;
 }
 
 /**
  * Return internal plugins for this editor; semi-implementation of hook_wysiwyg_plugin().
  */
 function wysiwyg_ckeditor_plugins($editor) {
-  $plugins = array(
+    $plugins = array(
     'default' => array(
-      'buttons' => array(
+        'buttons' => array(
         'Bold' => t('Bold'), 'Italic' => t('Italic'), 'Underline' => t('Underline'),
         'Strike' => t('Strike-through'),
         'JustifyLeft' => t('Align left'), 'JustifyCenter' => t('Align center'), 'JustifyRight' => t('Align right'), 'JustifyBlock' => t('Justify'),
@@ -317,17 +317,17 @@ 
         'Maximize' => t('Maximize'),
         'SpellChecker' => t('Check spelling'), 'Scayt' => t('Check spelling as you type'),
         'About' => t('About'),
-      ),
-      'internal' => TRUE,
+        ),
+        'internal' => TRUE,
     ),
-  );
+    );
 
-  if (version_compare($editor['installed version'], '3.1.0.4885', '<')) {
+    if (version_compare($editor['installed version'], '3.1.0.4885', '<')) {
     unset($plugins['default']['buttons']['CreateDiv']);
-  }
-  if (version_compare($editor['installed version'], '3.5.0.6260', '<')) {
+    }
+    if (version_compare($editor['installed version'], '3.5.0.6260', '<')) {
     unset($plugins['default']['buttons']['Iframe']);
-  }
-  return $plugins;
+    }
+    return $plugins;
 }
 

Spacing +10 added lines, -10 removed lines

@@ -58,7 +58,7 @@ 
  *   The installed editor version.
  */
 function wysiwyg_ckeditor_version($editor) {
-  $library = $editor['library path'] . '/ckeditor.js';
+  $library = $editor['library path'].'/ckeditor.js';
   if (!file_exists($library)) {
     return;
   }
@@ -72,7 +72,7 @@ 
       fclose($library);
       // Version numbers need to have three parts since 3.0.1.
       $version[1] = preg_replace('/^(\d+)\.(\d+)$/', '${1}.${2}.0', $version[1]);
-      return $version[1] . '.' . $version[2];
+      return $version[1].'.'.$version[2];
     }
     $max_lines--;
   }
@@ -93,11 +93,11 @@ 
  */
 function wysiwyg_ckeditor_themes($editor, $profile) {
   // @todo Skins are not themes but this will do for now.
-  $path = $editor['library path'] . '/skins/';
+  $path = $editor['library path'].'/skins/';
   if (file_exists($path) && ($dir_handle = opendir($path))) {
     $themes = array();
     while ($file = readdir($dir_handle)) {
-      if (is_dir($path . $file) && substr($file, 0, 1) != '.' && $file != 'CVS') {
+      if (is_dir($path.$file) && substr($file, 0, 1) != '.' && $file != 'CVS') {
         $themes[] = $file;
       }
     }
@@ -127,7 +127,7 @@ 
  */
 function wysiwyg_ckeditor_settings($editor, $config, $theme) {
   $settings = array(
-    'baseHref' => $GLOBALS['base_url'] . '/',
+    'baseHref' => $GLOBALS['base_url'].'/',
     'width' => '100%',
     // For better compatibility with smaller textareas.
     'resize_minWidth' => 450,
@@ -185,7 +185,7 @@ 
   }
   if (isset($config['resizing'])) {
     // CKEditor tests "!== false", so ensure it is a Boolean.
-    $settings['resize_enabled'] = (bool) $config['resizing'];
+    $settings['resize_enabled'] = (bool)$config['resizing'];
   }
   if (isset($config['toolbar_loc'])) {
     $settings['toolbarLocation'] = $config['toolbar_loc'];
@@ -251,11 +251,11 @@ 
       $settings[$name] = array();
       // Add path for native external plugins.
       if (empty($plugin['internal']) && isset($plugin['path'])) {
-        $settings[$name]['path'] = base_path() . $plugin['path'] . '/';
+        $settings[$name]['path'] = base_path().$plugin['path'].'/';
       }
       // Force native internal plugins to use the standard path.
       else {
-        $settings[$name]['path'] = base_path() . $editor['library path'] . '/plugins/' . $name . '/';
+        $settings[$name]['path'] = base_path().$editor['library path'].'/plugins/'.$name.'/';
       }
       // CKEditor defaults to 'plugin.js' on its own when filename is not set.
       if (!empty($plugin['filename'])) {
@@ -275,10 +275,10 @@ 
     // Populate required plugin settings.
     $settings[$name] = $plugin['dialog settings'] + array(
       'title' => $plugin['title'],
-      'icon' => base_path() . $plugin['icon path'] . '/' . $plugin['icon file'],
+      'icon' => base_path().$plugin['icon path'].'/'.$plugin['icon file'],
       'iconTitle' => $plugin['icon title'],
       // @todo These should only be set if the plugin defined them.
-      'css' => base_path() . $plugin['css path'] . '/' . $plugin['css file'],
+      'css' => base_path().$plugin['css path'].'/'.$plugin['css file'],
     );
   }
   return $settings;

Braces +4 added lines, -8 removed lines

@@ -105,8 +105,7 @@ 
     natcasesort($themes);
     $themes = array_values($themes);
     return !empty($themes) ? $themes : array('default');
-  }
-  else {
+  } else {
     return array('default');
   }
 }
@@ -165,16 +164,13 @@ 
     if (version_compare($editor['installed version'], '3.0.1.4391', '<')) {
       if ($config['css_setting'] == 'theme') {
         $settings['contentsCss'] = reset(wysiwyg_get_css());
-      }
-      elseif ($config['css_setting'] == 'self' && isset($config['css_path'])) {
+      } elseif ($config['css_setting'] == 'self' && isset($config['css_path'])) {
         $settings['contentsCss'] = strtr($config['css_path'], array('%b' => base_path(), '%t' => path_to_theme()));
       }
-    }
-    else {
+    } else {
       if ($config['css_setting'] == 'theme') {
         $settings['contentsCss'] = wysiwyg_get_css();
-      }
-      elseif ($config['css_setting'] == 'self' && isset($config['css_path'])) {
+      } elseif ($config['css_setting'] == 'self' && isset($config['css_path'])) {
         $settings['contentsCss'] = explode(',', strtr($config['css_path'], array('%b' => base_path(), '%t' => path_to_theme())));
       }
     }

drupal/sites/default/boinc/modules/contrib/wysiwyg/editors/fckeditor.inc

Doc Comments +2 added lines, -2 removed lines

@@ -47,7 +47,7 @@ 
  *   An array containing editor properties as returned from hook_editor().
  *
  * @return
- *   The installed editor version.
+ null|string   The installed editor version.
  */
 function wysiwyg_fckeditor_version($editor) {
   $library = $editor['library path'] . '/fckeditor.js';
@@ -75,7 +75,7 @@ 
  *   A wysiwyg editor profile.
  *
  * @return
- *   An array of theme names. The first returned name should be the default
+ string[]   An array of theme names. The first returned name should be the default
  *   theme name.
  */
 function wysiwyg_fckeditor_themes($editor, $profile) {

Indentation +99 added lines, -99 removed lines

@@ -9,15 +9,15 @@ 
  * Plugin implementation of hook_editor().
  */
 function wysiwyg_fckeditor_editor() {
-  $editor['fckeditor'] = array(
+    $editor['fckeditor'] = array(
     'title' => 'FCKeditor',
     'vendor url' => 'http://www.fckeditor.net',
     'download url' => 'http://www.fckeditor.net/download',
     'libraries' => array(
-      '' => array(
+        '' => array(
         'title' => 'Default',
         'files' => array('fckeditor.js'),
-      ),
+        ),
     ),
     'version callback' => 'wysiwyg_fckeditor_version',
     'themes callback' => 'wysiwyg_fckeditor_themes',
@@ -25,19 +25,19 @@ 
     'plugin callback' => 'wysiwyg_fckeditor_plugins',
     'plugin settings callback' => 'wysiwyg_fckeditor_plugin_settings',
     'proxy plugin' => array(
-      'drupal' => array(
+        'drupal' => array(
         'load' => TRUE,
         'proxy' => TRUE,
-      ),
+        ),
     ),
     'proxy plugin settings callback' => 'wysiwyg_fckeditor_proxy_plugin_settings',
     'versions' => array(
-      '2.6' => array(
+        '2.6' => array(
         'js files' => array('fckeditor-2.6.js'),
-      ),
+        ),
     ),
-  );
-  return $editor;
+    );
+    return $editor;
 }
 
 /**
@@ -50,20 +50,20 @@ 
  *   The installed editor version.
  */
 function wysiwyg_fckeditor_version($editor) {
-  $library = $editor['library path'] . '/fckeditor.js';
-  if (!file_exists($library)) {
+    $library = $editor['library path'] . '/fckeditor.js';
+    if (!file_exists($library)) {
     return;
-  }
-  $library = fopen($library, 'r');
-  $max_lines = 100;
-  while ($max_lines && $line = fgets($library, 60)) {
+    }
+    $library = fopen($library, 'r');
+    $max_lines = 100;
+    while ($max_lines && $line = fgets($library, 60)) {
     if (preg_match('@^FCKeditor.prototype.Version\s*= \'([\d\.]+)@', $line, $version)) {
-      fclose($library);
-      return $version[1];
+        fclose($library);
+        return $version[1];
     }
     $max_lines--;
-  }
-  fclose($library);
+    }
+    fclose($library);
 }
 
 /**
@@ -79,7 +79,7 @@ 
  *   theme name.
  */
 function wysiwyg_fckeditor_themes($editor, $profile) {
-  return array('default', 'office2003', 'silver');
+    return array('default', 'office2003', 'silver');
 }
 
 /**
@@ -97,7 +97,7 @@ 
  *   Drupal.settings.wysiwyg.configs.{editor}
  */
 function wysiwyg_fckeditor_settings($editor, $config, $theme) {
-  $settings = array(
+    $settings = array(
     'EditorPath' => base_path() . $editor['library path'] . '/',
     'SkinPath' => base_path() . $editor['library path'] . '/editor/skins/' . $theme . '/',
     'CustomConfigurationsPath' => base_path() . drupal_get_path('module', 'wysiwyg') . '/editors/js/fckeditor.config.js',
@@ -116,105 +116,105 @@ 
     // @todo Check whether completely disabling ProcessHTMLEntities is an option.
     'IncludeLatinEntities' => FALSE,
     'IncludeGreekEntities' => FALSE,
-  );
-  if (isset($config['block_formats'])) {
+    );
+    if (isset($config['block_formats'])) {
     $settings['FontFormats'] = strtr($config['block_formats'], array(',' => ';'));
-  }
-  if (isset($config['apply_source_formatting'])) {
+    }
+    if (isset($config['apply_source_formatting'])) {
     $settings['FormatOutput'] = $settings['FormatSource'] = $config['apply_source_formatting'];
-  }
-  if (isset($config['paste_auto_cleanup_on_paste'])) {
+    }
+    if (isset($config['paste_auto_cleanup_on_paste'])) {
     $settings['AutoDetectPasteFromWord'] = $config['paste_auto_cleanup_on_paste'];
-  }
+    }
 
-  if (isset($config['css_setting'])) {
+    if (isset($config['css_setting'])) {
     if ($config['css_setting'] == 'theme') {
-      $settings['EditorAreaCSS'] = implode(',', wysiwyg_get_css());
+        $settings['EditorAreaCSS'] = implode(',', wysiwyg_get_css());
     }
     else if ($config['css_setting'] == 'self' && isset($config['css_path'])) {
-      $settings['EditorAreaCSS'] = strtr($config['css_path'], array('%b' => base_path(), '%t' => path_to_theme()));
+        $settings['EditorAreaCSS'] = strtr($config['css_path'], array('%b' => base_path(), '%t' => path_to_theme()));
+    }
     }
-  }
 
-  // Use our custom toolbar set.
-  $settings['ToolbarSet'] = 'Wysiwyg';
-  // Populate our custom toolbar set for fckeditor.config.js.
-  $settings['buttons'] = array();
-  if (!empty($config['buttons'])) {
+    // Use our custom toolbar set.
+    $settings['ToolbarSet'] = 'Wysiwyg';
+    // Populate our custom toolbar set for fckeditor.config.js.
+    $settings['buttons'] = array();
+    if (!empty($config['buttons'])) {
     $plugins = wysiwyg_get_plugins($editor['name']);
     foreach ($config['buttons'] as $plugin => $buttons) {
-      foreach ($buttons as $button => $enabled) {
+        foreach ($buttons as $button => $enabled) {
         // Iterate separately over buttons and extensions properties.
         foreach (array('buttons', 'extensions') as $type) {
-          // Skip unavailable plugins.
-          if (!isset($plugins[$plugin][$type][$button])) {
+            // Skip unavailable plugins.
+            if (!isset($plugins[$plugin][$type][$button])) {
             continue;
-          }
-          // Add buttons.
-          if ($type == 'buttons') {
+            }
+            // Add buttons.
+            if ($type == 'buttons') {
             $settings['buttons'][] = $button;
-          }
-          // Allow plugins to add or override global configuration settings.
-          if (!empty($plugins[$plugin]['options'])) {
+            }
+            // Allow plugins to add or override global configuration settings.
+            if (!empty($plugins[$plugin]['options'])) {
             $settings = array_merge($settings, $plugins[$plugin]['options']);
-          }
+            }
         }
-      }
+        }
+    }
     }
-  }
-  // For now, all buttons are placed into one row.
-  $settings['buttons'] = array($settings['buttons']);
+    // For now, all buttons are placed into one row.
+    $settings['buttons'] = array($settings['buttons']);
 
-  return $settings;
+    return $settings;
 }
 
 /**
  * Build a JS settings array of native external plugins that need to be loaded separately.
  */
 function wysiwyg_fckeditor_plugin_settings($editor, $profile, $plugins) {
-  $settings = array();
-  foreach ($plugins as $name => $plugin) {
+    $settings = array();
+    foreach ($plugins as $name => $plugin) {
     // Register all plugins that need to be loaded.
     if (!empty($plugin['load'])) {
-      $settings[$name] = array();
-      // Add path for native external plugins; internal ones do not need a path.
-      if (empty($plugin['internal']) && isset($plugin['path'])) {
+        $settings[$name] = array();
+        // Add path for native external plugins; internal ones do not need a path.
+        if (empty($plugin['internal']) && isset($plugin['path'])) {
         // All native FCKeditor plugins use the filename fckplugin.js.
         $settings[$name]['path'] = base_path() . $plugin['path'] . '/';
-      }
-      if (!empty($plugin['languages'])) {
+        }
+        if (!empty($plugin['languages'])) {
         $settings[$name]['languages'] = $plugin['languages'];
-      }
+        }
+    }
     }
-  }
-  return $settings;
+    return $settings;
 }
 
 /**
  * Build a JS settings array for Drupal plugins loaded via the proxy plugin.
  */
 function wysiwyg_fckeditor_proxy_plugin_settings($editor, $profile, $plugins) {
-  $settings = array();
-  foreach ($plugins as $name => $plugin) {
+    $settings = array();
+    foreach ($plugins as $name => $plugin) {
     // Populate required plugin settings.
     $settings[$name] = $plugin['dialog settings'] + array(
-      'title' => $plugin['title'],
-      'icon' => base_path() . $plugin['icon path'] . '/' . $plugin['icon file'],
-      'iconTitle' => $plugin['icon title'],
-      // @todo These should only be set if the plugin defined them.
-      'css' => base_path() . $plugin['css path'] . '/' . $plugin['css file'],
+        'title' => $plugin['title'],
+        'icon' => base_path() . $plugin['icon path'] . '/' . $plugin['icon file'],
+        'iconTitle' => $plugin['icon title'],
+        // @todo These should only be set if the plugin defined them.
+        'css' => base_path() . $plugin['css path'] . '/' . $plugin['css file'],
     );
-  }
-  return $settings;
+    }
+    return $settings;
 }
 
 /**
  * Return internal plugins for this editor; semi-implementation of hook_wysiwyg_plugin().
  */
 function wysiwyg_fckeditor_plugins($editor) {
-  $plugins = array(
+    $plugins = array(
     'default' => array(
-      'buttons' => array(
+        'buttons' => array(
         'Bold' => t('Bold'), 'Italic' => t('Italic'), 'Underline' => t('Underline'),
         'StrikeThrough' => t('Strike-through'),
         'JustifyLeft' => t('Align left'), 'JustifyCenter' => t('Align center'), 'JustifyRight' => t('Align right'), 'JustifyFull' => t('Justify'),
@@ -240,39 +240,39 @@ 
         'Flash' => t('Flash'), 'Smiley' => t('Smiley'),
         'FitWindow' => t('FitWindow'),
         'SpellCheck' => t('Check spelling'),
-      ),
-      'internal' => TRUE,
+        ),
+        'internal' => TRUE,
     ),
     'autogrow' => array(
-      'path' => $editor['library path'] . '/editor/plugins',
-      'extensions' => array(
+        'path' => $editor['library path'] . '/editor/plugins',
+        'extensions' => array(
         'autogrow' => t('Autogrow'),
-      ),
-      'options' => array(
+        ),
+        'options' => array(
         'AutoGrowMax' => 800,
-      ),
-      'internal' => TRUE,
-      'load' => TRUE,
+        ),
+        'internal' => TRUE,
+        'load' => TRUE,
     ),
     'bbcode' => array(
-      'path' => $editor['library path'] . '/editor/plugins',
-      'extensions' => array(
+        'path' => $editor['library path'] . '/editor/plugins',
+        'extensions' => array(
         'bbcode' => t('BBCode'),
-      ),
-      'internal' => TRUE,
-      'load' => TRUE,
+        ),
+        'internal' => TRUE,
+        'load' => TRUE,
     ),
     'dragresizetable' => array(
-      'path' => $editor['library path'] . '/editor/plugins',
-      'extensions' => array(
+        'path' => $editor['library path'] . '/editor/plugins',
+        'extensions' => array(
         'dragresizetable' => t('Table drag/resize'),
-      ),
-      'internal' => TRUE,
-      'load' => TRUE,
+        ),
+        'internal' => TRUE,
+        'load' => TRUE,
     ),
     'tablecommands' => array(
-      'path' => $editor['library path'] . '/editor/plugins',
-      'buttons' => array(
+        'path' => $editor['library path'] . '/editor/plugins',
+        'buttons' => array(
         'TableCellProp' => t('Table: Cell properties'),
         'TableInsertRowAfter' => t('Table: Insert row after'),
         'TableInsertColumnAfter' => t('Table: Insert column after'),
@@ -282,11 +282,11 @@ 
         'TableDeleteCells' => t('Table: Delete cells'),
         'TableMergeCells' => t('Table: Merge cells'),
         'TableHorizontalSplitCell' => t('Table: Horizontal split cell'),
-      ),
-      'internal' => TRUE,
-      'load' => TRUE,
+        ),
+        'internal' => TRUE,
+        'load' => TRUE,
     ),
-  );
-  return $plugins;
+    );
+    return $plugins;
 }
 

Spacing +11 added lines, -11 removed lines

@@ -50,7 +50,7 @@ 
  *   The installed editor version.
  */
 function wysiwyg_fckeditor_version($editor) {
-  $library = $editor['library path'] . '/fckeditor.js';
+  $library = $editor['library path'].'/fckeditor.js';
   if (!file_exists($library)) {
     return;
   }
@@ -98,9 +98,9 @@ 
  */
 function wysiwyg_fckeditor_settings($editor, $config, $theme) {
   $settings = array(
-    'EditorPath' => base_path() . $editor['library path'] . '/',
-    'SkinPath' => base_path() . $editor['library path'] . '/editor/skins/' . $theme . '/',
-    'CustomConfigurationsPath' => base_path() . drupal_get_path('module', 'wysiwyg') . '/editors/js/fckeditor.config.js',
+    'EditorPath' => base_path().$editor['library path'].'/',
+    'SkinPath' => base_path().$editor['library path'].'/editor/skins/'.$theme.'/',
+    'CustomConfigurationsPath' => base_path().drupal_get_path('module', 'wysiwyg').'/editors/js/fckeditor.config.js',
     'Width' => '100%',
     'Height' => 420,
     'LinkBrowser' => FALSE,
@@ -180,7 +180,7 @@ 
       // Add path for native external plugins; internal ones do not need a path.
       if (empty($plugin['internal']) && isset($plugin['path'])) {
         // All native FCKeditor plugins use the filename fckplugin.js.
-        $settings[$name]['path'] = base_path() . $plugin['path'] . '/';
+        $settings[$name]['path'] = base_path().$plugin['path'].'/';
       }
       if (!empty($plugin['languages'])) {
         $settings[$name]['languages'] = $plugin['languages'];
@@ -199,10 +199,10 @@ 
     // Populate required plugin settings.
     $settings[$name] = $plugin['dialog settings'] + array(
       'title' => $plugin['title'],
-      'icon' => base_path() . $plugin['icon path'] . '/' . $plugin['icon file'],
+      'icon' => base_path().$plugin['icon path'].'/'.$plugin['icon file'],
       'iconTitle' => $plugin['icon title'],
       // @todo These should only be set if the plugin defined them.
-      'css' => base_path() . $plugin['css path'] . '/' . $plugin['css file'],
+      'css' => base_path().$plugin['css path'].'/'.$plugin['css file'],
     );
   }
   return $settings;
@@ -244,7 +244,7 @@ 
       'internal' => TRUE,
     ),
     'autogrow' => array(
-      'path' => $editor['library path'] . '/editor/plugins',
+      'path' => $editor['library path'].'/editor/plugins',
       'extensions' => array(
         'autogrow' => t('Autogrow'),
       ),
@@ -255,7 +255,7 @@ 
       'load' => TRUE,
     ),
     'bbcode' => array(
-      'path' => $editor['library path'] . '/editor/plugins',
+      'path' => $editor['library path'].'/editor/plugins',
       'extensions' => array(
         'bbcode' => t('BBCode'),
       ),
@@ -263,7 +263,7 @@ 
       'load' => TRUE,
     ),
     'dragresizetable' => array(
-      'path' => $editor['library path'] . '/editor/plugins',
+      'path' => $editor['library path'].'/editor/plugins',
       'extensions' => array(
         'dragresizetable' => t('Table drag/resize'),
       ),
@@ -271,7 +271,7 @@ 
       'load' => TRUE,
     ),
     'tablecommands' => array(
-      'path' => $editor['library path'] . '/editor/plugins',
+      'path' => $editor['library path'].'/editor/plugins',
       'buttons' => array(
         'TableCellProp' => t('Table: Cell properties'),
         'TableInsertRowAfter' => t('Table: Insert row after'),

Braces +1 added lines, -2 removed lines

@@ -130,8 +130,7 @@
   if (isset($config['css_setting'])) {
     if ($config['css_setting'] == 'theme') {
       $settings['EditorAreaCSS'] = implode(',', wysiwyg_get_css());
-    }
-    else if ($config['css_setting'] == 'self' && isset($config['css_path'])) {
+    } else if ($config['css_setting'] == 'self' && isset($config['css_path'])) {
       $settings['EditorAreaCSS'] = strtr($config['css_path'], array('%b' => base_path(), '%t' => path_to_theme()));
     }
   }

drupal/sites/default/boinc/modules/contrib/wysiwyg/editors/jwysiwyg.inc

Doc Comments +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
  *   An array containing editor properties as returned from hook_editor().
  *
  * @return
- *   The installed editor version.
+ null|string   The installed editor version.
  */
 function wysiwyg_jwysiwyg_version($editor) {
   $script = $editor['library path'] . '/jquery.wysiwyg.js';

Indentation +18 added lines, -18 removed lines

@@ -9,31 +9,31 @@ 
  * Plugin implementation of hook_editor().
  */
 function wysiwyg_jwysiwyg_editor() {
-  $editor['jwysiwyg'] = array(
+    $editor['jwysiwyg'] = array(
     'title' => 'jWYSIWYG',
     'vendor url' => 'http://code.google.com/p/jwysiwyg/',
     'download url' => 'http://code.google.com/p/jwysiwyg/downloads/list',
     'libraries' => array(
-      '' => array(
+        '' => array(
         'title' => 'Source',
         'files' => array('jquery.wysiwyg.js'),
-      ),
-      'pack' => array(
+        ),
+        'pack' => array(
         'title' => 'Packed',
         'files' => array('jquery.wysiwyg.pack.js'),
-      ),
+        ),
     ),
     'version callback' => 'wysiwyg_jwysiwyg_version',
     // @todo Wrong property; add separate properties for editor requisites.
     'css path' => wysiwyg_get_path('jwysiwyg'),
     'versions' => array(
-      '0.5' => array(
+        '0.5' => array(
         'js files' => array('jwysiwyg.js'),
         'css files' => array('jquery.wysiwyg.css'),
-      ),
+        ),
     ),
-  );
-  return $editor;
+    );
+    return $editor;
 }
 
 /**
@@ -46,17 +46,17 @@ 
  *   The installed editor version.
  */
 function wysiwyg_jwysiwyg_version($editor) {
-  $script = $editor['library path'] . '/jquery.wysiwyg.js';
-  if (!file_exists($script)) {
+    $script = $editor['library path'] . '/jquery.wysiwyg.js';
+    if (!file_exists($script)) {
     return;
-  }
-  $script = fopen($script, 'r');
-  fgets($script);
-  $line = fgets($script);
-  if (preg_match('@([0-9\.]+)$@', $line, $version)) {
+    }
+    $script = fopen($script, 'r');
+    fgets($script);
+    $line = fgets($script);
+    if (preg_match('@([0-9\.]+)$@', $line, $version)) {
     fclose($script);
     return $version[1];
-  }
-  fclose($script);
+    }
+    fclose($script);
 }
 

Spacing +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
  *   The installed editor version.
  */
 function wysiwyg_jwysiwyg_version($editor) {
-  $script = $editor['library path'] . '/jquery.wysiwyg.js';
+  $script = $editor['library path'].'/jquery.wysiwyg.js';
   if (!file_exists($script)) {
     return;
   }

drupal/sites/default/boinc/modules/contrib/wysiwyg/editors/markitup.inc

Doc Comments +2 added lines, -2 removed lines

@@ -44,7 +44,7 @@ 
  *   An array containing editor properties as returned from hook_editor().
  *
  * @return
- *   The installed editor version.
+ null|string   The installed editor version.
  */
 function wysiwyg_markitup_version($editor) {
   // Changelog was in markitup/markitup/readme.txt <= 1.1.5.
@@ -74,7 +74,7 @@ 
  *   A wysiwyg editor profile.
  *
  * @return
- *   An array of theme names. The first returned name should be the default
+ string[]   An array of theme names. The first returned name should be the default
  *   theme name.
  */
 function wysiwyg_markitup_themes($editor, $profile) {

Indentation +68 added lines, -68 removed lines

@@ -9,32 +9,32 @@ 
  * Plugin implementation of hook_editor().
  */
 function wysiwyg_markitup_editor() {
-  $editor['markitup'] = array(
+    $editor['markitup'] = array(
     'title' => 'markItUp',
     'vendor url' => 'http://markitup.jaysalvat.com',
     'download url' => 'http://markitup.jaysalvat.com/downloads',
     'library path' => wysiwyg_get_path('markitup'),
     'libraries' => array(
-      '' => array(
+        '' => array(
         'title' => 'Source',
         'files' => array('markitup/jquery.markitup.js'),
-      ),
-      'pack' => array(
+        ),
+        'pack' => array(
         'title' => 'Packed',
         'files' => array('markitup/jquery.markitup.pack.js'),
-      ),
+        ),
     ),
     'version callback' => 'wysiwyg_markitup_version',
     'themes callback' => 'wysiwyg_markitup_themes',
     'settings callback' => 'wysiwyg_markitup_settings',
     'plugin callback' => 'wysiwyg_markitup_plugins',
     'versions' => array(
-      '1.1.5' => array(
+        '1.1.5' => array(
         'js files' => array('markitup.js'),
-      ),
+        ),
     ),
-  );
-  return $editor;
+    );
+    return $editor;
 }
 
 /**
@@ -47,22 +47,22 @@ 
  *   The installed editor version.
  */
 function wysiwyg_markitup_version($editor) {
-  // Changelog was in markitup/markitup/readme.txt <= 1.1.5.
-  $changelog = $editor['library path'] . '/markitup/readme.txt';
-  if (!file_exists($changelog)) {
+    // Changelog was in markitup/markitup/readme.txt <= 1.1.5.
+    $changelog = $editor['library path'] . '/markitup/readme.txt';
+    if (!file_exists($changelog)) {
     // Changelog was moved up to markitup/CHANGELOG.md after 1.1.5.
     $changelog = $editor['library path'] . '/CHANGELOG.md';
     if (!file_exists($changelog)) {
-      return;
+        return;
+    }
     }
-  }
-  $changelog = fopen($changelog, 'r');
-  $line = fgets($changelog);
-  if (preg_match('@([0-9\.]+)@', $line, $version)) {
+    $changelog = fopen($changelog, 'r');
+    $line = fgets($changelog);
+    if (preg_match('@([0-9\.]+)@', $line, $version)) {
     fclose($changelog);
     return $version[1];
-  }
-  fclose($changelog);
+    }
+    fclose($changelog);
 }
 
 /**
@@ -78,7 +78,7 @@ 
  *   theme name.
  */
 function wysiwyg_markitup_themes($editor, $profile) {
-  return array('simple', 'markitup');
+    return array('simple', 'markitup');
 }
 
 /**
@@ -96,91 +96,91 @@ 
  *   Drupal.settings.wysiwyg.configs.{editor}
  */
 function wysiwyg_markitup_settings($editor, $config, $theme) {
-  // Whoever is guilty for adding this horrible CSS-file-without-filepath
-  // override "feature" to Drupal core... stand in the corner!
-  drupal_add_css($editor['library path'] . '/markitup/skins/' . $theme . '/style.css', 'theme');
+    // Whoever is guilty for adding this horrible CSS-file-without-filepath
+    // override "feature" to Drupal core... stand in the corner!
+    drupal_add_css($editor['library path'] . '/markitup/skins/' . $theme . '/style.css', 'theme');
 
-  $settings = array(
+    $settings = array(
     'root' => base_path() . $editor['library path'] . '/markitup/',
     'nameSpace' => $theme,
     'markupSet' => array(),
-  );
+    );
 
-  // Add configured buttons or all available.
-  $default_buttons = array(
+    // Add configured buttons or all available.
+    $default_buttons = array(
     'bold' => array(
-      'name' => t('Bold'),
-      'className' => 'markitup-bold',
-      'key' => 'B',
-      'openWith' => '(!(<strong>|!|<b>)!)',
-      'closeWith' => '(!(</strong>|!|</b>)!)',
+        'name' => t('Bold'),
+        'className' => 'markitup-bold',
+        'key' => 'B',
+        'openWith' => '(!(<strong>|!|<b>)!)',
+        'closeWith' => '(!(</strong>|!|</b>)!)',
     ),
     'italic' => array(
-      'name' => t('Italic'),
-      'className' => 'markitup-italic',
-      'key' => 'I',
-      'openWith' => '(!(<em>|!|<i>)!)',
-      'closeWith' => '(!(</em>|!|</i>)!)',
+        'name' => t('Italic'),
+        'className' => 'markitup-italic',
+        'key' => 'I',
+        'openWith' => '(!(<em>|!|<i>)!)',
+        'closeWith' => '(!(</em>|!|</i>)!)',
     ),
     'stroke' => array(
-      'name' => t('Strike-through'),
-      'className' => 'markitup-stroke',
-      'key' => 'S',
-      'openWith' => '<del>',
-      'closeWith' => '</del>',
+        'name' => t('Strike-through'),
+        'className' => 'markitup-stroke',
+        'key' => 'S',
+        'openWith' => '<del>',
+        'closeWith' => '</del>',
     ),
     'image' => array(
-      'name' => t('Image'),
-      'className' => 'markitup-image',
-      'key' => 'P',
-      'replaceWith' => '<img src="[![Source:!:http://]!]" alt="[![Alternative text]!]" />',
+        'name' => t('Image'),
+        'className' => 'markitup-image',
+        'key' => 'P',
+        'replaceWith' => '<img src="[![Source:!:http://]!]" alt="[![Alternative text]!]" />',
     ),
     'link' => array(
-      'name' => t('Link'),
-      'className' => 'markitup-link',
-      'key' => 'K',
-      'openWith' => '<a href="[![Link:!:http://]!]"(!( title="[![Title]!]")!)>',
-      'closeWith' => '</a>',
-      'placeHolder' => 'Your text to link...',
+        'name' => t('Link'),
+        'className' => 'markitup-link',
+        'key' => 'K',
+        'openWith' => '<a href="[![Link:!:http://]!]"(!( title="[![Title]!]")!)>',
+        'closeWith' => '</a>',
+        'placeHolder' => 'Your text to link...',
     ),
     // @todo
     // 'cleanup' => array('name' => t('Clean-up'), 'className' => 'markitup-cleanup', 'replaceWith' => 'function(markitup) { return markitup.selection.replace(/<(.*?)>/g, "") }'),
     'preview' => array(
-      'name' => t('Preview'),
-      'className' => 'markitup-preview',
-      'call' => 'preview',
+        'name' => t('Preview'),
+        'className' => 'markitup-preview',
+        'call' => 'preview',
     ),
-  );
-  $settings['markupSet'] = array();
-  if (!empty($config['buttons'])) {
+    );
+    $settings['markupSet'] = array();
+    if (!empty($config['buttons'])) {
     foreach ($config['buttons'] as $plugin) {
-      foreach ($plugin as $button => $enabled) {
+        foreach ($plugin as $button => $enabled) {
         if (isset($default_buttons[$button])) {
-          $settings['markupSet'][$button] = $default_buttons[$button];
+            $settings['markupSet'][$button] = $default_buttons[$button];
+        }
         }
-      }
     }
-  }
+    }
 
-  return $settings;
+    return $settings;
 }
 
 /**
  * Return internal plugins for this editor; semi-implementation of hook_wysiwyg_plugin().
  */
 function wysiwyg_markitup_plugins($editor) {
-  return array(
+    return array(
     'default' => array(
-      'buttons' => array(
+        'buttons' => array(
         'bold' => t('Bold'), 'italic' => t('Italic'),
         'stroke' => t('Strike-through'),
         'link' => t('Link'),
         'image' => t('Image'),
         // 'cleanup' => t('Clean-up'),
         'preview' => t('Preview'),
-      ),
-      'internal' => TRUE,
+        ),
+        'internal' => TRUE,
     ),
-  );
+    );
 }
 

Spacing +4 added lines, -4 removed lines

@@ -48,10 +48,10 @@ 
  */
 function wysiwyg_markitup_version($editor) {
   // Changelog was in markitup/markitup/readme.txt <= 1.1.5.
-  $changelog = $editor['library path'] . '/markitup/readme.txt';
+  $changelog = $editor['library path'].'/markitup/readme.txt';
   if (!file_exists($changelog)) {
     // Changelog was moved up to markitup/CHANGELOG.md after 1.1.5.
-    $changelog = $editor['library path'] . '/CHANGELOG.md';
+    $changelog = $editor['library path'].'/CHANGELOG.md';
     if (!file_exists($changelog)) {
       return;
     }
@@ -98,10 +98,10 @@ 
 function wysiwyg_markitup_settings($editor, $config, $theme) {
   // Whoever is guilty for adding this horrible CSS-file-without-filepath
   // override "feature" to Drupal core... stand in the corner!
-  drupal_add_css($editor['library path'] . '/markitup/skins/' . $theme . '/style.css', 'theme');
+  drupal_add_css($editor['library path'].'/markitup/skins/'.$theme.'/style.css', 'theme');
 
   $settings = array(
-    'root' => base_path() . $editor['library path'] . '/markitup/',
+    'root' => base_path().$editor['library path'].'/markitup/',
     'nameSpace' => $theme,
     'markupSet' => array(),
   );

drupal/sites/default/boinc/modules/contrib/wysiwyg/editors/nicedit.inc

Doc Comments +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
  *   An array containing editor properties as returned from hook_editor().
  *
  * @return
- *   The installed editor version.
+ string   The installed editor version.
  */
 function wysiwyg_nicedit_version($editor) {
   // @see http://nicedit.com/forums/viewtopic.php?t=425

Indentation +29 added lines, -29 removed lines

@@ -9,26 +9,26 @@ 
  * Plugin implementation of hook_editor().
  */
 function wysiwyg_nicedit_editor() {
-  $editor['nicedit'] = array(
+    $editor['nicedit'] = array(
     'title' => 'NicEdit',
     'vendor url' => 'http://nicedit.com',
     'download url' => 'http://nicedit.com/download.php',
     'libraries' => array(
-      '' => array(
+        '' => array(
         'title' => 'Source',
         'files' => array('nicEdit.js'),
-      ),
+        ),
     ),
     'version callback' => 'wysiwyg_nicedit_version',
     'settings callback' => 'wysiwyg_nicedit_settings',
     'plugin callback' => 'wysiwyg_nicedit_plugins',
     'versions' => array(
-      '0.9' => array(
+        '0.9' => array(
         'js files' => array('nicedit.js'),
-      ),
+        ),
     ),
-  );
-  return $editor;
+    );
+    return $editor;
 }
 
 /**
@@ -41,8 +41,8 @@ 
  *   The installed editor version.
  */
 function wysiwyg_nicedit_version($editor) {
-  // @see http://nicedit.com/forums/viewtopic.php?t=425
-  return '0.9';
+    // @see http://nicedit.com/forums/viewtopic.php?t=425
+    return '0.9';
 }
 
 /**
@@ -60,43 +60,43 @@ 
  *   Drupal.settings.wysiwyg.configs.{editor}
  */
 function wysiwyg_nicedit_settings($editor, $config, $theme) {
-  $settings = array(
+    $settings = array(
     'iconsPath' => base_path() . $editor['library path'] . '/nicEditorIcons.gif',
-  );
+    );
 
-  // Add configured buttons or all available.
-  $settings['buttonList'] = array();
-  if (!empty($config['buttons'])) {
+    // Add configured buttons or all available.
+    $settings['buttonList'] = array();
+    if (!empty($config['buttons'])) {
     $buttons = array();
     foreach ($config['buttons'] as $plugin) {
-      $buttons = array_merge($buttons, $plugin);
+        $buttons = array_merge($buttons, $plugin);
     }
     $settings['buttonList'] = array_keys($buttons);
-  }
+    }
 
-  // Add editor content stylesheet.
-  if (isset($config['css_setting'])) {
+    // Add editor content stylesheet.
+    if (isset($config['css_setting'])) {
     if ($config['css_setting'] == 'theme') {
-      $css = path_to_theme() . '/style.css';
-      if (file_exists($css)) {
+        $css = path_to_theme() . '/style.css';
+        if (file_exists($css)) {
         $settings['externalCSS'] = base_path() . $css;
-      }
+        }
     }
     else if ($config['css_setting'] == 'self' && isset($config['css_path'])) {
-      $settings['externalCSS'] = strtr($config['css_path'], array('%b' => base_path(), '%t' => path_to_theme()));
+        $settings['externalCSS'] = strtr($config['css_path'], array('%b' => base_path(), '%t' => path_to_theme()));
+    }
     }
-  }
 
-  return $settings;
+    return $settings;
 }
 
 /**
  * Return internal plugins for this editor; semi-implementation of hook_wysiwyg_plugin().
  */
 function wysiwyg_nicedit_plugins($editor) {
-  return array(
+    return array(
     'default' => array(
-      'buttons' => array(
+        'buttons' => array(
         'bold' => t('Bold'), 'italic' => t('Italic'), 'underline' => t('Underline'),
         'strikethrough' => t('Strike-through'),
         'left' => t('Align left'), 'center' => t('Align center'), 'right' => t('Align right'),
@@ -111,9 +111,9 @@ 
         'link' => t('Link'), 'unlink' => t('Unlink'),
         'fontFormat' => t('HTML block format'), 'fontFamily' => t('Font'), 'fontSize' => t('Font size'),
         'xhtml' => t('Source code'),
-      ),
-      'internal' => TRUE,
+        ),
+        'internal' => TRUE,
     ),
-  );
+    );
 }
 

Spacing +3 added lines, -3 removed lines

@@ -61,7 +61,7 @@ 
  */
 function wysiwyg_nicedit_settings($editor, $config, $theme) {
   $settings = array(
-    'iconsPath' => base_path() . $editor['library path'] . '/nicEditorIcons.gif',
+    'iconsPath' => base_path().$editor['library path'].'/nicEditorIcons.gif',
   );
 
   // Add configured buttons or all available.
@@ -77,9 +77,9 @@ 
   // Add editor content stylesheet.
   if (isset($config['css_setting'])) {
     if ($config['css_setting'] == 'theme') {
-      $css = path_to_theme() . '/style.css';
+      $css = path_to_theme().'/style.css';
       if (file_exists($css)) {
-        $settings['externalCSS'] = base_path() . $css;
+        $settings['externalCSS'] = base_path().$css;
       }
     }
     else if ($config['css_setting'] == 'self' && isset($config['css_path'])) {

Braces +1 added lines, -2 removed lines

@@ -108,8 +108,7 @@
       if (file_exists($css)) {
         $settings['externalCSS'] = base_path() . $css;
       }
-    }
-    else if ($config['css_setting'] == 'self' && isset($config['css_path'])) {
+    } else if ($config['css_setting'] == 'self' && isset($config['css_path'])) {
       $settings['externalCSS'] = strtr($config['css_path'], array('%b' => base_path(), '%t' => path_to_theme()));
     }
   }

drupal/sites/default/boinc/modules/contrib/wysiwyg/editors/openwysiwyg.inc

Doc Comments +2 added lines, -2 removed lines

@@ -41,7 +41,7 @@ 
  *   An array containing editor properties as returned from hook_editor().
  *
  * @return
- *   The installed editor version.
+ null|string   The installed editor version.
  */
 function wysiwyg_openwysiwyg_version($editor) {
   // 'library path' has '/scripts' appended already.
@@ -67,7 +67,7 @@ 
  *   A wysiwyg editor profile.
  *
  * @return
- *   An array of theme names. The first returned name should be the default
+ string[]   An array of theme names. The first returned name should be the default
  *   theme name.
  */
 function wysiwyg_openwysiwyg_themes($editor, $profile) {

Indentation +44 added lines, -44 removed lines

@@ -9,29 +9,29 @@ 
  * Plugin implementation of hook_editor().
  */
 function wysiwyg_openwysiwyg_editor() {
-  $editor['openwysiwyg'] = array(
+    $editor['openwysiwyg'] = array(
     'title' => 'openWYSIWYG',
     'vendor url' => 'http://www.openwebware.com',
     'download url' => 'http://www.openwebware.com/download.shtml',
     'library path' => wysiwyg_get_path('openwysiwyg') . '/scripts',
     'libraries' => array(
-      'src' => array(
+        'src' => array(
         'title' => 'Source',
         'files' => array('wysiwyg.js'),
-      ),
+        ),
     ),
     'version callback' => 'wysiwyg_openwysiwyg_version',
     'themes callback' => 'wysiwyg_openwysiwyg_themes',
     'settings callback' => 'wysiwyg_openwysiwyg_settings',
     'plugin callback' => 'wysiwyg_openwysiwyg_plugins',
     'versions' => array(
-      '1.4.7' => array(
+        '1.4.7' => array(
         'js files' => array('openwysiwyg.js'),
         'css files' => array('openwysiwyg.css'),
-      ),
+        ),
     ),
-  );
-  return $editor;
+    );
+    return $editor;
 }
 
 /**
@@ -44,18 +44,18 @@ 
  *   The installed editor version.
  */
 function wysiwyg_openwysiwyg_version($editor) {
-  // 'library path' has '/scripts' appended already.
-  $changelog = $editor['editor path'] . '/changelog';
-  if (!file_exists($changelog)) {
+    // 'library path' has '/scripts' appended already.
+    $changelog = $editor['editor path'] . '/changelog';
+    if (!file_exists($changelog)) {
     return;
-  }
-  $changelog = fopen($changelog, 'r');
-  $line = fgets($changelog, 20);
-  if (preg_match('@v([\d\.]+)@', $line, $version)) {
+    }
+    $changelog = fopen($changelog, 'r');
+    $line = fgets($changelog, 20);
+    if (preg_match('@v([\d\.]+)@', $line, $version)) {
     fclose($changelog);
     return $version[1];
-  }
-  fclose($changelog);
+    }
+    fclose($changelog);
 }
 
 /**
@@ -71,7 +71,7 @@ 
  *   theme name.
  */
 function wysiwyg_openwysiwyg_themes($editor, $profile) {
-  return array('default');
+    return array('default');
 }
 
 /**
@@ -89,58 +89,58 @@ 
  *   Drupal.settings.wysiwyg.configs.{editor}
  */
 function wysiwyg_openwysiwyg_settings($editor, $config, $theme) {
-  $settings = array(
+    $settings = array(
     'path' => base_path() . $editor['editor path'] . '/',
     'Width' => '100%',
-  );
+    );
 
-  if (isset($config['path_loc']) && $config['path_loc'] == 'none') {
+    if (isset($config['path_loc']) && $config['path_loc'] == 'none') {
     $settings['StatusBarEnabled'] = FALSE;
-  }
+    }
 
-  if (isset($config['css_setting'])) {
+    if (isset($config['css_setting'])) {
     if ($config['css_setting'] == 'theme') {
-      $settings['CSSFile'] = reset(wysiwyg_get_css());
+        $settings['CSSFile'] = reset(wysiwyg_get_css());
     }
     else if ($config['css_setting'] == 'self' && isset($config['css_path'])) {
-      $settings['CSSFile'] = strtr($config['css_path'], array('%b' => base_path(), '%t' => path_to_theme()));
+        $settings['CSSFile'] = strtr($config['css_path'], array('%b' => base_path(), '%t' => path_to_theme()));
+    }
     }
-  }
 
-  $settings['Toolbar'] = array();
-  if (!empty($config['buttons'])) {
+    $settings['Toolbar'] = array();
+    if (!empty($config['buttons'])) {
     $plugins = wysiwyg_get_plugins($editor['name']);
     foreach ($config['buttons'] as $plugin => $buttons) {
-      foreach ($buttons as $button => $enabled) {
+        foreach ($buttons as $button => $enabled) {
         foreach (array('buttons', 'extensions') as $type) {
-          // Skip unavailable plugins.
-          if (!isset($plugins[$plugin][$type][$button])) {
+            // Skip unavailable plugins.
+            if (!isset($plugins[$plugin][$type][$button])) {
             continue;
-          }
-          // Add buttons.
-          if ($type == 'buttons') {
+            }
+            // Add buttons.
+            if ($type == 'buttons') {
             $settings['Toolbar'][0][] = $button;
-          }
+            }
+        }
         }
-      }
     }
-  }
+    }
 
-  // @todo 
+    // @todo 
 //  if (isset($config['block_formats'])) {
 //    $settings['DropDowns']['headings']['elements'] = explode(',', $config['block_formats']);
 //  }
 
-  return $settings;
+    return $settings;
 }
 
 /**
  * Return internal plugins for this editor; semi-implementation of hook_wysiwyg_plugin().
  */
 function wysiwyg_openwysiwyg_plugins($editor) {
-  $plugins = array(
+    $plugins = array(
     'default' => array(
-      'buttons' => array(
+        'buttons' => array(
         'bold' => t('Bold'), 'italic' => t('Italic'), 'underline' => t('Underline'),
         'strikethrough' => t('Strike-through'),
         'justifyleft' => t('Align left'), 'justifycenter' => t('Align center'), 'justifyright' => t('Align right'), 'justifyfull' => t('Justify'),
@@ -164,10 +164,10 @@ 
         'print' => t('Print'),
         'inserttable' => t('Table'),
         'help' => t('Help'),
-      ),
-      'internal' => TRUE,
+        ),
+        'internal' => TRUE,
     ),
-  );
-  return $plugins;
+    );
+    return $plugins;
 }
 

Spacing +3 added lines, -3 removed lines

@@ -13,7 +13,7 @@ 
     'title' => 'openWYSIWYG',
     'vendor url' => 'http://www.openwebware.com',
     'download url' => 'http://www.openwebware.com/download.shtml',
-    'library path' => wysiwyg_get_path('openwysiwyg') . '/scripts',
+    'library path' => wysiwyg_get_path('openwysiwyg').'/scripts',
     'libraries' => array(
       'src' => array(
         'title' => 'Source',
@@ -45,7 +45,7 @@ 
  */
 function wysiwyg_openwysiwyg_version($editor) {
   // 'library path' has '/scripts' appended already.
-  $changelog = $editor['editor path'] . '/changelog';
+  $changelog = $editor['editor path'].'/changelog';
   if (!file_exists($changelog)) {
     return;
   }
@@ -90,7 +90,7 @@ 
  */
 function wysiwyg_openwysiwyg_settings($editor, $config, $theme) {
   $settings = array(
-    'path' => base_path() . $editor['editor path'] . '/',
+    'path' => base_path().$editor['editor path'].'/',
     'Width' => '100%',
   );
 

Braces +1 added lines, -2 removed lines

@@ -101,8 +101,7 @@
   if (isset($config['css_setting'])) {
     if ($config['css_setting'] == 'theme') {
       $settings['CSSFile'] = reset(wysiwyg_get_css());
-    }
-    else if ($config['css_setting'] == 'self' && isset($config['css_path'])) {
+    } else if ($config['css_setting'] == 'self' && isset($config['css_path'])) {
       $settings['CSSFile'] = strtr($config['css_path'], array('%b' => base_path(), '%t' => path_to_theme()));
     }
   }