Skip to content

Symfony 8#160

Merged
artgris merged 2 commits intoartgris:masterfrom
kpeu3u:symfony-8
Mar 5, 2026
Merged

Symfony 8#160
artgris merged 2 commits intoartgris:masterfrom
kpeu3u:symfony-8

Conversation

@kpeu3u
Copy link
Contributor

@kpeu3u kpeu3u commented Jan 20, 2026

Add support for Symfony 8

@pfpro
Copy link
Contributor

pfpro commented Feb 9, 2026

Hi Artgris, is there any chance you're working on the Symfony 8 topic? Your bundle is really helpful, and it would be a shame if this were the end, even though CK Editor 4 is also being discontinued.

@pfpro
Copy link
Contributor

pfpro commented Feb 15, 2026

https://github.com/jolicode/MediaBundle is the way

pfpro added a commit to pfpro/FileManagerBundle that referenced this pull request Mar 4, 2026
- Update Routing\Annotation\Route to Routing\Attribute\Route
- Add ^8.0 to all Symfony dependency constraints

Based on artgris#160

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
pfpro added a commit to pfpro/FileManagerBundle that referenced this pull request Mar 4, 2026
- Update Routing\Annotation\Route to Routing\Attribute\Route
- Add ^8.0 to all Symfony dependency constraints

Based on artgris#160

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@pfpro
Copy link
Contributor

pfpro commented Mar 4, 2026

... one more try: i created a new pull request, which covers the migration to symfony 8.0. please merge it:

#161

artgris pushed a commit that referenced this pull request Mar 5, 2026
* feat: add Symfony 8.0 support

- Update Routing\Annotation\Route to Routing\Attribute\Route
- Add ^8.0 to all Symfony dependency constraints

Based on #160

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* fix: resolve Symfony 7.2+ deprecations (Extension class, null handling)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* fix: replace removed Request::get() with Request::query->get()

Request::get() was deprecated in Symfony 6 and removed in Symfony 7,
causing HTTP 500 on the file manager page.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
@artgris artgris merged commit 106a18b into artgris:master Mar 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants