Index: modules/book/book.install
===================================================================
RCS file: /cvs/drupal/drupal/modules/book/book.install,v
retrieving revision 1.5
diff -u -p -r1.5 book.install
--- modules/book/book.install	20 Aug 2006 06:38:50 -0000	1.5
+++ modules/book/book.install	25 Aug 2006 03:58:38 -0000
@@ -1,30 +1,110 @@
 <?php
-// $Id: book.install,v 1.5 2006/08/20 06:38:50 dries Exp $
+// $Id: 
 
 function book_install() {
   switch ($GLOBALS['db_type']) {
     case 'mysql':
     case 'mysqli':
+      db_query("CREATE TABLE {book_pages} (
+        nid int unsigned NOT NULL default '0',
+        bid int unsigned NOT NULL default '0',
+        parent int unsigned NOT NULL default '0',
+        weight smallint NOT NULL default '0',
+        PRIMARY KEY (nid),
+        KEY bid (bid),
+        KEY parent (parent)
+      ) /*!40100 DEFAULT CHARACTER SET UTF8 */ ");
       db_query("CREATE TABLE {book} (
+        nid int unsigned NOT NULL default '0',
         vid int unsigned NOT NULL default '0',
+        book_author VARCHAR(255) NOT NULL default '',
+        book_license LONGTEXT NOT NULL default '',
+        weight smallint NOT NULL default '0',
+        toc_depth smallint unsigned NOT NULL default '1',
+        PRIMARY KEY  (nid, vid),
+        UNIQUE KEY vid (vid)
+      ) /*!40100 DEFAULT CHARACTER SET UTF8 */ ");
+      break;
+    case 'pgsql':
+      db_query("CREATE TABLE {book_pages} (
+        nid int_unsigned NOT NULL default '0',
+        bid int_unsigned NOT NULL default '0',
+        parent int_unsigned NOT NULL default '0',
+        weight smallint NOT NULL default '0',
+        PRIMARY KEY (nid)
+      ");
+      db_query("CREATE INDEX {book_pages}_xid_idx ON {book_pages} (bid)");
+      db_query("CREATE INDEX {book_pages}_parent_idx ON {book_pages} (parent)");
+      db_query("CREATE TABLE {book} (
+        nid int_unsigned NOT NULL default '0',
+        vid int_unsigned NOT NULL default '0',
+        book_author varchar(255) NOT NULL default '',
+        book_license text NOT NULL default '',
+        weight smallint NOT NULL default '0',
+        toc_depth smallint_unsigned NOT NULL default '1',
+        PRIMARY KEY (nid, vid),
+        UNIQUE (vid)
+      )");
+      break;
+  }
+}
+
+/**
+ * Update book table definitions.
+ */
+function book_update_1000() {
+  $ret = array();
+  
+  switch ($GLOBALS['db_type']) {
+    case 'mysql':
+    case 'mysqli':
+    $ret[] = update_sql("CREATE TABLE {book_pages} (
         nid int unsigned NOT NULL default '0',
-        parent int NOT NULL default '0',
+        bid int unsigned NOT NULL default '0',
+        parent int unsigned NOT NULL default '0',
         weight tinyint NOT NULL default '0',
-        PRIMARY KEY (vid),
-        KEY nid (nid),
+        PRIMARY KEY (nid),
+        KEY bid (bid),
         KEY parent (parent)
+        ) /*!40100 DEFAULT CHARACTER SET UTF8 */ ");
+      $ret[] = update_sql("INSERT INTO {book_pages} (nid, parent, weight) SELECT b.nid, b.parent, b.weight FROM {book} b INNER JOIN node n ON b.vid=n.vid");
+      $ret[] = update_sql("UPDATE {node} SET type = 'book_page' WHERE type = 'book'");
+      $ret[] = update_sql("DROP TABLE {book}");
+      $ret[] = update_sql("CREATE TABLE {book} (
+        nid int unsigned NOT NULL default '0',
+        vid int unsigned NOT NULL default '0',
+        book_author VARCHAR(255) NOT NULL default '',
+        book_license LONGTEXT NOT NULL default '',
+        weight smallint NOT NULL default '0',
+        toc_depth smallint unsigned NOT NULL default '1',
+        PRIMARY KEY  (nid, vid),
+        UNIQUE KEY vid (vid),
       ) /*!40100 DEFAULT CHARACTER SET UTF8 */ ");
       break;
     case 'pgsql':
-      db_query("CREATE TABLE {book} (
-        vid int_unsigned NOT NULL default '0',
+      $ret[] = update_sql("CREATE TABLE {book_pages} (
         nid int_unsigned NOT NULL default '0',
-        parent int NOT NULL default '0',
+        bid int_unsigned NOT NULL default '0',
+        parent int_unsigned NOT NULL default '0',
+        weight smallint NOT NULL default '0',
+        PRIMARY KEY (nid)
+      ");
+      $ret[] = update_sql("CREATE INDEX {book_pages}_xid_idx ON {book_pages} (bid)");
+      $ret[] = update_sql("CREATE INDEX {book_pages}_parent_idx ON {book_pages} (parent)");
+      $ret[] = update_sql("INSERT INTO {book_pages} (nid, parent, weight) SELECT b.nid, b.parent, b.weight FROM {book} b INNER JOIN node n ON b.vid=n.vid");
+      $ret[] = update_sql("UPDATE {node} SET type = 'book_page' WHERE type = 'book'");
+      $ret[] = update_sql("DROP TABLE {book}");
+      $ret[] = update_sql("CREATE TABLE {book} (
+        nid int_unsigned NOT NULL default '0',
+        vid int_unsigned NOT NULL default '0',
+        book_author varchar(255) NOT NULL default '',
+        book_license text NOT NULL default '',
         weight smallint NOT NULL default '0',
-        PRIMARY KEY (vid)
+        toc_depth smallint_unsigned NOT NULL default '1',
+        PRIMARY KEY (nid, vid),
+        UNIQUE (vid)
       )");
-      db_query("CREATE INDEX {book}_nid_idx ON {book} (nid)");
-      db_query("CREATE INDEX {book}_parent_idx ON {book} (parent)");
       break;
   }
+  return $ret;
 }
Index: modules/book/book.module
===================================================================
RCS file: /cvs/drupal/drupal/modules/book/book.module,v
retrieving revision 1.386
diff -u -p -r1.386 book.module
--- modules/book/book.module	23 Aug 2006 06:38:49 -0000	1.386
+++ modules/book/book.module	25 Aug 2006 03:58:39 -0000
@@ -11,11 +11,16 @@
  */
 function book_node_info() {
   return array(
-    'book' => array(
+    'book_page' => array(
       'name' => t('book page'),
+      'module' => 'book_page',
+      'description' => t("A book page is usually added to a book, but can also be independent. Books can represent a collaborative writing effort: users can collaborate writing the pages of the book, positioning the pages in the right order, and reviewing or modifying pages"),
+    ),
+    'book' => array(
+      'name' => t('book'),
       'module' => 'book',
-      'description' => t("A book is a collaborative writing effort: users can collaborate writing the pages of the book, positioning the pages in the right order, and reviewing or modifying pages previously written. So when you have some information to share or when you read a page of the book and you didn't like it, or if you think a certain page could have been written better, you can do something about it."),
-    )
+      'description' => t('Add a new book- this has a table of contents (for associated book pages), a list of authors, etc. Book pages (and other content) can then be added to this book'),
+    ),
   );
 }
 
@@ -27,24 +32,36 @@ function book_perm() {
 }
 
 /**
- * Implementation of hook_access().
+ * Implementation of hook_access() for a book cover.
  */
 function book_access($op, $node) {
   global $user;
 
   if ($op == 'create') {
-    // Only registered users can create book pages. Given the nature
-    // of the book module this is considered to be a good/safe idea.
-    return user_access('create book pages');
+    return user_access('create new books');
   }
-
   if ($op == 'update') {
-    // Only registered users can update book pages. Given the nature
-    // of the book module this is considered to be a good/safe idea.
-    // One can only update a book page if there are no suggested updates
-    // of that page waiting for approval. That is, only updates that
-    // don't overwrite the current or pending information are allowed.
+    if ($node->uid == $user->uid && user_access('edit own book pages')) {
+      return TRUE;
+    }
+    else {
+       // do nothing. node-access() will determine further access
+    }
+  }
+}
+
 
+/**
+ * Implementation of hook_access() for a book page.
+ */
+function book_page_access($op, $node) {
+  global $user;
+
+  if ($op == 'create') {
+    return user_access('create book pages');
+  }
+  if ($op == 'update') {
+    // users with 'edit book pages' permission may edit ALL book pages.
     if (user_access('edit book pages') || ($node->uid == $user->uid && user_access('edit own book pages'))) {
       return TRUE;
     }
@@ -61,12 +78,22 @@ function book_link($type, $node = NULL, 
 
   $links = array();
 
+  if ($type == 'node' && $node->type == 'book') {
+    if (book_page_access('create', $node)) {
+      $links['book_add_child'] = array(
+        'title' => t('add child page'),
+        'href' => "node/add/book_page/$node->nid/0"
+      );
+    }
+    return $links;
+  }
+
   if ($type == 'node' && isset($node->parent)) {
     if (!$teaser) {
-      if (book_access('create', $node)) {
+      if (book_page_access('create', $node)) {
         $links['book_add_child'] = array(
           'title' => t('add child page'),
-          'href' => "node/add/book/parent/$node->nid"
+          'href' => "node/add/book_page/$node->bid/$node->nid"
         );
       }
       if (user_access('see printer-friendly version')) {
@@ -108,10 +135,10 @@ function book_menu($may_cache) {
       'weight' => 8);
     $items[] = array(
       'path' => 'book',
-      'title' => t('books'),
-      'callback' => 'book_render',
-      'access' => user_access('access content'),
-      'type' => MENU_SUGGESTED_ITEM);
+      'title' => t('book shelf'),
+      'callback' => 'book_shelf',
+      'type' => MENU_NORMAL_ITEM,
+      'access' => user_access('access content'));
     $items[] = array(
       'path' => 'book/export',
       'callback' => 'book_export',
@@ -123,17 +150,24 @@ function book_menu($may_cache) {
     // We put this in !$may_cache so it's only added once per request
     drupal_add_css(drupal_get_path('module', 'book') .'/book.css');
 
+    $items[] = array(
+      'path' => 'node/add/book_page/select_book',
+      'title' => t('New page'),
+      'callback' => 'drupal_get_form',
+      'callback arguments' => array('_add_page_select_book'),
+      'access' => user_access('create book pages'),
+      'type' => MENU_LOCAL_TASK, 
+      'weight' => 2);
     // To avoid SQL overhead, check whether we are on a node page and whether the
     // user is allowed to outline posts in books.
     if (arg(0) == 'node' && is_numeric(arg(1)) && user_access('outline posts in books')) {
-      // Only add the outline-tab for non-book pages:
-      $result = db_query(db_rewrite_sql("SELECT n.nid FROM {node} n WHERE n.nid = %d AND n.type != 'book'"), arg(1));
-      if (db_num_rows($result) > 0) {
+      $type = db_result(db_query("SELECT n.type FROM {node} n WHERE n.nid=%d",arg(1)));
+      if (($type) && $type != 'book') {
         $items[] = array(
           'path' => 'node/'. arg(1) .'/outline',
           'title' => t('outline'),
           'callback' => 'drupal_get_form',
-          'callback arguments' => array('book_outline', arg(1)),
+          'callback arguments' => array('book_outline_form',arg(1)),
           'access' => user_access('outline posts in books'),
           'type' => MENU_LOCAL_TASK,
           'weight' => 2);
@@ -153,26 +187,38 @@ function book_menu($may_cache) {
 function book_block($op = 'list', $delta = 0) {
   $block = array();
   if ($op == 'list') {
-    $block[0]['info'] = t('Book navigation');
+    $block['navigation']['info'] = t('Book navigation');
+    $block['license']['info'] = t('Book license');
     return $block;
   }
   else if ($op == 'view') {
-    // Only display this block when the user is browsing a book:
+    // Book license block
+    // Only display this block when the user is browsing a node in the book outline (book_pages table):
+    if ($delta == 'license') {
+      if (arg(0) == 'node' && is_numeric(arg(1))) {
+        $result = db_query('SELECT b.book_license, n.title, nr.format FROM {book} b JOIN {book_pages} bp ON bp.bid = b.nid INNER JOIN {node} n ON n.vid = b.vid JOIN {node_revisions} nr ON nr.vid = n.vid WHERE bp.nid = %d', arg(1));
+        if (db_num_rows($result) > 0) {
+          $book = db_fetch_object($result);
+          $block['subject'] = t('%book_title license', array('%book_title' => $book->title));
+          $block['content'] = check_markup($book->book_license, $book->format, FALSE);
+        }
+      }
+      return $block;
+    }
+    // Book navigation block
+    // Only display this block when the user is browsing a node in the book outline (book_pages table):
     if (arg(0) == 'node' && is_numeric(arg(1))) {
-      $result = db_query(db_rewrite_sql('SELECT n.nid, n.title, b.parent FROM {node} n INNER JOIN {book} b ON n.vid = b.vid WHERE n.nid = %d'), arg(1));
-      if (db_num_rows($result) > 0) {
-        $node = db_fetch_object($result);
-
-        $path = book_location($node);
-        $path[] = $node;
-
+      $this_node = node_load(arg(1));
+      if (isset($this_node->bid) && $this_node->type != 'book') {          
+        $path = book_location($this_node);
+        $path[] = $this_node;
+   
         $expand = array();
         foreach ($path as $key => $node) {
           $expand[] = $node->nid;
         }
-
+        $block['content'] = book_page_menu_tree($this_node->bid,$path[0]->nid, 5, $expand);
         $block['subject'] = check_plain($path[0]->title);
-        $block['content'] = book_tree($expand[0], 5, $expand);
       }
     }
 
@@ -181,67 +227,161 @@ function book_block($op = 'list', $delta
 }
 
 /**
- * Implementation of hook_load().
+ * Implementation of hook_load() for a book cover.
  */
 function book_load($node) {
-  return db_fetch_object(db_query('SELECT * FROM {book} WHERE vid = %d', $node->vid));
+  return db_fetch_array(db_query('SELECT * FROM {book} WHERE vid = %d', $node->vid));
 }
 
 /**
- * Implementation of hook_insert().
+ * Implementation of hook_insert() for book nodes (covers).
  */
 function book_insert($node) {
-  db_query("INSERT INTO {book} (nid, vid, parent, weight) VALUES (%d, %d, %d, %d)", $node->nid, $node->vid, $node->parent, $node->weight);
+  db_query('INSERT INTO {book} (nid, vid, book_author, book_license, weight, toc_depth) VALUES (%d, %d, "%s", "%s", %d, %d)', $node->nid, $node->vid, $node->book_author, $node->book_license, $node->weight, $node->toc_depth);
+  $node->bid = $node->nid;
+  $node->parent = 0;
+  book_page_insert($node);
 }
 
 /**
- * Implementation of hook_update().
+ * Implementation of hook_insert() for book pages.
+ */
+function book_page_insert($node) {
+  db_query("INSERT INTO {book_pages} (nid, bid, parent, weight) VALUES (%d, %d, %d, %d)", $node->nid, $node->bid, $node->parent, $node->weight);
+}
+
+/**
+ * Implementation of hook_update() for book nodes (covers).
  */
 function book_update($node) {
   if ($node->revision) {
-    db_query("INSERT INTO {book} (nid, vid, parent, weight) VALUES (%d, %d, %d, %d)", $node->nid, $node->vid, $node->parent, $node->weight);
+    db_query('INSERT INTO {book} (nid, vid, book_author, book_license, weight, toc_depth) VALUES (%d, %d, "%s", "%s", %d, %d)', $node->nid, $node->vid, $node->book_author, $node->book_license, $node->weight, $node->toc_depth);
   }
   else {
-    db_query("UPDATE {book} SET parent = %d, weight = %d WHERE vid = %d", $node->parent, $node->weight, $node->vid);
+    db_query('UPDATE {book} SET book_author = "%s", weight = %d, book_license = "%s", toc_depth = %d WHERE vid = %d', $node->book_author, $node->weight, $node->book_license, $node->toc_depth, $node->vid);
   }
+  $node->bid = $node->nid;
+  $node->parent = 0;
+  book_page_update($node);
+}
+
+/**
+ * Implementation of hook_update().for book pages
+ */
+function book_page_update($node) {
+  db_query("UPDATE {book_pages} SET bid = %d, parent = %d, weight = %d WHERE nid = %d", $node->bid, $node->parent, $node->weight, $node->nid);  
 }
 
 /**
- * Implementation of hook_delete().
+ * Implementation of hook_delete() for book nodes (covers)
  */
 function book_delete(&$node) {
+  db_query('UPDATE {book_pages} SET bid = 0 WHERE bid = %d', $node->nid);
+  db_query('UPDATE {book_pages} SET parent = 0 WHERE parent = %d', $node->nid);
   db_query('DELETE FROM {book} WHERE nid = %d', $node->nid);
 }
 
 /**
- * Implementation of hook_submit().
+ * Implementation of hook_submit() for book pages.
  */
-function book_submit(&$node) {
+function book_page_submit(&$node) {
   global $user;
   // Set default values for non-administrators.
   if (!user_access('administer nodes')) {
-    $node->revision = 1;
-    $book->uid = $user->uid;
-    $book->name = $user->uid ? $user->name : '';
+    $node->revision = TRUE;
+    $node->uid = $user->uid; //TODO: do we want these settings?
   }
 }
 
 /**
- * Implementation of hook_form().
+ * Implementation of hook_form()
  */
 function book_form(&$node) {
+
+  $form['title'] = array('#type' => 'textfield',
+    '#title' => t('Title'),
+    '#required' => TRUE,
+    '#default_value' => $node->title,
+    '#weight' => -5,
+    '#description' => t('Title of the whole book.'),
+  );
+  $form['body_filter']['body'] = array('#type' => 'textarea',
+    '#title' => t('Body'),
+    '#default_value' => $node->body,
+    '#rows' => 20,
+    '#required' => TRUE,
+    '#description' => t('The content of book cover.'),
+  );
+  $form['weight'] = array(
+    '#type' => 'weight',
+    '#title' => t('Book weight'),
+    '#default_value' => (isset($node->weight) ? $node->weight : 0),
+    '#delta' => 15,
+    '#description' => t('Books are ordered first by weight and then by title.'),
+  ); 
+  $form['body_filter']['format'] = filter_form($node->format);
+  $form['book_author'] = array('#type' => 'textfield',
+    '#title' => t('Author'),
+    '#required' => FALSE,
+    '#default_value' => $node->book_author,
+    '#description' => t('The author of all the book pages that will be added to this book. The author can be a single person, or a collective. Leave blank if you do not wish to display the authoring information on the book cover.'),
+  );
+  $form['book_license'] = array('#type' => 'textarea',
+    '#title' => t('License'),
+    '#default_value' => $node->book_license,
+    '#rows' => 5,
+    '#required' => FALSE,
+    '#description' => t('The license of all the book pages within this book. This information can be displayed within a block on each relevant book page. Leave blank if you do not wish to display any license information. The input format is the same as the one selected for the body.'),
+  );
+  $form['toc_depth'] = array(
+    '#type' => 'select',
+    '#title' => t('Preferred depth for table of contents display'),
+    '#default_value' => (($node->toc_depth > 0) ? $node->toc_depth : 5),
+    '#options' => array(
+       1 => '1',
+       2 => '2',
+       3 => '3',
+       4 => '4',
+       5 => '5',
+       6 => '6',
+       7 => '7',
+       8 => '8',
+       9 => '9',
+       10 => '10',
+       15 => '15',
+       20 => '20', ),
+  '#description' => t('The table of contents on the book page will be displayed to this depth (depending upon your theme).'),
+  ); 
+
+  return $form;
+}
+
+
+/**
+ * Implementation of hook_form() for a book page.
+ */
+function book_page_form(&$node) {
+  
+  // looking for url like node/add/book-page/$bid/$parent
+  if ((arg(1)=='add') && is_numeric(arg(3)) && is_numeric(arg(4))) {
+      $node->bid = arg(3);
+      $node->parent = arg(4);
+  }
+  elseif (arg(2) == 'book-page') {
+    drupal_goto('node/add/book_page/select_book');
+  }
+
+  $form = _book_parent_and_log_form($node);
+  
+  $form['bid'] = array (
+    '#type' => 'value',
+    '#value' => $node->bid,
+  );
+
   $type = node_get_types('type', $node);
-  if ($node->nid && !$node->parent && !user_access('create new books')) {
-    $form['parent'] = array('#type' => 'value', '#value' => $node->parent);
-  }
-  else {
-    $form['parent'] = array('#type' => 'select',
-      '#title' => t('Parent'),
-      '#default_value' => ($node->parent ? $node->parent : arg(4)),
-      '#options' => book_toc($node->nid),
-      '#weight' => -4,
-      '#description' => user_access('create new books') ? t('The parent section in which to place this page. Note that each page whose parent is &lt;top-level&gt; is an independent, top-level book.') : t('The parent that this page belongs in.'),
-    );
+
+  if (!isset($node->parent)) {
+    $node->parent = 0;
   }
 
   $form['title'] = array('#type' => 'textfield',
@@ -258,102 +398,254 @@ function book_form(&$node) {
   );
   $form['body_filter']['format'] = filter_form($node->format);
 
+  return $form;
+}
+
+function _book_parent_and_log_form($node) {
+  $form=array();
+
+  if ($node->bid != 0 && $result = db_result(db_query('SELECT n.title FROM {node} n WHERE n.nid = %d', $node->bid))) {
+    $form_parent_title = t('Position in the book %book', array('%book' => $result));
+  }
+  else {
+    $node->bid = 0;
+    $form_parent_title = t('Parent');
+  }
+  
+  $form['parent'] = array(
+    '#type' => 'select',
+    '#title' => $form_parent_title,
+    '#default_value' => $node->parent,
+    '#options' => select_book_page_parent($node->nid, $node->bid),
+    '#weight' => -4,
+    '#description' => t('The parent that this page belongs under. "Top level" means the highest level within this book.'),
+  );
+  
   $form['log'] = array(
     '#type' => 'textarea',
     '#title' => t('Log message'),
+    '#rows' => 2,
     '#weight' => 5,
     '#description' => t('An explanation of the additions or updates being made to help other authors understand your motivations.'),
   );
 
-  if (user_access('administer nodes')) {
-    $form['weight'] = array('#type' => 'weight',
-      '#title' => t('Weight'),
+  if (user_access('outline posts in books')) {
+    $form['weight'] = array(
+      '#type' => 'weight',
+      '#title' => t('Book outline weight'),
       '#default_value' => $node->weight,
       '#delta' => 15,
-      '#weight' => 5,
+      '#weight' => 6,
       '#description' => t('Pages at a given level are ordered first by weight and then by title.'),
-    );
+    );  
   }
   else {
-    // If a regular user updates a book page, we preserve the node weight; otherwise
+   // If a regular user updates a book page, we preserve the node weight; otherwise
     // we use 0 as the default for new pages
     $form['weight'] = array(
-      '#type' => 'value',
-      '#value' => isset($node->weight) ? $node->weight : 0,
-    );
+      '#type' => 'value', 
+      '#value' => (isset($node->weight) ? $node->weight : 0),
+    ); 
   }
-
   return $form;
 }
 
 /**
- * Implementation of function book_outline()
- * Handles all book outline operations.
+ * helper function: generates a select form listing available books
+ * @param:
+ *  $default_bid: the default book in the select form.   0 corresponds to no
+ *  book (an independent top-level book page)
+ * @param
+ *  $nid: the Node ID (nid) of the node being acted upon
+ * @param
+ *  $note: a note to add to the form below the book select
  */
-function book_outline($nid) {
-  $node = node_load($nid);
-  $page = book_load($node);
+function _book_select_form($default_bid = 0, $nid, $note = NULL){
+  $result = db_query(db_rewrite_sql('SELECT b.nid, n.title FROM {book} b JOIN {node} n ON n.vid = b.vid ORDER BY b.weight, n.title'));
+  $books = array();
+  $books[0] = t('Independent, top-level book page');
+  while ($book = db_fetch_object($result)) {
+    $books[$book->nid] = $book->title;
+  }
 
-  $form['parent'] = array('#type' => 'select',
-    '#title' => t('Parent'),
-    '#default_value' => $page->parent,
-    '#options' => book_toc($node->nid),
-    '#description' => t('The parent page in the book.'),
-  );
-  $form['weight'] = array('#type' => 'weight',
-    '#title' => t('Weight'),
-    '#default_value' => $page->weight,
-    '#delta' => 15,
-    '#description' => t('Pages at a given level are ordered first by weight and then by title.'),
+  $form = array();
+  $form['bid'] = array(
+    '#type' => 'select',
+    '#title' => t('Select a book'),
+    '#default_value' => $default_bid,
+    '#options' => $books,
+    '#description' => t('Select the book you wish to add/move your page to.'),
+    '#weight' => 50,
+  ); 
+  $form['nid'] = array(
+    '#type' => 'value',
+    '#value' => $nid,
   );
-  $form['log'] = array('#type' => 'textarea',
-    '#title' => t('Log message'),
-    '#description' => t('An explanation to help other authors understand your motivations to put this post into the book.'),
+  if ($note) {
+    $form['note'] = array(
+      '#value' => '<p>'. $note. '</p>',
+      '#weight' => 52,
+    );
+  }
+  
+  return $form; 
+}
+
+/**
+ * Prints a list of books in which the user can add pages.
+ */
+function _add_page_select_book() {
+
+  $form = _book_select_form(0,0,t("After choosing a book, you will be able to edit your page and position it within the book."));
+  $form['submit'] = array(
+    '#type' => 'submit',
+    '#value' => t('Select book'),
+    '#weight' => 65,
   );
+  return $form;
+}
 
-  $form['nid'] = array('#type' => 'value', '#value' => $nid);
-  if ($page->nid) {
-    $form['update'] = array('#type' => 'submit',
+/**
+ * Menu call back: Select a book to move a page to.
+ */
+function _add_page_select_book_submit($form_id, $form) {
+
+   drupal_goto('node/add/book-page/'. $form['bid']. '/0');
+}
+
+/**
+ * Handles all book outline operations.
+ */
+function book_outline_form($nid){
+  
+  $node = node_load($nid);
+  
+  if ($node->type == 'book') {
+    return array(); 
+  }
+  
+  if (isset($node->bid)) { // node is already part of the book outline
+    
+    $form = _book_parent_and_log_form($node);        
+
+    $form['outline_heading'] = array(
+      '#value' => '<h2>'. t('Update book outline').'</h2>',
+      '#weight' => -10,
+    );
+    
+    $form['update'] = array(
+      '#type' => 'submit',
       '#value' => t('Update book outline'),
+      '#weight' => 10,
     );
-    $form['remove'] = array('#type' => 'submit',
-      '#value' => t('Remove from book outline'),
+
+    if ($node->type != 'book_page') {
+      $form['may_remove'] = array(
+        '#value' => '<hr class="book_outline"/><h2 class="book_outline">'. t('This node may be removed from the book outline; it is not a %book_page.',array('%book_page' => node_get_types('name','book_page'))). '</h2>',
+        '#weight' => 20,
+      );
+      $form['remove'] = array(
+        '#type' => 'submit',
+        '#value' => t('Remove from book outline'),
+        '#weight' => 21,
+      );     
+    }
+    $form['move_page'] = array(
+      '#value' => '<hr class="book_outline"/><h2 class="book_outline">'. t('Move this post to a different book.'). '</h2>',
+      '#weight' => 40,
+    );
+      $children_note = t("Note: All of this pages's children will be moved together to the new book. If you do not wish that, please update the outline of the child pages first.");
+    $form = array_merge($form,_book_select_form($node->bid, $node->nid, $children_note));
+    $form['select'] = array(
+      '#type' => 'submit',
+      '#value' => t('Select book'),
+      '#weight' => 55,
     );
   }
   else {
-    $form['add'] = array('#type' => 'submit', '#value' => t('Add to book outline'));
+    $form = _book_select_form(0, $node->nid);
+    $form['add_note'] = array(
+      '#value' => '<h2 class="book_outline">'. t('Add this post to the selected book.'). '</h2>',
+      '#weight' => 40,
+    );
+    $form['add'] = array(
+      '#type' => 'submit', 
+      '#value' => t('Add to book outline'),
+      '#weight' => 55,
+    );
   }
 
   drupal_set_title(check_plain($node->title));
+  
   return $form;
 }
 
+function _book_update_children_recurse($parent, $bid, $children) {
+  if (isset($children[$parent])) {
+    foreach($children[$parent] as $child) {
+      _book_update_children_recurse($child->nid, $bid, $children);
+      db_query("UPDATE {book_pages} SET bid = %d WHERE nid = %d", $bid, $child->nid);
+    }
+  }
+}
+
+/**
+ * Recursive function to update $bid for each child page
+ * when moving the parent to a new book.
+ * @param $parent is the nid of the parent that is being moved.
+ * @param $bid is the bid (book ID) of the book the parent is moving to.
+ * @param $children is the book hierarchy of the book the parent is moving from,
+ * as generated by function book_page_tree..
+ */
+function book_update_children_bid($parent, $bid, $children) { 
+
+  if (isset($children[$parent])) {
+    foreach($children[$parent] as $child) {
+      book_update_children_bid($child->nid, $bid, $children);
+      db_query("UPDATE {book_pages} SET bid = %d WHERE nid = %d", $bid, $child->nid);
+    }
+  }
+} 
+
 /**
  * Handles book outline form submissions.
  */
-function book_outline_submit($form_id, $form_values) {
+function book_outline_form_submit($form_id, $form_values) {
   $op = $_POST['op'];
   $node = node_load($form_values['nid']);
 
   switch ($op) {
     case t('Add to book outline'):
-      db_query('INSERT INTO {book} (nid, vid, parent, weight) VALUES (%d, %d, %d, %d)', $node->nid, $node->vid, $form_values['parent'], $form_values['weight']);
-      db_query("UPDATE {node_revisions} SET log = '%s' WHERE vid = %d", $form_values['log'], $node->vid);
-      drupal_set_message(t('The post has been added to the book.'));
+      db_query('INSERT INTO {book_pages} (nid, bid, parent, weight) VALUES (%d, %d, %d, %d)', $node->nid, $form_values['bid'], $form_values['parent'], $form_values['weight']);
+      drupal_set_message(t('The post has been added to the book.').'<br/>'.t('You can now update where in the book you want to locate the post.'));
+      drupal_goto('node/'. $node->nid. '/outline');
       break;
     case t('Update book outline'):
-      db_query('UPDATE {book} SET parent = %d, weight = %d WHERE vid = %d', $form_values['parent'], $form_values['weight'], $node->vid);
+      db_query('UPDATE {book_pages} SET parent = %d, weight = %d WHERE nid = %d', $form_values['parent'], $form_values['weight'], $node->nid);
       db_query("UPDATE {node_revisions} SET log = '%s' WHERE vid = %d", $form_values['log'], $node->vid);
       drupal_set_message(t('The book outline has been updated.'));
       break;
     case t('Remove from book outline'):
-      db_query('DELETE FROM {book} WHERE nid = %d', $node->nid);
-      drupal_set_message(t('The post has been removed from the book.'));
+      db_query('DELETE FROM {book_pages} WHERE nid = %d', $node->nid);
+      db_query("UPDATE {node_revisions} SET log = '%s' WHERE vid = %d", $form_values['log'], $node->vid);
+      drupal_set_message(t('The post has been removed from the book. Child pages may have been orphaned.'));
+      break;
+    case t('Select book'):
+      if ($form_values['bid']  != $node->bid) {
+        book_update_children_bid($node->nid, $form_values['bid'], book_page_tree($node->bid));
+        db_query('UPDATE {book_pages} SET bid = %d, parent = %d, weight = 0 WHERE nid = %d', $form_values['bid'], $form_values['bid'], $node->nid);
+        drupal_set_message(t('The post and its children have been relocated to the choosen book.').'<br/>'.t('You can now update where in the book you want to locate the post.'));
+        drupal_goto('node/'. $node->nid. '/outline');
+      }
+      else {
+        drupal_set_message(t('You selected the book that this post already belongs to. No changes were made.')); 
+      }
       break;
   }
   return "node/$node->nid";
 }
 
+
 /**
  * Given a node, this function returns an array of 'book node' objects
  * representing the path in the book tree from the root to the
@@ -366,20 +658,27 @@ function book_outline_submit($form_id, $
  * the node does not exist or is not part of a book hierarchy.
  *
  */
-function book_location($node, $nodes = array()) {
-  $parent = db_fetch_object(db_query(db_rewrite_sql('SELECT n.nid, n.title, b.parent, b.weight FROM {node} n INNER JOIN {book} b ON n.vid = b.vid WHERE n.nid = %d'), $node->parent));
-  if (isset($parent->title)) {
-    $nodes = book_location($parent, $nodes);
-    $nodes[] = $parent;
+function book_location($node) {
+  
+  $nodes = array();
+  
+  $pages = pages_of_book_by_nid($node->bid);
+  if (isset($pages[$node->parent])) {
+    $nodes[0]= $pages[$node->parent];
+    $i = 0;
+    while ($nodes[$i]->parent) {
+      $nodes[] = $pages[$nodes[$i++]->parent];
+    }
   }
-  return $nodes;
+  
+  return array_reverse($nodes);
 }
 
 /**
  * Accumulates the nodes up to the root of the book from the given node in the $nodes array.
  */
 function book_location_down($node, $nodes = array()) {
-  $last_direct_child = db_fetch_object(db_query(db_rewrite_sql('SELECT n.nid, n.title, b.parent, b.weight FROM {node} n INNER JOIN {book} b ON n.vid = b.vid WHERE n.status = 1 AND b.parent = %d ORDER BY b.weight DESC, n.title DESC'), $node->nid));
+  $last_direct_child = db_fetch_object(db_query(db_rewrite_sql('SELECT n.nid, n.title, bp.parent, bp.weight FROM {node} n INNER JOIN {book_pages} bp ON n.nid = bp.nid WHERE n.status = 1 AND bp.parent = %d ORDER BY bp.weight DESC, n.title DESC'), $node->nid));
   if ($last_direct_child) {
     $nodes[] = $last_direct_child;
     $nodes = book_location_down($last_direct_child, $nodes);
@@ -395,9 +694,11 @@ function book_prev($node) {
   if ($node->parent == 0) {
     return NULL;
   }
+  
+  //$children = book_page_tree($node->bid); TODO- use this instead
 
   // Previous on the same level:
-  $direct_above = db_fetch_object(db_query(db_rewrite_sql("SELECT n.nid, n.title, b.weight FROM {node} n INNER JOIN {book} b ON n.vid = b.vid WHERE b.parent = %d AND n.status = 1 AND (b.weight < %d OR (b.weight = %d AND n.title < '%s')) ORDER BY b.weight DESC, n.title DESC"), $node->parent, $node->weight, $node->weight, $node->title));
+  $direct_above = db_fetch_object(db_query(db_rewrite_sql("SELECT n.nid, n.title, bp.weight FROM {node} n INNER JOIN {book_pages} bp ON n.nid = bp.nid WHERE bp.parent = %d AND n.status = 1 AND (bp.weight < %d OR (bp.weight = %d AND n.title < '%s')) ORDER BY bp.weight DESC, n.title DESC"), $node->parent, $node->weight, $node->weight, $node->title));
   if ($direct_above) {
     // Get last leaf of $above.
     $path = book_location_down($direct_above);
@@ -406,7 +707,7 @@ function book_prev($node) {
   }
   else {
     // Direct parent:
-    $prev = db_fetch_object(db_query(db_rewrite_sql('SELECT n.nid, n.title FROM {node} n INNER JOIN {book} b ON n.vid = b.vid WHERE n.nid = %d AND n.status = 1'), $node->parent));
+    $prev = db_fetch_object(db_query(db_rewrite_sql('SELECT n.nid, n.title FROM {node} n INNER JOIN {book_pages} bp ON n.nid = bp.nid WHERE n.nid = %d AND n.status = 1'), $node->parent));
     return $prev;
   }
 }
@@ -416,17 +717,17 @@ function book_prev($node) {
  */
 function book_next($node) {
   // get first direct child
-  $child = db_fetch_object(db_query(db_rewrite_sql('SELECT n.nid, n.title, b.weight FROM {node} n INNER JOIN {book} b ON n.vid = b.vid WHERE b.parent = %d AND n.status = 1 ORDER BY b.weight ASC, n.title ASC'), $node->nid));
-  if ($child) {
-    return $child;
+  $children = book_page_tree($node->bid);  //= db_fetch_object(db_query(db_rewrite_sql('SELECT n.nid, n.title, bp.weight FROM {node} n INNER JOIN {book_pages} bp ON n.nid = bp.nid WHERE bp.parent = %d AND n.status = 1 ORDER BY bp.weight ASC, n.title ASC'), $node->nid));
+  if (isset($children[$node->nid][0])) {
+    return $children[$node->nid][0];
   }
 
   // No direct child: get next for this level or any parent in this book.
-  $path = book_location($node); // Path to top-level node including this one.
+  $path = book_location($node); // Path to top-level node including this one's parent.
   $path[] = $node;
 
   while (($leaf = array_pop($path)) && count($path)) {
-    $next = db_fetch_object(db_query(db_rewrite_sql("SELECT n.nid, n.title, b.weight FROM {node} n INNER JOIN {book} b ON n.vid = b.vid WHERE b.parent = %d AND n.status = 1 AND (b.weight > %d OR (b.weight = %d AND n.title > '%s')) ORDER BY b.weight ASC, n.title ASC"), $leaf->parent, $leaf->weight, $leaf->weight, $leaf->title));
+    $next = db_fetch_object(db_query(db_rewrite_sql("SELECT n.nid, n.title, bp.weight FROM {node} n INNER JOIN {book_pages} bp ON n.nid = bp.nid WHERE bp.parent = %d AND n.status = 1 AND (bp.weight > %d OR (bp.weight = %d AND n.title > '%s')) ORDER BY bp.weight ASC, n.title ASC"), $leaf->parent, $leaf->weight, $leaf->weight, $leaf->title));
     if ($next) {
       return $next;
     }
@@ -434,14 +735,59 @@ function book_next($node) {
 }
 
 /**
- * Returns the content of a given node. If $teaser if TRUE, returns
- * the teaser rather than full content. Displays the most recently
- * approved revision of a node (if any) unless we have to display this
- * page in the context of the moderation queue.
- */
-function book_content($node, $teaser = FALSE) {
-  // Return the page body.
-  return node_prepare($node, $teaser);
+* Theme the name of the Author(s) on the book cover
+*/
+function theme_book_author($author) {
+  return '<h2 class="book_author">'. t('A book by @author', array('@author' =>  $author)) .'</h2>';
+}
+
+/**
+* Theme the book license on the cover. 
+*/
+function theme_book_license($license) {
+  $output = '<h3 class="book_license">'. t('License') .'</h3>
+    <p class="book_license">'. $license .'</p>';
+  return $output;
+}
+
+/**
+* Implementation of hook_view() for a book cover.
+*/
+function book_view(&$node, $teaser = FALSE, $page = FALSE) {
+  $node = node_prepare($node, $teaser);
+  if (!empty($node->book_author)) {
+    $node->content['book_author'] = array(
+      '#value' => theme('book_author', $node->book_author),
+      '#weight' => -1,
+    );
+    $node->teaser = '<div class="book_author">'. $node->book_author .'</div>'. $node->teaser;
+  }
+  if (!empty($node->book_license)) {
+    $node->book_license = check_markup($node->book_license, $node->format, FALSE); 
+    $node->content['book_license'] = array(
+      '#value' => theme('book_license', $node->book_license) ,
+      '#weight' => 100,
+    );
+  }
+  if (!$teaser) {
+  $node->content['book_toc'] = array(
+      '#value' => theme('book_toc', $node),
+      '#weight' => 200,
+    );
+  }
+  return $node;
+}
+
+/**
+ * Theme function returning nested list items.
+ * Creates the Table of contents printed on book cover $bid.
+ * Returns formatted html.
+ */
+function theme_book_toc($node) {
+  $toc = '<hr /><div class="book_toc"><h3>' . t('Table of contents') . '</h3>';
+  $toc .= book_page_menu_tree($node->bid, $node->nid, $node->toc_depth);
+  $toc .= '</div><hr />';
+  return $toc;
 }
 
 /**
@@ -451,38 +797,53 @@ function book_content($node, $teaser = F
  */
 function book_nodeapi(&$node, $op, $teaser, $page) {
   switch ($op) {
+    case 'load': //all types handled here including book_page and book
+      return db_fetch_array(db_query('SELECT * FROM {book_pages} WHERE nid = %d', $node->nid));
+      break;
     case 'view':
       if (!$teaser) {
-        $book = db_fetch_array(db_query('SELECT * FROM {book} WHERE vid = %d', $node->vid));
-        if ($book) {
-          foreach ($book as $key => $value) {
-            $node->$key = $value;
-          }
-
-          $path = book_location($node);
-          // Construct the breadcrumb:
+        if ($node->type == 'book') {
           $node->breadcrumb = array(); // Overwrite the trail with a book trail.
-          foreach ($path as $level) {
-            $node->breadcrumb[] = array('path' => 'node/'. $level->nid, 'title' =>  $level->title);
-          }
-          $node->breadcrumb[] = array('path' => 'node/'. $node->nid);
-
-          $node->content['book_navigation'] = array(
-            '#value' => theme('book_navigation', $node),
-            '#weight' => 100,
-          );
-
-          if ($page) {
-            menu_set_location($node->breadcrumb);
+          $node->breadcrumb[] = array ('path' => 'book', 'title' => t('books'));
+          $node->breadcrumb[] = array ('path' => 'node/'. $node->nid, 'title' => $node->title);
+          menu_set_location($node->breadcrumb);
+        }
+        else { 
+          // check if it is a node added in the outline:
+          if (isset($node->bid)) {
+            $path = book_location($node);
+            // Construct the breadcrumb:
+            $node->breadcrumb = array(); // Overwrite the trail with a book trail.
+            $node->breadcrumb[] = array ('path' => 'book', 'title' => t('books'));
+            if ($node->bid) {
+              $cover = db_fetch_object(db_query('SELECT n.title, n.nid FROM {node} n WHERE n.nid = %d', $node->bid));
+              $node->breadcrumb[] = array ('path' => 'node/'. $cover->nid, 'title' => $cover->title);
+            }
+            
+            foreach ($path as $level) {
+              $node->breadcrumb[] = array('path' => 'node/'. $level->nid, 'title' =>  $level->title);
+            }
+            $node->breadcrumb[] = array('path' => 'node/'. $node->nid);
+            
+            if ($page) {
+              $node->content['book_navigation'] = array(
+                '#value' => theme('book_page_navigation', $node),
+                '#weight' => 100,
+              );
+              menu_set_location($node->breadcrumb);
+            }
           }
         }
       }
+      return $node;
       break;
     case 'delete revision':
-      db_query('DELETE FROM {book} WHERE vid = %d', $node->vid);
+      if ($node->type == 'book'){
+        db_query('DELETE FROM {book} WHERE vid = %d', $node->vid);
+      }
       break;
-    case 'delete':
-      db_query('DELETE FROM {book} WHERE nid = %d', $node->nid);
+    case 'delete': //all types handled here including book_page and book
+      db_query('DELETE FROM {book_pages} WHERE nid = %d', $node->nid);
       break;
   }
 }
@@ -493,20 +854,25 @@ function book_nodeapi(&$node, $op, $teas
  *
  * @ingroup themeable
  */
-function theme_book_navigation($node) {
+function theme_book_page_navigation($node) {
   $output = '';
   $links = '';
 
   if ($node->nid) {
-    $tree = book_tree($node->nid);
+    $tree = book_page_menu_tree($node->bid, $node->nid, 1);
 
     if ($prev = book_prev($node)) {
       drupal_add_link(array('rel' => 'prev', 'href' => url('node/'. $prev->nid)));
       $links .= l(t('‹ ') . $prev->title, 'node/'. $prev->nid, array('class' => 'page-previous', 'title' => t('Go to previous page')));
     }
-    if ($node->parent) {
+    if ($node->parent != 0) {
       drupal_add_link(array('rel' => 'up', 'href' => url('node/'. $node->parent)));
-      $links .= l(t('up'), 'node/'. $node->parent, array('class' => 'page-up', 'title' => t('Go to parent page')));
+      if ($node->parent != $node->bid){
+        $links .= l(t('up'), 'node/'. $node->parent, array('class' => 'page-up', 'title' => t('Go to parent page')));
+      }
+      else {
+        $links .= l(t('up'), 'node/'. $node->bid, array('class' => 'page-up', 'title' => t('Go to book cover')));
+      }
     }
     if ($next = book_next($node)) {
       drupal_add_link(array('rel' => 'next', 'href' => url('node/'. $next->nid)));
@@ -529,14 +895,65 @@ function theme_book_navigation($node) {
 }
 
 /**
- * This is a helper function for book_toc().
+ * Returns an array of titles and nid entries of book pages where the array keys are the nid values
+ */
+function pages_of_book_by_nid($bid) {
+ static $bynid = array();
+ 
+ if (!isset($bynid[$bid])){
+   $pages= pages_of_book($bid);
+   foreach ($pages as $node) {
+     $bynid[$bid][$node->nid] = $node;
+   }
+ }
+ return $bynid[$bid];
+}
+   
+/**
+ * Returns an array of titles and nid entries of book pages in table of contents order
+ */
+function pages_of_book($bid) {
+ static $book = array();
+ 
+ if (!isset($book[$bid])){
+   $book[$bid] = array();
+   $result = db_query(db_rewrite_sql('SELECT n.nid, n.title, bp.parent, bp.weight FROM {node} n INNER JOIN {book_pages} bp ON n.nid = bp.nid WHERE n.status = 1 AND bp.bid = %d ORDER BY bp.weight ASC, n.title ASC'), $bid);
+
+   while ($node = db_fetch_object($result)) {
+     $book[$bid][] = $node;
+   }
+ }
+ return $book[$bid];
+}
+
+/**
+ * Returns an array corresponding to the tree of titles and nid entries of book pages in table of contents order
+ */
+function book_page_tree($bid){
+  static $children = array();
+  
+  if (!isset($children[$bid])) {
+    $pages=pages_of_book($bid);
+    $children[$bid] = array();
+    foreach ($pages as $node) {
+      if (!$children[$bid][$node->parent]) {
+        $children[$bid][$node->parent] = array();
+      }
+      $children[$bid][$node->parent][] = $node;
+    }
+  }
+  return $children[$bid];
+}
+
+/**
+ * This is a helper function for select_book_page_parent
  */
-function book_toc_recurse($nid, $indent, $toc, $children, $exclude) {
+function _book_toc_recurse($nid, $indent, $toc, $children, $exclude) {
   if ($children[$nid]) {
-    foreach ($children[$nid] as $foo => $node) {
+    foreach ($children[$nid] as $node) {
       if (!$exclude || $exclude != $node->nid) {
         $toc[$node->nid] = $indent .' '. $node->title;
-        $toc = book_toc_recurse($node->nid, $indent .'--', $toc, $children, $exclude);
+        $toc = _book_toc_recurse($node->nid, $indent .'--', $toc, $children, $exclude);
       }
     }
   }
@@ -545,58 +962,55 @@ function book_toc_recurse($nid, $indent,
 }
 
 /**
- * Returns an array of titles and nid entries of book pages in table of contents order.
+ * Returns an array of titles and nid entries of book pages in table of contents order
+ * for use in a form select.
  */
-function book_toc($exclude = 0) {
-  $result = db_query(db_rewrite_sql('SELECT n.nid, n.title, b.parent, b.weight FROM {node} n INNER JOIN {book} b ON n.vid = b.vid WHERE n.status = 1 ORDER BY b.weight, n.title'));
-
-  $children = array();
-  while ($node = db_fetch_object($result)) {
-    if (!$children[$node->parent]) {
-      $children[$node->parent] = array();
-    }
-    $children[$node->parent][] = $node;
-  }
-
+function select_book_page_parent($exclude = 0,  $bid) {
+  
+  $children = book_page_tree($bid);
   $toc = array();
-  // If the user has permission to create new books, add the top-level book page to the menu;
-  if (user_access('create new books')) {
-    $toc[0] = '<'. t('top-level') .'>';
-  }
+  $toc[$bid] = '<'. t('top-level') .'>';
 
-  $toc = book_toc_recurse(0, '', $toc, $children, $exclude);
+  $toc = _book_toc_recurse($bid, '', $toc, $children, $exclude);
 
   return $toc;
 }
 
 /**
- * This is a helper function for book_tree()
+ * This is a default theme display function for book_tree()
  */
-function book_tree_recurse($nid, $depth, $children, $unfold = array()) {
+function theme_book_tree_display($nid, $depth, $children, $unfold = NULL) {
   $output = '';
   if ($depth > 0) {
     if (isset($children[$nid])) {
-      foreach ($children[$nid] as $foo => $node) {
-        if (in_array($node->nid, $unfold)) {
-          if ($tree = book_tree_recurse($node->nid, $depth - 1, $children, $unfold)) {
-            $output .= '<li class="expanded">';
-            $output .= l($node->title, 'node/'. $node->nid);
-            $output .= '<ul class="menu">'. $tree .'</ul>';
-            $output .= '</li>';
+      $output .= '<ul class="menu">';
+      foreach ($children[$nid] as $node) {
+        if (isset($children[$node->nid])) {
+          if (empty($unfold)){
+             if (depth == 1) {
+            $output .= '<li class="collapsed">'. l($node->title, 'node/'. $node->nid);
+            }
+            else {
+              $output .= '<li class="expanded">' . l($node->title, 'node/'. $node->nid);
+              $output .= theme_book_tree_display($node->nid, $depth - 1, $children, $unfold);            
+            }
           }
           else {
-            $output .= '<li class="leaf">'. l($node->title, 'node/'. $node->nid) .'</li>';
+            if (depth == 1 || !in_array($node->nid, $unfold)) {
+              $output .= '<li class="collapsed">'. l($node->title, 'node/'. $node->nid);
+            }
+            else {
+              $output .= '<li class="expanded">' . l($node->title, 'node/'. $node->nid);
+              $output .= theme_book_tree_display($node->nid, $depth - 1, $children, $unfold);
+            }
           }
         }
         else {
-          if ($tree = book_tree_recurse($node->nid, 1, $children)) {
-            $output .= '<li class="collapsed">'. l($node->title, 'node/'. $node->nid) .'</li>';
-          }
-          else {
-            $output .= '<li class="leaf">'. l($node->title, 'node/'. $node->nid) .'</li>';
-          }
+          $output .= '<li class="leaf">'. l($node->title, 'node/'. $node->nid);
         }
+        $output .= '</li>';
       }
+      $output .= '</ul>';
     }
   }
 
@@ -604,35 +1018,41 @@ function book_tree_recurse($nid, $depth,
 }
 
 /**
- * Returns an HTML nested list (wrapped in a menu-class div) representing the book nodes
+ * Returns an themed list representing the book nodes
  * as a tree.
  */
-function book_tree($parent = 0, $depth = 3, $unfold = array()) {
-  $result = db_query(db_rewrite_sql('SELECT n.nid, n.title, b.parent, b.weight FROM {node} n INNER JOIN {book} b ON n.vid = b.vid WHERE n.status = 1 ORDER BY b.weight, n.title'));
-
-  while ($node = db_fetch_object($result)) {
-    $list = isset($children[$node->parent]) ? $children[$node->parent] : array();
-    $list[] = $node;
-    $children[$node->parent] = $list;
-  }
-
-  if ($tree = book_tree_recurse($parent, $depth, $children, $unfold)) {
-    return '<ul class="menu">'. $tree .'</ul>';
-  }
+function book_page_menu_tree($bid, $parent = 0, $depth = 1, $unfold = array()) {
+  
+  $children = book_page_tree($bid);
+  return theme('book_tree_display',$parent, $depth, $children, $unfold);
 }
 
 /**
  * Menu callback; prints a listing of all books.
  */
-function book_render() {
-  $result = db_query(db_rewrite_sql('SELECT n.nid, n.title, b.weight FROM {node} n INNER JOIN {book} b ON n.vid = b.vid WHERE b.parent = 0 AND n.status = 1 ORDER BY b.weight, n.title'));
+function book_shelf() {
+  $result = db_query(db_rewrite_sql('SELECT b.nid, n.title FROM {book} b INNER JOIN {node} n ON n.vid = b.vid ORDER BY b.weight ASC, n.title ASC'));
+  $breadcrumb = array (l( t('Home'),''));
+  drupal_set_breadcrumb($breadcrumb);
 
+  $output = '';
   $books = array();
   while ($node = db_fetch_object($result)) {
     $books[] = l($node->title, 'node/'. $node->nid);
   }
+  $output .= theme('item_list', $books, t('books'));
+  
+  // Add to those the top-level book pages without a book_cover.
+  $result = db_query(db_rewrite_sql('SELECT bp.nid, n.title FROM {book_pages} bp INNER JOIN {node} n ON n.nid = bp.nid WHERE bp.parent = 0 AND bp.bid = 0 ORDER BY bp.weight ASC, n.title ASC'));
+  $books = array();
+  while ($node = db_fetch_object($result)) {
+    $books[] = l($node->title, 'node/'. $node->nid);
+  }
+  if (count($books)) {
+    $output .= theme('item_list', $books, t('Independent top level book pages'));
+  }
 
-  return theme('item_list', $books);
+  return $output;
 }
 
 /**
@@ -655,7 +1075,7 @@ function book_render() {
  */
 function book_export($type = 'html', $nid = 0) {
   $type = drupal_strtolower($type);
-  $node_result = db_query(db_rewrite_sql('SELECT n.nid, n.title, b.parent FROM {node} n INNER JOIN {book} b ON n.vid = b.vid WHERE n.nid = %d'), $nid);
+  $node_result = db_query(db_rewrite_sql('SELECT n.nid, n.title, bp.parent, bp.bid FROM {node} n INNER JOIN {book_pages} bp ON n.nid = bp.nid WHERE n.nid = %d'), $nid);
   if (db_num_rows($node_result) > 0) {
       $node = db_fetch_object($node_result);
   }
@@ -745,7 +1165,7 @@ function theme_book_export_html($title, 
  *  - the output generated in visiting each node
  */
 function book_recurse($nid = 0, $depth = 1, $visit_pre, $visit_post) {
-  $result = db_query(db_rewrite_sql('SELECT n.nid, n.title, b.weight FROM {node} n INNER JOIN {book} b ON n.vid = b.vid WHERE n.status = 1 AND n.nid = %d ORDER BY b.weight, n.title'), $nid);
+  $result = db_query(db_rewrite_sql('SELECT n.nid, n.title, bp.weight FROM {node} n INNER JOIN {book_pages} bp ON n.nid = bp.nid WHERE n.status = 1 AND n.nid = %d ORDER BY bp.weight ASC, n.title ASC'), $nid);
   while ($page = db_fetch_object($result)) {
     // Load the node:
     $node = node_load($page->nid);
@@ -758,7 +1178,7 @@ function book_recurse($nid = 0, $depth =
         $output .= book_node_visitor_html_pre($node, $depth, $nid);
       }
 
-      $children = db_query(db_rewrite_sql('SELECT n.nid, n.title, b.weight FROM {node} n INNER JOIN {book} b ON n.vid = b.vid WHERE n.status = 1 AND b.parent = %d ORDER BY b.weight, n.title'), $node->nid);
+      $children = db_query(db_rewrite_sql('SELECT n.nid, n.title, bp.weight FROM {node} n INNER JOIN {book_pages} bp ON n.nid = bp.nid WHERE n.status = 1 AND bp.parent = %d ORDER BY bp.weight ASC, n.title ASC'), $node->nid);
       while ($childpage = db_fetch_object($children)) {
           $childnode = node_load($childpage->nid);
           if ($childnode->nid != $node->nid) {
@@ -795,9 +1215,7 @@ function book_recurse($nid = 0, $depth =
  */
 function book_node_visitor_html_pre($node, $depth, $nid) {
   // Output the content:
-  if (node_hook($node, 'content')) {
-    $node = node_invoke($node, 'content');
-  }
+  $node = node_build_content($node, FALSE, FALSE);
   // Allow modules to change $node->body before viewing.
   node_invoke_nodeapi($node, 'print', $node->body, FALSE);
 
@@ -826,7 +1244,16 @@ function _book_admin_table($nodes = arra
   );
 
   foreach ($nodes as $node) {
-    $form = array_merge($form, _book_admin_table_tree($node, 0));
+    if ($node->type == 'book') {
+      $result = db_query('SELECT nid FROM {book_pages} WHERE parent = %d AND bid = %d', $node->bid, $node->bid);
+      while ($chapters = db_fetch_object($result)) {
+        $chapter = node_load($chapters->nid);
+        $form = array_merge($form, _book_admin_table_tree($chapter, 0));
+      }
+    }
+    else {
+      $form = array_merge($form, _book_admin_table_tree($node, 0));
+    }
   }
 
   return $form;
@@ -850,7 +1277,7 @@ function _book_admin_table_tree($node, $
     ),
   );
 
-  $children = db_query(db_rewrite_sql('SELECT n.nid, b.weight FROM {node} n INNER JOIN {book} b ON n.vid = b.vid WHERE b.parent = %d ORDER BY b.weight, n.title'), $node->nid);
+  $children = db_query(db_rewrite_sql('SELECT n.nid, bp.weight FROM {node} n INNER JOIN {book_pages} bp ON n.nid = bp.nid WHERE bp.parent = %d ORDER BY bp.weight ASC, n.title ASC'), $node->nid);
   while ($child = db_fetch_object($children)) {
     $form = array_merge($form, _book_admin_table_tree(node_load($child->nid), $depth + 1));
   }
@@ -859,7 +1286,7 @@ function _book_admin_table_tree($node, $
 }
 
 function theme_book_admin_table($form) {
-  $header = array(t('Title'), t('Weight'), array('data' => t('Operations'), 'colspan' => '3'));
+  $header = array(t('Title'), t('Weight'), array('data' => t('Operations'), 'colspan' => '4'));
 
   $rows = array();
   foreach (element_children($form) as $key) {
@@ -870,7 +1297,8 @@ function theme_book_admin_table($form) {
       drupal_render($form[$key]['weight']),
       l(t('view'), 'node/'. $nid),
       l(t('edit'), 'node/'. $nid .'/edit'),
-      l(t('delete'), 'node/'. $nid .'/delete', NULL, 'destination=admin/content/book'. (arg(3) == 'orphan' ? '/orphan' : '') . ($pid != $nid ? '/'.$pid : ''))
+      l(t('delete'), 'node/'. $nid .'/delete', NULL, 'destination=admin/content/book'. (arg(3) == 'orphan' ? '/orphan' : '') . ($pid != $nid ? '/'.$pid : '')),
+      l(t('outline'), 'node/'. $nid .'/outline'),
     );
   }
 
@@ -903,7 +1331,7 @@ function book_admin_edit($nid) {
  * Menu callback; displays a listing of all orphaned book pages.
  */
 function book_admin_orphan() {
-  $result = db_query(db_rewrite_sql('SELECT n.nid, n.title, n.status, b.parent FROM {node} n INNER JOIN {book} b ON n.vid = b.vid'));
+  $result = db_query(db_rewrite_sql('SELECT n.nid, n.title, n.status, bp.parent FROM {node} n INNER JOIN {book_pages} bp ON n.nid = bp.nid'));
 
   $pages = array();
   while ($page = db_fetch_object($result)) {
@@ -920,12 +1348,12 @@ function book_admin_orphan() {
   }
 
   if (count($orphans)) {
+ 
     $form['table'] = _book_admin_table($orphans);
     $form['save'] = array(
       '#type' => 'submit',
       '#value' => t('Save book pages'),
     );
-
   }
   else {
     $form['error'] = array('#value' => '<p>'. t('There are no orphan pages.') .'</p>');
@@ -963,7 +1391,7 @@ function book_admin_edit_submit($form_id
  */
 function book_admin($nid = 0) {
   if ($nid) {
-    return book_admin_edit($nid);
+    return drupal_get_form('book_admin_edit',$nid);
   }
   else {
     return book_admin_overview();
@@ -974,13 +1402,33 @@ function book_admin($nid = 0) {
  * Returns an administrative overview of all books.
  */
 function book_admin_overview() {
-  $result = db_query(db_rewrite_sql('SELECT n.nid, n.title, b.weight FROM {node} n INNER JOIN {book} b ON n.vid = b.vid WHERE b.parent = 0 ORDER BY b.weight, n.title'));
+  $output = '';
+
+  // List all book covers
+  $rows = array();
+  $result = db_query(db_rewrite_sql('SELECT b.nid, n.title FROM {book} b JOIN {node} n ON n.vid = b.vid ORDER BY weight, title'));
   while ($book = db_fetch_object($result)) {
     $rows[] = array(l($book->title, "node/$book->nid"), l(t('outline'), "admin/content/book/$book->nid"));
   }
-  $headers = array(t('Book'), t('Operations'));
 
-  return theme('table', $headers, $rows);
+  if (count($rows)) {
+    $headers = array(t('Book'), t('Operations'));
+    $output .= theme('table', $headers, $rows);
+    $output .= '<br />';
+  }
+  
+  // List all independent top-level book pages.
+  $rows = array();
+  $result = db_query(db_rewrite_sql('SELECT n.nid, n.title, bp.weight FROM {node} n INNER JOIN {book_pages} bp ON n.nid = bp.nid WHERE bp.parent = 0 AND bp.bid = 0 ORDER BY bp.weight, n.title'));
+  while ($book = db_fetch_object($result)) {
+    $rows[] = array(l($book->title, "node/$book->nid"), l(t('outline'), "admin/content/book/$book->nid"));
+  }
+  if (count($rows)) {
+    $headers = array(t('Independent book pages'), t('Operations'));
+    $output .= theme('table', $headers, $rows);
+  }
+  
+  return $output;
 }
 
 /**
@@ -989,20 +1437,23 @@ function book_admin_overview() {
 function book_help($section) {
   switch ($section) {
     case 'admin/help#book':
-      $output = '<p>'. t('The <em>book</em> content type is suited for creating structured, multi-page hypertexts such as site resource guides, manuals, and Frequently Asked Questions (FAQs). It permits a document to have chapters, sections, subsections, etc. Authors with suitable permissions can add pages to a collaborative book,  placing them into the existing document by adding them to a table of contents menu. ') .'</p>';
-      $output .= '<p>'. t('Books have additional <em>previous</em>, <em>up</em>, and <em>next</em> navigation elements at the bottom of each page for moving through the text. Additional navigation may be provided by enabling the <em>book navigation block</em> on the <a href="@admin-block">block administration page</a>.', array('@admin-block' => url('admin/build/block'))) .'</p>';
-      $output .= '<p>'. t('Users can select the <em>printer-friendly version</em> link visible at the bottom of a book page to generate a printer-friendly display of the page and all of its subsections. ') .'</p>';
-      $output .= '<p>'. t('Administrators can view a book outline, from which is it possible to change the titles of sections, and their <i>weight</i> (thus reordering sections). From this outline, it is also possible to edit and/or delete book pages. Many content types besides pages (for example, blog entries, stories, and polls) can be added to a collaborative book by choosing the <em>outline</em> tab when viewing the post.') .'</p>';
+      $output = '<p>'. t("The <em>book</em> module is suited for creating structured, multi-page hypertexts such as site resource guides, manuals, and Frequently Asked Questions (FAQs). It permits a document to have chapters, sections, subsections, etc. This module provides two content types: the 'book and 'book_page' types.  A 'book' node can be thought of as the book's cover and table of contents, while book pages are the organized content associated with a book.");
+      $output .= '<p>'.  t("Users with the 'outline books' permission can also add nodes of any type to a book, placing them into the existing document by adding them to the book outline via the %outline tab. This tab also provides a means for reorganizing the book one page at a time, including associating it with a different book or changing its positon inthe book relative to other pages.", array('%outline' => t('outline'))) .'</p>';
+      $output .= '<p>'. t('Book pages have additional <em>previous</em>, %up, and <em>next</em> navigation elements at the bottom of each page for moving through the text. Additional navigation may be provided by enabling the %navigation block on the <a href="!admin-block">block administration page</a>.', array('%up' => t('up'), '%navigation' => t('Book navigation'),'!admin-block' => url('admin/build/block'))) .'</p>';
+      $output .= '<p>'. t('Users can click the %child-page link visible at the bottom of a book page to add a new page to the current book and automatically position it as a sub-page of the page they are viewing',array('%child-page' => t('add child page'))). '</p>';
+      $output .= '<p>'. t('Select the %printer-friendly link visible at the bottom of a book page to generate a printer-friendly display of the page and all of its subsections. ',array('%printer-friendly' => t('printer-friendly version'))) .'</p>';
+      $output .= '<p>'. t('Administrators can view the book outline, from which is it possible to change the titles of pages, and their <i>weight</i> (thus reordering sections). From this outline, it is also possible to edit and/or delete book pages. Many content types besides pages (for example, blog entries, stories, and polls) can be added to a collaborative book by choosing the <em>outline</em> tab when viewing the post.') .'</p>';
       $output .= t('<p>You can</p>
 <ul>
-<li>create new book pages: <a href="@node-add-book">create content &gt;&gt; book page</a>.</li>
-<li>administer individual books (choose a book from list): <a href="@admin-node-book">administer &gt;&gt; content management &gt;&gt; books</a>.</li>
-<li>set workflow and other global book settings on the book configuration page: <a href="@admin-settings-content-types-book-page" title="book page content type">administer &gt;&gt; content management &gt;&gt; content types &gt;&gt; book page</a>.</li>
-<li>enable the book navigation block: <a href="@admin-block">administer &gt;&gt; site building &gt;&gt; blocks</a>.</li>
-<li>control who can create, edit, and outline posts in books by setting access permissions: <a href="@admin-access">administer &gt;&gt; user management &gt;&gt; access control</a>.</li>
+<li>create new books: <a href="!node-add-book">create content &gt;&gt; book</a>.</li>
+<li>create new book pages that will be added to a previously created book: <a href="!node-add-book-page">create content &gt;&gt; book page</a>.</li>
+<li>administer individual books (choose a book from list): <a href="!admin-content-book">administer &gt;&gt; content management &gt;&gt; books</a>.</li>
+<li>set workflow and other global book settings through the content type configuration pages: <a href="!admin-content-types-book" title="book content type">administer &gt;&gt; content management &gt;&gt; content types &gt;&gt; book</a> and <a href="!admin-content-types-book-page" title="book page content type">administer &gt;&gt; content management &gt;&gt; content types &gt;&gt; book page</a>.</li>
+<li>enable the %navigation or %license blocks: <a href="!admin-block">administer &gt;&gt; site building &gt;&gt; blocks</a>.</li>
+<li>control who can create, edit, and outline posts in books by setting access permissions: <a href="!admin-access">administer &gt;&gt; user management &gt;&gt; access control</a>.</li>
 </ul>
-', array('@node-add-book' => url('node/add/book'), '@admin-node-book' => url('admin/content/book'), '@admin-settings-content-types-book-page' => url('admin/content/types/book'), '@admin-block' => url('admin/build/block'), '@admin-access' => url('admin/user/access')));
-      $output .= '<p>'. t('For more information please read the configuration and customization handbook <a href="@book">Book page</a>.', array('@book' => 'http://drupal.org/handbook/modules/book/')) .'</p>';
+', array('!node-add-book' => url('node/add/book'), '!node-add-book-page' => url('node/add/book-page'), '!admin-content-book' => url('admin/content/book'), '!admin-content-types-book' => url('admin/content/types/book'), '!admin-content-types-book-page' => url('admin/content/types/book-page'), '%navigation' => t('Book navigation'), '%license' => t('Book license'), '!admin-block' => url('admin/build/block'), '!admin-access' => url('admin/user/access')));
+      $output .= '<p>'. t('For more information please read the <a href="!book">book module handbook section</a> on Drupal.org.', array('!book' => 'http://drupal.org/handbook/modules/book/')) .'</p>';
       return $output;
     case 'admin/settings/modules#description':
       return t('Allows users to collaboratively author a book.');
@@ -1013,8 +1464,21 @@ function book_help($section) {
   }
 
   if (arg(0) == 'node' && is_numeric(arg(1)) && arg(2) == 'outline') {
-    return t('The outline feature allows you to include posts in the <a href="@book">book hierarchy</a>.', array('@book' => url('book')));
+    return t('The outline feature allows you to organize posts in the <a href="!book">book hierarchy</a>.', array('!book' => url('book')));
   }
 }
 
-
+/**
+ * Implementation of hook_form_alter()
+ */
+function book_form_alter($form_id, &$form) {
+  switch($form_id) {
+    case 'node_delete_confirm':
+      $node_type = db_result(db_query('SELECT type FROM {node} WHERE nid = %d ', $form['nid']['#value']));
+      if ($node_type == 'book') {
+        $form['book'] = array(
+          '#value' => t('Note that all book pages belonging to this book (if any) will no longer be associated to any book. You may want to move remaining book pages within this book to another book first. '),
+        );
+      }
+  }
+}
