admin/node/book produces a blank page

The function book_admin_view() does not return a non-empty string if $nid is zero, which is the default value if /admin/node/book is visited.

The attached patch returns 'No book selected.'. There's probably a better way to do this.

Note that you have to wrap strings in t().

How about removing the books from the navigation menu, and returning an itemized list of books?

I also wrote a patch for this for issue http://drupal.org/node/21668

http://drupal.org/files/issues/show%20error%20on%20book%20id%20not%20fou...

This patch returns an error whenever the book ID cannot be found, then redirects the user back to admin with the message.

Robin

This patch corrects the function of admin/node/book.

Robin

Robin,

1. You don't have to pass book_admin_render() a $nid because it is known to be empty. A typo, I guess.

2. In book_admin_render(), you're not allowed to visualize a node like that. For example, you have to prepare the $node->teaser before it can be displayed, and you should allow other modules to modify/prepare it through the node API.

I'll fix it up for you, and commit it to HEAD.

Also,

3. You should not overwrite the page titles using drupal_set_title. Let the menu system do its job.

4. Don't capitialize the words in a title or header. ;-)

5. Is there a particular reason we prune unmoderated and non-published books from the administrator interface? I think I'll changed it so these are shown too. After all, this is the administration interface.

6. The book can be removed from the menu. I'll do that too.

Committed to HEAD. Thanks.