Your project must not contain invalid function or method calls
- Read doc
- Reliability
- Major
More information: https://insight.symfony.com/what-we-analyse/php.invalid_call
- $entityToOverride = $matrixRecord->getEntity();
- return null === $entityToOverride
- ? $criteria
- : $this->addCriteriaToOmitEntity($criteria, $entityToOverride);
Parameter #1 $criteria of method JG\BatchEntityImportBundle\Validator\Constraints\DatabaseEntityUniqueValidator::addCriteriaToOmitEntity() expects array<string, array<int, array{string, bool|float|int|string}>>, array<string, non-empty-list<array{'=', mixed}>> given.
Last edited by Jarosław Grygierek- }
- /**
- * @param array<string, array<int, array{0: string, 1: scalar}>> $criteria
- *
Your project should not use invalid return types 5
- Read doc
- Reliability
- Major
More information: https://insight.symfony.com/what-we-analyse/php.invalid_return_typehint
- $data = [];
- foreach ($fields as $field) {
- $data[$field] = $matrixRecord->$field;
- }
- return $data;
Method JG\BatchEntityImportBundle\Validator\Constraints\DatabaseEntityUniqueValidator::getMatrixRecordDataToCompare() should return array<string, bool|float|int|string> but returns array<string, mixed>.
Last edited by Jarek- }
- /**
- * @param array<string, scalar> $matrixDataToCompare
- */
- $criteria[ColumnNameHelper::toCamelCase($fieldName)][] = ['=', $value];
- }
- $entityToOverride = $matrixRecord->getEntity();
- return null === $entityToOverride
Method JG\BatchEntityImportBundle\Validator\Constraints\DatabaseEntityUniqueValidator::buildCriteria() should return array<string, array<int, array{string, bool|float|int|string}>> but returns array<string, array<int, array{string, mixed}>>.
Last edited by Jarosław Grygierek- ? $criteria
- : $this->addCriteriaToOmitEntity($criteria, $entityToOverride);
- }
- /**
- foreach ($primaryKeyData as $primaryKeyName => $primaryValue) {
- $criteria[$primaryKeyName][] = ['!=', $primaryValue];
- }
- return $criteria;
Method JG\BatchEntityImportBundle\Validator\Constraints\DatabaseEntityUniqueValidator::addCriteriaToOmitEntity() should return array<string, array<int, array{string, bool|float|int|string}>> but returns array<array<int, array{string, mixed}>>.
Last edited by Jarosław Grygierek- }
- /**
- * @param array<string, array<int, array{0: string, 1: scalar}>> $criteria
- */
- /**
- * @phpstan-return non-empty-string
- */
- private function getSeparator(FormFieldDefinition $fieldDefinition): string
- {
- return $fieldDefinition->getOptions()['separator'] ?? ArrayTextType::DEFAULT_SEPARATOR;
Method JG\BatchEntityImportBundle\Model\Configuration\AbstractImportConfiguration::getSeparator() should return non-empty-string but returns mixed.
Last edited by Jarosław Grygierek- }
- }
- $this->data[$name] = $value;
- }
- public function __get(string $name): int|string|null
- {
- return $this->data[$name];
Method JG\BatchEntityImportBundle\Model\Matrix\MatrixRecord::__get() should return int|string|null but returns mixed.
Last edited by Jarek- }
- }
Your project should not use invalid parameter and return typehints 8
- Read doc
- Reliability
- Major
More information: https://insight.symfony.com/what-we-analyse/php.invalid_typehint
- }
- /**
- * @return array<string, array<int, array{0: string, 1: scalar}>>
- */
- private function buildCriteria(MatrixRecord $matrixRecord, array $matrixDataToCompare): array
Method JG\BatchEntityImportBundle\Validator\Constraints\DatabaseEntityUniqueValidator::buildCriteria() has parameter $matrixDataToCompare with no value type specified in iterable type array.
Last edited by Jarek- {
- $criteria = [];
- foreach ($matrixDataToCompare as $fieldName => $value) {
- $criteria[ColumnNameHelper::toCamelCase($fieldName)][] = ['=', $value];
- }
- class MatrixRecord
- {
- private ?object $entity = null;
- private array $data = [];
- public function __construct(array $data = [], public readonly int|string|null $entityId = null)
Method JG\BatchEntityImportBundle\Model\Matrix\MatrixRecord::__construct() has parameter $data with no value type specified in iterable type array.
Last edited by Jarek- {
- foreach ($data as $name => $value) {
- if (!empty(\trim((string) $name))) {
- $this->data[\str_replace(' ', '_', (string) $name)] = $value;
- }
- $container
- ->registerForAutoconfiguration(ImportConfigurationAutoInjectInterface::class)
- ->addTag('batch_entity_import.controller');
- }
- private function setParameters(array $configs, ContainerBuilder $container): void
Method JG\BatchEntityImportBundle\DependencyInjection\BatchEntityImportExtension::setParameters() has parameter $configs with no value type specified in iterable type array.
Last edited by Jarek- {
- $processor = new Processor();
- $configuration = new Configuration();
- $config = $processor->processConfiguration($configuration, $configs);
- $this->addTemplatesConfig($rootNode->children());
- return $treeBuilder;
- }
- private function addTemplatesConfig(NodeBuilder $parentBuilder): void
Method JG\BatchEntityImportBundle\DependencyInjection\Configuration::addTemplatesConfig() has parameter $parentBuilder with generic class Symfony\Component\Config\Definition\Builder\NodeBuilder but does not specify its types: TParent
Last edited by Jarek- {
- $builder = $parentBuilder->arrayNode('templates')->addDefaultsIfNotSet()->children();
- $this->addNodeConfig($builder, 'select_file', '@BatchEntityImport/select_file.html.twig');
- $this->addNodeConfig($builder, 'edit_matrix', '@BatchEntityImport/edit_matrix.html.twig');
- $this->addNodeConfig($builder, 'select_file', '@BatchEntityImport/select_file.html.twig');
- $this->addNodeConfig($builder, 'edit_matrix', '@BatchEntityImport/edit_matrix.html.twig');
- $this->addNodeConfig($builder, 'layout', '@BatchEntityImport/layout.html.twig');
- }
- private function addNodeConfig(NodeBuilder $builder, string $name, string $value): void
Method JG\BatchEntityImportBundle\DependencyInjection\Configuration::addNodeConfig() has parameter $builder with generic class Symfony\Component\Config\Definition\Builder\NodeBuilder but does not specify its types: TParent
Last edited by Jarek- {
- $builder
- ->scalarNode($name)
- ->defaultValue($value)
- ->cannotBeEmpty()
- namespace JG\BatchEntityImportBundle\Model\Form;
- readonly class FormFieldDefinition
- {
- public function __construct(private string $class, private array $options = [])
Method JG\BatchEntityImportBundle\Model\Form\FormFieldDefinition::__construct() has parameter $options with no value type specified in iterable type array.
Last edited by Jarosław Grygierek- {
- }
- public function getClass(): string
- {
- protected function prepareContext(): void
- {
- $this->context->setNode($this->context->getValue(), $this->context->getObject(), $this->context->getMetadata(), '');
- }
- protected function addErrorToMatrixRecord(MatrixRecord $record, MatrixRecordUnique|DatabaseEntityUnique $constraint, int $index, array $fields): void
Method JG\BatchEntityImportBundle\Validator\Constraints\AbstractValidator::addErrorToMatrixRecord() has parameter $fields with no value type specified in iterable type array.
Last edited by Jarosław Grygierek- {
- $this->context
- ->buildViolation($constraint->message, ['%fields%' => implode(', ', $fields)])
- ->atPath(sprintf('children[records][%d][%s]', $index, $constraint->fields[0]))
- ->setInvalidValue($record)
- ->atPath(sprintf('children[records][%d][%s]', $index, $constraint->fields[0]))
- ->setInvalidValue($record)
- ->addViolation();
- }
- protected function getHash(array $data): string
Method JG\BatchEntityImportBundle\Validator\Constraints\AbstractValidator::getHash() has parameter $data with no value type specified in iterable type array.
Last edited by Jarek- {
- return md5(implode('--', $data));
- }
- }
Your project should use return types 6
- Read doc
- Reliability
- Major
More information: https://insight.symfony.com/what-we-analyse/php.missing_return_typehint
- public function setEntity(?object $entity): void
- {
- $this->entity = $entity;
- }
- public function getData(): array
Method JG\BatchEntityImportBundle\Model\Matrix\MatrixRecord::getData() return type has no value type specified in iterable type array.
Last edited by Jarek- {
- return $this->data;
- }
- public function __isset(string $name): bool
- public function getClass(): string
- {
- return $this->class;
- }
- public function getOptions(): array
Method JG\BatchEntityImportBundle\Model\Form\FormFieldDefinition::getOptions() return type has no value type specified in iterable type array.
Last edited by Jarek- {
- return $this->options;
- }
- }
- }
- return implode($this->separator, $value);
- }
- public function reverseTransform(mixed $value): array
Method JG\BatchEntityImportBundle\Form\Type\ArrayTextType::reverseTransform() return type has no value type specified in iterable type array.
Last edited by Pierre Svgnt- {
- if (!is_string($value)) {
- throw new UnexpectedValueException('Only strings are allowed');
- }
- public function getEntityTranslationRelationName(): ?string;
- /**
- * Use this method to pass constraints to the main Matrix form.
- */
- public function getMatrixConstraints(): array;
Method JG\BatchEntityImportBundle\Model\Configuration\ImportConfigurationInterface::getMatrixConstraints() return type has no value type specified in iterable type array.
Last edited by Jarek- /**
- * Use this method to set allowed file extensions.
- */
- public function getAllowedFileExtensions(): array;
- public function getMatrixConstraints(): array;
- /**
- * Use this method to set allowed file extensions.
- */
- public function getAllowedFileExtensions(): array;
Method JG\BatchEntityImportBundle\Model\Configuration\ImportConfigurationInterface::getAllowedFileExtensions() return type has no value type specified in iterable type array.
Last edited by Jarek- /**
- * Import data from matrix to database.
- */
- public function import(Matrix $matrix): void;
- }
- /**
- * @param string[] $fieldsUsedInDuplicationCheck
- */
- private function getMatrixRecordDataForDuplicationCheck(MatrixRecord $matrixRecord, array $fieldsUsedInDuplicationCheck): array
Method JG\BatchEntityImportBundle\Validator\Constraints\MatrixRecordUniqueValidator::getMatrixRecordDataForDuplicationCheck() return type has no value type specified in iterable type array.
Last edited by Jarek- {
- return array_intersect_key($matrixRecord->getData(), array_flip($fieldsUsedInDuplicationCheck));
- }
- }