#4516: newforms-admin: Move AdminModel's URL handling into a method
-----------------------------------------------------+----------------------
   Reporter:  Marty Alchin <[EMAIL PROTECTED]>  |                Owner:  nobody 
        
     Status:  new                                    |            Component:  
Admin interface
    Version:  newforms-admin                         |           Resolution:    
             
   Keywords:  newforms-admin handle_url              |                Stage:  
Unreviewed     
  Has_patch:  1                                      |           Needs_docs:  0 
             
Needs_tests:  0                                      |   Needs_better_patch:  0 
             
-----------------------------------------------------+----------------------
Changes (by brosner):

  * needs_better_patch:  => 0
  * version:  SVN => newforms-admin
  * needs_tests:  => 0
  * needs_docs:  => 0

Comment:

 +1.  I have already overridden ```__call__``` and it is pretty nasty and I
 end up losing the dependancy checks.  However, I think the new method
 needs to be improved slightly.  Based on the patch given here if I
 override ```handle_url``` I will still need to check for None and display
 the changelist page then call the parent.  If I call the parent first then
 ```change_view``` will always have to run before I can check for my own
 URL.  It seems to me that the URL mapping in ```AdminSite``` and
 ```ModelAdmin``` need to be refactored a little bit for better
 customization.

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