404 error and exception occur when the responsive_image module is disabled.

Open on Drupal.org →

Created on 21 April 2025, 3 months ago

Problem/Motivation

When accessing the "Image styles usage" report provided by the xray_audit module, a 404 error occurs if the responsive_image module is disabled. Additionally, the following error is logged in the watchdog log:

The "responsive_image_style" entity type does not exist.

This suggests that the module does not correctly check for the existence of the responsive_image entity type before attempting to load it, leading to a fatal error in environments where the module is not enabled.

Steps to reproduce

Ensure the responsive_image module is disabled.
Navigate to: /admin/reports/xray-audit/content-metric/image-styles-usage
Observe the 404 error and check the logs for the missing entity type exception.

Proposed resolution

Update the report builder logic to check if the responsive_image module is enabled before querying or interacting with the responsive_image_style entity type. This can be done using \Drupal::moduleHandler()->moduleExists('responsive_image') and conditionally adjusting the logic accordingly.

Remaining tasks

Implement the module existence check in the image style report generator.
Add a test to ensure the report does not fail when responsive_image is disabled.

User interface changes

No user interface changes expected.

API changes

No API changes expected.

Data model changes

No data model changes expected.

🐛 Bug report

Status

Active

Version

1.6

Component

Code

Created by

🇪🇸Spain Juanjol Navarra

Live updates comments and jobs are added and updated live.

Sign in to follow issues

Merge Requests

!78404 error and exception occur when the responsive_image module is disabled.
Open
Unnamed author
updated 6 days ago

Comments & Activities

Issue created by @Juanjol
Comment 8 days ago →
🇪🇸Spain antonio.nunez Spain
antonio nunez → made their first commit to this issue’s fork.
Assigned to antonio.nunez
Comment 8 days ago →
🇪🇸Spain antonio.nunez Spain
Merge request !78Draft: Issue #3520333: Fixed 404 error in the image styles report without activating... → (Open) created by Unnamed author
Comment 7 days ago →
🇪🇸Spain antonio.nunez Spain
Fixed two issues in the Image Styles Report:

Resolved 404 error when accessing the report without the responsive_image module enabled - the plugin now properly checks module availability before attempting to load responsive image styles

Addressed timeout issues by implementing conditional processing limits - when responsive_image is disabled, limits are applied (50 image styles, 100 view displays) to prevent timeouts; when enabled, all items are processed
without restrictions.

These changes ensure the report remains accessible and functional regardless of the responsive_image module status.
Comment 6 days ago →
🇪🇸Spain antonio.nunez Spain
Refactoring and debugging of the timeout management in the image styles report, as well as correcting the coding standards according to phpcs, phpstan and cspell.
Comment 6 days ago →
🇪🇸Spain antonio.nunez Spain
First commit to issue fork.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024