diff --git a/CHANGELOG.md b/CHANGELOG.md index 09ae7da..6547c00 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -7,6 +7,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0 ## [2.3.0] - 2024-02-23 - Make it possible to set [WeasyPrint options][2] via `WeasyTemplateResponse` (#79) +- Add support for non-hashed path when `ManifestStaticFilesStorage` is used with `DEBUG=True` [2]: https://doc.courtbouillon.org/weasyprint/stable/api_reference.html#weasyprint.DEFAULT_OPTIONS diff --git a/django_weasyprint/tests/__init__.py b/django_weasyprint/tests/__init__.py index 765a3b7..2d086b7 100644 --- a/django_weasyprint/tests/__init__.py +++ b/django_weasyprint/tests/__init__.py @@ -15,6 +15,7 @@ settings.configure( + DEBUG=False, SECRET_KEY=get_random_secret_key(), ALLOWED_HOSTS=['testserver'], ROOT_URLCONF=__name__, diff --git a/django_weasyprint/tests/test_utils.py b/django_weasyprint/tests/test_utils.py index e73dd4f..076e0ef 100644 --- a/django_weasyprint/tests/test_utils.py +++ b/django_weasyprint/tests/test_utils.py @@ -96,6 +96,18 @@ def test_manifest_static(self, mock_fetcher, mock_find, mock_open, hashed_files) mock_open.assert_called_once_with('/www/static/css/styles.css', 'rb') self.assert_data(data, '/www/static/css/styles.css', 'text/css') + mock_find.reset_mock() + mock_open.reset_mock() + + # ManifestStaticFilesStorage.url() returns un-hashed URL with DEBUG=True, + # django_url_fetcher() is still able to find the file. + with override_settings(DEBUG=True): + data = django_url_fetcher('file:///static/css/styles.css') + mock_fetcher.assert_not_called() + mock_find.assert_called_once_with('css/styles.css') + mock_open.assert_called_once_with('/www/static/css/styles.css', 'rb') + self.assert_data(data, '/www/static/css/styles.css', 'text/css') + @override_settings(STATIC_URL='/static/', STATIC_ROOT='/www/static') @mock.patch('django_weasyprint.utils.open') @mock.patch('django_weasyprint.utils.find', return_value=None) diff --git a/django_weasyprint/utils.py b/django_weasyprint/utils.py index ca90b99..e1f78c3 100644 --- a/django_weasyprint/utils.py +++ b/django_weasyprint/utils.py @@ -51,7 +51,7 @@ def django_url_fetcher(url, *args, **kwargs): # strip the STATIC_URL prefix to get the relative filesystem path relative_path = url_path.replace(settings.STATIC_URL, '', 1) # detect hashed files storage and get path with un-hashed filename - if hasattr(staticfiles_storage, 'hashed_files'): + if not settings.DEBUG and hasattr(staticfiles_storage, 'hashed_files'): log.debug('Hashed static files storage detected') relative_path = get_reversed_hashed_files()[relative_path] data['filename'] = Path(relative_path).name