Download & Extend

Update Entity cache support

Project:Entity Registration
Version:7.x-1.x-dev
Component:Registration Core
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

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.

AttachmentSize
registration_entity_cache_support_update.patch2.19 KB

Comments

#1

Status:active» needs review

Forgot to set status appriately

#2

Status:needs review» reviewed & tested by the community

Patch from OP works great for me.

#3

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.

#4

Status:needs work» needs review

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

AttachmentSize
entity_cache_support-1867564-4.patch 2.8 KB

#5

Status:needs review» fixed

#6

Status:fixed» closed (fixed)

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