Skip to content

Commit

Permalink
Remove psalm suppression according vimeo/psalm#10382
Browse files Browse the repository at this point in the history
  • Loading branch information
ostrolucky committed Jan 28, 2024
1 parent 8a6ccdf commit 28a4fe1
Showing 1 changed file with 8 additions and 3 deletions.
11 changes: 8 additions & 3 deletions DependencyInjection/DoctrineExtension.php
Expand Up @@ -82,6 +82,12 @@
* DoctrineExtension is an extension for the Doctrine DBAL and ORM library.
*
* @final since 2.9
* @psalm-type dbalConfig = {
* connections: array<string, array{logging: bool, profiling: bool, profiling_collect_backtrace: bool}>,
* driver_schemes: array<string, string>,
* default_connection: string,
* types: array<string, string>,
* }
*/
class DoctrineExtension extends AbstractDoctrineExtension

Check failure on line 92 in DependencyInjection/DoctrineExtension.php

View workflow job for this annotation

GitHub Actions / Static Analysis with Psalm

InvalidDocblock

DependencyInjection/DoctrineExtension.php:92:1: InvalidDocblock: Unexpected token { (see https://psalm.dev/008)
{
Expand Down Expand Up @@ -155,8 +161,8 @@ private function processConfigurationPrependingDefaults(ConfigurationInterface $
*
* <doctrine:dbal id="myconn" dbname="sfweb" user="root" />
*
* @param array<string, mixed> $config An array of configuration settings
* @param ContainerBuilder $container A ContainerBuilder instance
* @param dbalConfig $config An array of configuration settings

Check failure on line 164 in DependencyInjection/DoctrineExtension.php

View workflow job for this annotation

GitHub Actions / Static Analysis with Psalm

UndefinedDocblockClass

DependencyInjection/DoctrineExtension.php:164:15: UndefinedDocblockClass: Docblock-defined class, interface or enum named Doctrine\Bundle\DoctrineBundle\DependencyInjection\dbalConfig does not exist (see https://psalm.dev/200)

Check failure on line 164 in DependencyInjection/DoctrineExtension.php

View workflow job for this annotation

GitHub Actions / Static Analysis with Psalm

MismatchingDocblockParamType

DependencyInjection/DoctrineExtension.php:164:15: MismatchingDocblockParamType: Parameter $config has wrong type 'Doctrine\Bundle\DoctrineBundle\DependencyInjection\dbalConfig', should be 'array<array-key, mixed>' (see https://psalm.dev/141)
* @param ContainerBuilder $container A ContainerBuilder instance
*/
protected function dbalLoad(array $config, ContainerBuilder $container)
{
Expand All @@ -181,7 +187,6 @@ protected function dbalLoad(array $config, ContainerBuilder $container)
$connections = [];

foreach (array_keys($config['connections']) as $name) {
/** @psalm-suppress InvalidArrayOffset https://github.com/vimeo/psalm/issues/10382 */
$connections[$name] = sprintf('doctrine.dbal.%s_connection', $name);
}

Expand Down

0 comments on commit 28a4fe1

Please sign in to comment.