This was removed as a requirement in Drupal 6. It's a significant issue for me as it means that enabling the module never completes due to the number of nodes on the site.

Instead, it should call http://api.drupal.org/api/drupal/modules--node--node.module/function/nod...

CommentFileSizeAuthor
#1 1189648.1-node_access_needs_rebuild_7.x.patch1.22 KBdeviantintegral
#1 1189648.1-node_access_needs_rebuild_6.x.patch1.2 KBdeviantintegral

Comments

deviantintegral’s picture

deviantintegral commented
Title: hook_enable / hook_disable still call node_access_rebuild() » hook_enable / hook_disable still call node_access_rebuild() in user_relationship_node_access
Status: Active » Needs review
StatusFileSize
new 1189648.1-node_access_needs_rebuild_6.x.patch1.2 KB
new 1189648.1-node_access_needs_rebuild_7.x.patch1.22 KB

Tada!

Status: Needs review » Needs work

The last submitted patch, 1189648.1-node_access_needs_rebuild_6.x.patch, failed testing.

deviantintegral’s picture

deviantintegral commented
Status: Needs work » Needs review

Testbot is applying the 6.x patch to the wrong branch.

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Version: 7.x-1.x-dev » 6.x-1.x-dev

Commited.

The testbot applies patches to whatever version is specified in the version selection of the issues.

So the patch should apply now when triggering a re-test after changing the version.

berdir’s picture

berdir
Primary language German
Location Switzerland
commented

#1: 1189648.1-node_access_needs_rebuild_6.x.patch queued for re-testing.

deviantintegral’s picture

deviantintegral commented
Status: Needs review » Reviewed & tested by the community

Strange, I don't see a commit against 7.x. This passed this 6.x tests, so it should be good to commit there as well.

berdir’s picture

berdir
Primary language German
Location Switzerland
commented

Forgot to push, commit is now on d.o from 7.x

mrf’s picture

mrf commented
Issue tags: +6.x-1.1

Tagging issues for inclusion in 6.x-1.1

mrf’s picture

mrf commented
Status: Reviewed & tested by the community » Fixed

Committed to 6.x

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