LAG\AdminBundle\Bridge\Doctrine\ORM\Metadata\MetadataPropertyFactory::createProperties()

Complexity

Conditions	8
Paths	16

Size

Total Lines	34
Code Lines	19

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	1

<?php
    public function createProperties(string $resourceClass): array
    {
        $metadata = $this->metadataHelper->findMetadata($resourceClass);
        $properties = [];

        if ($metadata === null) {
            return $properties;
        }

        foreach ($metadata->getFieldNames() as $fieldName) {
            $fieldType = $metadata->getTypeOfField($fieldName);

            if (str_contains($fieldType, 'datetime')) {

It seems like $fieldType can also be of type null; however, parameter $haystack of str_contains() does only seem to accept string, maybe add an additional type check?

                $properties[$fieldName] = new DateProperty($fieldName);
            } elseif ($fieldType === 'boolean') {
                $properties[$fieldName] = new BooleanProperty($fieldName);
            } else {
                $properties[$fieldName] = new StringProperty($fieldName);
            }

            if (\count($properties) > 10) {
                return $properties;
            }
        }

        foreach ($metadata->getAssociationNames() as $associationName) {
            $properties[$associationName] = new CountProperty($associationName);

            if (\count($properties) > 10) {
                return $properties;
            }
        }

        return $properties;
    }