Skip to content

Commit 6597978

Browse files
authored
Merge pull request #5 from halloverden/feature/allow-symfony-6
Feature/allow symfony 6
2 parents 4a8ff5b + 2a8ca93 commit 6597978

File tree

5 files changed

+255
-498
lines changed

5 files changed

+255
-498
lines changed

EventListener/ExceptionListener.php

+6-26
Original file line numberDiff line numberDiff line change
@@ -13,34 +13,14 @@
1313

1414
class ExceptionListener implements EventSubscriberInterface {
1515

16-
/**
17-
* @var ExceptionConverterServiceInterface
18-
*/
19-
private $exceptionConverterService;
20-
21-
/**
22-
* @var ExceptionLogServiceInterface
23-
*/
24-
private $logExceptionService;
25-
26-
/**
27-
* @var ExceptionResponseServiceInterface
28-
*/
29-
private $exceptionResponseService;
30-
3116
/**
3217
* ExceptionListener constructor.
33-
*
34-
* @param ExceptionConverterServiceInterface $exceptionConverterService
35-
* @param ExceptionLogServiceInterface $logExceptionService
36-
* @param ExceptionResponseServiceInterface $exceptionResponseService
3718
*/
38-
public function __construct(ExceptionConverterServiceInterface $exceptionConverterService,
39-
ExceptionLogServiceInterface $logExceptionService,
40-
ExceptionResponseServiceInterface $exceptionResponseService) {
41-
$this->exceptionConverterService = $exceptionConverterService;
42-
$this->logExceptionService = $logExceptionService;
43-
$this->exceptionResponseService = $exceptionResponseService;
19+
public function __construct(
20+
private readonly ExceptionConverterServiceInterface $exceptionConverterService,
21+
private readonly ExceptionLogServiceInterface $logExceptionService,
22+
private readonly ExceptionResponseServiceInterface $exceptionResponseService
23+
) {
4424
}
4525

4626
/**
@@ -82,7 +62,7 @@ public function onKernelException(ExceptionEvent $event) {
8262
/**
8363
* @inheritDoc
8464
*/
85-
public static function getSubscribedEvents() {
65+
public static function getSubscribedEvents(): array {
8666
return array(
8767
KernelEvents::EXCEPTION => [
8868
['convertKernelException', -1],

Helpers/FlattenExceptionHelper.php

+3-3
Original file line numberDiff line numberDiff line change
@@ -16,14 +16,14 @@
1616
class FlattenExceptionHelper extends FlattenException {
1717

1818
/**
19-
* @var array<int, FlattenException>
19+
* @var array<int, FlattenExceptionHelper>
2020
*/
21-
private static $exceptions = [];
21+
private static array $exceptions = [];
2222

2323
/**
2424
* @inheritDoc
2525
*/
26-
public static function createFromThrowable(\Throwable $exception, int $statusCode = null, array $headers = []): FlattenException {
26+
public static function createFromThrowable(\Throwable $exception, int $statusCode = null, array $headers = []): static {
2727
$id = \spl_object_id($exception);
2828

2929
if (isset(static::$exceptions[$id])) {

Services/ExceptionResponseService.php

+1-1
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ public function __construct(bool $debug = false) {
3434
*/
3535
public function createResponseFromHttpException(HttpExceptionInterface $exception): Response {
3636
// Note: Encoding options needs to be set before data
37-
return JsonResponse::create(null, $exception->getStatusCode(), $exception->getHeaders())
37+
return (new JsonResponse(null, $exception->getStatusCode(), $exception->getHeaders()))
3838
->setEncodingOptions(JsonResponse::DEFAULT_ENCODING_OPTIONS | JSON_INVALID_UTF8_IGNORE)
3939
->setData($this->getData($exception));
4040
}

composer.json

+5-5
Original file line numberDiff line numberDiff line change
@@ -16,12 +16,12 @@
1616
"minimum-stability": "dev",
1717
"prefer-stable": true,
1818
"require": {
19-
"php":">=7.4",
20-
"symfony/http-kernel": "^v4.4.13|^5.2",
21-
"symfony/dependency-injection": "^v4.4.13|^5.2",
22-
"halloverden/symfony-http-exceptions": "^1.1.0",
19+
"php":">=8.1",
20+
"symfony/http-kernel": "^5.4|^6.1",
21+
"symfony/dependency-injection": "^5.4|^6.1",
22+
"halloverden/symfony-http-exceptions": "^1.0|^2.0",
2323
"ext-json": "*",
24-
"symfony/config": "^v4.4.13|^v5.2"
24+
"symfony/config": "^5.4|^6.1"
2525
},
2626
"autoload": {
2727
"psr-4": {

0 commit comments

Comments
 (0)