Three followups on 7075269:
- config/urls.py: drop dead try/except ImportError fallback.
The settings.py gate already guarantees debug_toolbar is
importable before we reach this line, so the except branch
was unreachable and the re-import of include/path was
redundant (both imported at top of file).
- config/settings.py: SHOW_TOOLBAR_CALLBACK now returns True
unconditionally. The triple gate passed at settings-load time,
so re-checking DEBUG and _IS_DEV inside the lambda was
redundant. Comment corrected — the callback has nothing to
do with "stale cached pages".
- requirements.txt: inline comment noting django-debug-toolbar
is dev-only and gated.
No behavioural change. Tests: 68/68.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Double-gated install: only loads when DEBUG=true AND USE_SQLITE=true,
never in prod. Lets us profile SQL query counts on the dashboard and
payroll pages before attacking N+1 hotspots.
requirements.txt adds django-debug-toolbar==6.0.0
config/settings.py conditionally appends to INSTALLED_APPS + MIDDLEWARE
config/urls.py conditionally includes __debug__ route
No behavioural change to production.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
