Re: #12804 - decorating admin views marked as invalid

2010-02-09 Thread Russell Keith-Magee
On Tue, Feb 9, 2010 at 10:51 PM, Luke Plant wrote: > On Tuesday 09 February 2010 02:40:03 Russell Keith-Magee wrote: > >> >> What do people think? >> > >> > Ugh. Assuming no one can come up with a brilliant fix so that the >> > auto_adapt stuff can work when combined with

Re: #12804 - decorating admin views marked as invalid

2010-02-09 Thread Luke Plant
On Tuesday 09 February 2010 02:40:03 Russell Keith-Magee wrote: > >> What do people think? > > > > Ugh. Assuming no one can come up with a brilliant fix so that the > > auto_adapt stuff can work when combined with arbitrary other > > decorators (I can't), I favor switching to a system that

Re: #12804 - decorating admin views marked as invalid

2010-02-09 Thread Florian Apolloner
On Feb 8, 2:44 pm, Luke Plant wrote: > What do people think? +1 for everything which removes the magic from auto_adapt_to_methods. Can't say much about the backwards incompatiblity issue, cause I don't care ;) Of course it would be nice, but given the options I would just

Re: #12804 - decorating admin views marked as invalid

2010-02-08 Thread Russell Keith-Magee
On Tue, Feb 9, 2010 at 7:19 AM, Karen Tracey wrote: > On Mon, Feb 8, 2010 at 8:44 AM, Luke Plant wrote: >> >> What do people think? > > Ugh. Assuming no one can come up with a brilliant fix so that the auto_adapt > stuff can work when combined with

Re: #12804 - decorating admin views marked as invalid

2010-02-08 Thread Karen Tracey
On Mon, Feb 8, 2010 at 8:44 AM, Luke Plant wrote: > What do people think? > Ugh. Assuming no one can come up with a brilliant fix so that the auto_adapt stuff can work when combined with arbitrary other decorators (I can't), I favor switching to a system that requires

Re: #12804 - decorating admin views marked as invalid

2010-02-08 Thread Luke Plant
On Monday 08 February 2010 08:21:27 Florian Apolloner wrote: > thx for your quick response, I'll update the ticket and reopen it > (I hope that's okay in this case…) That's fine, apologies for the premature closing. Here is my analysis, sorry in advance for the length. Unfortunately, we have

Re: #12804 - decorating admin views marked as invalid

2010-02-08 Thread Florian Apolloner
Hi, thx for your quick response, I'll update the ticket and reopen it (I hope that's okay in this case…) Cheers, Florian On Feb 8, 3:05 am, Luke Plant wrote: > Ignore my last post - my 'fix' only fixes one corner case, and not a > very useful one either.  I've got it

Re: #12804 - decorating admin views marked as invalid

2010-02-07 Thread Luke Plant
Ignore my last post - my 'fix' only fixes one corner case, and not a very useful one either. I've got it fairly sorted out now, I'll post tomorrow about this. Luke On Sunday 07 February 2010 21:11:42 Luke Plant wrote: > On Sunday 07 February 2010 10:45:29 Florian Apolloner wrote: > > Hi, > >

Re: #12804 - decorating admin views marked as invalid

2010-02-07 Thread Luke Plant
On Sunday 07 February 2010 10:45:29 Florian Apolloner wrote: > Hi, > > first of all, I agree with Luke that it's hard to fix this if it's > possible at all [1]. The only real problem I have with this ticket > beeing closed is the backwards compatibility issue. Decorating > admin views worked