Hi,

As you know I applied node_cache.patch and it is working fine for caching nodes... But, for example when using poll nodes, it caches those nodes, and then it returns cached version of poll node to any authenticated user.... For example, a user that did not vote on a poll would see results of a poll , as if he did vote on a poll (but he did not)... or user would see those radio buttons , options to vote... he makes his vote.... and it brings him back to the same choice, but drupal returns him a message that his vote was recorded.

This might not be a direct issue of your module, but do you think there could be some sort of a workaround for this? Just like the "cachexclude" module? it would be cool to somehow choose which types of nodes your module should not cache.

k.

Comments

kalashari’s picture

kalashari commented

Of course, when I empty cache with devel, this works fine (until next caching of a poll node).

robertdouglass’s picture

robertdouglass commented

I'm not familiar with cachexclude, but I imagine it would be easy for me to add a simple interface that lets you exclude a content type from being cached. I'll work on this at the next opportunity. Thanks for reporting.

robertdouglass’s picture

robertdouglass commented
Status: Active » Fixed

Fixed by allowing a variable, 'advcache_node_exclude_types', which contains an array of node types that cache should exclude. Default is array('poll'), so core Drupal should now be covered. If you need to add other types, there is no UI, but you can add it to $conf in settings.php:

$conf = array(
  'advcache_node_exclude_types' => array('poll', 'mytype1', 'mytype2' ....),
);
Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)
kalashari’s picture

kalashari commented

This still doesn't work for me. Poll nodes still behave as they are supposed to be cached. I am using 5.x-1.x-dev version of the module. I also tried setting $conf var in settings.php but that didn't make it work either.

dharamgollapudi’s picture

dharamgollapudi commented

Subscribing......

Crimson’s picture

Crimson commented
Title: Conflict: node_cache.patch and poll.module » Documentation: Exclude node types from advance cache
Version: 5.x-1.1 » 5.x-1.x-dev
Component: Miscellaneous » Documentation
Status: Closed (fixed) » Active

This option works great for dual node types like polls (and subsequently, advpoll). But is it possible to document this into the README.txt or some place where this is more visible?

firebus’s picture

firebus commented
Assigned: Unassigned » firebus
Status: Active » Fixed

i just committed a brief description of the variable to the README.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)

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