Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Fixed #17037 -- Added a --all option to diffsettings. #914

Closed
wants to merge 1 commit into from

1 participant

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Mar 17, 2013
  1. @aaugustin
This page is out of date. Refresh to see the latest.
View
16 django/core/management/commands/diffsettings.py
@@ -1,14 +1,22 @@
+from optparse import make_option
+
from django.core.management.base import NoArgsCommand
def module_to_dict(module, omittable=lambda k: k.startswith('_')):
- "Converts a module namespace to a Python dictionary. Used by get_settings_diff."
- return dict([(k, repr(v)) for k, v in module.__dict__.items() if not omittable(k)])
+ """Converts a module namespace to a Python dictionary."""
+ return dict((k, repr(v)) for k, v in module.__dict__.items() if not omittable(k))
class Command(NoArgsCommand):
help = """Displays differences between the current settings.py and Django's
default settings. Settings that don't appear in the defaults are
followed by "###"."""
+ option_list = NoArgsCommand.option_list + (
+ make_option('--all', action='store_true', dest='all', default=False,
+ help='Display all settings, regardless of their value.'
+ 'Default values are prefixed by "###".'),
+ )
+
requires_model_validation = False
def handle_noargs(self, **options):
@@ -22,9 +30,11 @@ def handle_noargs(self, **options):
default_settings = module_to_dict(global_settings)
output = []
- for key in sorted(user_settings.keys()):
+ for key in sorted(user_settings):
if key not in default_settings:
output.append("%s = %s ###" % (key, user_settings[key]))
elif user_settings[key] != default_settings[key]:
output.append("%s = %s" % (key, user_settings[key]))
+ elif options['all']:
+ output.append("### %s = %s" % (key, user_settings[key]))
return '\n'.join(output)
View
7 docs/ref/django-admin.txt
@@ -168,8 +168,11 @@ example, the default settings don't define :setting:`ROOT_URLCONF`, so
:setting:`ROOT_URLCONF` is followed by ``"###"`` in the output of
``diffsettings``.
-Note that Django's default settings live in ``django/conf/global_settings.py``,
-if you're ever curious to see the full list of defaults.
+The :djadminopt:`--all` option may be provided to display all settings, even
+if they have Django's default value.
+
+.. versionadded:: 1.6
+ The :djadminopt:`--all` option was added.
dumpdata <appname appname appname.Model ...>
--------------------------------------------
View
12 tests/admin_scripts/tests.py
@@ -1661,11 +1661,21 @@ def test_custom_project_template_with_non_ascii_templates(self):
class DiffSettings(AdminScriptTestCase):
"""Tests for diffsettings management command."""
+
def test_basic(self):
- "Runs without error and emits settings diff."
+ """Runs without error and emits settings diff."""
self.write_settings('settings_to_diff.py', sdict={'FOO': '"bar"'})
self.addCleanup(self.remove_settings, 'settings_to_diff.py')
args = ['diffsettings', '--settings=settings_to_diff']
out, err = self.run_manage(args)
self.assertNoOutput(err)
self.assertOutput(out, "FOO = 'bar' ###")
+
+ def test_all(self):
+ """The all options also shows settings with the default value."""
+ self.write_settings('settings_to_diff.py', sdict={'STATIC_URL': 'None'})
+ self.addCleanup(self.remove_settings, 'settings_to_diff.py')
+ args = ['diffsettings', '--settings=settings_to_diff', '--all']
+ out, err = self.run_manage(args)
+ self.assertNoOutput(err)
+ self.assertOutput(out, "### STATIC_URL = None")
Something went wrong with that request. Please try again.