Skip to content

Commit 1ee472f

Browse files
pfproclaude
andauthored
Add Symfony 7.2 / 8.0 support (#161)
* 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>
1 parent 5df601c commit 1ee472f

4 files changed

Lines changed: 21 additions & 21 deletions

File tree

Controller/ManagerController.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@
3131
use Symfony\Component\HttpFoundation\Request;
3232
use Symfony\Component\HttpFoundation\Response;
3333
use Symfony\Component\HttpKernel\Exception\BadRequestHttpException;
34-
use Symfony\Component\Routing\Annotation\Route;
34+
use Symfony\Component\Routing\Attribute\Route;
3535
use Symfony\Component\Routing\RouterInterface;
3636
use Symfony\Component\Validator\Constraints\NotBlank;
3737
use Symfony\Contracts\Translation\TranslatorInterface;
@@ -53,7 +53,7 @@ public function __construct(private FilemanagerService $filemanagerService, priv
5353
#[Route('/', name: 'file_manager')]
5454
public function indexAction(Request $request, FileTypeService $fileTypeService): JsonResponse|Response {
5555
$queryParameters = $request->query->all();
56-
$isJson = $request->get('json');
56+
$isJson = $request->query->get('json');
5757
if ($isJson) {
5858
unset($queryParameters['json']);
5959
}

DependencyInjection/ArtgrisFileManagerExtension.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
use Symfony\Component\Config\FileLocator;
66
use Symfony\Component\DependencyInjection\ContainerBuilder;
77
use Symfony\Component\DependencyInjection\Loader;
8-
use Symfony\Component\HttpKernel\DependencyInjection\Extension;
8+
use Symfony\Component\DependencyInjection\Extension\Extension;
99

1010
/**
1111
* @author Arthur Gribet <a.gribet@gmail.com>

Helpers/FileManager.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -151,7 +151,7 @@ public function getRoute(): ?string
151151

152152
public function getRoutePath(): ?string
153153
{
154-
return implode('/', array_map('rawurlencode', explode('/', $this->getRoute())));
154+
return implode('/', array_map('rawurlencode', explode('/', $this->getRoute() ?? '')));
155155
}
156156

157157
public function getBasePath(): bool|string

composer.json

Lines changed: 17 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -18,27 +18,27 @@
1818
"require": {
1919
"php": ">=8.1",
2020
"oomphinc/composer-installers-extender": "^2.0",
21-
"symfony/asset": "^6.0||^7.0",
22-
"symfony/event-dispatcher": "^6.0||^7.0",
23-
"symfony/expression-language": "^6.0||^7.0",
21+
"symfony/asset": "^6.0||^7.0||^8.0",
22+
"symfony/event-dispatcher": "^6.0||^7.0||^8.0",
23+
"symfony/expression-language": "^6.0||^7.0||^8.0",
2424
"symfony/flex": "^2.3",
25-
"symfony/form": "^6.0||^7.0",
26-
"symfony/framework-bundle": "^6.0||^7.0",
27-
"symfony/mime": "^6.0||^7.0",
28-
"symfony/string": "^6.0||^7.0",
29-
"symfony/translation": "^6.0||^7.0",
30-
"symfony/twig-bridge": "^6.0||^7.0",
31-
"symfony/twig-bundle": "^6.0||^7.0",
32-
"symfony/validator": "^6.0||^7.0"
25+
"symfony/form": "^6.0||^7.0||^8.0",
26+
"symfony/framework-bundle": "^6.0||^7.0||^8.0",
27+
"symfony/mime": "^6.0||^7.0||^8.0",
28+
"symfony/string": "^6.0||^7.0||^8.0",
29+
"symfony/translation": "^6.0||^7.0||^8.0",
30+
"symfony/twig-bridge": "^6.0||^7.0||^8.0",
31+
"symfony/twig-bundle": "^6.0||^7.0||^8.0",
32+
"symfony/validator": "^6.0||^7.0||^8.0"
3333
},
3434
"require-dev": {
3535
"phpunit/phpunit": "^9.5",
36-
"symfony/browser-kit": "^6.0||^7.0",
37-
"symfony/css-selector": "^6.0||^7.0",
38-
"symfony/phpunit-bridge": "^6.0||^7.0",
39-
"symfony/templating": "^6.0||^7.0",
40-
"symfony/var-dumper": "^6.0||^7.0",
41-
"symfony/yaml": "^6.0||^7.0"
36+
"symfony/browser-kit": "^6.0||^7.0||^8.0",
37+
"symfony/css-selector": "^6.0||^7.0||^8.0",
38+
"symfony/phpunit-bridge": "^6.0||^7.0||^8.0",
39+
"symfony/templating": "^6.0||^7.0||^8.0",
40+
"symfony/var-dumper": "^6.0||^7.0||^8.0",
41+
"symfony/yaml": "^6.0||^7.0||^8.0"
4242
},
4343
"config": {
4444
"sort-packages": true,

0 commit comments

Comments
 (0)