Analysis jargry91 / BatchEntityImportBundle #293

Your project must not contain invalid function or method calls 9

Read doc
Reliability
Major

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

in src/Model/Matrix/Matrix.php, line 63

    {
        $info = [];
        $checker = new PropertyExistenceChecker($className);
        foreach ($this->header as $name) {
            $info[$name] = $checker->propertyExists($name);Parameter #1 $name of method JG\BatchEntityImportBundle\Service\PropertyExistenceChecker::propertyExists() expects string, mixed given.
Time to fix: about 9 minutes
Read doc Permalink

                                    Last edited Tue, 21 Apr 2020 18:45:02 +0000 by Jarosław Grygierek
                    
        }
        return $info;
    }

in src/Model/Matrix/Matrix.php, line 39

        $this->header = $this->clearHeader($header);
        foreach ($recordsData as $data) {
            $clearedData = $this->clearRecordData($data);
            if ($clearedData) {
                $this->records[] = new MatrixRecord($clearedData, $this->getEntityIdValue($data));Parameter #1 $data of method JG\BatchEntityImportBundle\Model\Matrix\Matrix::getEntityIdValue() expects array, mixed given.
Time to fix: about 9 minutes
Read doc Permalink

                                    Last edited Mon, 17 Feb 2025 12:30:08 +0000 by Jarek
                    
            }
        }
    }
    public function getHeader(): array

in src/Model/Matrix/Matrix.php, line 83

    }
    private function clearHeader(array $header): array
    {
        $header = array_values(
            array_filter($header, $this->isColumnNameValid(...)),Parameter #2 $callback of function array_filter expects (callable(mixed): bool)|null, Closure(string|null): bool given.
Time to fix: about 9 minutes
Read doc Permalink

                                    Last edited Mon, 24 Nov 2025 01:59:29 +0000 by Jarek
                    
        );
        return \array_map(static fn (string $name): string => \str_replace(' ', '_', $name), $header);
    }

in src/Model/Matrix/Matrix.php, line 86

    {
        $header = array_values(
            array_filter($header, $this->isColumnNameValid(...)),
        );
        return \array_map(static fn (string $name): string => \str_replace(' ', '_', $name), $header);Parameter #1 $callback of function array_map expects (callable(mixed): mixed)|null, Closure(string): string given.
Time to fix: about 9 minutes
Read doc Permalink

                                    Last edited Mon, 24 Nov 2025 01:59:29 +0000 by Jarek
                    
    }
    private function clearRecordData(array $data): array
    {
        return array_filter($data, $this->isColumnNameValid(...), ARRAY_FILTER_USE_KEY);

in src/DependencyInjection/BatchEntityImportExtension.php, line 53

        $configuration = new Configuration();
        $config = $processor->processConfiguration($configuration, $configs);
        $templates = $config['templates'];
        $container->setParameter('batch_entity_import.templates', $templates);
        $container->setParameter('batch_entity_import.templates.select_file', $templates['select_file']);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 Permalink

                                    Last edited Sat, 16 May 2020 16:47:53 +0000 by Jarek
                    
        $container->setParameter('batch_entity_import.templates.edit_matrix', $templates['edit_matrix']);
        $container->setParameter('batch_entity_import.templates.layout', $templates['layout']);
    }
}

in src/DependencyInjection/BatchEntityImportExtension.php, line 55


        $templates = $config['templates'];
        $container->setParameter('batch_entity_import.templates', $templates);
        $container->setParameter('batch_entity_import.templates.select_file', $templates['select_file']);
        $container->setParameter('batch_entity_import.templates.edit_matrix', $templates['edit_matrix']);
        $container->setParameter('batch_entity_import.templates.layout', $templates['layout']);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 Permalink

                                    Last edited Thu, 16 Sep 2021 17:55:48 +0000 by Jarosław Grygierek
                    
    }
}

in src/DependencyInjection/BatchEntityImportExtension.php, line 54

        $config = $processor->processConfiguration($configuration, $configs);
        $templates = $config['templates'];
        $container->setParameter('batch_entity_import.templates', $templates);
        $container->setParameter('batch_entity_import.templates.select_file', $templates['select_file']);
        $container->setParameter('batch_entity_import.templates.edit_matrix', $templates['edit_matrix']);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 Permalink

                                    Last edited Sat, 16 May 2020 16:47:53 +0000 by Jarek
                    
        $container->setParameter('batch_entity_import.templates.layout', $templates['layout']);
    }
}

in src/DependencyInjection/BatchEntityImportExtension.php, line 52

        $processor = new Processor();
        $configuration = new Configuration();
        $config = $processor->processConfiguration($configuration, $configs);
        $templates = $config['templates'];
        $container->setParameter('batch_entity_import.templates', $templates);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 Permalink

                                    Last edited Sat, 16 May 2020 16:47:53 +0000 by Jarek
                    
        $container->setParameter('batch_entity_import.templates.select_file', $templates['select_file']);
        $container->setParameter('batch_entity_import.templates.edit_matrix', $templates['edit_matrix']);
        $container->setParameter('batch_entity_import.templates.layout', $templates['layout']);
    }
}

in src/Model/Configuration/AbstractImportConfiguration.php, line 74

            $setterName = ColumnNameHelper::getSetterName($name);
            if (isset($fieldDefinitions[$name])) {
                $fieldDefinition = $fieldDefinitions[$name];
                if (ArrayTextType::class === $fieldDefinition->getClass()) {
                    $value = $this->parseValueForArrayType($fieldDefinition, $value);Parameter #2 $value of method JG\BatchEntityImportBundle\Model\Configuration\AbstractImportConfiguration::parseValueForArrayType() expects string|null, mixed given.
Time to fix: about 9 minutes
Read doc Permalink

                                    Last edited Thu, 26 Sep 2024 10:04:57 +0000 by Pierre Svgnt
                    
                }
            }
            try {
                if (\interface_exists(TranslatableInterface::class) && $entity instanceof TranslatableInterface && $locale) {

Your project should use return types 3

Read doc
Reliability
Major

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

in src/Model/Matrix/Matrix.php, line 80

        }
        return null;
    }
    private function clearHeader(array $header): arrayMethod JG\BatchEntityImportBundle\Model\Matrix\Matrix::clearHeader() return type has no value type specified in iterable type array.
Time to fix: about 9 minutes
Read doc Permalink

                                    Last edited Sat, 02 May 2020 20:19:17 +0000 by Jarek
                    
    {
        $header = array_values(
            array_filter($header, $this->isColumnNameValid(...)),
        );

in src/Model/Configuration/AbstractImportConfiguration.php, line 151

    public function allowOverrideEntity(): bool
    {
        return true;
    }
    public function getAllowedFileExtensions(): arrayMethod JG\BatchEntityImportBundle\Model\Configuration\AbstractImportConfiguration::getAllowedFileExtensions() return type has no value type specified in iterable type array.
Time to fix: about 9 minutes
Read doc Permalink

                                    Last edited Fri, 20 Dec 2024 08:02:03 +0000 by Jarek
                    
    {
        return ['csv', 'xls', 'xlsx', 'ods'];
    }
    private function parseValueForArrayType(FormFieldDefinition $fieldDefinition, ?string $value): array

in src/Model/Configuration/AbstractImportConfiguration.php, line 41

    public function getEntityTranslationRelationName(): ?string
    {
        return null;
    }
    public function getMatrixConstraints(): arrayMethod JG\BatchEntityImportBundle\Model\Configuration\AbstractImportConfiguration::getMatrixConstraints() return type has no value type specified in iterable type array.
Time to fix: about 9 minutes
Read doc Permalink

                                    Last edited Sat, 27 Nov 2021 21:13:30 +0000 by Jarek
                    
    {
        return [];
    }
    public function import(Matrix $matrix): void

Your project should not use invalid return types

Read doc
Reliability
Major

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

in src/Model/Matrix/Matrix.php, line 73


    private function getEntityIdValue(array $data): int|string|null
    {
        foreach ($data as $name => $value) {
            if (self::RESERVED_ENTITY_ID_COLUMN_NAME === $name) {
                return $value;Method JG\BatchEntityImportBundle\Model\Matrix\Matrix::getEntityIdValue() should return int|string|null but returns mixed.
Time to fix: about 9 minutes
Read doc Permalink

                                    Last edited Mon, 17 Feb 2025 12:30:08 +0000 by Jarek
                    
            }
        }
        return null;
    }

Your project should not use invalid parameter and return typehints 2

Read doc
Reliability
Major

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

in src/Model/Matrix/Matrix.php, line 80

        }
        return null;
    }
    private function clearHeader(array $header): arrayMethod JG\BatchEntityImportBundle\Model\Matrix\Matrix::clearHeader() has parameter $header with no value type specified in iterable type array.
Time to fix: about 9 minutes
Read doc Permalink

                                    Last edited Sat, 02 May 2020 20:19:17 +0000 by Jarek
                    
    {
        $header = array_values(
            array_filter($header, $this->isColumnNameValid(...)),
        );

in src/Model/Matrix/Matrix.php, line 69

        }
        return $info;
    }
    private function getEntityIdValue(array $data): int|string|nullMethod JG\BatchEntityImportBundle\Model\Matrix\Matrix::getEntityIdValue() has parameter $data with no value type specified in iterable type array.
Time to fix: about 9 minutes
Read doc Permalink

                                    Last edited Mon, 17 Feb 2025 12:30:08 +0000 by Jarek
                    
    {
        foreach ($data as $name => $value) {
            if (self::RESERVED_ENTITY_ID_COLUMN_NAME === $name) {
                return $value;
            }