Issue status update for http://drupal.org/node/8253 Post a follow up: http://drupal.org/project/comments/add/8253 Project: Drupal Version: cvs Component: book.module Category: feature requests Priority: normal Assigned to: moshe weitzman Reported by: moshe weitzman Updated by: killes@www.drop.org -Status: active +Status: patch (code needs work) maybe somebody wants to look at it again. killes@www.drop.org Previous comments: ------------------------------------------------------------------------ Wed, 02 Jun 2004 23:39:34 +0000 : moshe weitzman Attachment: http://drupal.org/files/issues/drbook.patch (1.39 KB) in order to create a new book, the user currently must have 'administer nodes' permission. this is a bit too broad. this patch introduces a 'create books' permission and improves the help text a bit. ------------------------------------------------------------------------ Tue, 10 Aug 2004 01:54:25 +0000 : moshe weitzman this patch is old now, with new menu system. functionality still needed. ------------------------------------------------------------------------ Fri, 01 Oct 2004 15:10:15 +0000 : moshe weitzman Updated patch. It is far less risky to grant 'create new book' as opposed to 'administer nodes'. ------------------------------------------------------------------------ Sun, 03 Oct 2004 16:11:29 +0000 : nedjo +1, this makes sense as a distinct permission. Should the permission test however be if (user_access('create books') || user_access('administer nodes')) {, instead of if (user_access('administer nodes')) {, on the assumption that 'administer nodes' includes creating books? ------------------------------------------------------------------------ Sun, 13 Mar 2005 21:04:11 +0000 : killes@www.drop.org Makes sense to me, but the patch is outdated.