Block caching can considerably reduce server load, so I'm opening this issue to investigate whether there is any alternate way for this module to operate. If not, a warning on the project page and documentation might be a good idea.

Comments

girishmuraly’s picture

girishmuraly CreditAttribution: girishmuraly commented

+1 for a warning on the project page. Unfamiliar site builders could get stung by this indirect effect.

entendu’s picture

entendu CreditAttribution: entendu commented
Issue summary: View changes
Status: Active » Closed (fixed)

Adding the following text to the "Common Issues" section in the readme:

* Block caching: In D7, block caching is disabled for modules that implement
hook_node_grants, such as view_unpublished. There are a couple ways around this
issue, if block caching is valuable to you:

-The core patch in Issue #1930960: Block caching disable hardcoded on sites with
hook_node_grant() causes serious performance troubles when not necessary
Patch: https://drupal.org/comment/8647155#comment-8647155

-Block Cache Alter module: https://drupal.org/project/blockcache_alter

  • Commit baef1be on 7.x-1.x by entendu: 
    Issue #1957278: Added disclaimer about hoom_node_grants() to the readme.
hgoto’s picture

hgoto CreditAttribution: hgoto as a volunteer commented

@entendu, thank you. Your explanation helped me.