Closed (outdated)
Project:
Drupal core
Version:
9.4.x-dev
Component:
forum.module
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
20 Dec 2011 at 14:24 UTC
Updated:
10 Aug 2022 at 15:26 UTC
Jump to comment: Most recent
Comments
Comment #0.0
droplet commentedUpdated issue summary.
Comment #0.1
droplet commentedUpdated issue summary.
Comment #1
larowlanThe caching is for when a node access module rewrites the query
Comment #2
droplet commented@larowlan,
can you explain it in more details. Thanks.
EDIT:
patch above is totally remove the caches, anything I missed.
Comment #3
droplet commentedComment #4
larowlanHey droplet, you are correct - core forum does not need the caching with that index but the query has a node access tag and hence node access modules (eg forum access, acl and many others) may rewrite the query, turning our inexpensive core query into a more expensive one. In these circumstances query caching is needed - that's why the block is cached by query. I guess we could add a wrapper to the caching and a config option but that seems like overkill. Marking as postponed while we explore the merits of such an approach.
Lee
Comment #4.0
larowlanUpdated issue summary.
Comment #5
droplet commentedComment #17
smustgrave commentedMoving this to closed outdated as this hasn't had a follow up since moving into PNMI 11 years ago.
If you feel it's still an issue please reopen with an updated issue summary and steps for where you are seeing it