BOINC / boinc

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

@@ -81,8 +81,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()));
     }
   }

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

Doc Comments +2 added lines, -2 removed lines

@@ -77,7 +77,7 @@ 
  *   An array containing editor properties as returned from hook_editor().
  *
  * @return
- *   The installed editor version.
+ null|string   The installed editor version.
  */
 function wysiwyg_tinymce_version($editor) {
   $script = $editor['library path'] . '/tiny_mce.js';
@@ -104,7 +104,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_tinymce_themes($editor, $profile) {

Indentation +290 added lines, -290 removed lines

@@ -11,20 +11,20 @@ 
  * @todo wysiwyg_<editor>_alter() to add/inject optional libraries like gzip.
  */
 function wysiwyg_tinymce_editor() {
-  $editor['tinymce'] = array(
+    $editor['tinymce'] = array(
     'title' => 'TinyMCE',
     'vendor url' => 'http://tinymce.moxiecode.com',
     'download url' => 'http://tinymce.moxiecode.com/download.php',
     'library path' => wysiwyg_get_path('tinymce') . '/jscripts/tiny_mce',
     'libraries' => array(
-      '' => array(
+        '' => array(
         'title' => 'Minified',
         'files' => array('tiny_mce.js'),
-      ),
-      'src' => array(
+        ),
+        'src' => array(
         'title' => 'Source',
         'files' => array('tiny_mce_src.js'),
-      ),
+        ),
     ),
     'version callback' => 'wysiwyg_tinymce_version',
     'themes callback' => 'wysiwyg_tinymce_themes',
@@ -32,42 +32,42 @@ 
     'plugin callback' => 'wysiwyg_tinymce_plugins',
     'plugin settings callback' => 'wysiwyg_tinymce_plugin_settings',
     'proxy plugin' => array(
-      'drupal' => array(
+        'drupal' => array(
         'load' => TRUE,
         'proxy' => TRUE,
-      ),
+        ),
     ),
     'proxy plugin settings callback' => 'wysiwyg_tinymce_proxy_plugin_settings',
     'versions' => array(
-      '2.1' => array(
+        '2.1' => array(
         'js files' => array('tinymce-2.js'),
         'css files' => array('tinymce-2.css'),
         'download url' => 'http://sourceforge.net/project/showfiles.php?group_id=103281&package_id=111430&release_id=557383',
-      ),
-      // @todo Starting from 3.3, tiny_mce.js may support JS aggregation.
-      '3.1' => array(
+        ),
+        // @todo Starting from 3.3, tiny_mce.js may support JS aggregation.
+        '3.1' => array(
         'js files' => array('tinymce-3.js'),
         'css files' => array('tinymce-3.css'),
         'libraries' => array(
-          '' => array(
+            '' => array(
             'title' => 'Minified',
             'files' => array(
-              'tiny_mce.js' => array('preprocess' => FALSE),
+                'tiny_mce.js' => array('preprocess' => FALSE),
             ),
-          ),
-          'jquery' => array(
+            ),
+            'jquery' => array(
             'title' => 'jQuery',
             'files' => array('tiny_mce_jquery.js'),
-          ),
-          'src' => array(
+            ),
+            'src' => array(
             'title' => 'Source',
             'files' => array('tiny_mce_src.js'),
-          ),
+            ),
+        ),
         ),
-      ),
     ),
-  );
-  return $editor;
+    );
+    return $editor;
 }
 
 /**
@@ -80,19 +80,19 @@ 
  *   The installed editor version.
  */
 function wysiwyg_tinymce_version($editor) {
-  $script = $editor['library path'] . '/tiny_mce.js';
-  if (!file_exists($script)) {
+    $script = $editor['library path'] . '/tiny_mce.js';
+    if (!file_exists($script)) {
     return;
-  }
-  $script = fopen($script, 'r');
-  // Version is contained in the first 200 chars.
-  $line = fgets($script, 200);
-  fclose($script);
-  // 2.x: this.majorVersion="2";this.minorVersion="1.3"
-  // 3.x: majorVersion:'3',minorVersion:'2.0.1'
-  if (preg_match('@majorVersion[=:]["\'](\d).+?minorVersion[=:]["\']([\d\.]+)@', $line, $version)) {
+    }
+    $script = fopen($script, 'r');
+    // Version is contained in the first 200 chars.
+    $line = fgets($script, 200);
+    fclose($script);
+    // 2.x: this.majorVersion="2";this.minorVersion="1.3"
+    // 3.x: majorVersion:'3',minorVersion:'2.0.1'
+    if (preg_match('@majorVersion[=:]["\'](\d).+?minorVersion[=:]["\']([\d\.]+)@', $line, $version)) {
     return $version[1] . '.' . $version[2];
-  }
+    }
 }
 
 /**
@@ -108,7 +108,7 @@ 
  *   theme name.
  */
 function wysiwyg_tinymce_themes($editor, $profile) {
-  /*
+    /*
   $themes = array();
   $dir = $editor['library path'] . '/themes/';
   if (is_dir($dir) && $dh = opendir($dir)) {
@@ -122,7 +122,7 @@ 
   }
   return $themes;
   */
-  return array('advanced', 'simple');
+    return array('advanced', 'simple');
 }
 
 /**
@@ -140,7 +140,7 @@ 
  *   Drupal.settings.wysiwyg.configs.{editor}
  */
 function wysiwyg_tinymce_settings($editor, $config, $theme) {
-  $settings = array(
+    $settings = array(
     'button_tile_map' => TRUE, // @todo Add a setting for this.
     'document_base_url' => base_path(),
     'mode' => 'none',
@@ -158,49 +158,49 @@ 
     // HTML control characters and invisible characters. TinyMCE always converts
     // XML default characters '&', '<', '>'.
     'entities' => '160,nbsp,173,shy,8194,ensp,8195,emsp,8201,thinsp,8204,zwnj,8205,zwj,8206,lrm,8207,rlm',
-  );
-  if (isset($config['apply_source_formatting'])) {
+    );
+    if (isset($config['apply_source_formatting'])) {
     $settings['apply_source_formatting'] = $config['apply_source_formatting'];
-  }
-  if (isset($config['convert_fonts_to_spans'])) {
+    }
+    if (isset($config['convert_fonts_to_spans'])) {
     $settings['convert_fonts_to_spans'] = $config['convert_fonts_to_spans'];
-  }
-  if (isset($config['language'])) {
+    }
+    if (isset($config['language'])) {
     $settings['language'] = $config['language'];
-  }
-  if (isset($config['paste_auto_cleanup_on_paste'])) {
+    }
+    if (isset($config['paste_auto_cleanup_on_paste'])) {
     $settings['paste_auto_cleanup_on_paste'] = $config['paste_auto_cleanup_on_paste'];
-  }
-  if (isset($config['preformatted'])) {
+    }
+    if (isset($config['preformatted'])) {
     $settings['preformatted'] = $config['preformatted'];
-  }
-  if (isset($config['remove_linebreaks'])) {
+    }
+    if (isset($config['remove_linebreaks'])) {
     $settings['remove_linebreaks'] = $config['remove_linebreaks'];
-  }
-  if (isset($config['verify_html'])) {
+    }
+    if (isset($config['verify_html'])) {
     $settings['verify_html'] = (bool) $config['verify_html'];
-  }
+    }
 
-  if (!empty($config['css_classes'])) {
+    if (!empty($config['css_classes'])) {
     $settings['theme_advanced_styles'] = implode(';', array_filter(explode("\n", str_replace("\r", '', $config['css_classes']))));
-  }
+    }
 
-  if (isset($config['css_setting'])) {
+    if (isset($config['css_setting'])) {
     if ($config['css_setting'] == 'theme') {
-      $settings['content_css'] = implode(',', wysiwyg_get_css());
+        $settings['content_css'] = implode(',', wysiwyg_get_css());
     }
     else if ($config['css_setting'] == 'self' && isset($config['css_path'])) {
-      $settings['content_css'] = strtr($config['css_path'], array('%b' => base_path(), '%t' => path_to_theme()));
+        $settings['content_css'] = strtr($config['css_path'], array('%b' => base_path(), '%t' => path_to_theme()));
+    }
     }
-  }
 
-  // Find the enabled buttons and the button row they belong on.
-  // Also map the plugin metadata for each button.
-  // @todo What follows is a pain; needs a rewrite.
-  // $settings['buttons'] are stacked into $settings['theme_advanced_buttons1']
-  // later.
-  $settings['buttons'] = array();
-  if (!empty($config['buttons']) && is_array($config['buttons'])) {
+    // Find the enabled buttons and the button row they belong on.
+    // Also map the plugin metadata for each button.
+    // @todo What follows is a pain; needs a rewrite.
+    // $settings['buttons'] are stacked into $settings['theme_advanced_buttons1']
+    // later.
+    $settings['buttons'] = array();
+    if (!empty($config['buttons']) && is_array($config['buttons'])) {
     // Only array keys in $settings['extensions'] matter; added to
     // $settings['plugins'] later.
     $settings['extensions'] = array();
@@ -213,54 +213,54 @@ 
 
     $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;
-          }
-          // 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'])) {
             $settings['extensions'][_wysiwyg_tinymce_plugin_name('add', $button)] = 1;
-          }
-          // Add external plugins to the list of extensions.
-          else if ($type == 'buttons' && empty($plugins[$plugin]['internal'])) {
+            }
+            // Add external plugins to the list of extensions.
+            else if ($type == 'buttons' && empty($plugins[$plugin]['internal'])) {
             $settings['extensions'][_wysiwyg_tinymce_plugin_name('add', $plugin)] = 1;
-          }
-          // Add internal buttons that also need to be loaded as extension.
-          else if ($type == 'buttons' && !empty($plugins[$plugin]['load'])) {
+            }
+            // Add internal buttons that also need to be loaded as extension.
+            else if ($type == 'buttons' && !empty($plugins[$plugin]['load'])) {
             $settings['extensions'][$plugin] = 1;
-          }
-          // Add plain extensions.
-          else if ($type == 'extensions' && !empty($plugins[$plugin]['load'])) {
+            }
+            // Add plain extensions.
+            else if ($type == 'extensions' && !empty($plugins[$plugin]['load'])) {
             $settings['extensions'][$plugin] = 1;
-          }
-          // Allow plugins to add valid HTML elements.
-          if (!empty($plugins[$plugin]['extended_valid_elements'])) {
+            }
+            // Allow plugins to add valid HTML elements.
+            if (!empty($plugins[$plugin]['extended_valid_elements'])) {
             $settings['extended_valid_elements'] = array_merge($settings['extended_valid_elements'], $plugins[$plugin]['extended_valid_elements']);
-          }
-          // 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']);
-          }
+            }
+        }
         }
-      }
     }
     // Clean-up.
     $settings['extended_valid_elements'] = array_unique($settings['extended_valid_elements']);
     if ($settings['extensions']) {
-      $settings['plugins'] = array_keys($settings['extensions']);
+        $settings['plugins'] = array_keys($settings['extensions']);
     }
     unset($settings['extensions']);
-  }
+    }
 
-  // Add theme-specific settings.
-  switch ($theme) {
+    // Add theme-specific settings.
+    switch ($theme) {
     case 'advanced':
       $settings += array(
         'theme_advanced_resize_horizontal' => FALSE,
@@ -269,36 +269,36 @@ 
         'theme_advanced_resizing' => isset($config['resizing']) ? $config['resizing'] : 1,
         'theme_advanced_toolbar_location' => isset($config['toolbar_loc']) ? $config['toolbar_loc'] : 'top',
         'theme_advanced_toolbar_align' => isset($config['toolbar_align']) ? $config['toolbar_align'] : 'left',
-      );
-      if (isset($config['block_formats'])) {
+        );
+        if (isset($config['block_formats'])) {
         $settings['theme_advanced_blockformats'] = $config['block_formats'];
-      }
-      if (isset($settings['buttons'])) {
+        }
+        if (isset($settings['buttons'])) {
         // These rows explicitly need to be set to be empty, otherwise TinyMCE
         // loads its default buttons of the advanced theme for each row.
         $settings += array(
-          'theme_advanced_buttons1' => array(),
-          'theme_advanced_buttons2' => array(),
-          'theme_advanced_buttons3' => array(),
+            'theme_advanced_buttons1' => array(),
+            'theme_advanced_buttons2' => array(),
+            'theme_advanced_buttons3' => array(),
         );
         // @todo Allow to sort/arrange editor buttons.
         for ($i = 0; $i < count($settings['buttons']); $i++) {
-          $settings['theme_advanced_buttons1'][] = $settings['buttons'][$i];
+            $settings['theme_advanced_buttons1'][] = $settings['buttons'][$i];
         }
-      }
-      break;
-  }
-  unset($settings['buttons']);
+        }
+        break;
+    }
+    unset($settings['buttons']);
 
-  // Convert the config values into the form expected by TinyMCE.
-  $csv_settings = array('plugins', 'extended_valid_elements', 'theme_advanced_buttons1', 'theme_advanced_buttons2', 'theme_advanced_buttons3');
-  foreach ($csv_settings as $key) {
+    // Convert the config values into the form expected by TinyMCE.
+    $csv_settings = array('plugins', 'extended_valid_elements', 'theme_advanced_buttons1', 'theme_advanced_buttons2', 'theme_advanced_buttons3');
+    foreach ($csv_settings as $key) {
     if (isset($settings[$key]) && is_array($settings[$key])) {
-      $settings[$key] = implode(',', $settings[$key]);
+        $settings[$key] = implode(',', $settings[$key]);
+    }
     }
-  }
 
-  return $settings;
+    return $settings;
 }
 
 /**
@@ -308,46 +308,46 @@ 
  * directory) are loaded (once) upon initializing the editor.
  */
 function wysiwyg_tinymce_plugin_settings($editor, $profile, $plugins) {
-  $settings = array();
-  foreach ($plugins as $name => $plugin) {
+    $settings = array();
+    foreach ($plugins as $name => $plugin) {
     if (!empty($plugin['load'])) {
-      // Add path for native external plugins; internal ones are loaded
-      // automatically.
-      if (empty($plugin['internal']) && isset($plugin['path'])) {
+        // Add path for native external plugins; internal ones are loaded
+        // automatically.
+        if (empty($plugin['internal']) && isset($plugin['path'])) {
         // TinyMCE plugins commonly use the filename editor_plugin.js, but there
         // is no default. Previously, Wysiwyg's API documentation suggested to
         // have the 'path' contain the 'filename' property, so if 'filename' is
         // not defined, automatically extract it from 'path' for backwards
         // compatibility.
         if (!isset($plugin['filename'])) {
-          $parts = explode('/', $plugin['path']);
-          $plugin['filename'] = array_pop($parts);
-          $plugin['path'] = implode('/', $parts);
+            $parts = explode('/', $plugin['path']);
+            $plugin['filename'] = array_pop($parts);
+            $plugin['path'] = implode('/', $parts);
         }
         $settings[$name] = base_path() . $plugin['path'] . '/' . $plugin['filename'];
-      }
+        }
     }
-  }
-  return $settings;
+    }
+    return $settings;
 }
 
 /**
  * Build a JS settings array for Drupal plugins loaded via the proxy plugin.
  */
 function wysiwyg_tinymce_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'],
+        'title' => $plugin['title'],
+        'icon' => base_path() . $plugin['icon path'] . '/' . $plugin['icon file'],
+        'iconTitle' => $plugin['icon title'],
     );
     if (isset($plugin['css file'])) {
-      $settings[$name]['css'] = base_path() . $plugin['css path'] . '/' . $plugin['css file'];
+        $settings[$name]['css'] = base_path() . $plugin['css path'] . '/' . $plugin['css file'];
     }
-  }
-  return $settings;
+    }
+    return $settings;
 }
 
 /**
@@ -362,28 +362,28 @@ 
  *   A plugin name.
  */
 function _wysiwyg_tinymce_plugin_name($op, $name) {
-  if ($op == 'add') {
+    if ($op == 'add') {
     if (strpos($name, '-') !== 0) {
-      return '-' . $name;
+        return '-' . $name;
     }
     return $name;
-  }
-  else if ($op == 'remove') {
+    }
+    else if ($op == 'remove') {
     if (strpos($name, '-') === 0) {
-      return substr($name, 1);
+        return substr($name, 1);
     }
     return $name;
-  }
+    }
 }
 
 /**
  * Return internal plugins for this editor; semi-implementation of hook_wysiwyg_plugin().
  */
 function wysiwyg_tinymce_plugins($editor) {
-  $plugins = array(
+    $plugins = array(
     'default' => array(
-      'path' => $editor['library path'] . '/themes/advanced',
-      'buttons' => array(
+        'path' => $editor['library path'] . '/themes/advanced',
+        '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'),
@@ -402,217 +402,217 @@ 
         'removeformat' => t('Remove format'),
         'charmap' => t('Character map'),
         'help' => t('Help'),
-      ),
-      'internal' => TRUE,
+        ),
+        'internal' => TRUE,
     ),
     'advhr' => array(
-      'path' => $editor['library path'] . '/plugins/advhr',
-      'buttons' => array('advhr' => t('Advanced horizontal rule')),
-      'extended_valid_elements' => array('hr[class|width|size|noshade]'),
-      'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/advhr',
-      'internal' => TRUE,
-      'load' => TRUE,
+        'path' => $editor['library path'] . '/plugins/advhr',
+        'buttons' => array('advhr' => t('Advanced horizontal rule')),
+        'extended_valid_elements' => array('hr[class|width|size|noshade]'),
+        'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/advhr',
+        'internal' => TRUE,
+        'load' => TRUE,
     ),
     'advimage' => array(
-      'path' => $editor['library path'] . '/plugins/advimage',
-      'extensions' => array('advimage' => t('Advanced image')),
-      'extended_valid_elements' => array('img[src|alt|title|align|width|height|usemap|hspace|vspace|border|style|class|onmouseover|onmouseout|id|name]'),
-      'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/advimage',
-      'internal' => TRUE,
-      'load' => TRUE,
+        'path' => $editor['library path'] . '/plugins/advimage',
+        'extensions' => array('advimage' => t('Advanced image')),
+        'extended_valid_elements' => array('img[src|alt|title|align|width|height|usemap|hspace|vspace|border|style|class|onmouseover|onmouseout|id|name]'),
+        'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/advimage',
+        'internal' => TRUE,
+        'load' => TRUE,
     ),
     'advlink' => array(
-      'path' => $editor['library path'] . '/plugins/advlink',
-      'extensions' => array('advlink' => t('Advanced link')),
-      'extended_valid_elements' => array('a[name|href|target|title|class|onfocus|onblur|onclick|ondlbclick|onmousedown|onmouseup|onmouseover|onmouseout|onkeypress|onkeydown|onkeyup|id|style|rel]'),
-      'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/advlink',
-      'internal' => TRUE,
-      'load' => TRUE,
+        'path' => $editor['library path'] . '/plugins/advlink',
+        'extensions' => array('advlink' => t('Advanced link')),
+        'extended_valid_elements' => array('a[name|href|target|title|class|onfocus|onblur|onclick|ondlbclick|onmousedown|onmouseup|onmouseover|onmouseout|onkeypress|onkeydown|onkeyup|id|style|rel]'),
+        'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/advlink',
+        'internal' => TRUE,
+        'load' => TRUE,
     ),
     'autosave' => array(
-      'path' => $editor['library path'] . '/plugins/autosave',
-      'extensions' => array('autosave' => t('Auto save')),
-      'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/autosave',
-      'internal' => TRUE,
-      'load' => TRUE,
+        'path' => $editor['library path'] . '/plugins/autosave',
+        'extensions' => array('autosave' => t('Auto save')),
+        'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/autosave',
+        'internal' => TRUE,
+        'load' => TRUE,
     ),
     'contextmenu' => array(
-      'path' => $editor['library path'] . '/plugins/contextmenu',
-      'extensions' => array('contextmenu' => t('Context menu')),
-      'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/contextmenu',
-      'internal' => TRUE,
-      'load' => TRUE,
+        'path' => $editor['library path'] . '/plugins/contextmenu',
+        'extensions' => array('contextmenu' => t('Context menu')),
+        'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/contextmenu',
+        'internal' => TRUE,
+        'load' => TRUE,
     ),
     'directionality' => array(
-      'path' => $editor['library path'] . '/plugins/directionality',
-      'buttons' => array('ltr' => t('Left-to-right'), 'rtl' => t('Right-to-left')),
-      'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/directionality',
-      'internal' => TRUE,
-      'load' => TRUE,
+        'path' => $editor['library path'] . '/plugins/directionality',
+        'buttons' => array('ltr' => t('Left-to-right'), 'rtl' => t('Right-to-left')),
+        'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/directionality',
+        'internal' => TRUE,
+        'load' => TRUE,
     ),
     'emotions' => array(
-      'path' => $editor['library path'] . '/plugins/emotions',
-      'buttons' => array('emotions' => t('Emotions')),
-      'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/emotions',
-      'internal' => TRUE,
-      'load' => TRUE,
+        'path' => $editor['library path'] . '/plugins/emotions',
+        'buttons' => array('emotions' => t('Emotions')),
+        'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/emotions',
+        'internal' => TRUE,
+        'load' => TRUE,
     ),
     'font' => array(
-      'path' => $editor['library path'] . '/plugins/font',
-      'buttons' => array('formatselect' => t('HTML block format'), 'fontselect' => t('Font'), 'fontsizeselect' => t('Font size'), 'styleselect' => t('Font style')),
-      'extended_valid_elements' => array('font[face|size|color|style],span[class|align|style]'),
-      'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/font',
-      'internal' => TRUE,
+        'path' => $editor['library path'] . '/plugins/font',
+        'buttons' => array('formatselect' => t('HTML block format'), 'fontselect' => t('Font'), 'fontsizeselect' => t('Font size'), 'styleselect' => t('Font style')),
+        'extended_valid_elements' => array('font[face|size|color|style],span[class|align|style]'),
+        'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/font',
+        'internal' => TRUE,
     ),
     'fullscreen' => array(
-      'path' => $editor['library path'] . '/plugins/fullscreen',
-      'buttons' => array('fullscreen' => t('Fullscreen')),
-      'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/fullscreen',
-      'internal' => TRUE,
-      'load' => TRUE,
+        'path' => $editor['library path'] . '/plugins/fullscreen',
+        'buttons' => array('fullscreen' => t('Fullscreen')),
+        'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/fullscreen',
+        'internal' => TRUE,
+        'load' => TRUE,
     ),
     'inlinepopups' => array(
-      'path' => $editor['library path'] . '/plugins/inlinepopups',
-      'extensions' => array('inlinepopups' => t('Inline popups')),
-      'options' => array(
+        'path' => $editor['library path'] . '/plugins/inlinepopups',
+        'extensions' => array('inlinepopups' => t('Inline popups')),
+        'options' => array(
         'dialog_type' => array('modal'),
-      ),
-      'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/inlinepopups',
-      'internal' => TRUE,
-      'load' => TRUE,
+        ),
+        'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/inlinepopups',
+        'internal' => TRUE,
+        'load' => TRUE,
     ),
     'insertdatetime' => array(
-      'path' => $editor['library path'] . '/plugins/insertdatetime',
-      'buttons' => array('insertdate' => t('Insert date'), 'inserttime' => t('Insert time')),
-      'options' => array(
+        'path' => $editor['library path'] . '/plugins/insertdatetime',
+        'buttons' => array('insertdate' => t('Insert date'), 'inserttime' => t('Insert time')),
+        'options' => array(
         'plugin_insertdate_dateFormat' => '%Y-%m-%d',
         'plugin_insertdate_timeFormat' => '%H:%M:%S',
-      ),
-      'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/insertdatetime',
-      'internal' => TRUE,
-      'load' => TRUE,
+        ),
+        'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/insertdatetime',
+        'internal' => TRUE,
+        'load' => TRUE,
     ),
     'layer' => array(
-      'path' => $editor['library path'] . '/plugins/layer',
-      'buttons' => array('insertlayer' => t('Insert layer'), 'moveforward' => t('Move forward'), 'movebackward' => t('Move backward'), 'absolute' => t('Absolute')),
-      'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/layer',
-      'internal' => TRUE,
-      'load' => TRUE,
+        'path' => $editor['library path'] . '/plugins/layer',
+        'buttons' => array('insertlayer' => t('Insert layer'), 'moveforward' => t('Move forward'), 'movebackward' => t('Move backward'), 'absolute' => t('Absolute')),
+        'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/layer',
+        'internal' => TRUE,
+        'load' => TRUE,
     ),
     'paste' => array(
-      'path' => $editor['library path'] . '/plugins/paste',
-      'buttons' => array('pastetext' => t('Paste text'), 'pasteword' => t('Paste from Word'), 'selectall' => t('Select all')),
-      'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/paste',
-      'internal' => TRUE,
-      'load' => TRUE,
+        'path' => $editor['library path'] . '/plugins/paste',
+        'buttons' => array('pastetext' => t('Paste text'), 'pasteword' => t('Paste from Word'), 'selectall' => t('Select all')),
+        'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/paste',
+        'internal' => TRUE,
+        'load' => TRUE,
     ),
     'preview' => array(
-      'path' => $editor['library path'] . '/plugins/preview',
-      'buttons' => array('preview' => t('Preview')),
-      'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/preview',
-      'internal' => TRUE,
-      'load' => TRUE,
+        'path' => $editor['library path'] . '/plugins/preview',
+        'buttons' => array('preview' => t('Preview')),
+        'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/preview',
+        'internal' => TRUE,
+        'load' => TRUE,
     ),
     'print' => array(
-      'path' => $editor['library path'] . '/plugins/print',
-      'buttons' => array('print' => t('Print')),
-      'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/print',
-      'internal' => TRUE,
-      'load' => TRUE,
+        'path' => $editor['library path'] . '/plugins/print',
+        'buttons' => array('print' => t('Print')),
+        'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/print',
+        'internal' => TRUE,
+        'load' => TRUE,
     ),
     'searchreplace' => array(
-      'path' => $editor['library path'] . '/plugins/searchreplace',
-      'buttons' => array('search' => t('Search'), 'replace' => t('Replace')),
-      'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/searchreplace',
-      'internal' => TRUE,
-      'load' => TRUE,
+        'path' => $editor['library path'] . '/plugins/searchreplace',
+        'buttons' => array('search' => t('Search'), 'replace' => t('Replace')),
+        'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/searchreplace',
+        'internal' => TRUE,
+        'load' => TRUE,
     ),
     'style' => array(
-      'path' => $editor['library path'] . '/plugins/style',
-      'buttons' => array('styleprops' => t('Style properties')),
-      'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/style',
-      'internal' => TRUE,
-      'load' => TRUE,
+        'path' => $editor['library path'] . '/plugins/style',
+        'buttons' => array('styleprops' => t('Style properties')),
+        'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/style',
+        'internal' => TRUE,
+        'load' => TRUE,
     ),
     'table' => array(
-      'path' => $editor['library path'] . '/plugins/table',
-      'buttons' => array('tablecontrols' => t('Table')),
-      'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/table',
-      'internal' => TRUE,
-      'load' => TRUE,
+        'path' => $editor['library path'] . '/plugins/table',
+        'buttons' => array('tablecontrols' => t('Table')),
+        'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/table',
+        'internal' => TRUE,
+        'load' => TRUE,
     ),
-  );
-  if (version_compare($editor['installed version'], '3', '<')) {
+    );
+    if (version_compare($editor['installed version'], '3', '<')) {
     $plugins['flash'] = array(
-      'path' => $editor['library path'] . '/plugins/flash',
-      'buttons' => array('flash' => t('Flash')),
-      'extended_valid_elements' => array('img[class|src|alt|title|hspace|vspace|width|height|align|onmouseover|onmouseout|name|obj|param|embed]'),
-      'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/flash',
-      'internal' => TRUE,
-      'load' => TRUE,
+        'path' => $editor['library path'] . '/plugins/flash',
+        'buttons' => array('flash' => t('Flash')),
+        'extended_valid_elements' => array('img[class|src|alt|title|hspace|vspace|width|height|align|onmouseover|onmouseout|name|obj|param|embed]'),
+        'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/flash',
+        'internal' => TRUE,
+        'load' => TRUE,
     );
-  }
-  if (version_compare($editor['installed version'], '2.0.6', '>')) {
+    }
+    if (version_compare($editor['installed version'], '2.0.6', '>')) {
     $plugins['media'] = array(
-      'path' => $editor['library path'] . '/plugins/media',
-      'buttons' => array('media' => t('Media')),
-      'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/media',
-      'internal' => TRUE,
-      'load' => TRUE,
+        'path' => $editor['library path'] . '/plugins/media',
+        'buttons' => array('media' => t('Media')),
+        'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/media',
+        'internal' => TRUE,
+        'load' => TRUE,
     );
     $plugins['xhtmlxtras'] = array(
-      'path' => $editor['library path'] . '/plugins/xhtmlxtras',
-      'buttons' => array('cite' => t('Citation'), 'del' => t('Deleted'), 'abbr' => t('Abbreviation'), 'acronym' => t('Acronym'), 'ins' => t('Inserted'), 'attribs' => t('HTML attributes')),
-      'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/xhtmlxtras',
-      'internal' => TRUE,
-      'load' => TRUE,
+        'path' => $editor['library path'] . '/plugins/xhtmlxtras',
+        'buttons' => array('cite' => t('Citation'), 'del' => t('Deleted'), 'abbr' => t('Abbreviation'), 'acronym' => t('Acronym'), 'ins' => t('Inserted'), 'attribs' => t('HTML attributes')),
+        'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/xhtmlxtras',
+        'internal' => TRUE,
+        'load' => TRUE,
     );
-  }
-  if (version_compare($editor['installed version'], '3', '>')) {
+    }
+    if (version_compare($editor['installed version'], '3', '>')) {
     $plugins['bbcode'] = array(
-      'path' => $editor['library path'] . '/plugins/bbcode',
-      'extensions' => array('bbcode' => t('BBCode')),
-      'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/bbcode',
-      'internal' => TRUE,
-      'load' => TRUE,
+        'path' => $editor['library path'] . '/plugins/bbcode',
+        'extensions' => array('bbcode' => t('BBCode')),
+        'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/bbcode',
+        'internal' => TRUE,
+        'load' => TRUE,
     );
     if (version_compare($editor['installed version'], '3.3', '<')) {
-      $plugins['safari'] = array(
+        $plugins['safari'] = array(
         'path' => $editor['library path'] . '/plugins/safari',
         'extensions' => array('safari' => t('Safari compatibility')),
         'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/safari',
         'internal' => TRUE,
         'load' => TRUE,
-      );
+        );
     }
-  }
-  if (version_compare($editor['installed version'], '3.2.5', '>=')) {
+    }
+    if (version_compare($editor['installed version'], '3.2.5', '>=')) {
     $plugins['autoresize'] = array(
-      'path' => $editor['library path'] . '/plugins/autoresize',
-      'extensions' => array('autoresize' => t('Auto resize')),
-      'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/autoresize',
-      'internal' => TRUE,
-      'load' => TRUE,
+        'path' => $editor['library path'] . '/plugins/autoresize',
+        'extensions' => array('autoresize' => t('Auto resize')),
+        'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/autoresize',
+        'internal' => TRUE,
+        'load' => TRUE,
     );
-  }
-  if (version_compare($editor['installed version'], '3.3', '>=')) {
+    }
+    if (version_compare($editor['installed version'], '3.3', '>=')) {
     $plugins['advlist'] = array(
-      'path' => $editor['library path'] . '/plugins/advlist',
-      'extensions' => array('advlist' => t('Advanced list')),
-      'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/advlist',
-      'internal' => TRUE,
-      'load' => TRUE,
+        'path' => $editor['library path'] . '/plugins/advlist',
+        'extensions' => array('advlist' => t('Advanced list')),
+        'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/advlist',
+        'internal' => TRUE,
+        'load' => TRUE,
     );
-  }
-  if (version_compare($editor['installed version'], '3.2.6', '>=')) {
+    }
+    if (version_compare($editor['installed version'], '3.2.6', '>=')) {
     $plugins['wordcount'] = array(
-      'path' => $editor['library path'] . '/plugins/wordcount',
-      'extensions' => array('wordcount' => t('Word count')),
-      'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/wordcount',
-      'internal' => TRUE,
-      'load' => TRUE,
+        'path' => $editor['library path'] . '/plugins/wordcount',
+        'extensions' => array('wordcount' => t('Word count')),
+        'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/wordcount',
+        'internal' => TRUE,
+        'load' => TRUE,
     );
-  }
-  return $plugins;
+    }
+    return $plugins;
 }
 

Switch Indentation +23 added lines, -23 removed lines

@@ -261,32 +261,32 @@
 
   // Add theme-specific settings.
   switch ($theme) {
-    case 'advanced':
+  case 'advanced':
+    $settings += array(
+      'theme_advanced_resize_horizontal' => FALSE,
+      'theme_advanced_resizing_use_cookie' => FALSE,
+      'theme_advanced_path_location' => isset($config['path_loc']) ? $config['path_loc'] : 'bottom',
+      'theme_advanced_resizing' => isset($config['resizing']) ? $config['resizing'] : 1,
+      'theme_advanced_toolbar_location' => isset($config['toolbar_loc']) ? $config['toolbar_loc'] : 'top',
+      'theme_advanced_toolbar_align' => isset($config['toolbar_align']) ? $config['toolbar_align'] : 'left',
+    );
+    if (isset($config['block_formats'])) {
+      $settings['theme_advanced_blockformats'] = $config['block_formats'];
+    }
+    if (isset($settings['buttons'])) {
+      // These rows explicitly need to be set to be empty, otherwise TinyMCE
+      // loads its default buttons of the advanced theme for each row.
       $settings += array(
-        'theme_advanced_resize_horizontal' => FALSE,
-        'theme_advanced_resizing_use_cookie' => FALSE,
-        'theme_advanced_path_location' => isset($config['path_loc']) ? $config['path_loc'] : 'bottom',
-        'theme_advanced_resizing' => isset($config['resizing']) ? $config['resizing'] : 1,
-        'theme_advanced_toolbar_location' => isset($config['toolbar_loc']) ? $config['toolbar_loc'] : 'top',
-        'theme_advanced_toolbar_align' => isset($config['toolbar_align']) ? $config['toolbar_align'] : 'left',
+        'theme_advanced_buttons1' => array(),
+        'theme_advanced_buttons2' => array(),
+        'theme_advanced_buttons3' => array(),
       );
-      if (isset($config['block_formats'])) {
-        $settings['theme_advanced_blockformats'] = $config['block_formats'];
+      // @todo Allow to sort/arrange editor buttons.
+      for ($i = 0; $i < count($settings['buttons']); $i++) {
+        $settings['theme_advanced_buttons1'][] = $settings['buttons'][$i];
       }
-      if (isset($settings['buttons'])) {
-        // These rows explicitly need to be set to be empty, otherwise TinyMCE
-        // loads its default buttons of the advanced theme for each row.
-        $settings += array(
-          'theme_advanced_buttons1' => array(),
-          'theme_advanced_buttons2' => array(),
-          'theme_advanced_buttons3' => array(),
-        );
-        // @todo Allow to sort/arrange editor buttons.
-        for ($i = 0; $i < count($settings['buttons']); $i++) {
-          $settings['theme_advanced_buttons1'][] = $settings['buttons'][$i];
-        }
-      }
-      break;
+    }
+    break;
   }
   unset($settings['buttons']);
 

Spacing +35 added lines, -35 removed lines

@@ -15,7 +15,7 @@ 
     'title' => 'TinyMCE',
     'vendor url' => 'http://tinymce.moxiecode.com',
     'download url' => 'http://tinymce.moxiecode.com/download.php',
-    'library path' => wysiwyg_get_path('tinymce') . '/jscripts/tiny_mce',
+    'library path' => wysiwyg_get_path('tinymce').'/jscripts/tiny_mce',
     'libraries' => array(
       '' => array(
         'title' => 'Minified',
@@ -80,7 +80,7 @@ 
  *   The installed editor version.
  */
 function wysiwyg_tinymce_version($editor) {
-  $script = $editor['library path'] . '/tiny_mce.js';
+  $script = $editor['library path'].'/tiny_mce.js';
   if (!file_exists($script)) {
     return;
   }
@@ -91,7 +91,7 @@ 
   // 2.x: this.majorVersion="2";this.minorVersion="1.3"
   // 3.x: majorVersion:'3',minorVersion:'2.0.1'
   if (preg_match('@majorVersion[=:]["\'](\d).+?minorVersion[=:]["\']([\d\.]+)@', $line, $version)) {
-    return $version[1] . '.' . $version[2];
+    return $version[1].'.'.$version[2];
   }
 }
 
@@ -178,7 +178,7 @@ 
     $settings['remove_linebreaks'] = $config['remove_linebreaks'];
   }
   if (isset($config['verify_html'])) {
-    $settings['verify_html'] = (bool) $config['verify_html'];
+    $settings['verify_html'] = (bool)$config['verify_html'];
   }
 
   if (!empty($config['css_classes'])) {
@@ -324,7 +324,7 @@ 
           $plugin['filename'] = array_pop($parts);
           $plugin['path'] = implode('/', $parts);
         }
-        $settings[$name] = base_path() . $plugin['path'] . '/' . $plugin['filename'];
+        $settings[$name] = base_path().$plugin['path'].'/'.$plugin['filename'];
       }
     }
   }
@@ -340,11 +340,11 @@ 
     // 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'],
     );
     if (isset($plugin['css file'])) {
-      $settings[$name]['css'] = base_path() . $plugin['css path'] . '/' . $plugin['css file'];
+      $settings[$name]['css'] = base_path().$plugin['css path'].'/'.$plugin['css file'];
     }
   }
   return $settings;
@@ -364,7 +364,7 @@ 
 function _wysiwyg_tinymce_plugin_name($op, $name) {
   if ($op == 'add') {
     if (strpos($name, '-') !== 0) {
-      return '-' . $name;
+      return '-'.$name;
     }
     return $name;
   }
@@ -382,7 +382,7 @@ 
 function wysiwyg_tinymce_plugins($editor) {
   $plugins = array(
     'default' => array(
-      'path' => $editor['library path'] . '/themes/advanced',
+      'path' => $editor['library path'].'/themes/advanced',
       'buttons' => array(
         'bold' => t('Bold'), 'italic' => t('Italic'), 'underline' => t('Underline'),
         'strikethrough' => t('Strike-through'),
@@ -406,7 +406,7 @@ 
       'internal' => TRUE,
     ),
     'advhr' => array(
-      'path' => $editor['library path'] . '/plugins/advhr',
+      'path' => $editor['library path'].'/plugins/advhr',
       'buttons' => array('advhr' => t('Advanced horizontal rule')),
       'extended_valid_elements' => array('hr[class|width|size|noshade]'),
       'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/advhr',
@@ -414,7 +414,7 @@ 
       'load' => TRUE,
     ),
     'advimage' => array(
-      'path' => $editor['library path'] . '/plugins/advimage',
+      'path' => $editor['library path'].'/plugins/advimage',
       'extensions' => array('advimage' => t('Advanced image')),
       'extended_valid_elements' => array('img[src|alt|title|align|width|height|usemap|hspace|vspace|border|style|class|onmouseover|onmouseout|id|name]'),
       'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/advimage',
@@ -422,7 +422,7 @@ 
       'load' => TRUE,
     ),
     'advlink' => array(
-      'path' => $editor['library path'] . '/plugins/advlink',
+      'path' => $editor['library path'].'/plugins/advlink',
       'extensions' => array('advlink' => t('Advanced link')),
       'extended_valid_elements' => array('a[name|href|target|title|class|onfocus|onblur|onclick|ondlbclick|onmousedown|onmouseup|onmouseover|onmouseout|onkeypress|onkeydown|onkeyup|id|style|rel]'),
       'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/advlink',
@@ -430,49 +430,49 @@ 
       'load' => TRUE,
     ),
     'autosave' => array(
-      'path' => $editor['library path'] . '/plugins/autosave',
+      'path' => $editor['library path'].'/plugins/autosave',
       'extensions' => array('autosave' => t('Auto save')),
       'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/autosave',
       'internal' => TRUE,
       'load' => TRUE,
     ),
     'contextmenu' => array(
-      'path' => $editor['library path'] . '/plugins/contextmenu',
+      'path' => $editor['library path'].'/plugins/contextmenu',
       'extensions' => array('contextmenu' => t('Context menu')),
       'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/contextmenu',
       'internal' => TRUE,
       'load' => TRUE,
     ),
     'directionality' => array(
-      'path' => $editor['library path'] . '/plugins/directionality',
+      'path' => $editor['library path'].'/plugins/directionality',
       'buttons' => array('ltr' => t('Left-to-right'), 'rtl' => t('Right-to-left')),
       'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/directionality',
       'internal' => TRUE,
       'load' => TRUE,
     ),
     'emotions' => array(
-      'path' => $editor['library path'] . '/plugins/emotions',
+      'path' => $editor['library path'].'/plugins/emotions',
       'buttons' => array('emotions' => t('Emotions')),
       'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/emotions',
       'internal' => TRUE,
       'load' => TRUE,
     ),
     'font' => array(
-      'path' => $editor['library path'] . '/plugins/font',
+      'path' => $editor['library path'].'/plugins/font',
       'buttons' => array('formatselect' => t('HTML block format'), 'fontselect' => t('Font'), 'fontsizeselect' => t('Font size'), 'styleselect' => t('Font style')),
       'extended_valid_elements' => array('font[face|size|color|style],span[class|align|style]'),
       'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/font',
       'internal' => TRUE,
     ),
     'fullscreen' => array(
-      'path' => $editor['library path'] . '/plugins/fullscreen',
+      'path' => $editor['library path'].'/plugins/fullscreen',
       'buttons' => array('fullscreen' => t('Fullscreen')),
       'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/fullscreen',
       'internal' => TRUE,
       'load' => TRUE,
     ),
     'inlinepopups' => array(
-      'path' => $editor['library path'] . '/plugins/inlinepopups',
+      'path' => $editor['library path'].'/plugins/inlinepopups',
       'extensions' => array('inlinepopups' => t('Inline popups')),
       'options' => array(
         'dialog_type' => array('modal'),
@@ -482,7 +482,7 @@ 
       'load' => TRUE,
     ),
     'insertdatetime' => array(
-      'path' => $editor['library path'] . '/plugins/insertdatetime',
+      'path' => $editor['library path'].'/plugins/insertdatetime',
       'buttons' => array('insertdate' => t('Insert date'), 'inserttime' => t('Insert time')),
       'options' => array(
         'plugin_insertdate_dateFormat' => '%Y-%m-%d',
@@ -493,49 +493,49 @@ 
       'load' => TRUE,
     ),
     'layer' => array(
-      'path' => $editor['library path'] . '/plugins/layer',
+      'path' => $editor['library path'].'/plugins/layer',
       'buttons' => array('insertlayer' => t('Insert layer'), 'moveforward' => t('Move forward'), 'movebackward' => t('Move backward'), 'absolute' => t('Absolute')),
       'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/layer',
       'internal' => TRUE,
       'load' => TRUE,
     ),
     'paste' => array(
-      'path' => $editor['library path'] . '/plugins/paste',
+      'path' => $editor['library path'].'/plugins/paste',
       'buttons' => array('pastetext' => t('Paste text'), 'pasteword' => t('Paste from Word'), 'selectall' => t('Select all')),
       'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/paste',
       'internal' => TRUE,
       'load' => TRUE,
     ),
     'preview' => array(
-      'path' => $editor['library path'] . '/plugins/preview',
+      'path' => $editor['library path'].'/plugins/preview',
       'buttons' => array('preview' => t('Preview')),
       'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/preview',
       'internal' => TRUE,
       'load' => TRUE,
     ),
     'print' => array(
-      'path' => $editor['library path'] . '/plugins/print',
+      'path' => $editor['library path'].'/plugins/print',
       'buttons' => array('print' => t('Print')),
       'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/print',
       'internal' => TRUE,
       'load' => TRUE,
     ),
     'searchreplace' => array(
-      'path' => $editor['library path'] . '/plugins/searchreplace',
+      'path' => $editor['library path'].'/plugins/searchreplace',
       'buttons' => array('search' => t('Search'), 'replace' => t('Replace')),
       'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/searchreplace',
       'internal' => TRUE,
       'load' => TRUE,
     ),
     'style' => array(
-      'path' => $editor['library path'] . '/plugins/style',
+      'path' => $editor['library path'].'/plugins/style',
       'buttons' => array('styleprops' => t('Style properties')),
       'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/style',
       'internal' => TRUE,
       'load' => TRUE,
     ),
     'table' => array(
-      'path' => $editor['library path'] . '/plugins/table',
+      'path' => $editor['library path'].'/plugins/table',
       'buttons' => array('tablecontrols' => t('Table')),
       'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/table',
       'internal' => TRUE,
@@ -544,7 +544,7 @@ 
   );
   if (version_compare($editor['installed version'], '3', '<')) {
     $plugins['flash'] = array(
-      'path' => $editor['library path'] . '/plugins/flash',
+      'path' => $editor['library path'].'/plugins/flash',
       'buttons' => array('flash' => t('Flash')),
       'extended_valid_elements' => array('img[class|src|alt|title|hspace|vspace|width|height|align|onmouseover|onmouseout|name|obj|param|embed]'),
       'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/flash',
@@ -554,14 +554,14 @@ 
   }
   if (version_compare($editor['installed version'], '2.0.6', '>')) {
     $plugins['media'] = array(
-      'path' => $editor['library path'] . '/plugins/media',
+      'path' => $editor['library path'].'/plugins/media',
       'buttons' => array('media' => t('Media')),
       'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/media',
       'internal' => TRUE,
       'load' => TRUE,
     );
     $plugins['xhtmlxtras'] = array(
-      'path' => $editor['library path'] . '/plugins/xhtmlxtras',
+      'path' => $editor['library path'].'/plugins/xhtmlxtras',
       'buttons' => array('cite' => t('Citation'), 'del' => t('Deleted'), 'abbr' => t('Abbreviation'), 'acronym' => t('Acronym'), 'ins' => t('Inserted'), 'attribs' => t('HTML attributes')),
       'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/xhtmlxtras',
       'internal' => TRUE,
@@ -570,7 +570,7 @@ 
   }
   if (version_compare($editor['installed version'], '3', '>')) {
     $plugins['bbcode'] = array(
-      'path' => $editor['library path'] . '/plugins/bbcode',
+      'path' => $editor['library path'].'/plugins/bbcode',
       'extensions' => array('bbcode' => t('BBCode')),
       'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/bbcode',
       'internal' => TRUE,
@@ -578,7 +578,7 @@ 
     );
     if (version_compare($editor['installed version'], '3.3', '<')) {
       $plugins['safari'] = array(
-        'path' => $editor['library path'] . '/plugins/safari',
+        'path' => $editor['library path'].'/plugins/safari',
         'extensions' => array('safari' => t('Safari compatibility')),
         'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/safari',
         'internal' => TRUE,
@@ -588,7 +588,7 @@ 
   }
   if (version_compare($editor['installed version'], '3.2.5', '>=')) {
     $plugins['autoresize'] = array(
-      'path' => $editor['library path'] . '/plugins/autoresize',
+      'path' => $editor['library path'].'/plugins/autoresize',
       'extensions' => array('autoresize' => t('Auto resize')),
       'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/autoresize',
       'internal' => TRUE,
@@ -597,7 +597,7 @@ 
   }
   if (version_compare($editor['installed version'], '3.3', '>=')) {
     $plugins['advlist'] = array(
-      'path' => $editor['library path'] . '/plugins/advlist',
+      'path' => $editor['library path'].'/plugins/advlist',
       'extensions' => array('advlist' => t('Advanced list')),
       'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/advlist',
       'internal' => TRUE,
@@ -606,7 +606,7 @@ 
   }
   if (version_compare($editor['installed version'], '3.2.6', '>=')) {
     $plugins['wordcount'] = array(
-      'path' => $editor['library path'] . '/plugins/wordcount',
+      'path' => $editor['library path'].'/plugins/wordcount',
       'extensions' => array('wordcount' => t('Word count')),
       'url' => 'http://wiki.moxiecode.com/index.php/TinyMCE:Plugins/wordcount',
       'internal' => TRUE,

Braces +2 added lines, -4 removed lines

@@ -188,8 +188,7 @@ 
   if (isset($config['css_setting'])) {
     if ($config['css_setting'] == 'theme') {
       $settings['content_css'] = 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['content_css'] = strtr($config['css_path'], array('%b' => base_path(), '%t' => path_to_theme()));
     }
   }
@@ -367,8 +366,7 @@ 
       return '-' . $name;
     }
     return $name;
-  }
-  else if ($op == 'remove') {
+  } else if ($op == 'remove') {
     if (strpos($name, '-') === 0) {
       return substr($name, 1);
     }

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

Doc Comments +1 added lines, -1 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_whizzywig_version($editor) {
   $script = $editor['library path'] . '/whizzywig.js';

Indentation +48 added lines, -48 removed lines

@@ -9,32 +9,32 @@ 
  * Plugin implementation of hook_editor().
  */
 function wysiwyg_whizzywig_editor() {
-  $editor['whizzywig'] = array(
+    $editor['whizzywig'] = array(
     'title' => 'Whizzywig',
     'vendor url' => 'http://www.unverse.net',
     'download url' => 'http://www.unverse.net/whizzywig-download.html',
     'libraries' => array(
-      '' => array(
+        '' => array(
         'title' => 'Default',
         'files' => array('whizzywig.js', 'xhtml.js'),
-      ),
+        ),
     ),
     'version callback' => 'wysiwyg_whizzywig_version',
     'settings callback' => 'wysiwyg_whizzywig_settings',
     'plugin callback' => 'wysiwyg_whizzywig_plugins',
     'versions' => array(
-      '55' => array(
+        '55' => array(
         'js files' => array('whizzywig.js'),
-      ),
-      '56' => array(
+        ),
+        '56' => array(
         'js files' => array('whizzywig-56.js'),
-      ),
-      '60' => array(
+        ),
+        '60' => array(
         'js files' => array('whizzywig-60.js'),
-      ),
+        ),
     ),
-  );
-  return $editor;
+    );
+    return $editor;
 }
 
 /**
@@ -47,19 +47,19 @@ 
  *   The installed editor version.
  */
 function wysiwyg_whizzywig_version($editor) {
-  $script = $editor['library path'] . '/whizzywig.js';
-  if (!file_exists($script)) {
+    $script = $editor['library path'] . '/whizzywig.js';
+    if (!file_exists($script)) {
     return;
-  }
-  $script = fopen($script, 'r');
-  $line = fgets($script, 43);
-  // 55: Whizzywig v55i
-  // 60: Whizzywig 60
-  if (preg_match('@Whizzywig v?([0-9]+)@', $line, $version)) {
+    }
+    $script = fopen($script, 'r');
+    $line = fgets($script, 43);
+    // 55: Whizzywig v55i
+    // 60: Whizzywig 60
+    if (preg_match('@Whizzywig v?([0-9]+)@', $line, $version)) {
     fclose($script);
     return $version[1];
-  }
-  fclose($script);
+    }
+    fclose($script);
 }
 
 /**
@@ -77,53 +77,53 @@ 
  *   Drupal.settings.wysiwyg.configs.{editor}
  */
 function wysiwyg_whizzywig_settings($editor, $config, $theme) {
-  $settings = array();
+    $settings = array();
 
-  // Add path to button images, if available.
-  if (is_dir($editor['library path'] . '/btn')) {
+    // Add path to button images, if available.
+    if (is_dir($editor['library path'] . '/btn')) {
     $settings['buttonPath'] = base_path() . $editor['library path'] . '/btn/';
-  }
-  if (file_exists($editor['library path'] . '/WhizzywigToolbar.png')) {
+    }
+    if (file_exists($editor['library path'] . '/WhizzywigToolbar.png')) {
     $settings['toolbarImagePath'] = base_path() . $editor['library path'] . '/WhizzywigToolbar.png';
-  }
-  // Filename changed in version 60.
-  elseif (file_exists($editor['library path'] . '/icons.png')) {
+    }
+    // Filename changed in version 60.
+    elseif (file_exists($editor['library path'] . '/icons.png')) {
     $settings['toolbarImagePath'] = base_path() . $editor['library path'] . '/icons.png';
-  }
+    }
 
-  // Add configured buttons or all available.
-  $settings['buttons'] = array();
-  if (!empty($config['buttons'])) {
+    // Add configured buttons or all available.
+    $settings['buttons'] = array();
+    if (!empty($config['buttons'])) {
     $buttons = array();
     foreach ($config['buttons'] as $plugin) {
-      $buttons = array_merge($buttons, $plugin);
+        $buttons = array_merge($buttons, $plugin);
     }
     $settings['buttons'] = implode(' ', 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_whizzywig_plugins($editor) {
-  return array(
+    return array(
     'default' => array(
-      'buttons' => array(
+        'buttons' => array(
         'formatblock' => t('HTML block format'), 'fontname' => t('Font'), 'fontsize' => t('Font size'),
         'bold' => t('Bold'), 'italic' => t('Italic'), 'underline' => t('Underline'),
         'left' => t('Align left'), 'center' => t('Align center'), 'right' => t('Align right'),
@@ -139,9 +139,9 @@ 
         'clean' => t('Clean-up'),
         'html' => t('Source code'),
         'spellcheck' => t('Spell check'),
-      ),
-      'internal' => TRUE,
+        ),
+        'internal' => TRUE,
     ),
-  );
+    );
 }
 

Spacing +9 added lines, -9 removed lines

@@ -47,7 +47,7 @@ 
  *   The installed editor version.
  */
 function wysiwyg_whizzywig_version($editor) {
-  $script = $editor['library path'] . '/whizzywig.js';
+  $script = $editor['library path'].'/whizzywig.js';
   if (!file_exists($script)) {
     return;
   }
@@ -80,15 +80,15 @@ 
   $settings = array();
 
   // Add path to button images, if available.
-  if (is_dir($editor['library path'] . '/btn')) {
-    $settings['buttonPath'] = base_path() . $editor['library path'] . '/btn/';
+  if (is_dir($editor['library path'].'/btn')) {
+    $settings['buttonPath'] = base_path().$editor['library path'].'/btn/';
   }
-  if (file_exists($editor['library path'] . '/WhizzywigToolbar.png')) {
-    $settings['toolbarImagePath'] = base_path() . $editor['library path'] . '/WhizzywigToolbar.png';
+  if (file_exists($editor['library path'].'/WhizzywigToolbar.png')) {
+    $settings['toolbarImagePath'] = base_path().$editor['library path'].'/WhizzywigToolbar.png';
   }
   // Filename changed in version 60.
-  elseif (file_exists($editor['library path'] . '/icons.png')) {
-    $settings['toolbarImagePath'] = base_path() . $editor['library path'] . '/icons.png';
+  elseif (file_exists($editor['library path'].'/icons.png')) {
+    $settings['toolbarImagePath'] = base_path().$editor['library path'].'/icons.png';
   }
 
   // Add configured buttons or all available.
@@ -104,9 +104,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()));
     }
   }