Your project must not contain invalid function or method calls 18
- Read doc
- Reliability
- Major
More information: https://insight.symfony.com/what-we-analyse/php.invalid_call
- }
- try {
- if (\interface_exists(TranslatableInterface::class) && $entity instanceof TranslatableInterface && $locale) {
- $translatedEntity = $entity->translate($locale, false);
- if (method_exists($translatedEntity, $setterName)) {
Parameter #1 $object_or_class of function method_exists expects object|string, mixed given.
Last edited by Jarosław Grygierek- $translatedEntity->$setterName($value);
- } else {
- $translatedEntity->$propertyName = $value;
- }
- } elseif (!$locale) {
- }
- protected function dispatchEvents(): void
- {
- foreach ($this->updatedEntities as $entity) {
- $identifierValues = $this->em->getUnitOfWork()->getEntityIdentifier($entity);
Parameter #1 $entity of method Doctrine\ORM\UnitOfWork::getEntityIdentifier() expects object, mixed given.
Last edited by Jarek- $this->eventDispatcher->dispatch(
- new RecordImportedSuccessfullyEvent($this->getEntityClassName(), (string) \reset($identifierValues)),
- );
- }
- }
- private function parseValueForArrayType(FormFieldDefinition $fieldDefinition, ?string $value): array
- {
- return $value
- ? explode($fieldDefinition->getOptions()['separator'] ?? ArrayTextType::DEFAULT_SEPARATOR, $value)
Parameter #1 $separator of function explode expects non-empty-string, mixed given.
Last edited by Pierre Svgnt- : [];
- }
- }
- }
- #[Assert\Callback]
- public function validateExtensions(ExecutionContextInterface $context): void
- {
- $extensions = array_map(strtolower(...), $this->allowedExtensions);
Parameter #1 $callback of function array_map expects (callable(mixed): mixed)|null, Closure(string): lowercase-string given.
Last edited by Jarek- if (!in_array(strtolower($this->file->getClientOriginalExtension()), $extensions, true)) {
- $context->buildViolation('validation.file.extension', ['%extensions' => implode(', ', $extensions)])->addViolation();
- }
- }
- }
- #[Assert\Callback]
- public function validateExtensions(ExecutionContextInterface $context): void
- {
- $extensions = array_map(strtolower(...), $this->allowedExtensions);
- if (!in_array(strtolower($this->file->getClientOriginalExtension()), $extensions, true)) {
Cannot call method getClientOriginalExtension() on Symfony\Component\HttpFoundation\File\UploadedFile|null.
Last edited by Jarek- $context->buildViolation('validation.file.extension', ['%extensions' => implode(', ', $extensions)])->addViolation();
- }
- }
- }
- public function __construct(array $header = [], array $recordsData = [])
- {
- $this->header = $this->clearHeader($header);
- foreach ($recordsData as $data) {
- $clearedData = $this->clearRecordData($data);
Parameter #1 $data of method JG\BatchEntityImportBundle\Model\Matrix\Matrix::clearRecordData() expects array, mixed given.
Last edited by Jarek- if ($clearedData) {
- $this->records[] = new MatrixRecord($clearedData, $this->getEntityIdValue($data));
- }
- }
- }
- $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.
Last edited by Jarek- }
- }
- }
- public function getHeader(): array
- {
- $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.
Last edited by Jarosław Grygierek- }
- return $info;
- }
- }
- 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.
Last edited by Jarek- );
- return \array_map(static fn (string $name): string => \str_replace(' ', '_', $name), $header);
- }
- {
- $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.
Last edited by Jarek- }
- private function clearRecordData(array $data): array
- {
- return array_filter($data, $this->isColumnNameValid(...), ARRAY_FILTER_USE_KEY);
- $reader = self::getReader($file);
- $spreadsheet = $reader->load($file->getPathname());
- $data = $spreadsheet->getActiveSheet()->toArray();
- $header = array_shift($data);
- self::addKeysToRows($header, $data);
Parameter #1 $header of static method JG\BatchEntityImportBundle\Model\Matrix\MatrixFactory::addKeysToRows() expects array, array<mixed>|null given.
Last edited by Jarosław Grygierek- return new Matrix($header, $data);
- }
- public static function createFromPostData(array $data): Matrix
- return new Matrix($header, $data);
- }
- public static function createFromPostData(array $data): Matrix
- {
- return $data ? new Matrix(array_keys($data[0]), $data) : new Matrix();
Parameter #1 $array of function array_keys expects array, mixed given.
Last edited by Jarek- }
- private static function addKeysToRows(array $header, array &$data): void
- {
- array_walk(
- private static function addKeysToRows(array $header, array &$data): void
- {
- array_walk(
- $data,
- static function (array &$row) use ($header): void {
Parameter #2 $callback of function array_walk expects callable(mixed, int|string): mixed, Closure(array): void given.
Last edited by Jarosław Grygierek- $row = array_combine($header, $row);
- },
- );
- }
- private static function addKeysToRows(array $header, array &$data): void
- {
- array_walk(
- $data,
- static function (array &$row) use ($header): void {
- $row = array_combine($header, $row);
Parameter #1 $keys of function array_combine expects array<int|string>, array given.
Last edited by Jarek- },
- );
- }
- private static function getReader(UploadedFile $file): BaseReader
- foreach ($delimiters as $delimiter) {
- $delimiterCount[$delimiter] = substr_count($csvContent, (string) $delimiter);
- }
- return array_search(max($delimiterCount), $delimiterCount, true);
Parameter #1 ...$arg1 of function max expects non-empty-array, array<int<0, max>> given.
Last edited by Jarosław Grygierek- }
- }
- */
- public function __construct(string $entityClass)
- {
- $this->reflectionClass = new ReflectionClass($entityClass);
- if (\interface_exists(TranslatableInterface::class) && is_subclass_of($entityClass, TranslatableInterface::class)) {
- $this->translationReflectionClass = new ReflectionClass($this->reflectionClass->newInstanceWithoutConstructor()->translate());
Call to an undefined method object::translate().
Last edited by Micheal- }
- }
- public function propertyExists(string $name): bool
- {
- }
- private function getHashedMatrixRecordsDataForDuplicationCheck(array $records, array $fieldsUsedInDuplicationCheck): array
- {
- return array_map(
- fn (MatrixRecord $record): string => $this->getHash($this->getMatrixRecordDataForDuplicationCheck($record, $fieldsUsedInDuplicationCheck)),
Parameter #1 $callback of function array_map expects (callable(mixed): mixed)|null, Closure(JG\BatchEntityImportBundle\Model\Matrix\MatrixRecord): string given.
Last edited by Jarosław Grygierek- $records
- );
- }
- private function getMatrixRecordDataForDuplicationCheck(MatrixRecord $matrixRecord, array $fieldsUsedInDuplicationCheck): array
- );
- }
- private function getMatrixRecordDataForDuplicationCheck(MatrixRecord $matrixRecord, array $fieldsUsedInDuplicationCheck): array
- {
- return array_intersect_key($matrixRecord->getData(), array_flip($fieldsUsedInDuplicationCheck));
Parameter #1 $array of function array_flip expects array<int|string>, array given.
Last edited by Jarek- }
- }
Your project should not use invalid parameter and return typehints 32
- Read doc
- Reliability
- Major
More information: https://insight.symfony.com/what-we-analyse/php.invalid_typehint
- }
- $this->save();
- }
- protected function prepareRecord(MatrixRecord $record, array $headerInfo): void
Method JG\BatchEntityImportBundle\Model\Configuration\AbstractImportConfiguration::prepareRecord() has parameter $headerInfo with no value type specified in iterable type array.
Last edited by Jarosław Grygierek- {
- $entity = $this->getEntity($record);
- $data = $record->getData();
- $fieldDefinitions = $this->getFieldsDefinitions();
- {
- #[Assert\File]
- #[Assert\NotNull]
- private ?UploadedFile $file = null;
- public function __construct(private readonly array $allowedExtensions = [])
Method JG\BatchEntityImportBundle\Model\FileImport::__construct() has parameter $allowedExtensions with no value type specified in iterable type array.
Last edited by Jarek- {
- }
- public function getFile(): ?UploadedFile
- {
- new Assert\Type(MatrixRecord::class),
- ])]
- #[Assert\NotBlank]
- private array $records = [];
- public function __construct(array $header = [], array $recordsData = [])
Method JG\BatchEntityImportBundle\Model\Matrix\Matrix::__construct() has parameter $recordsData with no value type specified in iterable type array.
Last edited by Jarek- {
- $this->header = $this->clearHeader($header);
- foreach ($recordsData as $data) {
- $clearedData = $this->clearRecordData($data);
- new Assert\Type(MatrixRecord::class),
- ])]
- #[Assert\NotBlank]
- private array $records = [];
- public function __construct(array $header = [], array $recordsData = [])
Method JG\BatchEntityImportBundle\Model\Matrix\Matrix::__construct() has parameter $header with no value type specified in iterable type array.
Last edited by Jarek- {
- $this->header = $this->clearHeader($header);
- foreach ($recordsData as $data) {
- $clearedData = $this->clearRecordData($data);
- }
- return $info;
- }
- private function getEntityIdValue(array $data): int|string|null
Method JG\BatchEntityImportBundle\Model\Matrix\Matrix::getEntityIdValue() has parameter $data with no value type specified in iterable type array.
Last edited by Jarek- {
- foreach ($data as $name => $value) {
- if (self::RESERVED_ENTITY_ID_COLUMN_NAME === $name) {
- return $value;
- }
- }
- return null;
- }
- private function clearHeader(array $header): array
Method JG\BatchEntityImportBundle\Model\Matrix\Matrix::clearHeader() has parameter $header with no value type specified in iterable type array.
Last edited by Jarek- {
- $header = array_values(
- array_filter($header, $this->isColumnNameValid(...)),
- );
- );
- return \array_map(static fn (string $name): string => \str_replace(' ', '_', $name), $header);
- }
- private function clearRecordData(array $data): array
Method JG\BatchEntityImportBundle\Model\Matrix\Matrix::clearRecordData() has parameter $data with no value type specified in iterable type array.
Last edited by Jarek- {
- return array_filter($data, $this->isColumnNameValid(...), ARRAY_FILTER_USE_KEY);
- }
- private function isColumnNameValid(?string $name): bool
- self::addKeysToRows($header, $data);
- return new Matrix($header, $data);
- }
- public static function createFromPostData(array $data): Matrix
Method JG\BatchEntityImportBundle\Model\Matrix\MatrixFactory::createFromPostData() has parameter $data with no value type specified in iterable type array.
Last edited by Jarek- {
- return $data ? new Matrix(array_keys($data[0]), $data) : new Matrix();
- }
- private static function addKeysToRows(array $header, array &$data): void
- public static function createFromPostData(array $data): Matrix
- {
- return $data ? new Matrix(array_keys($data[0]), $data) : new Matrix();
- }
- private static function addKeysToRows(array $header, array &$data): void
Method JG\BatchEntityImportBundle\Model\Matrix\MatrixFactory::addKeysToRows() has parameter $header with no value type specified in iterable type array.
Last edited by Jarek- {
- array_walk(
- $data,
- static function (array &$row) use ($header): void {
- $row = array_combine($header, $row);
- public static function createFromPostData(array $data): Matrix
- {
- return $data ? new Matrix(array_keys($data[0]), $data) : new Matrix();
- }
- private static function addKeysToRows(array $header, array &$data): void
Method JG\BatchEntityImportBundle\Model\Matrix\MatrixFactory::addKeysToRows() has parameter $data with no value type specified in iterable type array.
Last edited by Jarek- {
- array_walk(
- $data,
- static function (array &$row) use ($header): void {
- $row = array_combine($header, $row);
- private function translationPropertyExists(string $name): bool
- {
- return $this->translationReflectionClass instanceof ReflectionClass && $this->isPropertyWritable($this->translationReflectionClass, $name);
- }
- private function isPropertyWritable(ReflectionClass $entity, string $name): bool
Method JG\BatchEntityImportBundle\Service\PropertyExistenceChecker::isPropertyWritable() has parameter $entity with generic class ReflectionClass but does not specify its types: T
Last edited by Jarosław Grygierek- {
- $setterName = ColumnNameHelper::getSetterName($name);
- return ($entity->hasProperty($name) && $entity->getProperty($name)->isPublic())
- || ($entity->hasMethod($setterName) && $entity->getMethod($setterName)->isPublic());
- }
- parent::validateArguments($value, $constraint);
- }
- private function getHashedMatrixRecordsDataForDuplicationCheck(array $records, array $fieldsUsedInDuplicationCheck): array
Method JG\BatchEntityImportBundle\Validator\Constraints\MatrixRecordUniqueValidator::getHashedMatrixRecordsDataForDuplicationCheck() has parameter $records with no value type specified in iterable type array.
Last edited by Jarek- {
- return array_map(
- fn (MatrixRecord $record): string => $this->getHash($this->getMatrixRecordDataForDuplicationCheck($record, $fieldsUsedInDuplicationCheck)),
- $records
- );
- }
- parent::validateArguments($value, $constraint);
- }
- private function getHashedMatrixRecordsDataForDuplicationCheck(array $records, array $fieldsUsedInDuplicationCheck): array
Method JG\BatchEntityImportBundle\Validator\Constraints\MatrixRecordUniqueValidator::getHashedMatrixRecordsDataForDuplicationCheck() has parameter $fieldsUsedInDuplicationCheck with no value type specified in iterable type array.
Last edited by Jarek- {
- return array_map(
- fn (MatrixRecord $record): string => $this->getHash($this->getMatrixRecordDataForDuplicationCheck($record, $fieldsUsedInDuplicationCheck)),
- $records
- );
- fn (MatrixRecord $record): string => $this->getHash($this->getMatrixRecordDataForDuplicationCheck($record, $fieldsUsedInDuplicationCheck)),
- $records
- );
- }
- private function getMatrixRecordDataForDuplicationCheck(MatrixRecord $matrixRecord, array $fieldsUsedInDuplicationCheck): array
Method JG\BatchEntityImportBundle\Validator\Constraints\MatrixRecordUniqueValidator::getMatrixRecordDataForDuplicationCheck() has parameter $fieldsUsedInDuplicationCheck with no value type specified in iterable type array.
Last edited by Jarek- {
- return array_intersect_key($matrixRecord->getData(), array_flip($fieldsUsedInDuplicationCheck));
- }
- }
- }
- parent::validateArguments($value, $constraint);
- }
- private function getMatrixRecordDataToCompare(MatrixRecord $matrixRecord, array $fields): array
Method JG\BatchEntityImportBundle\Validator\Constraints\DatabaseEntityUniqueValidator::getMatrixRecordDataToCompare() has parameter $fields with no value type specified in iterable type array.
Last edited by Jarek- {
- $data = [];
- foreach ($fields as $field) {
- $data[$field] = $matrixRecord->$field;
- }
- }
- return $data;
- }
- private function isDuplicate(array $matrixDataToCompare): bool
Method JG\BatchEntityImportBundle\Validator\Constraints\DatabaseEntityUniqueValidator::isDuplicate() has parameter $matrixDataToCompare with no value type specified in iterable type array.
Last edited by Jarek- {
- return array_key_exists($this->getHash($matrixDataToCompare), $this->duplicatedRecords);
- }
- private function isCorrectRecord(array $matrixDataToCompare): bool
- private function isDuplicate(array $matrixDataToCompare): bool
- {
- return array_key_exists($this->getHash($matrixDataToCompare), $this->duplicatedRecords);
- }
- private function isCorrectRecord(array $matrixDataToCompare): bool
Method JG\BatchEntityImportBundle\Validator\Constraints\DatabaseEntityUniqueValidator::isCorrectRecord() has parameter $matrixDataToCompare with no value type specified in iterable type array.
Last edited by Jarek- {
- return array_key_exists($this->getHash($matrixDataToCompare), $this->correctRecords);
- }
- private function addDuplicate(array $matrixDataToCompare): void
- private function isCorrectRecord(array $matrixDataToCompare): bool
- {
- return array_key_exists($this->getHash($matrixDataToCompare), $this->correctRecords);
- }
- private function addDuplicate(array $matrixDataToCompare): void
Method JG\BatchEntityImportBundle\Validator\Constraints\DatabaseEntityUniqueValidator::addDuplicate() has parameter $matrixDataToCompare with no value type specified in iterable type array.
Last edited by Jarek- {
- $this->duplicatedRecords[$this->getHash($matrixDataToCompare)] = true;
- }
- private function markAsCorrectRecord(array $matrixDataToCompare): void
- private function addDuplicate(array $matrixDataToCompare): void
- {
- $this->duplicatedRecords[$this->getHash($matrixDataToCompare)] = true;
- }
- private function markAsCorrectRecord(array $matrixDataToCompare): void
Method JG\BatchEntityImportBundle\Validator\Constraints\DatabaseEntityUniqueValidator::markAsCorrectRecord() has parameter $matrixDataToCompare with no value type specified in iterable type array.
Last edited by Jarek- {
- $this->correctRecords[$this->getHash($matrixDataToCompare)] = true;
- }
- private function buildCriteria(MatrixRecord $matrixRecord, array $matrixDataToCompare): array
- private function markAsCorrectRecord(array $matrixDataToCompare): void
- {
- $this->correctRecords[$this->getHash($matrixDataToCompare)] = true;
- }
- 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];
- }
- }
- return $criteria;
- }
- private function addCriteriaToOmitEntity(array &$criteria, object $entityToOverride): void
Method JG\BatchEntityImportBundle\Validator\Constraints\DatabaseEntityUniqueValidator::addCriteriaToOmitEntity() has parameter $criteria with no value type specified in iterable type array.
Last edited by Jarek- {
- $primaryKeyData = $this->entityManager->getUnitOfWork()->getEntityIdentifier($entityToOverride);
- foreach ($primaryKeyData as $primaryKeyName => $primaryValue) {
- $criteria[$primaryKeyName][] = ['!=', $primaryValue];
- foreach ($primaryKeyData as $primaryKeyName => $primaryValue) {
- $criteria[$primaryKeyName][] = ['!=', $primaryValue];
- }
- }
- private function isRecordDuplicatedInDatabase(EntityManagerInterface $em, string $class, array $criteria): bool
Method JG\BatchEntityImportBundle\Validator\Constraints\DatabaseEntityUniqueValidator::isRecordDuplicatedInDatabase() has parameter $criteria with no value type specified in iterable type array.
Last edited by Jarek- {
- $query = $em->createQuery($this->buildDQL($class, $criteria));
- $this->passParametersToQuery($query, $criteria);
- return !empty($query->getArrayResult());
- $this->passParametersToQuery($query, $criteria);
- return !empty($query->getArrayResult());
- }
- private function buildDQL(string $class, array $criteria): string
Method JG\BatchEntityImportBundle\Validator\Constraints\DatabaseEntityUniqueValidator::buildDQL() has parameter $criteria with no value type specified in iterable type array.
Last edited by Jarek- {
- $sql = /* @lang DQL */
- "SELECT c FROM $class c";
- $nmb = 0;
- }
- return $sql;
- }
- private function passParametersToQuery(AbstractQuery $query, array $criteria): void
Method JG\BatchEntityImportBundle\Validator\Constraints\DatabaseEntityUniqueValidator::passParametersToQuery() has parameter $criteria with no value type specified in iterable type array.
Last edited by Jarek- {
- $nmb = 0;
- foreach ($criteria as $data) {
- foreach ($data as [$operator, $value]) {
- $query->setParameter('param_' . $nmb++, $value);
- use Twig\TwigFunction;
- use UnexpectedValueException;
- class TemplateExtension extends AbstractExtension
- {
- public function __construct(private readonly array $availableTemplates = [])
Method JG\BatchEntityImportBundle\Twig\TemplateExtension::__construct() has parameter $availableTemplates with no value type specified in iterable type array.
Last edited by Jarosław Grygierek- {
- }
- public function getFunctions(): array
- {
- $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;
- class FormFieldDefinition
- {
- public function __construct(private readonly string $class, private readonly 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
- {
- 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;
- }
- 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("children[records][$index][{$constraint->fields[0]}]")
- ->setInvalidValue($record)
- ->atPath("children[records][$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 19
- Read doc
- Reliability
- Major
More information: https://insight.symfony.com/what-we-analyse/php.missing_return_typehint
- public function getEntityTranslationRelationName(): ?string
- {
- return null;
- }
- public function getMatrixConstraints(): array
Method JG\BatchEntityImportBundle\Model\Configuration\AbstractImportConfiguration::getMatrixConstraints() return type has no value type specified in iterable type array.
Last edited by Jarek- {
- return [];
- }
- public function import(Matrix $matrix): void
- public function allowOverrideEntity(): bool
- {
- return true;
- }
- public function getAllowedFileExtensions(): array
Method JG\BatchEntityImportBundle\Model\Configuration\AbstractImportConfiguration::getAllowedFileExtensions() return type has no value type specified in iterable type array.
Last edited by Jarek- {
- return ['csv', 'xls', 'xlsx', 'ods'];
- }
- private function parseValueForArrayType(FormFieldDefinition $fieldDefinition, ?string $value): array
- public function getAllowedFileExtensions(): array
- {
- return ['csv', 'xls', 'xlsx', 'ods'];
- }
- private function parseValueForArrayType(FormFieldDefinition $fieldDefinition, ?string $value): array
Method JG\BatchEntityImportBundle\Model\Configuration\AbstractImportConfiguration::parseValueForArrayType() return type has no value type specified in iterable type array.
Last edited by Pierre Svgnt- {
- return $value
- ? explode($fieldDefinition->getOptions()['separator'] ?? ArrayTextType::DEFAULT_SEPARATOR, $value)
- : [];
- }
- $this->records[] = new MatrixRecord($clearedData, $this->getEntityIdValue($data));
- }
- }
- }
- public function getHeader(): array
Method JG\BatchEntityImportBundle\Model\Matrix\Matrix::getHeader() return type has no value type specified in iterable type array.
Last edited by Jarek- {
- return $this->header;
- }
- /**
- public function getRecords(): array
- {
- return $this->records;
- }
- public function getHeaderInfo(string $className): array
Method JG\BatchEntityImportBundle\Model\Matrix\Matrix::getHeaderInfo() return type has no value type specified in iterable type array.
Last edited by Jarosław Grygierek- {
- $info = [];
- $checker = new PropertyExistenceChecker($className);
- foreach ($this->header as $name) {
- }
- return null;
- }
- private function clearHeader(array $header): array
Method JG\BatchEntityImportBundle\Model\Matrix\Matrix::clearHeader() return type has no value type specified in iterable type array.
Last edited by Jarek- {
- $header = array_values(
- array_filter($header, $this->isColumnNameValid(...)),
- );
- );
- return \array_map(static fn (string $name): string => \str_replace(' ', '_', $name), $header);
- }
- private function clearRecordData(array $data): array
Method JG\BatchEntityImportBundle\Model\Matrix\Matrix::clearRecordData() return type has no value type specified in iterable type array.
Last edited by Jarek- {
- return array_filter($data, $this->isColumnNameValid(...), ARRAY_FILTER_USE_KEY);
- }
- private function isColumnNameValid(?string $name): bool
- }
- parent::validateArguments($value, $constraint);
- }
- private function getHashedMatrixRecordsDataForDuplicationCheck(array $records, array $fieldsUsedInDuplicationCheck): array
Method JG\BatchEntityImportBundle\Validator\Constraints\MatrixRecordUniqueValidator::getHashedMatrixRecordsDataForDuplicationCheck() return type has no value type specified in iterable type array.
Last edited by Jarek- {
- return array_map(
- fn (MatrixRecord $record): string => $this->getHash($this->getMatrixRecordDataForDuplicationCheck($record, $fieldsUsedInDuplicationCheck)),
- $records
- );
- fn (MatrixRecord $record): string => $this->getHash($this->getMatrixRecordDataForDuplicationCheck($record, $fieldsUsedInDuplicationCheck)),
- $records
- );
- }
- 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));
- }
- }
- }
- parent::validateArguments($value, $constraint);
- }
- private function getMatrixRecordDataToCompare(MatrixRecord $matrixRecord, array $fields): array
Method JG\BatchEntityImportBundle\Validator\Constraints\DatabaseEntityUniqueValidator::getMatrixRecordDataToCompare() return type has no value type specified in iterable type array.
Last edited by Jarek- {
- $data = [];
- foreach ($fields as $field) {
- $data[$field] = $matrixRecord->$field;
- }
- private function markAsCorrectRecord(array $matrixDataToCompare): void
- {
- $this->correctRecords[$this->getHash($matrixDataToCompare)] = true;
- }
- private function buildCriteria(MatrixRecord $matrixRecord, array $matrixDataToCompare): array
Method JG\BatchEntityImportBundle\Validator\Constraints\DatabaseEntityUniqueValidator::buildCriteria() return type has no value type specified in iterable type array.
Last edited by Jarek- {
- $criteria = [];
- foreach ($matrixDataToCompare as $fieldName => $value) {
- $criteria[ColumnNameHelper::toCamelCase($fieldName)][] = ['=', $value];
- }
- 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;
- }
- }
- 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
- enum CsvDelimiterEnum: string
- {
- case SEMICOLON = ';';
- case COMMA = ',';
- public static function asValues(): array
Method JG\BatchEntityImportBundle\Enums\CsvDelimiterEnum::asValues() return type has no value type specified in iterable type array.
Last edited by Jarosław Grygierek- {
- return [
- self::SEMICOLON->value,
- self::COMMA->value,
- ];
- }
- 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;
- public function getDefaultOption(): string
- {
- return 'entityClassName';
- }
- public function getRequiredOptions(): array
Method JG\BatchEntityImportBundle\Validator\Constraints\DatabaseEntityUnique::getRequiredOptions() return type has no value type specified in iterable type array.
Last edited by Jarek- {
- return ['entityClassName', 'fields'];
- }
- }
- public function getDefaultOption(): string
- {
- return 'fields';
- }
- public function getRequiredOptions(): array
Method JG\BatchEntityImportBundle\Validator\Constraints\MatrixRecordUnique::getRequiredOptions() return type has no value type specified in iterable type array.
Last edited by Jarek- {
- return ['fields'];
- }
- }
Your project should not use invalid return types 4
- Read doc
- Reliability
- Major
More information: https://insight.symfony.com/what-we-analyse/php.invalid_return_typehint
- 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.
Last edited by Jarek- }
- }
- return null;
- }
- $reader->setDelimiter($detectedDelimiter->value);
- } elseif ($reader instanceof Xls || $reader instanceof Xlsx) {
- $reader->setIgnoreRowsWithNoCells(true);
- }
- return $reader;
Method JG\BatchEntityImportBundle\Model\Matrix\MatrixFactory::getReader() should return PhpOffice\PhpSpreadsheet\Reader\BaseReader but returns object.
Last edited by Jarosław Grygierek- }
- }
- foreach ($delimiters as $delimiter) {
- $delimiterCount[$delimiter] = substr_count($csvContent, (string) $delimiter);
- }
- return array_search(max($delimiterCount), $delimiterCount, true);
Method JG\BatchEntityImportBundle\Service\CsvDelimiterDetector::detectDelimiter() should return string but returns int|string|false.
Last edited by Jarosław Grygierek- }
- }
- }
- public function getTemplate(string $name): string
- {
- if (array_key_exists($name, $this->availableTemplates)) {
- return $this->availableTemplates[$name];
Method JG\BatchEntityImportBundle\Twig\TemplateExtension::getTemplate() should return string but returns mixed.
Last edited by Jarek- }
- throw new UnexpectedValueException("Template $name not found.");
- }
- }
Your project must not contain invalid instantiations 3
- Read doc
- Reliability
- Major
More information: https://insight.symfony.com/what-we-analyse/php.invalid_instantiation
- $data = $spreadsheet->getActiveSheet()->toArray();
- $header = array_shift($data);
- self::addKeysToRows($header, $data);
- return new Matrix($header, $data);
Parameter #1 $header of class JG\BatchEntityImportBundle\Model\Matrix\Matrix constructor expects array, array<mixed>|null given.
Last edited by Jarek- }
- public static function createFromPostData(array $data): Matrix
- {
- return $data ? new Matrix(array_keys($data[0]), $data) : new Matrix();
- /**
- * @throws ReflectionException
- */
- public function __construct(string $entityClass)
- {
- $this->reflectionClass = new ReflectionClass($entityClass);
Parameter #1 $objectOrClass of class ReflectionClass constructor expects class-string<T of object>|T of object, string given.
Last edited by Micheal- if (\interface_exists(TranslatableInterface::class) && is_subclass_of($entityClass, TranslatableInterface::class)) {
- $this->translationReflectionClass = new ReflectionClass($this->reflectionClass->newInstanceWithoutConstructor()->translate());
- }
- }
- */
- public function __construct(string $entityClass)
- {
- $this->reflectionClass = new ReflectionClass($entityClass);
- if (\interface_exists(TranslatableInterface::class) && is_subclass_of($entityClass, TranslatableInterface::class)) {
- $this->translationReflectionClass = new ReflectionClass($this->reflectionClass->newInstanceWithoutConstructor()->translate());
Parameter #1 $objectOrClass of class ReflectionClass constructor expects class-string<T of object>|T of object, mixed given.
Last edited by Micheal- }
- }
- public function propertyExists(string $name): bool
- {
Your project should only use iterable types in foreach loops 3
- Read doc
- Reliability
- Major
More information: https://insight.symfony.com/what-we-analyse/php.invalid_foreach_iterable
- {
- /** @var MatrixRecord $entity */
- $entity = $form->getData();
- $selectedValue = $entity->entityId;
- foreach ($view['entity']->vars['choices'] ?? [] as $index => $choice) {
Argument of an invalid type mixed supplied for foreach, only iterables are supported.
Last edited by Jarek- if ($choice->value === $selectedValue) {
- $view['entity']->vars['choices'][$index]->attr['selected'] = 'selected';
- }
- }
- }
- $sql = /* @lang DQL */
- "SELECT c FROM $class c";
- $nmb = 0;
- foreach ($criteria as $fieldName => $data) {
- foreach ($data as [$operator, $value]) {
Argument of an invalid type mixed supplied for foreach, only iterables are supported.
Last edited by Jarek- $sql .= $nmb > 0 ? ' AND' : ' WHERE';
- $sql .= " c.$fieldName $operator :param_" . $nmb++;
- }
- }
- private function passParametersToQuery(AbstractQuery $query, array $criteria): void
- {
- $nmb = 0;
- foreach ($criteria as $data) {
- foreach ($data as [$operator, $value]) {
Argument of an invalid type mixed supplied for foreach, only iterables are supported.
Last edited by Jarek- $query->setParameter('param_' . $nmb++, $value);
- }
- }
- }
- }