Re: Manager-model reassignment on abstract subclassing

2008-05-23 Thread Sebastian Noack
Because of I needed the patch more urgent I actually thought by myself, I have already uploaded a new version including test cases. I found a bug by the way, that breaks the ability to derive from a not abstract model, but I have fixed it. If you are already using the previous version of my patch,

Re: Manager-model reassignment on abstract subclassing

2008-05-22 Thread Sebastian Noack
> > > There's a similar ticket to this at 7154  (http:// > > > code.djangoproject.com/ticket/7154). It has a patch that works > > > but I am not sure if it is the most elegant method. I am the author of #7154 and its patch. I thought first that it is possible to fix this problem with less code if

Re: Manager-model reassignment on abstract subclassing

2008-05-17 Thread Thejaswi Puthraya
On May 17, 10:57 am, "[EMAIL PROTECTED]" <[EMAIL PROTECTED]> wrote: > > There's a similar ticket to this at 7154  (http:// > > code.djangoproject.com/ticket/7154). It has a patch that works but I > > am not sure if it is the most elegant method. > > Whoops!  Sounds like we're talking about the sam

Re: Manager-model reassignment on abstract subclassing

2008-05-16 Thread [EMAIL PROTECTED]
> There's a similar ticket to this at 7154  (http:// > code.djangoproject.com/ticket/7154). It has a patch that works but I > am not sure if it is the most elegant method. Whoops! Sounds like we're talking about the same thing. I thought based on the summary that it was a different problem, bec

Re: Manager-model reassignment on abstract subclassing

2008-05-16 Thread Thejaswi Puthraya
On May 17, 5:38 am, "[EMAIL PROTECTED]" <[EMAIL PROTECTED]> wrote: > I've opened up a ticket 7252 [1] with patch, but I'm not sure my patch > addresses the problem from the right angle, so if someone knows a > better way to go about fixing it, I'd like to open the discussion here > for better ways

Manager-model reassignment on abstract subclassing

2008-05-16 Thread [EMAIL PROTECTED]
I've opened up a ticket 7252 [1] with patch, but I'm not sure my patch addresses the problem from the right angle, so if someone knows a better way to go about fixing it, I'd like to open the discussion here for better ways to fix this. Essentially what's happening is that the manager is being in