A new beta of Commons distro is out and there's a PDOException issue which happens when you click "Attend" an event then "Cancel":

#1867318: PDOException Canceling event registration with Entity Cache

I tracked it down to this module which doesn't seem to support the latest Entity cache configuration settings. I guessed this as there's a previous issue which provides support but no longer works:

#1663596-1: Registration: Wrong variable used in hook_entity_info()'s entitycache support

I've attached a patch which removes the error but I'm not sure if you're wanting to cache all the things I set, or if this is indeed the "correct" way to do things, but thought it'd be a start.

CommentFileSizeAuthor
#4 entity_cache_support-1867564-4.patch2.8 KBMatt V.
registration_entity_cache_support_update.patch2.19 KBAnonymous (not verified)
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Anonymous’s picture

Status: Active » Needs review

Forgot to set status appriately

jpontani’s picture

Status: Needs review » Reviewed & tested by the community

Patch from OP works great for me.

levelos’s picture

Status: Reviewed & tested by the community » Needs work

Thanks @stevepurkiss. The patch does not apply because the diff paths are wrong (there's an extra /registration in there). More importantly, we're missing a hook_update_N() implementation to add the schema tables. Ref. http://drupal.org/node/150220.

Matt V.’s picture

Status: Needs work » Needs review
FileSize
2.8 KB

Here's an updated patch, tested against the latest 7.x-3.x-dev release of Commons.

levelos’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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

  • levelos committed 5d87832 on 7.x-1.x, panels, any-entity, slots, integrations, hold_state
    #1867564: Provide entity cache support.