Psi\Component\ContentType\Metadata\Driver\XmlDriver::loadMetadataFromFile()

Complexity

Conditions	4
Paths	4

Size

Total Lines	36
Code Lines	23

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function loadMetadataFromFile(\ReflectionClass $class, $path)
    {
        $classMetadata = new ClassMetadata($class->getName());

Consider using $class->name. There is an issue with getName() and APC-enabled PHP versions.

        $dom = new \DOMDocument('1.0');
        $dom->load($path);
        $xpath = new \DOMXpath($dom);
        $xpath->registerNamespace('psict', self::XML_NAMESPACE);

        foreach ($xpath->query('//psict:class') as $classEl) {
            $classAttr = $classEl->getAttribute('name');

            if ($classAttr !== $class->getName()) {

Consider using $class->name. There is an issue with getName() and APC-enabled PHP versions.

                throw new \InvalidArgumentException(sprintf(
                    'Expected class name to be "%s" but it is mapped as "%s"',
                    $class->getName(), $classAttr

Consider using $class->name. There is an issue with getName() and APC-enabled PHP versions.

                ));
            }

            foreach ($xpath->query('./psict:field', $classEl) as $fieldEl) {
                $options = $this->extractOptions($xpath, $fieldEl);
                $propertyMetadata = new PropertyMetadata(
                    $class->getName(),

Consider using $class->name. There is an issue with getName() and APC-enabled PHP versions.

                    $fieldEl->getAttribute('name'),
                    $fieldEl->getAttribute('type'),
                    $fieldEl->getAttribute('role'),
                    $fieldEl->getAttribute('group'),
                    $options
                );

                $classMetadata->addPropertyMetadata($propertyMetadata);
            }
        }

        return $classMetadata;
    }