Beans default to DRUPAL_CACHE_PER_ROLE caching. It would be handy to specify this per bean type, or per block. There may be another module out there that lets you do this on a per block basis, but I'm not aware of one and don't seem to be able to find it if it does exist. If not, this functionality may be better off in a separate module.

CommentFileSizeAuthor
#6 bean_type_cache-1216580-6.patch3.62 KBalexweber
#2 bean_type_cache-1216580-2.patch3.66 KBindytechcook
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mrfelton’s picture

mrfelton CreditAttribution: mrfelton commented

Just found this http://drupal.org/project/blockcache_alter. However it may still be good to allow people to specify the default type of caching for custom block types.

indytechcook’s picture

indytechcook CreditAttribution: indytechcook commented
Status: Active » Needs review
FileSize
bean_type_cache-1216580-2.patch3.66 KB

Checkout this patch. It's pretty much untested

indytechcook’s picture

indytechcook CreditAttribution: indytechcook commented
Status: Needs review » Closed (duplicate)

Closing in favor of #1164056: Add module to override core block add

indytechcook’s picture

indytechcook CreditAttribution: indytechcook commented
Status: Closed (duplicate) » Needs review

I'm reopening this because it's a small step in the core block add replacement.

alexweber’s picture

alexweber CreditAttribution: alexweber commented
Status: Needs review » Needs work

Patch doesn't apply anymore against 7.x-1.x

alexweber’s picture

alexweber CreditAttribution: alexweber commented
Status: Needs work » Needs review
FileSize
bean_type_cache-1216580-6.patch3.62 KB

Patch re-rolled to apply cleanly on 7.x-1.x, not sure whether it works at all.

barraponto’s picture

barraponto CreditAttribution: barraponto commented

Can't we provide an optional dependency on http://drupal.org/project/blockcache_alter ? Maybe we can integrate the interface, but I don't think we should reinvent the wheel (unless core blocks are the wheel).

indytechcook’s picture

indytechcook CreditAttribution: indytechcook commented

@barraponto,

I do like the idea of not repeating ourselves. I took a look at blockcache_alter and it just does form_alters on the block configure form. There really isn't an API to hook into.

@alexweber, THanks!

alexweber’s picture

alexweber CreditAttribution: alexweber commented
Status: Needs review » Needs work

@indytechcook np! :) The patch actually applies but doesn't work as advertised, it seems the original patch was a work in progress...