I have a problem with the book module.. i know what has caused it by i want it to go away.

I wanted to create a number of content types similar to a book (in my case for an academic website, these were Thesis, Reports, Papers). So i copied the book module and edited its text by replacing all the words "book" to "Thesis" or "Report" or "Paper". This worked well but as a result i have now got the links for "add child page" and "Printer-friendly version" appearing 4 times below my page. This would have been a minor problem of aesthetics if only all of these links would point to the content type of the node... but instead the first set of links point to the content type book, the second to "paper" and so on...

i would appreciate it if someone can tell me how to fix this... i am using drupal 4.7.x

Comments

dwees’s picture

dwees commented
Priority: Critical » Normal
Status: Active » Closed (fixed)

Although this is not a problem related to this module, I think the solution is reasonably straight forward.

You need to adjust the following function:


/**
 * Implementation of hook_link().
 */
function book_link($type, $node = NULL, $teaser = FALSE) {

  $links = array();

  if ($type == 'node' && isset($node->parent)) {
    if (!$teaser) {
      if (book_access('create', $node) && $node->status == 1) {
        $links['book_add_child'] = array(
          'title' => t('Add child page'),
          'href' => "node/add/book/parent/$node->nid"
        );
      }
      if (user_access('see printer-friendly version')) {
        $links['book_printer'] = array(
          'title' => t('Printer-friendly version'),
          'href' => 'book/export/html/'. $node->nid,
          'attributes' => array('title' => t('Show a printer-friendly version of this book page and its sub-pages.'))
        );
      }
    }
  }

  return $links;
}

which I think you want to modify to:


/**
 * Implementation of hook_link().
 */
function book_link($type, $node = NULL, $teaser = FALSE) {

  $links = array();

  if ($type == 'node' && isset($node->parent)  && $node->type == 'book') {
    if (!$teaser) {
      if (book_access('create', $node) && $node->status == 1) {
        $links['book_add_child'] = array(
          'title' => t('Add child page'),
          'href' => "node/add/book/parent/$node->nid"
        );
      }
      if (user_access('see printer-friendly version')) {
        $links['book_printer'] = array(
          'title' => t('Printer-friendly version'),
          'href' => 'book/export/html/'. $node->nid,
          'attributes' => array('title' => t('Show a printer-friendly version of this book page and its sub-pages.'))
        );
      }
    }
  }

  return $links;
}

Note that you'd have to make this switch for all 4 of your modules, and substitute book for the appropriate term you used in the other 3 modules.

Does this make sense?

Dave