diff -urp /home/ebola/contributions/modules/nodehierarchy/nodehierarchy.install nodehierarchy/nodehierarchy.install
--- /home/ebola/contributions/modules/nodehierarchy/nodehierarchy.install	2008-05-14 18:22:00.000000000 -0400
+++ nodehierarchy/nodehierarchy.install	2008-08-17 02:52:26.000000000 -0400
@@ -2,6 +2,11 @@
 // $Id: nodehierarchy.install,v 1.1.4.2 2008/05/14 22:22:00 ronan Exp $
 
 /**
+ * @file
+ * Node Hierarchy installer.
+ */
+
+/**
  * Implementation of hook_install().
  */
 function nodehierarchy_install() {
@@ -75,4 +80,4 @@ function nodehierarchy_add_default_paren
     $order_by++;
   }
   return $out;
-}
\ No newline at end of file
+}
diff -urp /home/ebola/contributions/modules/nodehierarchy/nodehierarchy.module nodehierarchy/nodehierarchy.module
--- /home/ebola/contributions/modules/nodehierarchy/nodehierarchy.module	2008-07-18 21:56:28.000000000 -0400
+++ nodehierarchy/nodehierarchy.module	2008-08-17 02:57:38.000000000 -0400
@@ -2,6 +2,12 @@
 // $Id: nodehierarchy.module,v 1.4.2.8 2008/07/19 01:56:28 ronan Exp $
 
 /**
+ * @file
+ * Node Hierarchy allows nodes to be children of other nodes creating a
+ * tree-like hierarchy of content.
+ */
+
+/**
  * Implementation of hook_init().
  */
 function nodehierarchy_init() {
@@ -24,7 +30,7 @@ function nodehierarchy_init() {
 }
 
 /**
- * Implemenation of hook_help().
+ * Implementation of hook_help().
  */
 function nodehierarchy_help($path, $arg) {
   switch ($path) {
@@ -59,7 +65,7 @@ function nodehierarchy_menu() {
     'page callback' => 'nodehierarchy_callback_ajax'
   );
   $items['admin/content/nodehierarchy'] = array(
-    'title' => t('Site Outline'),
+    'title' => 'Site Outline',
     'description' => 'Display an hierarchical outline of the site.',
     'page callback' => 'nodehierarchy_site_outline',
     'access arguments' => array('view site outline'),
@@ -91,7 +97,7 @@ function nodehierarchy_menu() {
     'type' => MENU_CALLBACK,
   );
 
-   $items['node/%node/children'] = array(
+  $items['node/%node/children'] = array(
     'title' => 'Children',
     'page callback' => 'nodehierarchy_view_children',
     'page arguments' => array(1),
@@ -210,7 +216,7 @@ function nodehierarchy_form_alter(&$form
 }
 
 /**
- * Implmentation of hook_nodeapi().
+ * Implementation of hook_nodeapi().
  */
 function nodehierarchy_nodeapi(&$node, $op, $teaser = NULL, $page = NULL) {
   switch ($op) {
@@ -457,7 +463,7 @@ function nodehierarchy_callback_ajax() {
       case "getchildren":
         $node = node_load($nid);
         $children = nodehierarchy_get_node_children_list($node->nid, TRUE, 20);
-        if($node && node_access('view', $node)) {
+        if ($node && node_access('view', $node)) {
           print theme("nodehierarchy_children_list", $node, $children, FALSE);
           print theme("nodehierarchy_children_list_more_link", $node, 20, _nodehierarchy_get_children_count($node->nid));
         }
@@ -465,7 +471,7 @@ function nodehierarchy_callback_ajax() {
       break;
       case "up":
       case "down":
-        if(user_access('reorder children')) {
+        if (user_access('reorder children')) {
           $node = node_load($nid);
           nodehierarchy_movechild($node, $action);
         }
@@ -650,7 +656,7 @@ function nodehierarchy_set_breadcrumbs($
   $parent = $node;
   $homepage = drupal_get_normal_path(variable_get('site_frontpage', 'node'));
   // If any of the node's decendents have a menu, set it to current.
-  $router_item = null;
+  $router_item = NULL;
   while ($parent->nid) {
     $item = menu_get_item("node/". $parent->nid);
     if (!$router_item && _nodehierarchy_get_menu($parent->nid)) {
@@ -882,7 +888,7 @@ function _nodehierarchy_create_menu(&$no
     // the module is called after this one.
     unset($node->menu);
 
-    // Prevent the this module from attempting to create a menu twice.
+    // Prevent this module from attempting to create a menu twice.
     $node->nodehierarchy_create_menu = FALSE;
   }
 }
@@ -919,9 +925,9 @@ function _nodehierarchy_get_menu($nid) {
  */
 function _nodehierarchy_get_parent_menu($parent, $nid) {
   // If the item has a parent node, get the parent's menu item (if any).
-  if( $parent ) {
+  if ($parent) {
     return _nodehierarchy_get_menu($nid);
-  } 
+  }
   // Otherwise get the parent menu of the item (if any).
   else {
     return db_fetch_array(db_query_range("SELECT plid FROM {menu_links} WHERE link_path = '%s'", "node/$nid", 0, 1));
@@ -970,10 +976,10 @@ function _nodehierarchy_get_parent_pulld
 /**
  * Get the items for the parent selector pulldown.
  */
-function _nodehierarchy_get_parent_pulldown_items( $parent_id, $types, $child_node = null, $depth = 0 ) {
+function _nodehierarchy_get_parent_pulldown_items( $parent_id, $types, $child_node = NULL, $depth = 0) {
   $out = array();
   $query = "SELECT * FROM {node} n, {nodehierarchy} h WHERE h.nid = n.nid AND h.parent = %d AND n.type IN (". implode( ",", $types ) .") ORDER BY h.order_by ASC";
-  $result = db_query(db_rewrite_sql($query), $parent_id);  
+  $result = db_query(db_rewrite_sql($query), $parent_id);
   while ($hierarchylist = db_fetch_object($result)) {
     if ($hierarchylist->nid != $child_node && node_access('update', $hierarchylist)) {
       $out[$hierarchylist->nid] = str_repeat('--', $depth) .' '. $hierarchylist->title;
@@ -997,4 +1003,4 @@ function _nodehierarchy_get_parent_types
     }
   }
   return $types;
-}
\ No newline at end of file
+}
diff -urp /home/ebola/contributions/modules/nodehierarchy/nodehierarchy_theme.inc nodehierarchy/nodehierarchy_theme.inc
--- /home/ebola/contributions/modules/nodehierarchy/nodehierarchy_theme.inc	2008-05-14 18:22:00.000000000 -0400
+++ nodehierarchy/nodehierarchy_theme.inc	2008-08-16 01:29:10.000000000 -0400
@@ -83,7 +83,7 @@ function theme_nodehierarchy_new_child_l
       if (node_access('create', $key) && variable_get('nh_child_'. $key, FALSE)) {
         $destination = drupal_get_destination() ."&edit[parent]=$node->nid";
         $title = t('Add a new %s.', array('%s' => $type->name));
-        $create_links[] = l($type->name, "node/add/$key", array('query' => $destination, 'attributes' => array('title' => $title)));
+        $create_links[] = l($type->name, 'node/add/'. str_replace('_', '-', $key), array('query' => $destination, 'attributes' => array('title' => $title)));
       }
     }
     if ($create_links) {
@@ -220,4 +220,4 @@ function theme_nodehierarchy_children_li
     $output .= '</div>';
   }
   return $output;
-}
\ No newline at end of file
+}
diff -urp /home/ebola/contributions/modules/nodehierarchy/nodehierarchy_views/nodehierarchy_views.install nodehierarchy/nodehierarchy_views/nodehierarchy_views.install
--- /home/ebola/contributions/modules/nodehierarchy/nodehierarchy_views/nodehierarchy_views.install	2008-03-23 02:18:40.000000000 -0400
+++ nodehierarchy/nodehierarchy_views/nodehierarchy_views.install	2008-08-17 02:39:23.000000000 -0400
@@ -1,6 +1,11 @@
 <?php
 // $Id: nodehierarchy_views.install,v 1.1.4.1 2008/03/23 06:18:40 ronan Exp $
 
+/**
+ * @file
+ * Views.module integration for nodehierarchy.module installer
+ */
+
 // Create the database table on install (MySQL only for now)
 function nodehierarchy_views_install() {
   drupal_install_schema('nodehierarchy_views');
@@ -33,4 +38,4 @@ function nodehierarchy_views_schema() {
     'primary key' => array('nid'),
   );
   return $schema;
-}
\ No newline at end of file
+}
diff -urp /home/ebola/contributions/modules/nodehierarchy/nodehierarchy.views_default.inc nodehierarchy/nodehierarchy.views_default.inc
--- /home/ebola/contributions/modules/nodehierarchy/nodehierarchy.views_default.inc	2008-05-14 18:22:00.000000000 -0400
+++ nodehierarchy/nodehierarchy.views_default.inc	2008-08-17 02:44:18.000000000 -0400
@@ -2,12 +2,17 @@
 // $Id: nodehierarchy.views_default.inc,v 1.1.2.2 2008/05/14 22:22:00 ronan Exp $
 
 /**
+ * @file
+ * Views.module Node Hierarchy Views defaults installer
+ */
+
+/**
  * Implementation of hook_views_default_views().
  */
 function nodehierarchy_views_views_default_views() {
   $view = new view;
   $view->name = 'nodehierarchy_children_teasers';
-  $view->description = 'Display a node\'s children as teasers';
+  $view->description = 'Display a node\'s children as teasers.';
   $view->tag = '';
   $view->view_php = '';
   $view->base_table = 'node';
@@ -101,7 +106,7 @@ function nodehierarchy_views_views_defau
         ),
       );
   $view->name = 'nodehierarchy_children_list';
-  $view->description = t('Display a node\'s children as a list');
+  $view->description = t('Display a node\'s children as a list.');
   $views[$view->name] = $view;
 
 
diff -urp /home/ebola/contributions/modules/nodehierarchy/nodehierarchy.views.inc nodehierarchy/nodehierarchy.views.inc
--- /home/ebola/contributions/modules/nodehierarchy/nodehierarchy.views.inc	2008-05-14 18:22:00.000000000 -0400
+++ nodehierarchy/nodehierarchy.views.inc	2008-08-17 02:47:27.000000000 -0400
@@ -1,9 +1,14 @@
 <?php
 // $Id: nodehierarchy.views.inc,v 1.1.2.2 2008/05/14 22:22:00 ronan Exp $
 
+/**
+ * @file
+ * Views.module integration includes for nodehierarchy.module
+ */
+
 // Implementations of Views Hooks
 /**
- * Implementation of hook_views_data()
+ * Implementation of hook_views_data().
  */
 function nodehierarchy_views_views_data() {
   // Basic table information.
@@ -193,4 +198,4 @@ class views_handler_field_nodehierarchy_
     $node->nid = $values->{$this->field_alias};
     return theme("nodehierarchy_actions", $node);
   }
-}
\ No newline at end of file
+}
diff -urp /home/ebola/contributions/modules/nodehierarchy/nodehierarchy_workflow_ng.inc nodehierarchy/nodehierarchy_workflow_ng.inc
--- /home/ebola/contributions/modules/nodehierarchy/nodehierarchy_workflow_ng.inc	2008-05-14 18:22:00.000000000 -0400
+++ nodehierarchy/nodehierarchy_workflow_ng.inc	2008-08-17 02:40:07.000000000 -0400
@@ -7,7 +7,7 @@
  */
 
 /**
- * Implementation of hook_condition_info()
+ * Implementation of hook_condition_info().
  */
 function nodehierarchy_condition_info() {
   return array(
@@ -27,7 +27,7 @@ function nodehierarchy_has_parent($node,
 
 
 /**
- * Implementation of hook_action_info()
+ * Implementation of hook_action_info().
  */
 function nodehierarchy_action_info() {
   return array(