#24188: Update django.contrib.auth.decorators.py
--------------------------------------+--------------------
     Reporter:  syndbg                |      Owner:  nobody
         Type:  Cleanup/optimization  |     Status:  new
    Component:  contrib.auth          |    Version:  1.7
     Severity:  Normal                |   Keywords:
 Triage Stage:  Unreviewed            |  Has patch:  1
Easy pickings:  0                     |      UI/UX:  0
--------------------------------------+--------------------
 Having a default login_url of settings.LOGIN_URL is a bit more explicit
 and readable than giving it a default value of None and later in
 _wrapped_view resolving the login_url or settings.LOGIN_URL if the first
 was none.

 login_required and permission_required don't really need to be changed,
 but it's better for consistency.

 Addressed in https://github.com/django/django/pull/3959

--
Ticket URL: <https://code.djangoproject.com/ticket/24188>
Django <https://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 unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/049.944b406c50a60802d6864294235c8bf5%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to