Your project should not use dependencies with incompatible licenses

More information: https://insight.symfony.com/what-we-analyse/composer.incompatible_license

The dependency "mpdf/mpdf" uses a copyleft license (GPL-2.0-only) which is incompatible with your project license (MIT).
Time to fix: about 2 days
Read doc Open Issue Permalink Copy Prompt
Collective

Your project controllers should only contains actions as public methods

More information: https://insight.symfony.com/what-we-analyse/symfony.controller.non_action_public_method_in_controller_class

in src/Controller/Crudit/ConfigController.php, line 29 
  1.         private EntityManagerInterface $em,
  2.     ) {
  3.         $this->config = $config;
  4.     }
  6.     #[Route('/refresh-cache')]
    
    
    Non-action public method found in controller class
    
    
    
                    
    Time to fix: about 1 hour
    
                Read doc
                                    
                
                Open Issue
            
                 Permalink
         Copy Prompt
    
    

            
    
            
    
                                    Last edited  by Nathan
                    
                            
    
        
    
    
  7.     public function refreshCache(): Response
  8.     {
  9.         $this->denyAccessUnlessGranted('ROLE_CONFIG_REFRESH_CACHE');
  11.         $configs = $this->em->getRepository(ConfigInterface::class)

Your project must not contain invalid function or method calls 10

More information: https://insight.symfony.com/what-we-analyse/php.invalid_call

in src/Command/WarmupCommand.php, line 34 
  1.     {
  2.         /** @var AbstractConfigRepository $configRepository */
  3.         $configRepository = $this->em->getRepository(ConfigInterface::class);
  5.         foreach ($this->warmups as $warmup) {
  6.             $warmup->warmup($configRepository);
    
    
    Cannot call method warmup() on mixed.
    
    
    
                    
    Time to fix: about 9 minutes
    
                Read doc
                                    
                
                Open Issue
            
                 Permalink
         Copy Prompt
    
    

            
    
            
    
                                    Last edited  by Nathan
                    
                            
    
        
    
    
  7.         }
  9.         $this->em->flush();
  11.         $io = new SymfonyStyle($input, $output);
in src/Crudit/Datasource/ConfigDatasource.php, line 118 
  1.                     ->setValueString($resource->getValueString())
  2.                     ->setValueText($resource->getValueText())
  3.                     ->setValueInt($resource->getValueInt())
  4.                     ->setValueFloat($resource->getValueFloat())
  5.                     ->setTri($resource->getTri())
  6.                     ->setTenantId($this->tenantService ? $this->tenantService->getTenantId() : null);
    
    
    Parameter #1 $tenantId of method Lle\ConfigBundle\Contracts\ConfigInterface::setTenantId() expects int, int|null given.
    
    
    
                    
    Time to fix: about 9 minutes
    
                Read doc
                                    
                
                Open Issue
            
                 Permalink
         Copy Prompt
    
    

            
    
            
    
                                    Last edited  by Artahs
                    
                            
    
        
    
    
  8.                 $this->entityManager->refresh($resource);
  10.                 parent::save($config);
in src/DependencyInjection/LleConfigExtension.php, line 28 
  1.         $loader->load("services.yaml");
  3.         $configuration = new Configuration();
  4.         $processedConfig = $this->processConfiguration($configuration, $configs);
  6.         $container->setParameter('lle_config.using_tenant', $processedConfig['using_tenant']);
    
    
    Parameter #2 $value of method Symfony\Component\DependencyInjection\Container::setParameter() expects array|bool|float|int|string|UnitEnum|null, mixed given.
    
    
    
                    
    Time to fix: about 9 minutes
    
                Read doc
                                    
                
                Open Issue
            
                 Permalink
         Copy Prompt
    
    

            
    
            
    
                                    Last edited  by Valentin HELIES
                    
                            
    
        
    
    
  7.         if ($processedConfig['tenant_service']) {
  8.             $container->setAlias('lle_config.tenant_service', $processedConfig['tenant_service']);
  9.         }
  10.     }
  11. }
in src/DependencyInjection/LleConfigExtension.php, line 30 
  1.         $configuration = new Configuration();
  2.         $processedConfig = $this->processConfiguration($configuration, $configs);
  4.         $container->setParameter('lle_config.using_tenant', $processedConfig['using_tenant']);
  5.         if ($processedConfig['tenant_service']) {
  6.             $container->setAlias('lle_config.tenant_service', $processedConfig['tenant_service']);
    
    
    Parameter #2 $id of method Symfony\Component\DependencyInjection\ContainerBuilder::setAlias() expects string|Symfony\Component\DependencyInjection\Alias, mixed given.
    
    
    
                    
    Time to fix: about 9 minutes
    
                Read doc
                                    
                
                Open Issue
            
                 Permalink
         Copy Prompt
    
    

            
    
            
    
                                    Last edited  by valentin-helies
                    
                            
    
        
    
    
  7.         }
  8.     }
  9. }
in src/Repository/AbstractConfigRepository.php, line 385 
  1.         $item = new $configClass();
  2.         $item
  3.             ->setGroup($group)
  4.             ->setLabel($label)
  5.             ->setValueType($valueType)
  6.             ->setTenantId($tenantId);
    
    
    Parameter #1 $tenantId of method Lle\ConfigBundle\Contracts\ConfigInterface::setTenantId() expects int, int|null given.
    
    
    
                    
    Time to fix: about 9 minutes
    
                Read doc
                                    
                
                Open Issue
            
                 Permalink
         Copy Prompt
    
    

            
    
            
    
                                    Last edited  by Artahs
                    
                            
    
        
    
    
  8.         return $item;
  9.     }
  11.     #[Required]
in src/Twig/ConfigExtension.php, line 31 
  1.         /** @var AbstractConfigRepository $configRepository */
  2.         $configRepository = $this->em->getRepository(ConfigInterface::class);
  4.         switch ($type) {
  5.             case ConfigInterface::BOOL:
  6.                 return $configRepository->getBool($group, $name, $default);
    
    
    Parameter #3 $default of method Lle\ConfigBundle\Repository\AbstractConfigRepository::getBool() expects bool, mixed given.
    
    
    
                    
    Time to fix: about 9 minutes
    
                Read doc
                                    
                
                Open Issue
            
                 Permalink
         Copy Prompt
    
    

            
    
            
    
                                    Last edited  by Quentin
                    
                            
    
        
    
    
  7.             case ConfigInterface::STRING:
  8.                 return $configRepository->getString($group, $name, $default);
  9.             case ConfigInterface::TEXT:
  10.                 return $configRepository->getText($group, $name, $default);
  11.             case ConfigInterface::INT:
in src/Twig/ConfigExtension.php, line 33 
  2.         switch ($type) {
  3.             case ConfigInterface::BOOL:
  4.                 return $configRepository->getBool($group, $name, $default);
  5.             case ConfigInterface::STRING:
  6.                 return $configRepository->getString($group, $name, $default);
    
    
    Parameter #3 $default of method Lle\ConfigBundle\Repository\AbstractConfigRepository::getString() expects string, mixed given.
    
    
    
                    
    Time to fix: about 9 minutes
    
                Read doc
                                    
                
                Open Issue
            
                 Permalink
         Copy Prompt
    
    

            
    
            
    
                                    Last edited  by Quentin
                    
                            
    
        
    
    
  7.             case ConfigInterface::TEXT:
  8.                 return $configRepository->getText($group, $name, $default);
  9.             case ConfigInterface::INT:
  10.                 return $configRepository->getInt($group, $name, $default);
  11.             case ConfigInterface::FLOAT:
in src/Twig/ConfigExtension.php, line 35 
  1.             case ConfigInterface::BOOL:
  2.                 return $configRepository->getBool($group, $name, $default);
  3.             case ConfigInterface::STRING:
  4.                 return $configRepository->getString($group, $name, $default);
  5.             case ConfigInterface::TEXT:
  6.                 return $configRepository->getText($group, $name, $default);
    
    
    Parameter #3 $default of method Lle\ConfigBundle\Repository\AbstractConfigRepository::getText() expects string, mixed given.
    
    
    
                    
    Time to fix: about 9 minutes
    
                Read doc
                                    
                
                Open Issue
            
                 Permalink
         Copy Prompt
    
    

            
    
            
    
                                    Last edited  by Quentin
                    
                            
    
        
    
    
  7.             case ConfigInterface::INT:
  8.                 return $configRepository->getInt($group, $name, $default);
  9.             case ConfigInterface::FLOAT:
  10.                 return $configRepository->getFloat($group, $name, $default);
  11.             default:
in src/Twig/ConfigExtension.php, line 37 
  1.             case ConfigInterface::STRING:
  2.                 return $configRepository->getString($group, $name, $default);
  3.             case ConfigInterface::TEXT:
  4.                 return $configRepository->getText($group, $name, $default);
  5.             case ConfigInterface::INT:
  6.                 return $configRepository->getInt($group, $name, $default);
    
    
    Parameter #3 $default of method Lle\ConfigBundle\Repository\AbstractConfigRepository::getInt() expects int, mixed given.
    
    
    
                    
    Time to fix: about 9 minutes
    
                Read doc
                                    
                
                Open Issue
            
                 Permalink
         Copy Prompt
    
    

            
    
            
    
                                    Last edited  by Quentin
                    
                            
    
        
    
    
  7.             case ConfigInterface::FLOAT:
  8.                 return $configRepository->getFloat($group, $name, $default);
  9.             default:
  10.                 return null;
  11.         }
in src/Twig/ConfigExtension.php, line 39 
  1.             case ConfigInterface::TEXT:
  2.                 return $configRepository->getText($group, $name, $default);
  3.             case ConfigInterface::INT:
  4.                 return $configRepository->getInt($group, $name, $default);
  5.             case ConfigInterface::FLOAT:
  6.                 return $configRepository->getFloat($group, $name, $default);
    
    
    Parameter #3 $default of method Lle\ConfigBundle\Repository\AbstractConfigRepository::getFloat() expects float, mixed given.
    
    
    
                    
    Time to fix: about 9 minutes
    
                Read doc
                                    
                
                Open Issue
            
                 Permalink
         Copy Prompt
    
    

            
    
            
    
                                    Last edited  by Nathan
                    
                            
    
        
    
    
  7.             default:
  8.                 return null;
  9.         }
  10.     }
  11. }

Your project should not use invalid parameter and return typehints

More information: https://insight.symfony.com/what-we-analyse/php.invalid_typehint

in src/Command/WarmupCommand.php, line 19 
  1. use Symfony\Component\DependencyInjection\Attribute\AutowireIterator;
  3. #[AsCommand('lle:config:warmup')]
  4. class WarmupCommand extends Command
  5. {
  6.     public function __construct(
    
    
    Method Lle\ConfigBundle\Command\WarmupCommand::__construct() has parameter $warmups with no value type specified in iterable type iterable.
    
    
    
                    
    Time to fix: about 9 minutes
    
                Read doc
                                    
                
                Open Issue
            
                 Permalink
         Copy Prompt
    
    

            
    
            
    
                                    Last edited  by Nathan
                    
                            
    
        
    
    
  7.         private EntityManagerInterface $em,
  8.         #[AutowireIterator('lle.config.warmup')]
  9.         /** @var iterable<WarmupInterface> */
  10.         private iterable $warmups,
  11.     ) {

Your project should use return types 4

More information: https://insight.symfony.com/what-we-analyse/php.missing_return_typehint

in src/Crudit/Datasource/ConfigDatasource.php, line 35 
  1.         $metadata = $this->entityManager->getClassMetadata(ConfigInterface::class);
  3.         return $metadata->name;
  4.     }
  6.     public function list(?DatasourceParams $requestParams): iterable
    
    
    Method Lle\ConfigBundle\Crudit\Datasource\ConfigDatasource::list() return type has no value type specified in iterable type iterable.
    
    
    
                    
    Time to fix: about 9 minutes
    
                Read doc
                                    
                
                Open Issue
            
                 Permalink
         Copy Prompt
    
    

            
    
            
    
                                    Last edited  by Valentin HELIES
                    
                            
    
        
    
    
  7.     {
  8.         /** @var ConfigInterface[] $list */
  9.         $list = parent::list($requestParams);
  11.         $resultWithTenant = [];
in src/Crudit/Config/ConfigCrudConfig.php, line 21 
  1.     )
  2.     {
  3.         $this->datasource = $datasource;
  4.     }
  6.     public function getFields(string $key): array
    
    
    Method Lle\ConfigBundle\Crudit\Config\ConfigCrudConfig::getFields() return type has no value type specified in iterable type array.
    
    
    
                    
    Time to fix: about 9 minutes
    
                Read doc
                                    
                
                Open Issue
            
                 Permalink
         Copy Prompt
    
    

            
    
            
    
                                    Last edited  by Bastien
                    
                            
    
        
    
    
  7.     {
  8.         $group = Field::new('group')->setRuptGroup(1);
  9.         $label = Field::new('label')->setLabel('field.code');
  10.         $transLabel = Field::new('label')
  11.             ->setTemplate('@LleConfig/_trans_label.html.twig')
in src/Crudit/Config/ConfigCrudConfig.php, line 47 
  1.     public function getNbItems(): int
  2.     {
  3.         return 500;
  4.     }
  6.     public function getListActions(): array
    
    
    Method Lle\ConfigBundle\Crudit\Config\ConfigCrudConfig::getListActions() return type has no value type specified in iterable type array.
    
    
    
                    
    Time to fix: about 9 minutes
    
                Read doc
                                    
                
                Open Issue
            
                 Permalink
         Copy Prompt
    
    

            
    
            
    
                                    Last edited  by Nathan
                    
                            
    
        
    
    
  7.     {
  8.         return [];
  9.     }
  11.     public function getItemActions(): array
in src/Crudit/Config/ConfigCrudConfig.php, line 52 
  1.     public function getListActions(): array
  2.     {
  3.         return [];
  4.     }
  6.     public function getItemActions(): array
    
    
    Method Lle\ConfigBundle\Crudit\Config\ConfigCrudConfig::getItemActions() return type has no value type specified in iterable type array.
    
    
    
                    
    Time to fix: about 9 minutes
    
                Read doc
                                    
                
                Open Issue
            
                 Permalink
         Copy Prompt
    
    

            
    
            
    
                                    Last edited  by Nathan
                    
                            
    
        
    
    
  7.     {
  8.         return [];
  9.     }
  11.     public function getRootRoute(): string

Your project uses non-strict array lookups 5

More information: https://insight.symfony.com/what-we-analyse/php.strict_array_lookup

in src/Crudit/Datasource/ConfigDatasource.php, line 49 
  1.             }
  2.         }
  4.         $result = [];
  5.         foreach ($list as $item) {
  6.             if ($item->getTenantId() || !in_array($item->getLabel() . '-' . $item->getGroup(), $resultWithTenant)) {
    
    
    in_array() should be called with the third parameter set to true to enable strict comparison and avoid type juggling bugs.
    
    
    
                    
    Time to fix: about 15 minutes
    
                Read doc
                                    
                
                Open Issue
            
                 Permalink
         Copy Prompt
    
    

            
    
            
    
                                    Last edited  by Valentin HELIES
                    
                            
    
        
    
    
  7.                 $result[] = $item;
  8.             }
  9.         }
  11.         return $result;
in src/Crudit/Datasource/ConfigDatasource.php, line 63 
  1.      * @param ConfigInterface $resource
  2.      */
  3.     public function save(object $resource): bool
  4.     {
  5.         // input type password or string set null on empty field
  6.         if (in_array($resource->getValueType(), [ConfigInterface::PASSWORD, ConfigInterface::STRING]) && !$resource->getValueString()) {
    
    
    in_array() should be called with the third parameter set to true to enable strict comparison and avoid type juggling bugs.
    
    
    
                    
    Time to fix: about 15 minutes
    
                Read doc
                                    
                
                Open Issue
            
                 Permalink
         Copy Prompt
    
    

            
    
            
    
                                    Last edited  by Bastien MUNCK
                    
                            
    
        
    
    
  7.             $resource->setValueString('');
  8.         }
  9.         if (in_array($resource->getValueType(), [ConfigInterface::TEXT]) && !$resource->getValueText()) {
  10.             $resource->setValueText('');
  11.         }
in src/Crudit/Datasource/ConfigDatasource.php, line 66 
  1.     {
  2.         // input type password or string set null on empty field
  3.         if (in_array($resource->getValueType(), [ConfigInterface::PASSWORD, ConfigInterface::STRING]) && !$resource->getValueString()) {
  4.             $resource->setValueString('');
  5.         }
  6.         if (in_array($resource->getValueType(), [ConfigInterface::TEXT]) && !$resource->getValueText()) {
    
    
    in_array() should be called with the third parameter set to true to enable strict comparison and avoid type juggling bugs.
    
    
    
                    
    Time to fix: about 15 minutes
    
                Read doc
                                    
                
                Open Issue
            
                 Permalink
         Copy Prompt
    
    

            
    
            
    
                                    Last edited  by Bastien
                    
                            
    
        
    
    
  7.             $resource->setValueText('');
  8.         }
  9.         if (in_array($resource->getValueType(), [ConfigInterface::INT]) && !$resource->getValueInt()) {
  10.             return true;
  11.         }
in src/Crudit/Datasource/ConfigDatasource.php, line 69 
  1.             $resource->setValueString('');
  2.         }
  3.         if (in_array($resource->getValueType(), [ConfigInterface::TEXT]) && !$resource->getValueText()) {
  4.             $resource->setValueText('');
  5.         }
  6.         if (in_array($resource->getValueType(), [ConfigInterface::INT]) && !$resource->getValueInt()) {
    
    
    in_array() should be called with the third parameter set to true to enable strict comparison and avoid type juggling bugs.
    
    
    
                    
    Time to fix: about 15 minutes
    
                Read doc
                                    
                
                Open Issue
            
                 Permalink
         Copy Prompt
    
    

            
    
            
    
                                    Last edited  by Bastien
                    
                            
    
        
    
    
  7.             return true;
  8.         }
  9.         if (in_array($resource->getValueType(), [ConfigInterface::FLOAT]) && !$resource->getValueFloat()) {
  10.             return true;
  11.         }
in src/Crudit/Datasource/ConfigDatasource.php, line 72 
  1.             $resource->setValueText('');
  2.         }
  3.         if (in_array($resource->getValueType(), [ConfigInterface::INT]) && !$resource->getValueInt()) {
  4.             return true;
  5.         }
  6.         if (in_array($resource->getValueType(), [ConfigInterface::FLOAT]) && !$resource->getValueFloat()) {
    
    
    in_array() should be called with the third parameter set to true to enable strict comparison and avoid type juggling bugs.
    
    
    
                    
    Time to fix: about 15 minutes
    
                Read doc
                                    
                
                Open Issue
            
                 Permalink
         Copy Prompt
    
    

            
    
            
    
                                    Last edited  by Bastien
                    
                            
    
        
    
    
  7.             return true;
  8.         }
  10.         /** @var bool $usingTenant */
  11.         $usingTenant = $this->parameterBag->get('lle_config.using_tenant');

Your project should not use invalid return types

More information: https://insight.symfony.com/what-we-analyse/php.invalid_return_typehint

in src/Service/CacheManager.php, line 64 
  1.             (int) $tenantId
  2.         );
  4.         $item = $this->cache->getItem($cacheKey);
  5.         if ($item->isHit()) {
  6.             return $item->get();
    
    
    Method Lle\ConfigBundle\Service\CacheManager::get() should return bool|float|int|string|null but returns mixed.
    
    
    
                    
    Time to fix: about 9 minutes
    
                Read doc
                                    
                
                Open Issue
            
                 Permalink
         Copy Prompt
    
    

            
    
            
    
                                    Last edited  by Nathan
                    
                            
    
        
    
    
  7.         }
  9.         return null;
  10.     }

Text files should end with a valid new line character. 4

More information: https://insight.symfony.com/what-we-analyse/missing_e_o_l

in .github/workflows/phpstan.yml, line 32 
  2.             -   name: 'PHPStan Static Analysis'
  3.                 uses: php-actions/phpstan@v3
  4.                 with:
  5.                     configuration: tests/phpstan.neon
  6.                     memory_limit: 256M
    
    
    This file ends with no newline character, or with a different newline character than other files in your project. It won't render properly on a terminal, and it's considered a bad practice.
    
    
    
                    
    Time to fix: about 15 minutes
    
                Read doc
                                    
                
                Open Issue
            
                 Permalink
         Copy Prompt
    
    

            
    
            
    
                                    Last edited  by artahs
in .github/workflows/phpunit.yml, line 35 
  1.                 with:
  2.                     php_version: ${{ matrix.php_version }}
  3.                     php_extensions: xdebug
  4.                     version: 10.3
  5.                     configuration: tests/phpunit/phpunit.xml
  6.                     bootstrap: vendor/autoload.php
    
    
    This file ends with no newline character, or with a different newline character than other files in your project. It won't render properly on a terminal, and it's considered a bad practice.
    
    
    
                    
    Time to fix: about 15 minutes
    
                Read doc
                                    
                
                Open Issue
            
                 Permalink
         Copy Prompt
    
    

            
    
            
    
                                    Last edited  by artahs
in .github/workflows/validate.yml, line 41 
  1.                     php_version: ${{ matrix.php_version }}
  2.                     version: 2
  3.                     args: --profile --ignore-platform-reqs
  5.             -   name: 'PHP CodeSniffer Validations'
  6.                 run: vendor/bin/phpcs
    
    
    This file ends with no newline character, or with a different newline character than other files in your project. It won't render properly on a terminal, and it's considered a bad practice.
    
    
    
                    
    Time to fix: about 15 minutes
    
                Read doc
                                    
                
                Open Issue
            
                 Permalink
         Copy Prompt
    
    

            
    
            
    
                                    Last edited  by artahs
in src/Resources/views/_trans_label.html.twig, line 4 
  1. {% set transLabel = resource.getTransLabel|trans %}
  2. {% if 'config.' not in transLabel  %}
  3.     {{ transLabel }}
  4. {% endif %}
    
    
    This file ends with no newline character, or with a different newline character than other files in your project. It won't render properly on a terminal, and it's considered a bad practice.
    
    
    
                    
    Time to fix: about 15 minutes
    
                Read doc
                                    
                
                Open Issue
            
                 Permalink
         Copy Prompt
    
    

            
    
            
    
                                    Last edited  by theo