Index: skinr.handlers.inc
===================================================================
RCS file: /cvs/drupal-contrib/contributions/modules/skinr/skinr.handlers.inc,v
retrieving revision 1.6
diff -u -p -F '^f' -r1.6 skinr.handlers.inc
--- skinr.handlers.inc	10 Nov 2010 19:36:36 -0000	1.6
+++ skinr.handlers.inc	10 Nov 2010 20:41:43 -0000
@@ -2,7 +2,7 @@
 // $Id: skinr.handlers.inc,v 1.6 2010/11/10 19:36:36 jgirlygirl Exp $
 /**
  * @file
- * Defines the various handler objects to support Skinr.
+ * Defines the various default handler functions to support Skinr.
  */
 
 /*
@@ -10,14 +10,15 @@
  *
  * @param $op
  *   What kind of action is being performed. Possible values:
- *   - "access skinr": access to edit skinr's selector
- *   - "access skinr classes": access to edit skinr's additional classes
+ *   - 'access skinr': access to edit Skinr's settings.
+ *   - 'access skinr classes': access to edit Skinr's additional classes.
  * @param &$form
  *   Passes in the $form parameter from hook_form_alter().
  * @param $form_state
  *   Passes in the $form_state parameter from hook_form_alter().
+ *
  * @return
- *   TRUE if we get access, FALSE if we don't.
+ *   TRUE if we get access, FALSE otherwise.
  */
 function skinr_access_handler($op, &$form, $form_state) {
   switch ($op) {
@@ -33,14 +34,15 @@ function skinr_access_handler($op, &$for
  *
  * @param $op
  *   What kind of action is being performed. Possible values:
- *   - "form": the form elements for Skinr are being inserted in a form
- *   - "submit": the form has been submitted.
+ *   - 'form': the form elements for Skinr are being inserted in a form
+ *   - 'submit': the form has been submitted.
  * @param &$form
- *   - For "form", passes in the $form parameter from hook_form_alter().
- *   - For "submit", passes in the $form parameter from hook_form_submit().
+ *   - For 'form', passes in the $form parameter from hook_form_alter().
+ *   - For 'submit', passes in the $form parameter from hook_form_submit().
  * @param $form_state
- *   - For "form", passes in the $form_state parameter from hook_form_alter().
- *   - For "submit", passes in the $form_state parameter from hook_form_submit().
+ *   - For 'form', passes in the $form_state parameter from hook_form_alter().
+ *   - For 'submit', passes in the $form_state parameter from hook_form_submit().
+ *
  * @return
  *   The index where we can find our values in Skinrs data structure.
  */
@@ -62,20 +64,23 @@ function skinr_index_handler($op, &$form
   }
 }
 
-/*
+/**
  * Skinr data handler.
  *
  * @param &$form
  *   Passes in the $form parameter from hook_form_submit().
  * @param $form_state
  *   Passes in the $form_state parameter from hook_form_submit().
+ * @param $theme
+ *   The theme that is currently being processed.
  * @param $module
  *   The module that is currently being processed.
  * @param $form_settings
  *   The settings from hook_skinr_config() for the form that's currently being
  *   processed.
+ *
  * @return
- *   TRUE if we get access, FALSE if we don't.
+ *   A skinr settings object.
  */
 function skinr_data_handler(&$form, $form_state, $theme, $module, $form_settings) {
   // Ensure we have the required index_handler
@@ -99,8 +104,6 @@ function skinr_data_handler(&$form, $for
  * @param $form_settings
  *   The settings from hook_skinr_config() for the form that's currently being
  *   processed.
- * @return
- *   TRUE if we get access, FALSE if we don't.
  */
 function skinr_submit_handler(&$form, $form_state, $module, $form_settings) {
   if (isset($form_state['values']['skinr_settings'][$module . '_group'])) {
@@ -139,7 +142,7 @@ function skinr_submit_handler(&$form, $f
 // Declare API compatibility on behalf of core modules:
 
 /**
- * Implementation of hook_skinr_api().
+ * Implements hook_skinr_api().
  *
  * This one is used as the base to reduce errors when updating.
  */
Index: skinr.install
===================================================================
RCS file: /cvs/drupal-contrib/contributions/modules/skinr/skinr.install,v
retrieving revision 1.10
diff -u -p -F '^f' -r1.10 skinr.install
--- skinr.install	29 Oct 2010 06:24:37 -0000	1.10
+++ skinr.install	10 Nov 2010 20:41:43 -0000
@@ -7,7 +7,7 @@
  */
 
 /**
- * Implementation of hook_schema.
+ * Implements hook_schema().
  */
 function skinr_schema() {
   $schema['skinr'] = array(
@@ -181,7 +181,7 @@ function skinr_schema() {
 }
 
 /**
- * Implementation of hook_uninstall().
+ * Implements hook_uninstall().
  */
 function skinr_uninstall() {
   // Remove all skinr variables.
Index: skinr.module
===================================================================
RCS file: /cvs/drupal-contrib/contributions/modules/skinr/skinr.module,v
retrieving revision 1.31
diff -u -p -F '^f' -r1.31 skinr.module
--- skinr.module	10 Nov 2010 19:36:36 -0000	1.31
+++ skinr.module	10 Nov 2010 20:41:43 -0000
@@ -2,7 +2,12 @@
 // $Id: skinr.module,v 1.31 2010/11/10 19:36:36 jgirlygirl Exp $
 
 /**
- * Implementation of hook_help().
+ * @file
+ * Handles core Skinr functionality.
+ */
+
+/**
+ * Implements hook_help().
  */
 function skinr_help($path, $arg) {
   switch ($path) {
@@ -30,7 +35,7 @@ function skinr_module_implements_alter(&
 }
 
 /**
- * Implementation of hook_init().
+ * Implements hook_init().
  */
 function skinr_init() {
   module_load_include('inc', 'skinr', 'skinr.handlers');
@@ -38,7 +43,7 @@ function skinr_init() {
 }
 
 /**
- * Implementation of hook_preprocess().
+ * Implements hook_preprocess().
  */
 function skinr_preprocess(&$variables, $hook) {
   // Fix for update script.
@@ -73,7 +78,7 @@ function skinr_preprocess(&$variables, $
         }
         $variables['classes_array'] = array_merge($variables['classes_array'], $extracted['classes']);
 
-        // Need to add a hook for skinr_ui to plug into.
+        // Add a hook for skinr_ui to plug into: hook_skinr_preprocess_alter().
         $data = array(
           'hook' => $hook,
           'variables' => &$variables,
@@ -85,6 +90,24 @@ function skinr_preprocess(&$variables, $
   }
 }
 
+/**
+ * Helper function to extract the classes, js and css files, and templates
+ * that we are applying.
+ *
+ * @param $module
+ *   The module these settings applies to.
+ * @param $sids
+ *   The ID, or an array of IDs, for these settings.
+ * @param $settings
+ *   Skinr configuration data for the desired module.
+ * @param $theme
+ *   The theme these settings apply to. If no theme name is provided, the
+ *   current theme will be used.
+ *
+ * @return
+ *   An array containing the module, sids, classes, css, js, template to be
+ *   applied.
+ */
 function skinr_skin_extract($module, $sids, $settings, $theme = NULL) {
   if (empty($sids)) {
     return FALSE;
@@ -242,9 +265,17 @@ function skinr_skin_get_files($skins, $t
 }
 
 /**
- * Helper fuction to add CSS and JS files.
+ * Helper fuction to add CSS and JS files using drupal_add_css()
+ * and drupal_add_js().
+ *
+ * @param $filename
+ *   The name of the file to add.
+ * @param $type
+ *   The type of file. Options include: 'css' and 'js'.
+ * @param $media
+ *   (optional) The media type for the stylesheet, e.g., 'all', 'print',
+ *   'screen'. Only required if $type is set to 'css'.
  *
- * The function checks an array of paths for the existence of the file to account for base themes.
  */
 function _skinr_add_file($filename, $type, $media = NULL) {
   if ($type == 'css') {
@@ -257,6 +288,12 @@ function _skinr_add_file($filename, $typ
 
 /**
  * Helper function to flatten an array of classes settings.
+ *
+ * @param $skins
+ *   The array of skins to flatten.
+ *
+ * @return
+ *   An array of classes.
  */
 function skinr_flatten_skins_array($skins) {
   $return = array();
@@ -277,10 +314,16 @@ function skinr_flatten_skins_array($skin
 }
 
 // ------------------------------------------------------------------
-// Page rule functions.
+// Rule functions.
 
 /**
- * Save a skinr page rule object.
+ * Save a skinr rule object.
+ *
+ * @param $rule
+ *   A rule object.
+ *
+ * @return
+ *   The rule ID.
  */
 function skinr_rule_save($rule) {
   drupal_write_record('skinr_rules', $rule, !empty($rule->rid) ? array('rid') : array());
@@ -288,7 +331,14 @@ function skinr_rule_save($rule) {
 }
 
 /**
- * Load a skinr page rule object.
+ * Load a skinr rule object.
+ *
+ * @param $rid
+ *   The rule ID.
+ *
+ * @return
+ *   A rule object. If no $rid is specified an array of all rules will be
+ *   returned.
  */
 function skinr_rule_load($rid = NULL) {
   $rids = (isset($rid) ? array($rid) : array());
@@ -297,7 +347,16 @@ function skinr_rule_load($rid = NULL) {
 }
 
 /**
- * Loads multiple skinr page rule objects.
+ * Loads multiple skinr rule objects.
+ *
+ * @param $rids
+ *   An array of rule IDs. Optional.
+ * @param $conditions
+ *   An array of conditions on the {skinr_rules} table in the form 'field' => $value.
+ *
+ * @return
+ *   An array of rule objects indexed by rid. If $rids is not provided, all
+ *   rules are returned.
  */
 function skinr_rule_load_multiple($rids = array(), $conditions = array()) {
   $rules = array();
@@ -316,7 +375,10 @@ function skinr_rule_load_multiple($rids 
 }
 
 /**
- * Delete a skinr page rule object.
+ * Delete a skinr rule object.
+ *
+ * @param $rid
+ *   The rule ID.
  */
 function skinr_rule_delete($rid) {
   if ($rule = skinr_rule_load($rid)) {
@@ -330,6 +392,15 @@ function skinr_rule_delete($rid) {
   }
 }
 
+/**
+ * Determin whether a rule is visible to the current page or not.
+ *
+ * @param $rid
+ *   The rule ID.
+ *
+ * @return
+ *   TRUE if the rule is visible to the current page, FALSE otherwise.
+ */
 function skinr_rule_visible($rid) {
   global $user;
 
@@ -370,7 +441,12 @@ function skinr_rule_visible($rid) {
 // Include file helpers.
 
 /**
- * Load skinr files on behalf of modules.
+ * Include files as necessary.
+ *
+ * @param $file
+ *   The file to include. The file needs to be prefixed with the
+ *   implementing module's name, but do not include the module name
+ *   prefix (i.e. MODULE.$file).
  */
 function skinr_module_include($file) {
   foreach (skinr_get_module_apis() as $module => $info) {
@@ -408,10 +484,10 @@ function skinr_get_module_apis() {
 // Skinr data handling functions.
 
 /**
- * Validate a skinr object.
+ * Validate a skinr settings object.
  *
  * @param $skinr
- *   A skinr object.
+ *   The skinr settings object to validate.
  *
  * @return
  *   TRUE on success, FALSE on failure.
@@ -434,7 +510,7 @@ function skinr_validate(&$skinr) {
 }
 
 /**
- * Save a skinr object.
+ * Save a skinr settings object.
  *
  * @param $skinr
  *   A skinr object.
@@ -472,12 +548,21 @@ function skinr_set($skinr) {
 }
 
 /**
- * Retrieves the desired skinr object.
+ * Retrieves the desired skinr settings object.
+ *
+ * @param $theme
+ *   The theme these settings apply to. If no theme name is provided, the
+ *   current theme will be used.
+ * @param $module
+ *   The module these settings apply to.
+ * @param $sid
+ *   The element ID these settings apply to.
  *
  * @return
- *   A skinr object if both $module and $sid are specified. An array of skinr
- *   objects if only $module is specified. An array of all skinr objects for a
- *   theme if neither $module nor $sid is specified. FALSE on failure.
+ *   A skinr settings object if both $module and $sid are specified. An array
+ *   of skinr settings objects if only $module is specified. An array of all
+ *   skinr settings objects for a theme if neither $module nor $sid is
+ *   specified. FALSE on failure.
  */
 function skinr_get($theme = NULL, $module = NULL, $sid = NULL) {
   $cache = &drupal_static(__FUNCTION__, array());
@@ -543,6 +628,12 @@ function skinr_get($theme = NULL, $modul
 
 /**
  * Helper function to remove empty skins from an array.
+ *
+ * @param $array
+ *   A single- or multi-dimensional array to strip of empty values.
+ *
+ * @return
+ *   An array stripped of empty values.
  */
 function _skinr_array_strip_empty($array) {
   $new_array = array();
@@ -565,6 +656,9 @@ function _skinr_array_strip_empty($array
  * @param $exculde_admin_theme
  *   Optional. Set to TRUE to exclude the admin theme from posible themes to
  *   return.
+ *
+ * @return
+ *   The current theme name.
  */
 function skinr_current_theme($exclude_admin_theme = FALSE) {
   global $user, $custom_theme;
@@ -583,7 +677,7 @@ function skinr_current_theme($exclude_ad
 }
 
 /**
- * Prepare defaults for skins.
+ * Prepare defaults for skinsets.
  *
  * @return
  *   An array of default skinset settings.
@@ -598,10 +692,10 @@ function skinr_skins_default() {
 }
 
 /**
- * Prepare defaults for skinr options.
+ * Prepare defaults for groups.
  *
  * @return
- *   An array of default skinset options settings.
+ *   An array of default group settings.
  */
 function skinr_group_default() {
   return array(
@@ -617,7 +711,7 @@ function skinr_group_default() {
  * Prepare defaults for skins.
  *
  * @return
- *   An array of default skins settings.
+ *   An array of default skin settings.
  */
 function skinr_skin_default() {
   return array(
@@ -636,8 +730,11 @@ function skinr_skin_default() {
 /**
  * Helper function to process a skin or theme .info file.
  *
+ * @param $info
+ *   An array of data as gotten from a skin or theme .info file.
+ *
  * @return
- *    A skinset.
+ *    A processed set of skins.
  */
 function _skinr_skinset($info) {
   $skinset = array(
@@ -745,7 +842,7 @@ function _skinr_add_path_to_files($files
  * Helper function to process an array of skins.
  *
  * @return
- *    An array of skinsets.
+ *    An array of skins.
  */
 function skinr_skinsets() {
   $skinsets = &drupal_static(__FUNCTION__, array());
@@ -791,10 +888,10 @@ function skinr_skinsets() {
 }
 
 /**
- * Return an array of statuses of each skin in a skinset for each theme.
+ * Return an array of statuses for each skin in a skinset for each theme.
  *
  * @param $skinset_name
- *   The name of the skinset of which to load the statuses.
+ *   The name of the skinset for which to load the statuses.
  *
  * @return
  *   Array of statuses for individual skins in a skinset.
@@ -815,7 +912,7 @@ function skinr_skinset_statuses($skinset
 }
 
 /**
- * Themes are allowed to set Skinr skins in their .info files.
+ * Processes and compiles a list of all skins.
  *
  * @return
  *    An array of skinsets keyed by themename.
@@ -871,7 +968,10 @@ function skinr_skin_data() {
 }
 
 /**
- * Fetch Skinr configuration data from modules.
+ * Fetch Skinr configuration data from Skinr functionality plugins.
+ *
+ * @return
+ *   An array of all configuration data.
  */
 function skinr_fetch_config() {
   $cache = &drupal_static(__FUNCTION__);
@@ -888,7 +988,15 @@ function skinr_fetch_config() {
 }
 
 /**
- * Fetch default configuration data for modules.
+ * Fetch default configuration data for Skinr functionality plugins.
+ *
+ * @param @setting
+ *   The value 'form'. There are no other valide options.
+ *
+ * @return
+ *   Returns an array of default values.
+ *
+ * @todo Remove the $setting parameter. There is only ever one option used: 'form'.
  */
 function _skinr_fetch_config_defaults($setting) {
   switch ($setting) {
@@ -912,6 +1020,63 @@ function _skinr_fetch_config_defaults($s
 
 /**
  * Execute a module's data handler.
+ *
+ * @param $type
+ *   The type of handler to execute. Possible values:
+ *   - 'access_handler':
+ *   - 'contextual_links':
+ *   - 'data_handler':
+ *   - 'form_index_handler':
+ *   - 'preprocess_index_handler':
+ *   - 'preprocess_hook_callback':
+ *   - 'submit_handler':
+ * @param $op
+ *   For 'access_handler' the possible values are 'access skinr'
+ *     and 'access skinr classes'.
+ *   For 'contextual_links' an empty string is passed.
+ *   For 'data_handler' the possible values are 'form' and 'submit'.
+ *   For 'form_index_handler' the possible values are 'form' and 'submit'.
+ *   For 'preprocess_index_handler' the possible values are 'preprocess'.
+ *   For 'preprocess_hook_callback' an empty string is passed.
+ *   For 'submit_handler' an empty string is passed.
+ * @param $handler
+ *   The function name for this handler as gotten from skinr_fetch_config().
+ * @param $a3
+ *   For 'access_handler', passes in the $form parameter as provided to a
+ *     form function.
+ *   For 'contextual_links', passes in the $variables parameter
+ *     from skinr_preprocess().
+ *   For 'data_handler', passes in the $form parameter from hook_form_submit().
+ *   For 'form_index_handler':
+ *   - For $op 'form', passes in the $form parameter from hook_form_alter().
+ *   - For $op 'submit', passes in the $form parameter from hook_form_submit().
+ *   For 'preprocess_index_handler', passes in the $variables parameter
+ *     from module_preprocess().
+ *   For 'preprocess_hook_callback', passes in the $form parameter
+ *     from hook_form_alter().
+ *   For 'submit_handler', passes in the $form parameter
+ *     from hook_form_alter().
+ * @param $a4
+ *   For 'access_handler', passes in the $form_state array as provided to a
+ *     form function.
+ *   For 'data_handler', passes in the $form_state parameter
+ *     from hook_form_submit().
+ *   For 'form_index_handler':
+ *   - For $op 'form', passes in the $form_state parameter from hook_form_alter().
+ *   - For $op 'submit', passes in the $form_state parameter from hook_form_submit().
+ *   For 'preprocess_hook_callback', passes in the $form_state parameter
+ *     from hook_form_alter().
+ *   For 'submit_handler', passes in the $form_state parameter
+ *     from hook_form_alter().
+ * @param $a5
+ *   For 'data_handler', passes in the module that is currently being processed.
+ *   For 'submit_handler', passes in the module that is currently being processed.
+ * @param $a6
+ *   For 'data_handler', passes in the settings from hook_skinr_config() for
+ *     the form that's currently being processed.
+ *   For 'submit_handler', passes in the settings from hook_skinr_config() for
+ *     the form that's currently being processed.
+ * @param $a7
  */
 function skinr_handler($type, $op, $handler, &$a3, $a4 = NULL, $a5 = NULL, $a6 = NULL, $a7 = NULL) {
   if (is_callable($handler)) {
Index: skinr_ui.admin.inc
===================================================================
RCS file: /cvs/drupal-contrib/contributions/modules/skinr/skinr_ui.admin.inc,v
retrieving revision 1.16
diff -u -p -F '^f' -r1.16 skinr_ui.admin.inc
--- skinr_ui.admin.inc	29 Oct 2010 06:42:59 -0000	1.16
+++ skinr_ui.admin.inc	10 Nov 2010 20:41:43 -0000
@@ -3,7 +3,7 @@
 
 /**
  * @file
- * Admin page callbacks for the skinr module.
+ * Admin page callbacks for the Skinr module.
  */
 
 /**
@@ -21,6 +21,9 @@ function skinr_ui_skinr_operations() {
 
 /**
  * List skinr administration filters that can be applied.
+ *
+ * @return
+ *   An array of filters.
  */
 function skinr_ui_filters() {
   // Theme filter.
@@ -58,6 +61,10 @@ function skinr_ui_filters() {
 
 /**
  * Return form for skinr administration filters.
+ * 
+ * @ingroup forms
+ *
+ * @todo This function is missing the $form and $form_state parameters.
  */
 function skinr_ui_filter_form() {
   $session = &$_SESSION['skinr_ui_overview_filter'];
@@ -157,7 +164,7 @@ function theme_skinr_ui_filters($variabl
 }
 
 /**
- * Process result from node administration filter form.
+ * Process skinr_ui_filter_form() submissions.
  */
 function skinr_ui_filter_form_submit($form, &$form_state) {
   $filters = skinr_ui_filters();
@@ -187,6 +194,8 @@ function skinr_ui_filter_form_submit($fo
 
 /**
  * Menu callback; displays the skinr listing page.
+ * 
+ * @ingroup forms
  */
 function skinr_ui_list($form, &$form_state) {
   if (isset($form_state['values']['operation']) && $form_state['values']['operation'] == 'delete') {
@@ -203,6 +212,8 @@ function skinr_ui_list($form, &$form_sta
 
 /**
  * Form builder: Builds the node administration overview.
+ * 
+ * @ingroup forms
  */
 function skinr_ui_admin_skins() {
   $session = !empty($_SESSION['skinr_ui_overview_filter']) && is_array($_SESSION['skinr_ui_overview_filter']) ? $_SESSION['skinr_ui_overview_filter'] : array();
@@ -326,7 +337,7 @@ function skinr_ui_admin_skins() {
 }
 
 /**
- * Validate skinr_ui_list form submissions.
+ * Validate skinr_ui_list() form submissions.
  *
  * Check if any skinr settings have been selected to perform the chosen
  * 'Update option' on.
@@ -339,7 +350,7 @@ function skinr_ui_admin_skins_validate($
 }
 
 /**
- * Process skinr_ui_list form submissions.
+ * Process skinr_ui_list() form submissions.
  *
  * Execute the chosen 'Update option' on the selected skinr settings.
  */
@@ -367,6 +378,14 @@ function skinr_ui_admin_skins_submit($fo
   }
 }
 
+/**
+ * Menu callback; displays the delete confirmation for multiple deletions.
+ *
+ * @param $skins
+ *   An array of skins to delete.
+ * 
+ * @ingroup forms
+ */
 function skinr_ui_multiple_delete_confirm($form, &$form_state, $skins) {
   $themes = list_themes();
 
@@ -392,6 +411,9 @@ function skinr_ui_multiple_delete_confir
                     t('Delete'), t('Cancel'));
 }
 
+/**
+ * Process skinr_ui_multiple_delete_confirm() submissions.
+ */
 function skinr_ui_multiple_delete_confirm_submit($form, &$form_state) {
   if ($form_state['values']['confirm']) {
     foreach ($form_state['values']['skins'] as $skin => $value) {
@@ -420,6 +442,8 @@ function skinr_ui_multiple_delete_confir
 
 /**
  * Menu callback; displays the skinr settings administration form.
+ * 
+ * @ingroup forms
  */
 function skinr_ui_settings($form, &$form_state) {
   $form = array();
@@ -446,7 +470,7 @@ function skinr_ui_settings($form, &$form
 }
 
 /**
- * Validation handler for skinr settings administration form.
+ * Validate skinr_ui_settings() submissions.
  */
 function skinr_ui_settings_validate(&$form, &$form_state) {
   if (empty($form_state['values']['skinr_overlay_width'])) {
@@ -602,13 +626,22 @@ function _skinr_ui_admin_skinsets_build_
 
 /**
  * Helper function to sort skinsets by the name in their .info file.
+ *
+ * @param $a
+ *   The first skinset to compare.
+ * @param $b
+ *   The second skinset to compare.
+ *
+ * @return
+ *   Returns < 0 if $a is less than $b; > 0 if $a is greater than $b, and 0
+ *   if they are equal.
  */
 function skinr_ui_sort_by_info_name($a, $b) {
   return strcasecmp($a->info['name'], $b->info['name']);
 }
 
 /**
- * Process skinr_ui_admin_skinsets form submissions.
+ * Process skinr_ui_admin_skinsets() form submissions.
  */
 function skinr_ui_admin_skinsets_submit($form, &$form_state) {
   // Store list of previously enabled themes and disable all themes.
@@ -870,7 +903,7 @@ function theme_skinr_ui_admin_skinsets_s
 }
 
 /**
- * Process skinr_ui_admin_skinsets_settings form submissions.
+ * Process skinr_ui_admin_skinsets_settings() form submissions.
  */
 function skinr_ui_admin_skinsets_settings_submit($form, &$form_state) {
   if ($form_state['values']['op'] == t('Save configuration')) {
@@ -918,7 +951,13 @@ function skinr_ui_admin_skinsets_setting
 }
 
 /**
- * Menu callback; displays Skinr settings export form.
+ * Menu callback; displays the Skinr settings export form.
+ *
+ * @param $theme
+ *   (optional) The name of the theme to export Skinr settings for. If no
+ *   theme name is provided a theme selector is shown.
+ *
+ * @ingroup forms
  */
 function skinr_ui_export_form($form, &$form_state, $theme = NULL) {
   $form = array();
@@ -996,7 +1035,7 @@ function skinr_ui_export_form($form, &$f
 }
 
 /**
- * Validation handler for skinr settings export form.
+ * Validate skinr_ui_export_form() submissions.
  */
 function skinr_ui_export_form_validate(&$form, &$form_state) {
   if (!empty($form_state['values']['theme']) && preg_match('/[^a-zA-Z0-9_]/', $form_state['values']['theme'])) {
@@ -1005,14 +1044,16 @@ function skinr_ui_export_form_validate(&
 }
 
 /**
- * Submit handler for skinr settings export form.
+ * Process skinr_ui_export_form() submissions.
  */
 function skinr_ui_export_form_submit(&$form, &$form_state) {
   drupal_goto('admin/appearance/skinr/export/' . str_replace('_', '-', $form_state['values']['theme']));
 }
 
 /**
- * Menu callback; displays Skinr settings import form.
+ * Menu callback; displays the Skinr settings import form.
+ *
+ * @ingroup forms
  */
 function skinr_ui_import_form($form, &$form_state) {
   $form = array();
@@ -1061,7 +1102,7 @@ function skinr_ui_import_form($form, &$f
 }
 
 /**
- * Validation handler for Skinr settings import form.
+ * Validate skinr_ui_import_form() submissions.
  */
 function skinr_ui_import_form_validate(&$form, &$form_state) {
   if (empty($form_state['values']['skinr_settings'])) {
@@ -1103,7 +1144,7 @@ function skinr_ui_import_form_validate(&
 }
 
 /**
- * Submit handler for skinr settings import form.
+ * Process skinr_ui_import_form() submissions.
  */
 function skinr_ui_import_form_submit(&$form, &$form_state) {
   foreach ($form_state['skinrs'] as $module => $skinrs) {
@@ -1119,7 +1160,16 @@ function skinr_ui_import_form_submit(&$f
 }
 
 /**
- * Menu callback; displays the delete confirmation for a skinr page rule.
+ * Menu callback; displays the delete confirmation for a skinr setting.
+ *
+ * @param $theme
+ *   The name of the theme to delete a setting for.
+ * @param $module
+ *   The module to delete a setting for.
+ * @param $sid
+ *   The ID of the setting to delete.
+ *
+ * @ingroup forms
  */
 function skinr_ui_delete_confirm($form, &$form_state, $theme, $module, $sid) {
   $form['theme'] = array(
@@ -1146,6 +1196,9 @@ function skinr_ui_delete_confirm($form, 
   );
 }
 
+/**
+ * Process skinr_ui_delete_confirm() submissions.
+ */
 function skinr_ui_delete_confirm_submit($form, &$form_state) {
   if ($form_state['values']['confirm']) {
     $skinr = new stdClass();
Index: skinr_ui.install
===================================================================
RCS file: /cvs/drupal-contrib/contributions/modules/skinr/skinr_ui.install,v
retrieving revision 1.4
diff -u -p -F '^f' -r1.4 skinr_ui.install
--- skinr_ui.install	27 Sep 2010 16:09:10 -0000	1.4
+++ skinr_ui.install	10 Nov 2010 20:41:43 -0000
@@ -7,7 +7,7 @@
  */
 
 /**
- * Implementation of hook_install().
+ * Implementats hook_install().
  */
 function skinr_ui_install() {
   // Skinr should go before contextual module.
@@ -18,7 +18,7 @@ function skinr_ui_install() {
 }
 
 /**
- * Implementation of hook_uninstall().
+ * Implementats hook_uninstall().
  */
 function skinr_ui_uninstall() {
   variable_del('skinr_overlay_width');
@@ -27,6 +27,10 @@ function skinr_ui_uninstall() {
 
 /**
  * Delete a few unused variables.
+ *
+ * @todo This is a badly converted function it should be
+ *       skinr_ui_update_6001(). Remove this function and implement
+ *       skinr_update_last_removed() instead.
  */
 function skinr_ui_update_7000() {
   variable_del('skinr_overlay_autofit');
Index: skinr_ui.module
===================================================================
RCS file: /cvs/drupal-contrib/contributions/modules/skinr/skinr_ui.module,v
retrieving revision 1.18
diff -u -p -F '^f' -r1.18 skinr_ui.module
--- skinr_ui.module	10 Nov 2010 19:36:36 -0000	1.18
+++ skinr_ui.module	10 Nov 2010 20:41:43 -0000
@@ -2,7 +2,12 @@
 // $Id: skinr_ui.module,v 1.18 2010/11/10 19:36:36 jgirlygirl Exp $
 
 /**
- * Implementation of hook_permission().
+ * @file
+ * Handles Skinr UI functionality allowing users to apply skins to their site.
+ */
+
+/**
+ * Implements hook_permission().
  */
 function skinr_ui_permission() {
   return array(
@@ -22,7 +27,7 @@ function skinr_ui_permission() {
 }
 
 /**
- * Implementation of hook_menu().
+ * Implements hook_menu().
  */
 function skinr_ui_menu() {
   $items['admin/appearance/skinr'] = array(
@@ -208,7 +213,7 @@ if (!function_exists('dialog_js_load')) 
 }
 
 /**
- * Implementation of hook_theme().
+ * Implements hook_theme().
  */
 function skinr_ui_theme() {
   return array(
@@ -240,7 +245,7 @@ function skinr_ui_theme() {
 }
 
 /**
- * Implementation of hook_help().
+ * Implements hook_help().
  */
 function skinr_ui_help($path, $arg) {
   if (module_exists('advanced_help')) {
@@ -265,7 +270,10 @@ function skinr_ui_help($path, $arg) {
 }
 
 /**
- * Check if overlay functionality is enabled.
+ * Checks if overlay functionality is enabled.
+ *
+ * @return
+ *   TRUE if enabled, FALE otherwise.
  */
 function skinr_ui_overlay_enabled() {
   static $warned = FALSE;
@@ -323,15 +331,14 @@ function skinr_ui_edit_contextual_title(
  * Menu callback; prepares some variables and displays a Skinr edit form.
  *
  * @param $js
- *   ...
+ *   TRUE if called from javascript, FALSE otherwise.
  * @param $module
  *   The module that we're editing settings of.
  * @param $sid
  *   The sid of the object we're editing settings of.
  * @param $sids
- *   ...
- *
- * @todo Enter description of parameters.
+ *   An array of $sid when more than one is returned from the preprocess
+ *   index handler. Used by the javascript UI to update all elements involved.
  */
 function skinr_ui_edit($js = FALSE, $module, $sid, $sids = NULL) {
   global $gskinr;
@@ -351,7 +358,12 @@ function skinr_ui_edit($js = FALSE, $mod
 }
 
 /**
- * Skinr form.
+ * Form builder for the skinr settings configuration form.
+ * 
+ * @param $arguments
+ *   An array of arguments as passed in by skinr_ui_edit().
+ * 
+ * @ingroup forms
  */
 function skinr_ui_form($form, &$form_state, $arguments) {
   $form = array(
@@ -383,7 +395,7 @@ function skinr_ui_form($form, &$form_sta
 }
 
 /**
- * Implementation of hook_form_alter().
+ * Implements hook_form_alter().
  */
 function skinr_ui_form_alter(&$form, $form_state, $form_id) {
   // Fix for update script.
@@ -594,7 +606,7 @@ function skinr_ui_form_alter(&$form, $fo
 }
 
 /**
- * Validation handler.
+ * Validate skinr_ui_form_alter() submissions.
  */
 function skinr_ui_form_validate($form, &$form_state) {
   $form_id = $form_state['values']['form_id'];
@@ -643,7 +655,7 @@ function skinr_ui_form_validate($form, &
 }
 
 /**
- * Submit handler.
+ * Process skinr_ui_form_alter() submissions.
  */
 function skinr_ui_form_submit($form, &$form_state) {
   $form_id = $form_state['values']['form_id'];
@@ -661,7 +673,7 @@ function skinr_ui_form_submit($form, &$f
 }
 
 /**
- * Implementation of hook_preprocess().
+ * Implements hook_preprocess().
  */
 function skinr_ui_preprocess(&$variables, $hook) {
   $skinr_config = skinr_fetch_config();
@@ -685,8 +697,15 @@ function skinr_ui_preprocess(&$variables
 }
 
 /**
- * Set contextual menu items for skinr. We couldn't use hook_block_view_alter()
- * because not all blocks are allowing the context to be set from there.
+ * Set contextual menu items for skinr.
+ *
+ * @param $variables
+ *   The $variables parameter from a preprocess function.
+ * @param $hook
+ *   The $hook parameter from a preprocess function.
+ * @param $contextual_links
+ *   An array of contextual links data as returned from Skinr's contextual
+ *   links handler.
  */
 function skinr_ui_contextual_links(&$variables, $hook, $contextual_links) {
   _skinr_ui_set_contextual_links($hook, $contextual_links);
@@ -712,10 +731,22 @@ function skinr_ui_contextual_links(&$var
   }
 }
 
+/**
+ * Get all contextual links as returned from Skinr's contextual links handler.
+ *
+ * @return
+ *   An array of contextual links data.
+ */
 function skinr_ui_get_contextual_links() {
   return _skinr_ui_set_contextual_links();
 }
 
+/**
+ * Store contextual links internally for future use.
+ *
+ * @return
+ *   An array of contextual links data.
+ */
 function _skinr_ui_set_contextual_links($hook = NULL, $links = NULL) {
   static $contextual_links = array();
 
@@ -729,8 +760,16 @@ function _skinr_ui_set_contextual_links(
 }
 
 /**
- * Helper function determine whether one of a set of hooks exists in a list of
- * required features.
+ * Helper function to determine whether one of a set of hooks exists in a list
+ * of required theme hooks.
+ *
+ * @param $preprocess_hooks
+ *   An array of theme hooks available to this element.
+ * @param $features
+ *   An array of allowed theme hooks.
+ *
+ * @return
+ *   TRUE if an overlap is found, FALSE otherwise.
  */
 function _skinr_is_featured($preprocess_hooks, $features) {
   foreach ($preprocess_hooks as $preprocess_hook) {
@@ -743,6 +782,9 @@ function _skinr_is_featured($preprocess_
 
 /**
  * Helper function to retrieve a unique id for each skinr class. Used by AJAX.
+ *
+ * @return
+ *   A unique ID number.
  */
 function _skinr_ui_ajax_id() {
   static $skinr_id = 0;
@@ -750,8 +792,14 @@ function _skinr_ui_ajax_id() {
 }
 
 /**
- * Helper function to convert an array of options, as specified in the info
- * file, into an array usable by form api.
+ * Helper function to convert an array of options, as specified in the .info
+ * file, into an array usable by Form API.
+ *
+ * @param $options
+ *   An array containing at least the 'class' and 'label' keys.
+ *
+ * @return
+ *   A Form API compatible array of options.
  */
 function skinr_ui_info_options_to_form_options($options) {
   $form_options = array();
Index: skinr_ui.rules.inc
===================================================================
RCS file: /cvs/drupal-contrib/contributions/modules/skinr/skinr_ui.rules.inc,v
retrieving revision 1.6
diff -u -p -F '^f' -r1.6 skinr_ui.rules.inc
--- skinr_ui.rules.inc	27 May 2010 14:07:17 -0000	1.6
+++ skinr_ui.rules.inc	10 Nov 2010 20:41:43 -0000
@@ -3,7 +3,7 @@
 
 /**
  * @file
- * Admin page callbacks for the skinr module.
+ * Admin page callbacks for Skinr module's rules.
  */
 
 /**
@@ -54,6 +54,8 @@ function skinr_rules() {
 
 /**
  * Menu callback; displays the edit form for a skinr rule.
+ * 
+ * @ingroup forms
  */
 function skinr_rule_add($form, &$form_state) {
   $form = array();
@@ -96,6 +98,9 @@ function skinr_rule_add($form, &$form_st
   return $form;
 }
 
+/**
+ * Process skinr_rule_add() submissions.
+ */
 function skinr_rule_add_submit($form, &$form_state) {
   $rule = new stdClass();
   $rule->rid = NULL;
@@ -114,6 +119,11 @@ function skinr_rule_add_submit($form, &$
 
 /**
  * Menu callback; displays the edit form for a skinr rule.
+ *
+ * @param $rid
+ *   The rule ID.
+ * 
+ * @ingroup forms
  */
 function skinr_rule_edit($form, &$form_state, $rid = NULL) {
   $form = array();
@@ -216,6 +226,9 @@ function skinr_rule_edit($form, &$form_s
   return $form;
 }
 
+/**
+ * Process skinr_rule_edit() submissions.
+ */
 function skinr_rule_edit_submit($form, &$form_state) {
   $rule = new stdClass();
   $rule->rid = !empty($form_state['values']['rid']) ? $form_state['values']['rid'] : NULL;
@@ -232,6 +245,11 @@ function skinr_rule_edit_submit($form, &
   $form_state['redirect'] = 'admin/appearance/skinr/rules';
 }
 
+/**
+ * Called from within the rule edit form; redirects to skinr_rule_delete_confirm().
+ * 
+ * @ingroup forms
+ */
 function skinr_rule_delete_submit($form, &$form_state) {
   $destination = '';
   if (isset($_REQUEST['destination'])) {
@@ -243,6 +261,11 @@ function skinr_rule_delete_submit($form,
 
 /**
  * Menu callback; displays the delete confirmation for a skinr rule.
+ *
+ * @param $rid
+ *   The rule ID.
+ * 
+ * @ingroup forms
  */
 function skinr_rule_delete_confirm($form, &$form_state, $rid) {
   $form['rid'] = array(
@@ -260,6 +283,9 @@ function skinr_rule_delete_confirm($form
   );
 }
 
+/**
+ * Process skinr_rule_delete_confirm() submissions.
+ */
 function skinr_rule_delete_confirm_submit($form, &$form_state) {
   if ($form_state['values']['confirm']) {
     skinr_rule_delete($form_state['values']['rid']);
Index: modules/block.skinr.inc
===================================================================
RCS file: /cvs/drupal-contrib/contributions/modules/skinr/modules/block.skinr.inc,v
retrieving revision 1.12
diff -u -p -F '^f' -r1.12 block.skinr.inc
--- modules/block.skinr.inc	15 Oct 2010 21:11:55 -0000	1.12
+++ modules/block.skinr.inc	10 Nov 2010 20:41:43 -0000
@@ -12,7 +12,7 @@
  */
 
 /**
- * Implementation of hook_skinr_config().
+ * Implements hook_skinr_config().
  */
 function block_skinr_config() {
   $data['block']['form']['block_admin_configure'] = array(
@@ -49,14 +49,15 @@ function block_skinr_config() {
  *
  * @param $op
  *   What kind of action is being performed. Possible values:
- *   - "form": the form elements for Skinr are being inserted in a form
- *   - "submit": the form has been submitted.
+ *   - 'form': the form elements for Skinr are being inserted in a form
+ *   - 'submit': the form has been submitted.
  * @param &$form
- *   - For "form", passes in the $form parameter from hook_form_alter().
- *   - For "submit", passes in the $form parameter from hook_form_submit().
+ *   - For 'form', passes in the $form parameter from hook_form_alter().
+ *   - For 'submit', passes in the $form parameter from hook_form_submit().
  * @param $form_state
- *   - For "form", passes in the $form_state parameter from hook_form_alter().
- *   - For "submit", passes in the $form_state parameter from hook_form_submit().
+ *   - For 'form', passes in the $form_state parameter from hook_form_alter().
+ *   - For 'submit', passes in the $form_state parameter from hook_form_submit().
+ *
  * @return
  *   The index where we can find our values in Skinrs data structure.
  */
@@ -71,14 +72,15 @@ function block_skinr_form_index_handler(
 }
 
 /**
- * Skinr preprocess_hook_callback.
+ * Skinr preprocess hook callback.
  *
  * @param &$form
  *   Passes in the $form parameter from hook_form_alter().
  * @param $form_state
  *   Passes in the $form_state parameter from hook_form_alter().
+ *
  * @return
- *   The preprocess_hook we wish to use.
+ *   An array of preprocess hooks we wish to use.
  */
 function block_skinr_preprocess_hook_callback(&$form, $form_state) {
   $preprocess_hooks = array();
@@ -107,8 +109,9 @@ function block_skinr_preprocess_hook_cal
  *
  * @param &$variables
  *   Passes in the $variables parameter from module_preprocess().
+ *
  * @return
- *   The index where we can find our values in Skinrs data structure. If an
+ *   The index where we can find our values in Skinr's data structure. If an
  *   array is returned, it will loop through each index in Skinr's data
  *   structure and merge the returned classes.
  */
@@ -121,6 +124,7 @@ function block_skinr_preprocess_index_ha
  *
  * @param &$variables
  *   Passes in the $variables parameter from skinr_preprocess().
+ *
  * @return
  *   An array. Each value is an array that forms the function arguments for
  *   menu_contextual_links(). For example:
Index: modules/comment.skinr.inc
===================================================================
RCS file: /cvs/drupal-contrib/contributions/modules/skinr/modules/comment.skinr.inc,v
retrieving revision 1.9
diff -u -p -F '^f' -r1.9 comment.skinr.inc
--- modules/comment.skinr.inc	2 Jun 2010 13:16:58 -0000	1.9
+++ modules/comment.skinr.inc	10 Nov 2010 20:41:43 -0000
@@ -12,7 +12,7 @@
  */
 
 /**
- * Implementation of hook_skinr_config().
+ * Implements hook_skinr_config().
  */
 function comment_skinr_config() {
   $data['comment']['form']['node_type_form'] = array(
@@ -42,16 +42,17 @@ function comment_skinr_config() {
  *
  * @param $op
  *   What kind of action is being performed. Possible values:
- *   - "form": the form elements for Skinr are being inserted in a form
- *   - "submit": the form has been submitted.
+ *   - 'form': the form elements for Skinr are being inserted in a form
+ *   - 'submit': the form has been submitted.
  * @param &$form
- *   - For "form", passes in the $form parameter from hook_form_alter().
- *   - For "submit", passes in the $form parameter from hook_form_submit().
+ *   - For 'form', passes in the $form parameter from hook_form_alter().
+ *   - For 'submit', passes in the $form parameter from hook_form_submit().
  * @param $form_state
- *   - For "form", passes in the $form_state parameter from hook_form_alter().
- *   - For "submit", passes in the $form_state parameter from hook_form_submit().
+ *   - For 'form', passes in the $form_state parameter from hook_form_alter().
+ *   - For 'submit', passes in the $form_state parameter from hook_form_submit().
+ *
  * @return
- *   The index where we can find our values in Skinrs data structure.
+ *   The index where we can find our values in Skinr's data structure.
  */
 function comment_skinr_form_index_handler($op, &$form, &$form_state) {
   switch ($op) {
@@ -76,14 +77,15 @@ function comment_skinr_form_index_handle
 }
 
 /**
- * Skinr preprocess_hook_callback.
+ * Skinr preprocess hook callback.
  *
  * @param &$form
  *   Passes in the $form parameter from hook_form_alter().
  * @param $form_state
  *   Passes in the $form_state parameter from hook_form_alter().
+ *
  * @return
- *   The preprocess_hook we wish to use.
+ *   An array of preprocess hooks we wish to use.
  */
 function comment_skinr_preprocess_hook_callback(&$form, $form_state) {
   $preprocess_hooks = array();
@@ -104,8 +106,9 @@ function comment_skinr_preprocess_hook_c
  *
  * @param &$variables
  *   Passes in the $variables parameter from skinr_preprocess().
+ *
  * @return
- *   The index where we can find our values in Skinrs data structure. If an
+ *   The index where we can find our values in Skinr's data structure. If an
  *   array is returned, it will loop through each index in Skinr's data
  *   structure and merge the returned classes.
  */
@@ -118,6 +121,7 @@ function comment_skinr_preprocess_index_
  *
  * @param &$variables
  *   Passes in the $variables parameter from skinr_preprocess().
+ *
  * @return
  *   An array. Each value is an array that forms the function arguments for
  *   menu_contextual_links(). For example:
Index: modules/node.skinr.inc
===================================================================
RCS file: /cvs/drupal-contrib/contributions/modules/skinr/modules/node.skinr.inc,v
retrieving revision 1.9
diff -u -p -F '^f' -r1.9 node.skinr.inc
--- modules/node.skinr.inc	2 Jun 2010 13:16:58 -0000	1.9
+++ modules/node.skinr.inc	10 Nov 2010 20:41:43 -0000
@@ -12,7 +12,7 @@
  */
 
 /**
- * Implementation of hook_skinr_config().
+ * Implements hook_skinr_config().
  */
 function node_skinr_config() {
   $data['node']['form']['node_type_form'] = array(
@@ -40,16 +40,17 @@ function node_skinr_config() {
  *
  * @param $op
  *   What kind of action is being performed. Possible values:
- *   - "form": the form elements for Skinr are being inserted in a form
- *   - "submit": the form has been submitted.
+ *   - 'form': the form elements for Skinr are being inserted in a form
+ *   - 'submit': the form has been submitted.
  * @param &$form
- *   - For "form", passes in the $form parameter from hook_form_alter().
- *   - For "submit", passes in the $form parameter from hook_form_submit().
+ *   - For 'form', passes in the $form parameter from hook_form_alter().
+ *   - For 'submit', passes in the $form parameter from hook_form_submit().
  * @param $form_state
- *   - For "form", passes in the $form_state parameter from hook_form_alter().
- *   - For "submit", passes in the $form_state parameter from hook_form_submit().
+ *   - For 'form', passes in the $form_state parameter from hook_form_alter().
+ *   - For 'submit', passes in the $form_state parameter from hook_form_submit().
+ *
  * @return
- *   The index where we can find our values in Skinrs data structure.
+ *   The index where we can find our values in Skinr's data structure.
  */
 function node_skinr_form_index_handler($op, &$form, $form_state) {
   switch ($op) {
@@ -74,14 +75,15 @@ function node_skinr_form_index_handler($
 }
 
 /**
- * Skinr preprocess_hook_callback.
+ * Skinr preprocess hook callback.
  *
  * @param &$form
  *   Passes in the $form parameter from hook_form_alter().
  * @param $form_state
  *   Passes in the $form_state parameter from hook_form_alter().
+ *
  * @return
- *   The preprocess_hook we wish to use.
+ *   An array of preprocess hooks we wish to use.
  */
 function node_skinr_preprocess_hook_callback(&$form, $form_state) {
   $preprocess_hooks = array();
@@ -102,8 +104,9 @@ function node_skinr_preprocess_hook_call
  *
  * @param &$variables
  *   Passes in the $variables parameter from module_preprocess().
+ *
  * @return
- *   The index where we can find our values in Skinrs data structure. If an
+ *   The index where we can find our values in Skinr's data structure. If an
  *   array is returned, it will loop through each index in Skinr's data
  *   structure and merge the returned classes.
  */
Index: modules/panels.skinr.inc
===================================================================
RCS file: /cvs/drupal-contrib/contributions/modules/skinr/modules/panels.skinr.inc,v
retrieving revision 1.16
diff -u -p -F '^f' -r1.16 panels.skinr.inc
--- modules/panels.skinr.inc	10 Nov 2010 19:36:36 -0000	1.16
+++ modules/panels.skinr.inc	10 Nov 2010 20:41:43 -0000
@@ -12,7 +12,7 @@
  */
 
 /**
- * Implementation of hook_skinr_config().
+ * Implements hook_skinr_config().
  */
 function panels_skinr_config() {
   $data['panels']['form']['panels_edit_style_settings_form'] = array(
@@ -57,16 +57,17 @@ function panels_skinr_config() {
  *
  * @param $op
  *   What kind of action is being performed. Possible values:
- *   - "form": the form elements for Skinr are being inserted in a form
- *   - "submit": the form has been submitted.
+ *   - 'form': the form elements for Skinr are being inserted in a form
+ *   - 'submit': the form has been submitted.
  * @param &$form
- *   - For "form", passes in the $form parameter from hook_form_alter().
- *   - For "submit", passes in the $form parameter from hook_form_submit().
+ *   - For 'form', passes in the $form parameter from hook_form_alter().
+ *   - For 'submit', passes in the $form parameter from hook_form_submit().
  * @param $form_state
- *   - For "form", passes in the $form_state parameter from hook_form_alter().
- *   - For "submit", passes in the $form_state parameter from hook_form_submit().
+ *   - For 'form', passes in the $form_state parameter from hook_form_alter().
+ *   - For 'submit', passes in the $form_state parameter from hook_form_submit().
+ *
  * @return
- *   The index where we can find our values in Skinrs data structure.
+ *   The index where we can find our values in Skinr's data structure.
  */
 function panels_skinr_form_index_handler($op, &$form, $form_state) {
   switch ($op) {
@@ -88,14 +89,15 @@ function panels_skinr_form_index_handler
  *
  * @param $op
  *   What kind of action is being performed. Possible values:
- *   - "access skinr": access to edit skinr's selector
- *   - "access skinr classes": access to edit skinr's additional classes
+ *   - 'access skinr': access to edit Skinr's settings.
+ *   - 'access skinr classes': access to edit Skinr's additional classes.
  * @param &$form
  *   Passes in the $form parameter from hook_form_alter().
  * @param $form_state
  *   Passes in the $form_state parameter from hook_form_alter().
+ *
  * @return
- *   TRUE if we get access, FALSE if we don't.
+ *   TRUE if we get access, FALSE otherwise.
  */
 function panels_skinr_access_handler_display($op, &$form, $form_state) {
   // We don't want the skinr settings form to appear on this form. We only want
@@ -171,8 +173,9 @@ function panels_skinr_access_handler_dis
  * @param $form_settings
  *   The settings from hook_skinr_config() for the form that's currently being
  *   processed.
+ *
  * @return
- *   TRUE if we get access, FALSE if we don't.
+ *   A skinr settings object.
  */
 function panels_skinr_data_handler(&$form, $form_state, $theme, $module, $form_settings) {
   // Ensure we have the required index_handler.
@@ -200,16 +203,15 @@ function panels_skinr_data_handler(&$for
 /**
  * Skinr submit handler.
  *
- * @param $op
- *   What kind of action is being performed. Possible values:
- *   - "access skinr": access to edit skinr's selector
- *   - "access skinr classes": access to edit skinr's additional classes
  * @param &$form
- *   Passes in the $form parameter from hook_form_alter().
+ *   Passes in the $form parameter from hook_form_submit().
  * @param $form_state
- *   Passes in the $form_state parameter from hook_form_alter().
- * @return
- *   TRUE if we get access, FALSE if we don't.
+ *   Passes in the $form_state parameter from hook_form_submit().
+ * @param $module
+ *   The module that is currently being processed.
+ * @param $form_settings
+ *   The settings from hook_skinr_config() for the form that's currently being
+ *   processed.
  */
 function panels_skinr_submit_handler_settings(&$form, $form_state, $module, $form_settings) {
   foreach ($form_state['values']['skinr_settings'][$module . '_group'] as $theme_name => $theme) {
@@ -261,7 +263,7 @@ function panels_skinr_submit_handler_set
 }
 
 /**
- * @todo remove values from skinr if a display is deleted
+ * @todo Remove values from skinr if a display is deleted.
  */
 function panels_skinr_submit_handler_display(&$form, $form_state, $module, $form_settings) {
   $form_id = $form_state['values']['form_id'];
@@ -324,14 +326,15 @@ function _panels_skinr_save_and_clear_ca
 }
 
 /**
- * Skinr preprocess_hook_callback.
+ * Skinr preprocess hook callback.
  *
  * @param &$form
  *   Passes in the $form parameter from hook_form_alter().
  * @param $form_state
  *   Passes in the $form_state parameter from hook_form_alter().
+ *
  * @return
- *   The preprocess_hook we wish to use.
+ *   An array of preprocess hooks we wish to use.
  */
 function panels_skinr_preprocess_hook_callback(&$form, $form_state) {
   switch ($form_state['type']) {
@@ -345,14 +348,15 @@ function panels_skinr_preprocess_hook_ca
 }
 
 /**
- * Skinr preprocess_hook_callback.
+ * Skinr preprocess hook callback.
  *
  * @param &$form
  *   Passes in the $form parameter from hook_form_alter().
  * @param $form_state
  *   Passes in the $form_state parameter from hook_form_alter().
+ *
  * @return
- *   The preprocess_hook we wish to use.
+ *   An array of preprocess hooks we wish to use.
  */
 function panels_skinr_ajax_preprocess_hook_callback(&$form, $form_state) {
   if (strpos($form['skinr']['sid']['#value'], '-region-') !== FALSE) {
@@ -371,8 +375,9 @@ function panels_skinr_ajax_preprocess_ho
  *
  * @param &$variables
  *   Passes in the $variables parameter from module_preprocess().
+ *
  * @return
- *   The index where we can find our values in Skinrs data structure. If an
+ *   The index where we can find our values in Skinr's data structure. If an
  *   array is returned, it will loop through each index in Skinr's data
  *   structure and merge the returned classes.
  */
@@ -388,7 +393,7 @@ function panels_skinr_preprocess_index_h
 // Panels hooks.
 
 /**
- * Implementation of hook_ctools_plugin_directory() to let the system know
+ * Implementats hook_ctools_plugin_directory() to let the system know
  * we implement panels plugins.
  */
 function skinr_ctools_plugin_directory($module, $plugin) {
Index: modules/skinr.skinr.inc
===================================================================
RCS file: /cvs/drupal-contrib/contributions/modules/skinr/modules/skinr.skinr.inc,v
retrieving revision 1.7
diff -u -p -F '^f' -r1.7 skinr.skinr.inc
--- modules/skinr.skinr.inc	24 Oct 2010 02:51:58 -0000	1.7
+++ modules/skinr.skinr.inc	10 Nov 2010 20:41:43 -0000
@@ -48,14 +48,14 @@ function skinr_skinr_config() {
  *
  * @param $op
  *   What kind of action is being performed. Possible values:
- *   - "form": the form elements for Skinr are being inserted in a form
- *   - "submit": the form has been submitted.
+ *   - 'form': the form elements for Skinr are being inserted in a form
+ *   - 'submit': the form has been submitted.
  * @param &$form
- *   - For "form", passes in the $form parameter from hook_form_alter().
- *   - For "submit", passes in the $form parameter from hook_form_submit().
+ *   - For 'form', passes in the $form parameter from hook_form_alter().
+ *   - For 'submit', passes in the $form parameter from hook_form_submit().
  * @param $form_state
- *   - For "form", passes in the $form_state parameter from hook_form_alter().
- *   - For "submit", passes in the $form_state parameter from hook_form_submit().
+ *   - For 'form', passes in the $form_state parameter from hook_form_alter().
+ *   - For 'submit', passes in the $form_state parameter from hook_form_submit().
  * @return
  *   The index where we can find our values in Skinr's data structure.
  */
Index: modules/views.skinr.inc
===================================================================
RCS file: /cvs/drupal-contrib/contributions/modules/skinr/modules/views.skinr.inc,v
retrieving revision 1.11
diff -u -p -F '^f' -r1.11 views.skinr.inc
--- modules/views.skinr.inc	10 Nov 2010 20:16:55 -0000	1.11
+++ modules/views.skinr.inc	10 Nov 2010 20:41:43 -0000
@@ -13,7 +13,7 @@
  */
 
 /**
- * Implementation of hook_skinr_config().
+ * Implements hook_skinr_config().
  */
 function views_skinr_config() {
   $data['views']['form']['views_ui_edit_display_form'] = array(
@@ -48,16 +48,17 @@ function views_skinr_config() {
  *
  * @param $op
  *   What kind of action is being performed. Possible values:
- *   - "form": the form elements for Skinr are being inserted in a form
- *   - "submit": the form has been submitted.
+ *   - 'form': the form elements for Skinr are being inserted in a form
+ *   - 'submit': the form has been submitted.
  * @param &$form
- *   - For "form", passes in the $form parameter from hook_form_alter().
- *   - For "submit", passes in the $form parameter from hook_form_submit().
+ *   - For 'form', passes in the $form parameter from hook_form_alter().
+ *   - For 'submit', passes in the $form parameter from hook_form_submit().
  * @param $form_state
- *   - For "form", passes in the $form_state parameter from hook_form_alter().
- *   - For "submit", passes in the $form_state parameter from hook_form_submit().
+ *   - For 'form', passes in the $form_state parameter from hook_form_alter().
+ *   - For 'submit', passes in the $form_state parameter from hook_form_submit().
+ *
  * @return
- *   The index where we can find our values in Skinrs data structure.
+ *   The index where we can find our values in Skinr's data structure.
  */
 function views_skinr_form_index_handler($op, &$form, $form_state) {
   switch ($op) {
@@ -87,14 +88,15 @@ function views_skinr_form_index_handler(
  *
  * @param $op
  *   What kind of action is being performed. Possible values:
- *   - "access skinr": access to edit skinr's selector
- *   - "access skinr classes": access to edit skinr's additional classes
+ *   - 'access skinr': access to edit skinr's settings.
+ *   - 'access skinr classes': access to edit skinr's additional classes.
  * @param &$form
  *   Passes in the $form parameter from hook_form_alter().
  * @param $form_state
  *   Passes in the $form_state parameter from hook_form_alter().
+ *
  * @return
- *   TRUE if we get access, FALSE if we don't.
+ *   TRUE if we get access, FALSE otherwise.
  */
 function views_skinr_access_handler_display($op, &$form, $form_state) {
   if ($access = skinr_access_handler($op, $form, $form_state)) {
@@ -147,8 +149,9 @@ function views_skinr_access_handler_view
  * @param $form_settings
  *   The settings from hook_skinr_config() for the form that's currently being
  *   processed.
+ *
  * @return
- *   A skinr object.
+ *   A skinr settings object.
  */
 function views_skinr_data_handler(&$form, $form_state, $theme, $module, $form_settings) {
   // Ensure we have the required index_handler
@@ -182,16 +185,15 @@ function views_skinr_data_handler(&$form
 /**
  * Skinr submit handler.
  *
- * @param $op
- *   What kind of action is being performed. Possible values:
- *   - "access skinr": access to edit skinr's selector
- *   - "access skinr classes": access to edit skinr's additional classes
  * @param &$form
- *   Passes in the $form parameter from hook_form_alter().
+ *   Passes in the $form parameter from hook_form_submit().
  * @param $form_state
- *   Passes in the $form_state parameter from hook_form_alter().
- * @return
- *   TRUE if we get access, FALSE if we don't.
+ *   Passes in the $form_state parameter from hook_form_submit().
+ * @param $module
+ *   The module that is currently being processed.
+ * @param $form_settings
+ *   The settings from hook_skinr_config() for the form that's currently being
+ *   processed.
  */
 function views_skinr_submit_handler_display(&$form, $form_state, $module, $form_settings) {
   foreach ($form_state['values']['skinr_settings'][$module .'_group'] as $theme_name => $theme) {
@@ -268,14 +270,15 @@ function views_skinr_submit_handler_view
 }
 
 /**
- * Skinr form preprocess_hook callback.
+ * Skinr form preprocess hook callback.
  *
  * @param &$form
  *   Passes in the $form parameter from hook_form_alter().
  * @param $form_state
  *   Passes in the $form_state parameter from hook_form_alter().
+ *
  * @return
- *   The preprocess_hook we wish to use.
+ *   An array of preprocess hooks we wish to use.
  */
 function views_skinr_preprocess_hook_callback(&$form, $form_state) {
   $preprocess_hooks = array('views_view');
@@ -323,6 +326,7 @@ function _views_skinr_sidinfo($sid) {
  *
  * @param &$variables
  *   Passes in the $variables parameter from module_preprocess().
+ *
  * @return
  *   The index where we can find our values in Skinrs data structure. If an
  *   array is returned, it will loop through each index in Skinr's data