Closed (fixed)
Project:
Book access
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
17 Feb 2009 at 22:09 UTC
Updated:
22 Jan 2010 at 06:10 UTC
Jump to comment: Most recent file
Comments
Comment #1
Neodym commentedHi, it's me again.
Discard the patch above, this one is the real version. Tested against latest Drupal 6.9 release.
So long,
Neodym
PS: Why is it not possible to edit one's own filed issue?
Comment #2
pfaocleThanks for this, I've modified this patch to "fix" the "Add child page" link display for D5 with a slightly simplified version.
Comment #3
avpadernoThe patch should be created for the development snapshot.
The patch as it is doesn't follow the coding standards; then, the code that changes the access callback of the menu should also change the access arguments.
Comment #4
avpadernoThe development snapshot already has the code to remove the link ; therefore that part of the patch is not necessary.
Comment #5
avpadernoI started to change the code added by the patch, and applying it in chunks.
For now, I moved the code that was in
hook_nodeapi()intohook_menu_link_alter(); I also updated the code that verifies if the user has the permission to update the book as granted by the realm book_access_rule.Comment #6
avpadernoI added the rest of the code, after I changed it to follow more the coding standards, and after I fixed the code (removing a book page from a book doesn't mean to delete it).
Thanks for the report, and the help with the code.