Just in case OG, OG Access and OG Access Roles outlive the maintainership of Book Manager we should allow them to supply their own hook_book_manager_batch_operate() functions without having a namespace collision.

Maybe don't need to check for module_exists() in the attached patch?

CommentFileSizeAuthor
book-manager-batch-function-exists.patch1.5 KBmarcp

Comments

jgraham’s picture

jgraham commented
Status: Active » Fixed

looks good commited to DRUPAL-6--2 and HEAD.

I left in the module_exists as it doesn't make sense to define those functions if the corresponding module is not enabled. That is, if we don't have to let's not invoke the batch API.

Status: Fixed » Closed (fixed)

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