[1.2.X] Fixed #13610 -- Improved error reporting when the ENGINE setting is ommitted from a database configuration. Thanks to Gregor Müllegger for the report and patch.
Backport of r13489 from trunk. git-svn-id: http://code.djangoproject.com/svn/django/branches/releases/1.2.X@13490 bcc190cf-cafb-0310-a4f2-bffc1f526a37
This commit is contained in:
parent
e10e3007d1
commit
9ff58e30c1
1
AUTHORS
1
AUTHORS
|
@ -336,6 +336,7 @@ answer newbie questions, and generally made Django that much better:
|
|||
Aljosa Mohorovic <aljosa.mohorovic@gmail.com>
|
||||
Ramiro Morales <rm0@gmx.net>
|
||||
Eric Moritz <http://eric.themoritzfamily.com/>
|
||||
Gregor Müllegger <gregor@muellegger.de>
|
||||
Robin Munn <http://www.geekforgod.com/>
|
||||
James Murty
|
||||
msundstr
|
||||
|
|
|
@ -250,15 +250,15 @@ class ManagementUtility(object):
|
|||
"""
|
||||
try:
|
||||
app_name = get_commands()[subcommand]
|
||||
if isinstance(app_name, BaseCommand):
|
||||
# If the command is already loaded, use it directly.
|
||||
klass = app_name
|
||||
else:
|
||||
klass = load_command_class(app_name, subcommand)
|
||||
except KeyError:
|
||||
sys.stderr.write("Unknown command: %r\nType '%s help' for usage.\n" % \
|
||||
(subcommand, self.prog_name))
|
||||
sys.exit(1)
|
||||
if isinstance(app_name, BaseCommand):
|
||||
# If the command is already loaded, use it directly.
|
||||
klass = app_name
|
||||
else:
|
||||
klass = load_command_class(app_name, subcommand)
|
||||
return klass
|
||||
|
||||
def autocomplete(self):
|
||||
|
|
|
@ -35,6 +35,8 @@ if DEFAULT_DB_ALIAS not in settings.DATABASES:
|
|||
raise ImproperlyConfigured("You must default a '%s' database" % DEFAULT_DB_ALIAS)
|
||||
|
||||
for alias, database in settings.DATABASES.items():
|
||||
if 'ENGINE' not in database:
|
||||
raise ImproperlyConfigured("You must specify a 'ENGINE' for database '%s'" % alias)
|
||||
if database['ENGINE'] in ("postgresql", "postgresql_psycopg2", "sqlite3", "mysql", "oracle"):
|
||||
import warnings
|
||||
if 'django.contrib.gis' in settings.INSTALLED_APPS:
|
||||
|
|
Loading…
Reference in New Issue