Skip to content

Commit a98e260

Browse files
arjanfransDominicLuidold
authored andcommitted
misc: Fix codestyle errors + require php >=8.2 for all extensions
1 parent d76e275 commit a98e260

File tree

8 files changed

+14
-13
lines changed

8 files changed

+14
-13
lines changed

Diff for: README.md

+2-2
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
[![License](https://img.shields.io/packagist/l/fusonic/api-documentation-bundle?color=blue)](https://github.com/fusonic/php-api-documentation-bundle/blob/master/LICENSE)
44
[![Latest Version](https://img.shields.io/github/tag/fusonic/php-api-documentation-bundle.svg?color=blue)](https://github.com/fusonic/php-api-documentation-bundle/releases)
55
[![Total Downloads](https://img.shields.io/packagist/dt/fusonic/api-documentation-bundle.svg?color=blue)](https://packagist.org/packages/fusonic/api-documentation-bundle)
6-
![php 8.1+](https://img.shields.io/badge/php-min%208.1-blue.svg)
6+
![php 8.2+](https://img.shields.io/badge/php-min%208.1-blue.svg)
77

88
* [About](#about)
99
* [Install](#install)
@@ -34,7 +34,7 @@ composer require fusonic/api-documentation-bundle
3434

3535
Requirements:
3636

37-
- PHP 8.1+
37+
- PHP 8.2+
3838
- Symfony 5.4+
3939

4040
In case Symfony did not add the bundle to the bundle configuration, add the following (by default located

Diff for: composer.json

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
{
22
"name": "fusonic/api-documentation-bundle",
33
"license": "MIT",
4-
"version": "0.0.16",
4+
"version": "0.1.0",
55
"description": "Symfony bundle for automated documentation with NelmioApiDocBundle.",
66
"type": "symfony-bundle",
77
"authors": [
@@ -38,7 +38,7 @@
3838
"tomasvotruba/type-coverage": "^0.3"
3939
},
4040
"require": {
41-
"php": ">=8.1",
41+
"php": ">=8.2",
4242
"nelmio/api-doc-bundle": "^4.11",
4343
"symfony/config": "^5.4 || ^6.0 || ^7.0",
4444
"symfony/dependency-injection": "^5.4 || ^6.0 || ^7.0",

Diff for: phpstan.neon

+1
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ parameters:
99
return: 100
1010
param: 100
1111
property: 100
12+
constant: 0 # FIXME change to 100 once everything requires at least PHP 8.3
1213

1314
includes:
1415
- vendor/phpstan/phpstan-deprecation-rules/rules.neon

Diff for: src/AnnotationBuilder/AnnotationBuilder.php

+2-2
Original file line numberDiff line numberDiff line change
@@ -221,7 +221,7 @@ public function getOutputAnnotation(string $httpMethod): ?AbstractAnnotation
221221

222222
private function builtinTypeDescription(string $httpMethod): string
223223
{
224-
return sprintf(
224+
return \sprintf(
225225
'%s %s%s',
226226
$httpMethod,
227227
$this->output,
@@ -235,7 +235,7 @@ private function classTypeDescription(string $httpMethod): string
235235
$output = $this->output;
236236
$outputClassBasename = (new \ReflectionClass($output))->getShortName();
237237

238-
return sprintf(
238+
return \sprintf(
239239
'%s %s%s',
240240
$httpMethod,
241241
$outputClassBasename,

Diff for: src/DependencyInjection/Compiler/ConfigurationCompilerPass.php

+4-4
Original file line numberDiff line numberDiff line change
@@ -21,22 +21,22 @@ final class ConfigurationCompilerPass implements CompilerPassInterface
2121
public function process(ContainerBuilder $container): void
2222
{
2323
if (!$container->hasExtension('nelmio_api_doc')) {
24-
throw new LogicException(sprintf('%s is not configured.', NelmioApiDocBundle::class));
24+
throw new LogicException(\sprintf('%s is not configured.', NelmioApiDocBundle::class));
2525
}
2626

2727
/** @var string[] $areas */
2828
$areas = $container->getParameter('nelmio_api_doc.areas');
2929

3030
foreach ($areas as $area) {
31-
$container->register(sprintf('fusonic_api_documentation.describers.openapi_php.%s', $area), DocumentedRouteDescriber::class)
31+
$container->register(\sprintf('fusonic_api_documentation.describers.openapi_php.%s', $area), DocumentedRouteDescriber::class)
3232
->setPublic(false)
3333
->setArguments([
34-
new Reference(sprintf('nelmio_api_doc.routes.%s', $area)),
34+
new Reference(\sprintf('nelmio_api_doc.routes.%s', $area)),
3535
new Reference('nelmio_api_doc.controller_reflector'),
3636
new Reference('logger'),
3737
$container->getParameter('fusonic_api_documentation.request_object_class'),
3838
])
39-
->addTag(sprintf('nelmio_api_doc.describer.%s', $area), ['priority' => -199]);
39+
->addTag(\sprintf('nelmio_api_doc.describer.%s', $area), ['priority' => -199]);
4040
}
4141
}
4242
}

Diff for: src/Describer/DocumentedRouteDescriber.php

+1-1
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ public function __construct(
4343
?string $requestObjectClass = null,
4444
) {
4545
if (null !== $requestObjectClass && !(class_exists($requestObjectClass) || interface_exists($requestObjectClass))) {
46-
throw new \InvalidArgumentException(sprintf('Class %s does not exist.', $requestObjectClass));
46+
throw new \InvalidArgumentException(\sprintf('Class %s does not exist.', $requestObjectClass));
4747
} elseif (null !== $requestObjectClass) {
4848
$this->requestObjectReflectionClass = new \ReflectionClass($requestObjectClass);
4949
}

Diff for: src/Exception/DuplicateAttributesException.php

+1-1
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,6 @@ final class DuplicateAttributesException extends \LogicException
1313
{
1414
public function __construct(string $attributeClass)
1515
{
16-
parent::__construct(sprintf('Attribute `%s` can only be set once.', $attributeClass));
16+
parent::__construct(\sprintf('Attribute `%s` can only be set once.', $attributeClass));
1717
}
1818
}

Diff for: src/Extractor/MethodReturnTypePhpDocExtractor.php

+1-1
Original file line numberDiff line numberDiff line change
@@ -104,7 +104,7 @@ public function getTypes(string $class, string $property, array $context = []):
104104

105105
private function getDocBlock(string $class, string $property): ?DocBlock
106106
{
107-
$propertyHash = sprintf('%s::%s', $class, $property);
107+
$propertyHash = \sprintf('%s::%s', $class, $property);
108108

109109
if (isset($this->docBlocks[$propertyHash])) {
110110
return $this->docBlocks[$propertyHash];

0 commit comments

Comments
 (0)