ClassMetadataFactory::fetchClassMetadata() - Code Metrics - egeloen/ivory-serializer - Measure and Improve Code Quality continuously with Scrutinizer

ClassMetadataFactory::fetchClassMetadata() A
last analyzed 2017-02-27 21:52 UTC

↳ Parent: ClassMetadataFactory

Complexity

Conditions	4
Paths	8

Size

Total Lines	14
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	9
CRAP Score	4

Importance

Changes

Metric	Value
dl	0
loc	14
ccs	9
cts	9
cp	1
rs	9.2
c	0
b	0
f	0
cc	4
eloc	8
nc	8
nop	1
crap	4

<?php

/*
 * This file is part of the Ivory Serializer package.
 *
 * (c) Eric GELOEN <[email protected]>
 *
 * For the full copyright and license information, please read the LICENSE
 * file that was distributed with this source code.
 */

namespace Ivory\Serializer\Mapping\Factory;

use Doctrine\Common\Annotations\AnnotationReader;
use Ivory\Serializer\Mapping\ClassMetadata;
use Ivory\Serializer\Mapping\ClassMetadataInterface;
use Ivory\Serializer\Mapping\Loader\AnnotationClassMetadataLoader;
use Ivory\Serializer\Mapping\Loader\ChainClassMetadataLoader;
use Ivory\Serializer\Mapping\Loader\ClassMetadataLoaderInterface;
use Ivory\Serializer\Mapping\Loader\ReflectionClassMetadataLoader;
use phpDocumentor\Reflection\ClassReflector;
use Symfony\Component\PropertyInfo\Extractor\PhpDocExtractor;
use Symfony\Component\PropertyInfo\Extractor\ReflectionExtractor;
use Symfony\Component\PropertyInfo\PropertyInfoExtractor;

/**
 * @author GeLo <[email protected]>
 */
class ClassMetadataFactory extends AbstractClassMetadataFactory
{
    /**
     * @var ClassMetadataLoaderInterface
     */
    private $loader;

    /**
     * @param ClassMetadataLoaderInterface $loader
     */
    public function __construct(ClassMetadataLoaderInterface $loader)
    {
        $this->loader = $loader;
    }

    /**
     * @param ClassMetadataLoaderInterface[] $loaders
     *
     * @return ClassMetadataFactoryInterface
     */
    public static function create(array $loaders = [])
    {
        if (empty($loaders)) {
            $extractor = null;

            if (class_exists(PropertyInfoExtractor::class)) {
                $extractors = $typeExtractors = [new ReflectionExtractor()];

                if (class_exists(ClassReflector::class)) {
                    array_unshift($typeExtractors, new PhpDocExtractor());
                }

                $extractor = new PropertyInfoExtractor($extractors, $typeExtractors, [], $extractors);
            }

            $loaders = [new ReflectionClassMetadataLoader($extractor)];

            if (class_exists(AnnotationReader::class)) {
                $loaders[] = new AnnotationClassMetadataLoader(new AnnotationReader());
            }
        }

        return new static(count($loaders) > 1 ? new ChainClassMetadataLoader($loaders) : array_shift($loaders));
/** @return stdClass|null */
function mayReturnNull() { }

function doesNotAcceptNull(stdClass $x) { }

// With potential error.
function withoutCheck() {
    $x = mayReturnNull();
    doesNotAcceptNull($x); // Potential error here.
}

// Safe - Alternative 1
function withCheck1() {
    $x = mayReturnNull();
    if ( ! $x instanceof stdClass) {
        throw new \LogicException('$x must be defined.');
    }
    doesNotAcceptNull($x);
}

// Safe - Alternative 2
function withCheck2() {
    $x = mayReturnNull();
    if ($x instanceof stdClass) {
        doesNotAcceptNull($x);
    }
}
    }

    /**
     * {@inheritdoc}
     */
    protected function fetchClassMetadata($class)
    {
        $classMetadata = new ClassMetadata($class);
        $found = false;

        if (($parentMetadata = $this->getParentClassMetadata($class)) !== null) {
            $classMetadata->merge($parentMetadata);
            $found = true;
        }

        $found = $this->loader->loadClassMetadata($classMetadata) || $found;

        return $found ? $classMetadata : null;
    }

    /**
     * @param string $class
     *
     * @return ClassMetadataInterface|null
     */
    private function getParentClassMetadata($class)
    {
        if (($parent = get_parent_class($class)) !== false) {
            return $this->getClassMetadata($parent);
        }
    }
}


1		<?php
2
3		/*
4		* This file is part of the Ivory Serializer package.
5		*
6		* (c) Eric GELOEN <[email protected]>
7		*
8		* For the full copyright and license information, please read the LICENSE
9		* file that was distributed with this source code.
10		*/
11
12		namespace Ivory\Serializer\Mapping\Factory;
13
14		use Doctrine\Common\Annotations\AnnotationReader;
15		use Ivory\Serializer\Mapping\ClassMetadata;
16		use Ivory\Serializer\Mapping\ClassMetadataInterface;
17		use Ivory\Serializer\Mapping\Loader\AnnotationClassMetadataLoader;
18		use Ivory\Serializer\Mapping\Loader\ChainClassMetadataLoader;
19		use Ivory\Serializer\Mapping\Loader\ClassMetadataLoaderInterface;
20		use Ivory\Serializer\Mapping\Loader\ReflectionClassMetadataLoader;
21		use phpDocumentor\Reflection\ClassReflector;
22		use Symfony\Component\PropertyInfo\Extractor\PhpDocExtractor;
23		use Symfony\Component\PropertyInfo\Extractor\ReflectionExtractor;
24		use Symfony\Component\PropertyInfo\PropertyInfoExtractor;
25
26		/**
27		* @author GeLo <[email protected]>
28		*/
29		class ClassMetadataFactory extends AbstractClassMetadataFactory
30		{
31		/**
32		* @var ClassMetadataLoaderInterface
33		*/
34		private $loader;
35
36		/**
37		* @param ClassMetadataLoaderInterface $loader
38		*/
39	1512	public function __construct(ClassMetadataLoaderInterface $loader)
40		{
41	1512	$this->loader = $loader;
42	1512	}
43
44		/**
45		* @param ClassMetadataLoaderInterface[] $loaders
46		*
47		* @return ClassMetadataFactoryInterface
48		*/
49	1492	public static function create(array $loaders = [])
50		{
51	1492	if (empty($loaders)) {
52	1492	$extractor = null;
53
54	1492	if (class_exists(PropertyInfoExtractor::class)) {
55	1492	$extractors = $typeExtractors = [new ReflectionExtractor()];
56
57	1492	if (class_exists(ClassReflector::class)) {
58	1492	array_unshift($typeExtractors, new PhpDocExtractor());
59	746	}
60
61	1492	$extractor = new PropertyInfoExtractor($extractors, $typeExtractors, [], $extractors);
62	746	}
63
64	1492	$loaders = [new ReflectionClassMetadataLoader($extractor)];
65
66	1492	if (class_exists(AnnotationReader::class)) {
67	1492	$loaders[] = new AnnotationClassMetadataLoader(new AnnotationReader());
68	746	}
69	746	}
70
71	1492	return new static(count($loaders) > 1 ? new ChainClassMetadataLoader($loaders) : array_shift($loaders));
		0 ignored issues – show Bug introduced 2016-09-21 00:53 UTC by Report Bug Copy Issue Report It seems like `count($loaders) > 1 ? ne...: array_shift($loaders)` can be `null`; however, `__construct()` does not accept `null`, maybe add an additional type check? Unless you are absolutely sure that the expression can never be null because of other conditions, we strongly recommend to add an additional type check to your code: /** @return stdClass\|null */ function mayReturnNull() { } function doesNotAcceptNull(stdClass $x) { } // With potential error. function withoutCheck() { $x = mayReturnNull(); doesNotAcceptNull($x); // Potential error here. } // Safe - Alternative 1 function withCheck1() { $x = mayReturnNull(); if ( ! $x instanceof stdClass) { throw new \LogicException('$x must be defined.'); } doesNotAcceptNull($x); } // Safe - Alternative 2 function withCheck2() { $x = mayReturnNull(); if ($x instanceof stdClass) { doesNotAcceptNull($x); } } Loading history...
72		}
73
74		/**
75		* {@inheritdoc}
76		*/
77	1104	protected function fetchClassMetadata($class)
78		{
79	1104	$classMetadata = new ClassMetadata($class);
80	1100	$found = false;
81
82	1100	if (($parentMetadata = $this->getParentClassMetadata($class)) !== null) {
83	4	$classMetadata->merge($parentMetadata);
84	4	$found = true;
85	2	}
86
87	1100	$found = $this->loader->loadClassMetadata($classMetadata) \|\| $found;
88
89	1100	return $found ? $classMetadata : null;
90		}
91
92		/**
93		* @param string $class
94		*
95		* @return ClassMetadataInterface\|null
96		*/
97	1100	private function getParentClassMetadata($class)
98		{
99	1100	if (($parent = get_parent_class($class)) !== false) {
100	4	return $this->getClassMetadata($parent);
101		}
102	1100	}
103		}
104

egeloen / ivory-serializer

ClassMetadataFactory::fetchClassMetadata() A last analyzed 2017-02-27 21:52 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

ClassMetadataFactory::fetchClassMetadata() A
last analyzed 2017-02-27 21:52 UTC