| 2008-08-26 20:37:01 |
Marius Gedminas |
description |
There's this application that's using zope.annotation.factory to adapt objects that are wrapped in LocationProxies. This causes certain problems (those transient proxies get stored in the ZODB with write-on-read sematics, causing frequent conflict errors).
It is my strong opinion that zope.annotation.factory should never assign a proxy to an object's __parent__. I'm attaching a patch against zope.annotation 3.4 branch that fixes this. I'd feel better if someone else reviewed it before I commit. |
There's this application that's using zope.annotation.factory to adapt objects that are wrapped in LocationProxies. This causes certain problems (those transient proxies get stored in the ZODB with write-on-read sematics, causing frequent conflict errors).
It is my strong opinion that zope.annotation.factory should never assign a proxy to a persistent object's __parent__. I'm attaching a patch against zope.annotation 3.4 branch that fixes this. I'd feel better if someone else reviewed it before I commit. |
|
