Problem/Motivation

BC was missed for core/modules/image/src/Plugin/Field/FieldFormatter/ImageFormatter.php #3567618: Create service for image_path_flush, image_style_options and constant for IMAGE_DERIVATIVE_TOKEN

Let's add it here.

Steps to reproduce

Use svg image.
Upgrade to 11.4

Proposed resolution

Add BC for the constructor changes.

Remaining tasks

Review

User interface changes

N/A

Introduced terminology

N/A

API changes

N/A

Data model changes

N/A

Release notes snippet

N/A

Issue fork drupal-3609124

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

nicxvan created an issue. See original summary.

nicxvan’s picture

nicxvan commented
Related issues: +#3567618: Create service for image_path_flush, image_style_options and constant for IMAGE_DERIVATIVE_TOKEN

nicxvan opened merge request !16246

nicxvan’s picture

nicxvan commented
Issue summary: View changes
catch’s picture

catch
he/him
Primary language English
commented
Status: Active » Reviewed & tested by the community

Looks great I don't think we need anything else.

  • longwave committed 340ccd3f on 11.4.x 
    fix: #3609124 Add proper BC for ImageFormatter

By: nicxvan
(cherry...

  • longwave committed e455f285 on 11.x 
    fix: #3609124 Add proper BC for ImageFormatter

By: nicxvan
longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Version: main » 11.4.x-dev
Status: Reviewed & tested by the community » Fixed

Committed and pushed e455f2852dc to 11.x and 340ccd3fd7c to 11.4.x. Thanks!

Didn't commit to main as it would only mean removing it again later, the code looks correct already there to me.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

longwave closed merge request !16246