#3185: Allow django.contrib.auth's LOGIN_URL, LOGOUT_URL and ACCOUNT_URL to be 
set
in settings.py
------------------------------------------------------+---------------------
   Reporter:  Vasily Sulatskov <[EMAIL PROTECTED]>  |                Owner:  
adrian      
     Status:  new                                     |            Component:  
Contrib apps
    Version:  SVN                                     |           Resolution:   
           
   Keywords:                                          |                Stage:  
Accepted    
  Has_patch:  1                                       |           Needs_docs:  
0           
Needs_tests:  0                                       |   Needs_better_patch:  
1           
------------------------------------------------------+---------------------
Changes (by mtredinnick):

  * needs_better_patch:  0 => 1
  * stage:  Ready for checkin => Accepted

Comment:

 The most recent patch still has a few subtle problems:
 
  1. Anything that is configurable should always be read from settings, not
 shoved in as a constant into some other module (it's not a constant any
 longer, after all). I realise LOGIN_URL is already a constant in the auth
 module, but that now becomes a proxy for the value from settings for
 backwards compatibility reasons. Everything else comes straight out of
 settings (including all current internal uses of LOGIN_URL).
 
  2. We should not be reading anything from settings at import time; only
 at execution time. The reason for this is manually configured settings.
 The developer needs to be able to import the necessary modules and then
 call {{{settings.configure()}}}. The current code reads from settings at
 import time and so will raise a configuration error. This problem exists
 in other parts of Django as well, but that's not a reason to perpetuate
 the problem.

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