Okay, I've submitted #7537 with three patches. I manually split out sub-changes from a previous version, so some things may be broken, but hopefully in simple ways.
The first patch at least could go in almost as-is (one little design- decision FIXME) and would be immediately helpful. The second is icing on the cake, but the actual code change is very minor. The third patch is -- let me say it again -- example only. I don't understand that core nfa logic, so it's actually broken. But one feature about it to note is how I've resolved (for another project) the issue of decorating instance methods. For example, in the nfa sites.py, if the never_cache decorator actually tried to do something with `request`, it would break because the first parameter it sees is `self` (the AdminSite instance), not `request`. But using instance methods as views is a powerful pattern (see my post http://lethain.com/entry/2008/jun/22/json-object-oriented-views-and-starting-a-real-app/), so I figured out a way to work around that problem: the instance_decorator descriptor that I pasted at the top of that patch. -Ken On Jun 25, 9:41 am, "Gary Wilson Jr." <[EMAIL PROTECTED]> wrote: > Ken Arnold wrote: > > I wanted to get any feedback on this change before submitting a > > ticket, but if people think it's a good idea, I can clean up the patch > > and make a ticket. > > Anything that gets REST projects and nfa off the ad-hoc dispatchers > sounds like a good idea to me. Along with a ticket and patch, it would > be nice to see what nfa's dispatching looks like with the changes. If > you feel like it, please put those changes in the patch as well or > attach a separate patch. > > Thanks, > Gary --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Django developers" group. To post to this group, send email to django-developers@googlegroups.com To unsubscribe from this group, send email to [EMAIL PROTECTED] For more options, visit this group at http://groups.google.com/group/django-developers?hl=en -~----------~----~----~----~------~----~------~--~---
