diff options
| author | Aymeric Augustin <aymeric.augustin@m4x.org> | 2015-10-15 22:03:25 +0200 |
|---|---|---|
| committer | Aymeric Augustin <aymeric.augustin@m4x.org> | 2015-10-18 10:34:04 +0200 |
| commit | df0a446fd4c864c003e4f941b5b7abd6f10c9427 (patch) | |
| tree | f80865b225991bcf1b032e4276d80bb883530b76 | |
| parent | 6c95b134e9b2d5641c123551c080305e90e6a89d (diff) | |
Fixed #25510 -- Allowed runserver to start with incorrect INSTALLED_APPS.
In that case, the content of INSTALLED_APPS will be ignored until it's
fixed and the autoreloader kicks in. I confirmed this behavior manually.
As explained on the ticket it's hard to write a test for this case
| -rw-r--r-- | django/core/management/__init__.py | 11 | ||||
| -rw-r--r-- | docs/releases/1.8.6.txt | 4 |
2 files changed, 11 insertions, 4 deletions
diff --git a/django/core/management/__init__.py b/django/core/management/__init__.py index eb49cce375..f9e11b48a6 100644 --- a/django/core/management/__init__.py +++ b/django/core/management/__init__.py @@ -1,9 +1,9 @@ from __future__ import unicode_literals -import collections import os import pkgutil import sys +from collections import OrderedDict, defaultdict from importlib import import_module import django @@ -142,7 +142,7 @@ class ManagementUtility(object): "", "Available subcommands:", ] - commands_dict = collections.defaultdict(lambda: []) + commands_dict = defaultdict(lambda: []) for name, app in six.iteritems(get_commands()): if app == 'django.core': app = 'django' @@ -312,8 +312,11 @@ class ManagementUtility(object): autoreload.check_errors(django.setup)() except Exception: # The exception will be raised later in the child process - # started by the autoreloader. - pass + # started by the autoreloader. Pretend it didn't happen by + # loading an empty list of applications. + apps.all_models = defaultdict(OrderedDict) + apps.app_configs = OrderedDict() + apps.apps_ready = apps.models_ready = apps.ready = True # In all other cases, django.setup() is required to succeed. else: diff --git a/docs/releases/1.8.6.txt b/docs/releases/1.8.6.txt index 178d714f01..8fb9e16be5 100644 --- a/docs/releases/1.8.6.txt +++ b/docs/releases/1.8.6.txt @@ -25,3 +25,7 @@ Bugfixes * Allowed filtering over a ``RawSQL`` annotation (:ticket:`25506`). * Made the ``Concat`` database function idempotent on SQLite (:ticket:`25517`). + +* Avoided a confusing stack trace when starting :djadmin:`runserver` with an + invalid :setting:`INSTALLED_APPS` setting (:ticket:`25510`). This regression + appeared in 1.8.5 as a side effect of fixing :ticket:`24704`. |
