Entity cache is basically non-optionally for loading entities with fields, like beans, so I think that bean module should have some lovely integration.

Comments

steven jones’s picture

Assigned: steven jones » Unassigned
Status: Active » Needs review
StatusFileSize
new1.61 KB

Here's a patch for consideration.

steven jones’s picture

StatusFileSize
new1.73 KB

Quick follow up to that patch, we should inform hook_flush_caches about our cache bin.

damienmckenna’s picture

Status: Needs review » Needs work

EntityCache shouldn't be a dependency, it should just be encouraged.

steven jones’s picture

Status: Needs work » Needs review

To be clear here, this doesn't add a dependency on Entity Cache to Bean, but rather adds a new module to the bean package that depends on both bean and Entity Cache. So you can enable Bean without having Entity Cache, and also, you can enable both Bean and Entity Cache and nothing will change, but you could then also enable this module provided in this patch, which integrates the two.

damienmckenna’s picture

Ok, sorry, I misread the patch file. I'll test it out soon, thank you for the patch.

saltednut’s picture

Status: Needs review » Fixed
StatusFileSize
new153.24 KB

Working well on an existing site with 50+ modules AND a clean D7 7.21 with stable Entity API (1.0) and ctools (1.2). Feeling ok moving this on...

Screen Shot 2013-03-25 at 11.20.43 PM.png

http://drupalcode.org/project/bean.git/commit/1d5da63

Status: Fixed » Closed (fixed)

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