#4491: [newforms-admin] addAnother link does not work
-------------------------------------------------+--------------------------
   Reporter:  Honza Král <[EMAIL PROTECTED]>  |                Owner:  
jkocherhans    
     Status:  assigned                           |            Component:  Admin 
interface
    Version:  newforms-admin                     |           Resolution:        
         
   Keywords:  newforms admin addAnother          |                Stage:  
Accepted       
  Has_patch:  1                                  |           Needs_docs:  0     
         
Needs_tests:  0                                  |   Needs_better_patch:  1     
         
-------------------------------------------------+--------------------------
Changes (by jkocherhans):

  * owner:  adrian => jkocherhans
  * needs_better_patch:  0 => 1
  * status:  new => assigned
  * stage:  Unreviewed => Accepted

Comment:

 Thanks for the ideas Honza Král. It makes perfect sense for both of those
 classes to know about which {{{AdminSite}}} object they are attached to. I
 think the newest patch I posted is pretty close, but don't like having to
 create a class for every single inline. Turning {{{StackedInline}}} and
 {{{TabulatInline}}} into factory functions would work, but then you have
 to do some trickery to hang custom methods on those classes and it just
 feels wrong. I'll think about this some more and maybe get some input at
 the sprint next week.

-- 
Ticket URL: <http://code.djangoproject.com/ticket/4491#comment:3>
Django Code <http://code.djangoproject.com/>
The web framework for perfectionists with deadlines
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at 
http://groups.google.com/group/django-updates?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to