I just had a quick read of the code. webchick's is hyping this module at http://groups.drupal.org/node/16134#comment-55161.

  • if ($node->type == 'book') {. huh? we only protect nodes of type=book? i must be missing something
  • cruft: "* Checks to see if maybe we're being disabled."
  • _book_access_restrict_options() - the second loop can be replaced by array_diff() I think.

Comments

harry slaughter’s picture

harry slaughter
Location Omaha NE USA
commented
Assigned: moshe weitzman » harry slaughter

-> if ($node->type == 'book')

this is a relic of the D5 version of the module.

the D6 port was never properly completed or tested.

i'm going over the module right now to remedy that and will integrate this ticket.

tks moshe

harry slaughter’s picture

harry slaughter
Location Omaha NE USA
commented
Status: Needs review » Fixed

addressed in rc2

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.