django-sphinx icon indicating copy to clipboard operation
django-sphinx copied to clipboard

Patch fixed utils.config._get_database_engine for django 1.2.x

Open Apkawa opened this issue 15 years ago • 1 comments

Hi. I written path http://paste.pocoo.org/show/263575/

In django new config as DATABASES[key]['ENGINE']

Apkawa avatar Sep 18 '10 14:09 Apkawa

This patch does not detect MySQL correctly - settings.DATABASES['default']['ENGINE'] is a dotted-notation string like "django.db.backends.mysql" so comparing with "mysql" is wrong.

Also, "default" should be replaced with DEFAULT_DB_ALIAS.

Also, you make an unrelated change to the end of the file.

lamby avatar Sep 12 '11 11:09 lamby