zope.annotation.factory should handle proxies

Bug #261620 reported by Marius Gedminas
2
Affects Status Importance Assigned to Milestone
Zope 3
Fix Released
Undecided
Unassigned

Bug 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 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.

Revision history for this message
Marius Gedminas (mgedmin) wrote :
description: updated
Revision history for this message
Marius Gedminas (mgedmin) wrote :

Committed in rev 90365, will be released in zope.annotation 3.4.1.

Changed in zope3:
status: New → Fix Committed
Revision history for this message
Marius Gedminas (mgedmin) wrote :

Released in zope.annotation 3.4.1

Changed in zope3:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.