Skip to content

Commit 1012b5e

Browse files
committed
phpstan issues
1 parent bab5b57 commit 1012b5e

File tree

7 files changed

+13
-15
lines changed

7 files changed

+13
-15
lines changed

phpstan.neon

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -75,6 +75,3 @@ parameters:
7575
}
7676
'''
7777
inferPrivatePropertyTypeFromConstructor: true
78-
ignoreErrors:
79-
# False positive
80-
- '#Call to an undefined method Symfony\\Component\\Config\\Definition\\Builder\\NodeDefinition::children\(\)\.#'

src/AttributeGenerator/ApiPlatformCoreAttributeGenerator.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -148,6 +148,8 @@ private static function extractParameters(string $type, array $values): array
148148
],
149149
[]
150150
);
151+
152+
// @phpstan-ignore-next-line
151153
if (isset(self::$parameterTypes[$type])) {
152154
$types = self::$parameterTypes[$type];
153155
} else {

src/Command/GenerateCommand.php

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -130,13 +130,13 @@ protected function execute(InputInterface $input, OutputInterface $output): int
130130
$configuration = $this->processConfiguration($configContent, $outputDir, $dir === $defaultOutput ? $this->namespacePrefix : null);
131131

132132
(new SchemaGenerator())->generate($configuration, $output, $io);
133-
(new OpenApiGenerator())->generate($configuration, $configArgument ?? self::DEFAULT_CONFIG_FILE, $output, $io);
133+
(new OpenApiGenerator())->generate($configuration, $configArgument ?? self::DEFAULT_CONFIG_FILE, $output, $io); // @phpstan-ignore-line
134134

135135
return Command::SUCCESS;
136136
}
137137

138138
/**
139-
* @return Configuration
139+
* @return array<string, mixed>
140140
*/
141141
private function processConfiguration(string $configContent, string $outputDir, ?string $defaultNamespacePrefix): array
142142
{
@@ -146,7 +146,6 @@ private function processConfiguration(string $configContent, string $outputDir,
146146

147147
$processor = new Processor();
148148
$configuration = new SchemaGeneratorConfiguration($defaultNamespacePrefix);
149-
/** @var Configuration $processedConfiguration */
150149
$processedConfiguration = $processor->processConfiguration($configuration, [$config]);
151150
$processedConfiguration['output'] = $outputDir;
152151
if (!$processedConfiguration['output']) {

src/OpenApi/Generator.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@
2626
final class Generator
2727
{
2828
/**
29-
* @param Configuration $configuration
29+
* @param array<string, mixed> $configuration
3030
*/
3131
public function generate(array $configuration, string $configurationPath, OutputInterface $output, SymfonyStyle $io): void
3232
{
@@ -51,12 +51,12 @@ public function generate(array $configuration, string $configurationPath, Output
5151

5252
$classGenerator = new ClassGenerator($inflector, new PhpTypeConverter());
5353
$classGenerator->setLogger($logger);
54-
$classes = $classGenerator->generate($openApi, $configuration);
54+
$classes = $classGenerator->generate($openApi, $configuration); // @phpstan-ignore-line
5555

56-
$twig = (new TwigBuilder())->build($configuration);
56+
$twig = (new TwigBuilder())->build($configuration); // @phpstan-ignore-line
5757

5858
$filesGenerator = new FilesGenerator($inflector, new Printer(), $twig, $io);
5959
$filesGenerator->setLogger($logger);
60-
$filesGenerator->generate($classes, $configuration);
60+
$filesGenerator->generate($classes, $configuration); // @phpstan-ignore-line
6161
}
6262
}

src/Schema/Generator.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@
2929
final class Generator
3030
{
3131
/**
32-
* @param Configuration $configuration
32+
* @param array<string, mixed> $configuration
3333
*/
3434
public function generate(array $configuration, OutputInterface $output, SymfonyStyle $io): void
3535
{
@@ -66,12 +66,12 @@ public function generate(array $configuration, OutputInterface $output, SymfonyS
6666
);
6767
$entitiesGenerator->setLogger($logger);
6868

69-
$classes = $entitiesGenerator->generate($graphs, $configuration);
69+
$classes = $entitiesGenerator->generate($graphs, $configuration); // @phpstan-ignore-line
7070

71-
$twig = (new TwigBuilder())->build($configuration);
71+
$twig = (new TwigBuilder())->build($configuration); // @phpstan-ignore-line
7272

7373
$filesGenerator = new FilesGenerator($inflector, new Printer(), $twig, $io);
7474
$filesGenerator->setLogger($logger);
75-
$filesGenerator->generate($classes, $configuration);
75+
$filesGenerator->generate($classes, $configuration); // @phpstan-ignore-line
7676
}
7777
}

src/SchemaGeneratorConfiguration.php

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,6 @@ public function getConfigTreeBuilder(): TreeBuilder
5252
array_keys($nodeConfig)
5353
);
5454

55-
// @phpstan-ignore-next-line node is not null
5655
$attributesNode = static fn () => (new NodeBuilder())
5756
->arrayNode('attributes')
5857
->info('Attributes (merged with generated attributes)')

src/TypesGenerator.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -174,6 +174,7 @@ public function generate(array $graphs, array $config): array
174174
?? ($class->hasChild && !$class->isReferencedBy);
175175

176176
// When including all properties, ignore properties already set on parent
177+
// @phpstan-ignore-next-line
177178
if (($config['types'][$class->name()]['allProperties'] ?? true) && isset($classes[$class->parent()])) {
178179
$type = $class->resource();
179180
foreach ($propertiesMap[$type->getUri()] as $property) {

0 commit comments

Comments
 (0)