#9122: Inline admin on generic relations ignores exclude and max_num
-----------------------------------+----------------------------------------
          Reporter:  danielr       |         Owner:  nobody                  
            Status:  new           |     Milestone:                          
         Component:  Contrib apps  |       Version:  1.0                     
        Resolution:                |      Keywords:  admin, generic relations
             Stage:  Accepted      |     Has_patch:  1                       
        Needs_docs:  0             |   Needs_tests:  1                       
Needs_better_patch:  0             |  
-----------------------------------+----------------------------------------
Changes (by alexr):

 * cc: alexr (added)

Comment:

 Ok, I would like to see this bug corrected. I am willing to write the
 tests but I am not really sure how.

 The class we are correcting is GenericInlineModelAdmin, whose constructor
 has the signature(self, parent_model, admin_site). To test this I guess I
 have to create an admin site instance to pass to this item. How should I
 do that? I wondered how the tests worked for the InlineModelAdmin, hoping
 I could copy them. However, django/contrib/admin doesn't have any tests
 and a recursive grep of trunk/tests doesn't show anything for
 InlineModelAdmin.

 Any ideas for where to start? (Or tips for other test code that might lead
 me in the right direction.)

-- 
Ticket URL: <http://code.djangoproject.com/ticket/9122#comment:3>
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 [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