Skip to content

[3.x] Migrate to Intervention Image v4#4544

Open
olivervogel wants to merge 2 commits intoflarum:2.xfrom
olivervogel:intervention-image-v4
Open

[3.x] Migrate to Intervention Image v4#4544
olivervogel wants to merge 2 commits intoflarum:2.xfrom
olivervogel:intervention-image-v4

Conversation

@olivervogel
Copy link
Copy Markdown

@olivervogel olivervogel commented Apr 11, 2026

This PR makes the necessary changes to migrate to the just released Intervention Image Version 4.

Please note that the new version requires at least PHP 8.3.

I recommend a thorough review. However, I think this is a good start and takes care of most of the work for the update.

If you have any questions, feel free to reach out to me.

PS: For your reference, here is the upgrade guide

Reviewers should focus on:

Necessity

  • For core PRs, does this need to be in core, or could it be in an extension?
  • Are we willing to maintain this for years / potentially forever?

Confirmed

  • Frontend changes: tested on a local Flarum installation.
  • Backend changes: tests are green (run composer test).
  • Core developer confirmed locally this works as intended.
  • Tests have been added, or are not appropriate here.

@olivervogel olivervogel requested a review from a team as a code owner April 11, 2026 08:17
@imorland imorland changed the title Migrate to Intervention Image v4 [2.x] Migrate to Intervention Image v4 Apr 11, 2026
@imorland
Copy link
Copy Markdown
Member

Thanks for this Oliver — the migration work looks good and this is something we definitely want to do.

The blocker is that Intervention Image is used directly by extensions in the ecosystem — fof/upload being a notable example. The v3 → v4 API changes mean this would be a silent breaking change for any extension that instantiates ImageManager or uses v3 methods directly, and we can't coordinate that at a minor version bump.

We're going to add this to the 3.0 roadmap where we can signal the breaking change properly and give extension authors time to migrate alongside us. We'll keep this PR open for reference when that work begins.

@imorland imorland added this to the 3.0 milestone Apr 11, 2026
@imorland imorland changed the title [2.x] Migrate to Intervention Image v4 [3.x] Migrate to Intervention Image v4 Apr 11, 2026
@imorland imorland added dependencies Pull requests that update a dependency file org/keep Issues we want to keep open breaking-changes extension-breaking labels Apr 11, 2026
@olivervogel
Copy link
Copy Markdown
Author

We're going to add this to the 3.0 roadmap where we can signal the breaking change properly and give extension authors time to migrate alongside us. We'll keep this PR open for reference when that work begins.

Okay, I assumed it would still work in 2.0, since it's only in beta so far.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking-changes dependencies Pull requests that update a dependency file extension-breaking org/keep Issues we want to keep open

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants