Can't seem to make it work with Group filter

node/add/book?gids[]=54 is renamed in node/ognodeadd?type=book&gids[]=54, which doesn't match the expression '/node\/add\/book\?gids\[\]\=(.*)/' .
I may be wrong, but maybe if you used

$group_node = og_get_group_context();
$gid = $group_node->nid;

to get the gid, instead of the URL, the module would work better.

Indeed, I tried replacing
preg_match('/node\/add\/book\?gids\[\]\=(.*)/', $request, $match);
preg_match('/node\/ognodeadd\?type=book&gids\[\]\=(.*)/', $request, $match);
And it worked better.
Would it be possible to update the module to avoid parsing URL to retrieve GID, and use instead og_get_group_context() ?

Thanks,
David

I also tried this:

  if (($uid != 1)&&(module_exists('og'))) {
//  if ($match[1]) {
//    $output = og_book_get_options($match[1]);
// instead of parsing URL, we call og_get_group_context()
    $group_node = og_get_group_context();
    if (isset( $group_node->nid )) {
      $output = og_book_get_options($group_node->nid);
//(...)

And it works great, but I'm not sure it is the only thing to change to make it work perfectly...

Please let me know... ;)

David

Another problem I just raised: clicking on "add child page" link when viewing a group book page looses context, and allows creating a book page anywhere in the books hierarchy.