Update django.contrib.auth.decorators.py

[Anton Antonov]

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

[Claude Paroz]

I guess the current code is written that way to prevent accessing settings at import time, as settings should be kept lazy as much as possible.

[Tim Graham]

I agree with Claude and don't see much advantage to changing it, especially as it seems to cause many test failures. e74edb4d53b089ec57ec4830eeba98607283a092 is version before the current version where settings was also imported only if needed.

[Anton Antonov]

Alright.