[drupal-devel] [feature] new permission: create new books

killes drupal-devel at drupal.org
Wed Aug 10 23:46:57 UTC 2005


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 at www.drop.org
-Status:       active
+Status:       patch (code needs work)

maybe somebody wants to look at it again.




killes at 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 at www.drop.org

Makes sense to me, but the patch is outdated.







More information about the drupal-devel mailing list