#30329: ImproperlyConfigured exceptions should be raised immediately.
---------------------------------+------------------------------------
Reporter: Alasdair Nicol | Owner: (none)
Type: Bug | Status: new
Component: Error reporting | Version: master
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
---------------------------------+------------------------------------
Old description:
> When an older version of sqlite is installed, the final line of the
> traceback is unrelated.
>
> {{{
> raise LookupError(message)
> LookupError: No installed app with label 'admin'.
> }}}
>
> If you look further up the traceback, you can see what the problem is,
> but this can be tricky for newer users.
>
> {{{
> raise ImproperlyConfigured('SQLite 3.8.3 or later is required (found
> %s).' % Database.sqlite_version)
> django.core.exceptions.ImproperlyConfigured: SQLite 3.8.3 or later is
> required (found 3.7.17).
> }}}
>
> I think that if we made the error message clearer it would be helpful for
> new users. Perhaps we could do this by checking the sqlite version in a
> system check, or maybe there's a better approach.
>
> Related Stack Overflow question:
> https://stackoverflow.com/questions/55512244/no-installed-app-with-label-
> admin-in-empty-django-2-2-project/
>
> See
> [https://github.com/django/django/blob/755673e1bca7edb6bee7a958f40d9ae54d85d44c/django/apps/registry.py#L131-L134
> django/apps/registry.py].
New description:
When an older version of sqlite is installed, the final line of the
traceback in runserver is unrelated.
{{{
raise LookupError(message)
LookupError: No installed app with label 'admin'.
}}}
If you look further up the traceback, you can see what the problem is, but
this can be tricky for newer users.
{{{
raise ImproperlyConfigured('SQLite 3.8.3 or later is required (found
%s).' % Database.sqlite_version)
django.core.exceptions.ImproperlyConfigured: SQLite 3.8.3 or later is
required (found 3.7.17).
}}}
I think that if we made the error message clearer it would be helpful for
new users. Perhaps we could do this by checking the sqlite version in a
system check, or maybe there's a better approach.
Related Stack Overflow question:
https://stackoverflow.com/questions/55512244/no-installed-app-with-label-
admin-in-empty-django-2-2-project/
See
[https://github.com/django/django/blob/755673e1bca7edb6bee7a958f40d9ae54d85d44c/django/apps/registry.py#L131-L134
django/apps/registry.py].
--
Comment (by Alasdair Nicol):
I ran git bisect and I believe the behaviour changed in #27685 when the
autoloader changed.
Only `runserver` shows the `LookupError` last - `manage.py check`,
`manage.py migrate` and so on all show
`django.core.exceptions.ImproperlyConfigured`.
--
Ticket URL: <https://code.djangoproject.com/ticket/30329#comment:4>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
--
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit
https://groups.google.com/d/msgid/django-updates/071.344565379bbe0f6a4e0e785969e29382%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
