[sqlalchemy] Re: Merge fails with ComparableProperty

2008-06-03 Thread Michael Bayer
a bug, fixed in r4834 of 0.4 branch and r4835 of the trunk. On Jun 3, 2008, at 10:15 AM, Martin Pengelly-Phillips wrote: Hi there, In SA 0.4.6 I get the following failure when attempting to merge an object that contains a ComparableProperty: prop.merge(self, instance, merged,

[sqlalchemy] Re: Merge fails with ComparableProperty

2008-06-03 Thread Martin Pengelly-Phillips
Thanks as always for the quick response and fix. As a slight aside; I am using a MapperExtension to create an attribute on an object. When the object is stored, reloaded and then merged back into a session this attribute is no longer available. Any thoughts on why this might be? ta Martin

[sqlalchemy] Re: Merge fails with ComparableProperty

2008-06-03 Thread Martin Pengelly-Phillips
Upgraded to 0.5 trunk and all works fine now. Thank you again for your help. Martin On Jun 3, 5:23 pm, Michael Bayer [EMAIL PROTECTED] wrote: merge() makes a copy of the object in a similar manner as which it would be loaded from the DB for the first time, using __new__ instead of

[sqlalchemy] Re: Merge fails with ComparableProperty

2008-06-03 Thread Michael Bayer
er, what's the reconsistute hook you're using ? we havent made a public API for it yet. we're discussing it on ticket #1070 since I'd like to get one in there for you. On Jun 3, 2008, at 1:47 PM, Martin Pengelly-Phillips wrote: Upgraded to 0.5 trunk and all works fine now. Thank

[sqlalchemy] Re: Merge fails with ComparableProperty

2008-06-03 Thread Martin Pengelly-Phillips
Hmmm...I read your last mail as populate_instance works better in 0.5, but looking again I see that wasn't what you meant. As it is I just left my code as is and after upgrading to 0.5 it just worked... On Jun 3, 7:51 pm, Michael Bayer [EMAIL PROTECTED] wrote: er, what's the reconsistute hook