Skip to content

Commit eb20dc8

Browse files
ulgensSaptakS
authored andcommitted
Remove redundant if condition from django debug toolbar init
DEBUG is defined as True in the same file. Because the variable value is hardcoded, `if DEBUG` will always give same result - hence not needed
1 parent cdb2e39 commit eb20dc8

File tree

1 file changed

+18
-19
lines changed

1 file changed

+18
-19
lines changed

djangoproject/settings/dev.py

Lines changed: 18 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -44,25 +44,24 @@
4444

4545
PUSH_SSL_CALLBACK = False
4646

47-
# Enable optional components
48-
49-
if DEBUG:
50-
try:
51-
import debug_toolbar # NOQA
52-
except ImportError:
53-
pass
54-
else:
55-
INSTALLED_APPS.append("debug_toolbar")
56-
INTERNAL_IPS = ["127.0.0.1"]
57-
MIDDLEWARE.insert(
58-
MIDDLEWARE.index("django.middleware.common.CommonMiddleware") + 1,
59-
"debug_toolbar.middleware.DebugToolbarMiddleware",
60-
)
61-
MIDDLEWARE.insert(
62-
MIDDLEWARE.index("debug_toolbar.middleware.DebugToolbarMiddleware") + 1,
63-
"djangoproject.middleware.CORSMiddleware",
64-
)
65-
47+
# django-debug-toolbar initialization
48+
try:
49+
import debug_toolbar # NOQA
50+
except ImportError:
51+
pass
52+
else:
53+
INSTALLED_APPS.append("debug_toolbar")
54+
INTERNAL_IPS = ["127.0.0.1"]
55+
MIDDLEWARE.insert(
56+
MIDDLEWARE.index("django.middleware.common.CommonMiddleware") + 1,
57+
"debug_toolbar.middleware.DebugToolbarMiddleware",
58+
)
59+
MIDDLEWARE.insert(
60+
MIDDLEWARE.index("debug_toolbar.middleware.DebugToolbarMiddleware") + 1,
61+
"djangoproject.middleware.CORSMiddleware",
62+
)
63+
64+
# django-recaptcha settings
6665
SILENCED_SYSTEM_CHECKS = SILENCED_SYSTEM_CHECKS + [
6766
# Default test keys for development.
6867
"django_recaptcha.recaptcha_test_key_error"

0 commit comments

Comments
 (0)