Problem/Motivation

Untangle interdependencies of ResponsiveImageStyle from Breakpoints (more specifically breakpoint manager). Identified around #3267870-54: Order image mappings by breakpoint ID and numeric multiplier and the next few comments.

Steps to reproduce

\Drupal::service('breakpoint.manager')->getBreakpointsByGroup($this->getBreakpointGroup());
Calling out to the service of another module directly from the entity class is a bit strange, but unless we only do this on save (in which case it could happen in the presave hook) I don't see another way.

To add more the debate, calling \Drupal::service('breakpoint.manager') _already_ happens in calculateDependencies. If we want to remove that DI failure, I think we need to open a follow-up to see how we can untangle the required dependencies between breakpoints and responsive images.

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

Comments

heddn created an issue. See original summary.

Version: 10.1.x-dev » 11.x-dev

Drupal core is moving towards using a “main” branch. As an interim step, a new 11.x branch has been opened, as Drupal.org infrastructure cannot currently fully support a branch named main. New developments and disruptive changes should now be targeted for the 11.x branch, which currently accepts only minor-version allowed changes. For more information, see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

Version: 11.x-dev » main

Drupal core is now using the main branch as the primary development branch. New developments and disruptive changes should now be targeted to the main branch.

Read more in the announcement.