Download & Extend

Use Batch Option to Rebuild Node Access Table

Project:Taxonomy Access Control Lite
Version:6.x-1.4
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:active

Issue Summary

Maybe this is fixed in a dev version, not sure. Anyway, I noticed that when content access permissions are rebuilt, it is called directly versus in a batch. This is causing some issues on a site with lots of nodes. There's a quick fix for it though, since node_access_rebuild() has the option to invoke via batch already. Tested it briefly, seemed to work fine. I'd have submitted as a patch, but I'm short on time today and it's a tiny change. Sorry. :)

--- tac_lite.module (revision 634)
+++ tac_lite.module (working copy)
@@ -139,7 +139,7 @@
   // First, save settings the default way.
   system_settings_form_submit($form, $form_state);
   // Next, rebuild the node_access table.
-  node_access_rebuild();
+  node_access_rebuild(TRUE);
   //drupal_set_message(t('The content access permissions have been rebuilt.'));
   // And rebuild menus, in case the number of schemes has changed.
   menu_rebuild();

Comments

#1

I think this is already in dev branch.

Thanks for posting, though.