#15569: wrong index assigned when creating a new inline
------------------------------------------------+-----------------------
               Reporter:  sbaechler             |        Owner:  nobody
                 Status:  new                   |    Milestone:  1.3
              Component:  django.contrib.admin  |      Version:  1.2
             Resolution:                        |     Keywords:  blocker
           Triage Stage:  Ready for checkin     |    Has patch:  1
    Needs documentation:  0                     |  Needs tests:  1
Patch needs improvement:  1                     |
------------------------------------------------+-----------------------

Comment (by russellm):

 Two comments:

  1. Is it really too much to ask that when you report a bug, you provide a
 clear set of instructions for how to reproduce the bug? I've spent half an
 hour spelunking through admin source code trying to work out what magic
 incantations would allow me to get a UL in an inline. For future reference
 -- have a model field with choices, and add radio_fields to the
 InlineModelAdmin.

  2. Neither proposed patch fixes the problem without reverting the fix
 introduced by r15422. Moving the increment corrects the id number for the
 UL, but doesn't fix the numbering of LABEL elements associated with the
 associated with LIs on the UL. Using a glob to capture child elements gets
 the counter consistent for all elements, but gets the count wrong when you
 add three rows, delete one, add a new row (the last added new row will
 have an id 1 less than it should be).

-- 
Ticket URL: <http://code.djangoproject.com/ticket/15569#comment:10>
Django <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 django-updates@googlegroups.com.
To unsubscribe from this group, send email to 
django-updates+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/django-updates?hl=en.

Reply via email to