#35822 closed Bug (invalid)
App name hard-coded in collectstatic command forces name of overriding app name to be 'staticfiles'
Reported by: | fishfin | Owned by: | |
---|---|---|---|
Component: | contrib.staticfiles | Version: | 5.1 |
Severity: | Normal | Keywords: | collectstatic |
Cc: | fishfin | Triage Stage: | Unreviewed |
Has patch: | no | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description (last modified by )
In django.contrib.staticfiles.management.commands.collectstatic
, Command.set_options()
, there is:
ignore_patterns += apps.get_app_config("staticfiles").ignore_patterns
This looks for config class StaticFilesConfig
in the app trying to overwrite ignore_patterns
. Because Django forces the class name to match config name (in this case StaticFiles
, the name of the app is forced to be staticfiles
, it cannot be anything else. Either the hard-coding in collectstatic command set_options() should be changed, or document at https://docs.djangoproject.com/en/5.1/ref/contrib/staticfiles/#customizing-the-ignored-pattern-list should be changed for the example which shows the Config can be named MyStaticFilesConfig
, which is clearly cannot be.
I have just started on Django (3-4 months) and I have looked at number of places, this info is missing, hence raising this ticket.
Change History (6)
comment:1 by , 5 weeks ago
Description: | modified (diff) |
---|
comment:2 by , 5 weeks ago
Resolution: | → worksforme |
---|---|
Status: | new → closed |
comment:3 by , 5 weeks ago
These are my steps on Windows 11, Python 3.12.7, Django 5.1.2, I use pipenv: I know some of these steps don't matter, but I am unable to recreate what you said, so listing everything.
$ mkdir myvenv & cd myvenv & mkdir .venv & pipenv --python 3.12 & pipenv shell $ pipenv install django $ django-admin.exe startproject myproj & cd myproj $ django-admin.exe startapp myapp
Edit myvenv/myproj/myproj/settings.py to:
1) Add STATIC_ROOT = "static"
to the file
2) Replace item 'django.contrib.staticfiles'
in INSTALLED_APPS with 'myapp.apps.MyappConfig'
(pointing directly to the Config class instead of just app name).
Edit myvenv/myproj/myapp/apps.py to:
1) Replace
from django.apps import AppConfig class MyappConfig(AppConfig): default_auto_field = 'django.db.models.BigAutoField' name = 'myapp'
with
from django.contrib.staticfiles.apps import StaticFilesConfig class MyappConfig(StaticFilesConfig): name = 'myapp' ignore_patterns = ["testpattern"]
Open shell:
$ cd myvenv/myproj $ py manage.py shell >>> from django.apps import apps >>> apps.get_app_config('staticfiles').ignore_patterns LookupError: No installed app with label 'staticfiles'. >>> apps.get_app_config('myapp').ignore_patterns ['testpattern']
When you removed 'django.contrib.staticfiles'
from settings.py, I am not sure how it is still able to find an app with that name. I am getting an error as I had expected as seen above. Changing MyappConfig.name to 'staticfiles' doesn't help either. Changing parent class of MyappConfig from StaticFilesConfig to AppConfig does not seem to be right.
Now, since we removed 'django.contrib.staticfiles'
, django doesn't know about its management/commands directory, so py manage.py collectstatic
won't work.
$ py manage.py collectstatic Unknown command: 'collectstatic' Type 'manage.py help' for usage.
To make it work, I did the following:
$ mkdir myvenv/myproj/myapp/management/commands
Create following empty files:
myvenv/myproj/myapp/management/__init__.py myvenv/myproj/myapp/management/commands/__init__.py myvenv/myproj/myapp/management/commands/collectstatic.py myvenv/myproj/myapp/management/commands/findstatic.py
Edit myvenv/myproj/myapp/management/commands/collectstatic.py
and add following code:
from django.contrib.staticfiles.management.commands.collectstatic import Command
Edit myvenv/myproj/myapp/management/commands/findstatic.py
and add following code:
from django.contrib.staticfiles.management.commands.findstatic import Command
Now:
$ py manage.py collectstatic File "...\myvenv\.venv\Lib\site-packages\django\contrib\staticfiles\management\commands\collectstatic.py", line 103, in set_options ignore_patterns += apps.get_app_config("staticfiles").ignore_patterns LookupError: No installed app with label 'staticfiles'.
...which is the same error I got in django shell. If I am doing something wrong, please let me know.
comment:4 by , 5 weeks ago
Resolution: | worksforme |
---|---|
Status: | closed → new |
Please see my previous comment, listed all steps to recreate.
Additionally:
>>> from django.apps import apps >>> apps.app_configs {'admin': <AdminConfig: admin>, 'auth': <AuthConfig: auth>, 'contenttypes': <ContentTypesConfig: contenttypes>, 'sessions': <SessionsConfig: sessions>, 'messages': <MessagesConfig: messages>, 'myapp': <MyappConfig: myapp>}
There's no staticfiles
app.
comment:5 by , 5 weeks ago
Resolution: | → invalid |
---|---|
Status: | new → closed |
You added a name.
This
class MyappConfig(StaticFilesConfig): name = 'myapp' ignore_patterns = ["testpattern"]
should be:
class MyappConfig(StaticFilesConfig): ignore_patterns = ["testpattern"]
It inherits the name for StaticFilesConfig
So it works as documented
comment:6 by , 5 weeks ago
I removed the name.
from django.contrib.staticfiles.apps import StaticFilesConfig class MyappConfig(StaticFilesConfig): # StaticFilesConfig.name = "django.contrib.staticfiles" ignore_patterns = ["testpattern"]
Works fine.
I missed a number of things here: INSTALLED_APPS can point to a specific AppConfig class, which in turn may have a name, which is the app Django loads. This also loads the management commands in django.contrib.staticfiles
, so no overrides of commands were required. I removed myapp
as there's no need create one just for staticfiles commands now, just shifted the overriding class to a file in myproj, and changed settings.INSTALLED_APPS likewise.
Thank you.
I followed the documented example and it works as expected
These were my steps:
STATIC_ROOT = "static"
python manage.py startapp app1
Then followed the instructions
ignore_patterns
'django.contrib.staticfiles'
with that class path ('app1.apps.MyStaticFilesConfig'
) inINSTALLED_APPS
settingCommand uses new custom setting
Also in the shell