Django >= 1.7 compatibility for requires_system_checks

[nealtodd]

Hi @dcramer,

Devserver will fail to start up on Django 1.7 with the error:

django.core.exceptions.ImproperlyConfigured: Command Command defines both "requires_model_validation" and "requires_system_checks", which is illegal. Use only "requires_system_checks".

The django-devserver runserver management command inherits from the django core runserver management command which set requires_system_checks = False. The django-devserver runserver management command then sets requires_model_validation = False.

This pull request ensures only the appropriate attribute is set to False, so is backwards compatible with Django < 1.7.

The implementation is only suggested. If you have another way you'd prefer to do it that's fine, this was just a convenient way to illustrate the issue.

[nealtodd]

PS. An alternative is to just delete line 70 (at the expense of explicitness) because the superclass sets the appropriate attribute to False anyway.

[gtaylor]

Looks like this is tripping me up, too.

[mnaser]

Just ran into this too, be nice if it can be pulled in.

[seocam]

Same here. @dcramer any chance we can get this merged?

[sutyrin]

+1. Gotta use personal fork for this, please merge this :)

[ramusus]

+1 waiting for merge

[egrubbs]

+1 please merge

[keimlink]

Thanks @dcramer 🎆

[seocam]

👍

[nealtodd]

Thanks @dcramer. When you get a chance could you make a 0.8.1 release to PyPI please, so we can get it from there in a pip install?

[coagulant]

Merged fix forces validation for Django 1.6 and below, is that on purpose?
#99 (comment)

[GitFree]

+1 for

Thanks @dcramer. When you get a chance could you make a 0.8.1 release to PyPI please, so we can get it from there in a pip install?

[Pewpewarrows]

Shouldn't this be changing requires_system_checks and requires_model_validation on self, instead of creating locally scoped variables inside __init__? Or am I understanding this wrong?