Fix all Django system check warnings in the Review Board codebase.
Review Request #11071 — Created July 6, 2020 and submitted — Latest diff uploaded
Django had a few warnings about code deprecations and recommendations,
which it would show every time the dev server loaded, or when running
other certain management commands.This change satisfies all warnings (aside from one category, which is
silenced). This just consists of removingnull=True
from
ManyToManyFields
(which doesn't do anything), and fully removing the
legacysettings.TEMPLATE_*
settings in favor ofsettings.TEMPLATES
,The silenced warning is just a suggestion to change a
ForeignKey
with
unique=True
to aOneToOneField
. This makes sense, but would require
new work in Django Evolution to understand the equivalency of this
change, and is not worth the time. So instead, we're just telling Django
to ignore this warning.
Ran some standard tests in Review Board. Didn't see any issues, or any
warnings.Unit tests pass.