Issue status update for http://drupal.org/node/21441 Project: Drupal Version: cvs Component: book.module Category: bug reports Priority: normal -Assigned to: Anonymous +Assigned to: RobinMonks Reported by: moshe weitzman Updated by: RobinMonks Status: patch Attachment: http://drupal.org/files/issues/show error on book id not found_0.patch (2.04 KB) This patch corrects the function of admin/node/book. Robin RobinMonks Previous comments: ------------------------------------------------------------------------ April 27, 2005 - 19:20 : moshe weitzman probably a result of the recent patch where we return content instead of print. ------------------------------------------------------------------------ April 29, 2005 - 00:41 : puregin Attachment: http://drupal.org/files/issues/book.module_blank_admin_page.patch (488 bytes) 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. ------------------------------------------------------------------------ May 1, 2005 - 16:03 : Dries 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? ------------------------------------------------------------------------ May 2, 2005 - 13:30 : RobinMonks 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%20found.p... This patch returns an error whenever the book ID cannot be found, then redirects the user back to admin with the message. Robin