Completed
Pull Request — master (#1304)
by Tuan
07:52
created

ClassMetadataFactory::initializeReflection()   A

Complexity

Conditions 1
Paths 1

Size

Total Lines 3
Code Lines 1

Duplication

Lines 0
Ratio 0 %

Code Coverage

Tests 2
CRAP Score 1
Metric Value
dl 0
loc 3
ccs 2
cts 2
cp 1
rs 10
cc 1
eloc 1
nc 1
nop 2
crap 1
1
<?php
2
/*
3
 * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
4
 * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
5
 * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
6
 * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
7
 * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
8
 * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
9
 * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
10
 * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
11
 * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
12
 * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
13
 * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
14
 *
15
 * This software consists of voluntary contributions made by many individuals
16
 * and is licensed under the MIT license. For more information, see
17
 * <http://www.doctrine-project.org>.
18
 */
19
20
namespace Doctrine\ODM\MongoDB\Mapping;
21
22
use Doctrine\Common\Persistence\Mapping\AbstractClassMetadataFactory;
23
use Doctrine\Common\Persistence\Mapping\ClassMetadata as ClassMetadataInterface;
24
use Doctrine\Common\Persistence\Mapping\ReflectionService;
25
use Doctrine\ODM\MongoDB\Configuration;
26
use Doctrine\ODM\MongoDB\DocumentManager;
27
use Doctrine\ODM\MongoDB\Events;
28
use Doctrine\ODM\MongoDB\Mapping\ClassMetadata;
29
use Doctrine\ODM\MongoDB\Mapping\MappingException;
30
31
/**
32
 * The ClassMetadataFactory is used to create ClassMetadata objects that contain all the
33
 * metadata mapping informations of a class which describes how a class should be mapped
34
 * to a document database.
35
 *
36
 * @since       1.0
37
 * @author      Jonathan H. Wage <[email protected]>
38
 * @author      Roman Borschel <[email protected]>
39
 */
40
class ClassMetadataFactory extends AbstractClassMetadataFactory
41
{
42
    protected $cacheSalt = "\$MONGODBODMCLASSMETADATA";
43
44
    /** @var DocumentManager The DocumentManager instance */
45
    private $dm;
46
47
    /** @var Configuration The Configuration instance */
48
    private $config;
49
50
    /** @var \Doctrine\Common\Persistence\Mapping\Driver\MappingDriver The used metadata driver. */
51
    private $driver;
52
53
    /** @var \Doctrine\Common\EventManager The event manager instance */
54
    private $evm;
55
56
    /**
57
     * Sets the DocumentManager instance for this class.
58
     *
59
     * @param DocumentManager $dm The DocumentManager instance
60
     */
61 915
    public function setDocumentManager(DocumentManager $dm)
62
    {
63 915
        $this->dm = $dm;
64 915
    }
65
66
    /**
67
     * Sets the Configuration instance
68
     *
69
     * @param Configuration $config
70
     */
71 915
    public function setConfiguration(Configuration $config)
72
    {
73 915
        $this->config = $config;
74 915
    }
75
76
    /**
77
     * Lazy initialization of this stuff, especially the metadata driver,
78
     * since these are not needed at all when a metadata cache is active.
79
     */
80 732
    protected function initialize()
81
    {
82 732
        $this->driver = $this->config->getMetadataDriverImpl();
83 732
        $this->evm = $this->dm->getEventManager();
84 732
        $this->initialized = true;
85 732
    }
86
87
    /**
88
     * {@inheritDoc}
89
     */
90
    protected function getFqcnFromAlias($namespaceAlias, $simpleClassName)
91
    {
92
        return $this->config->getDocumentNamespace($namespaceAlias) . '\\' . $simpleClassName;
93
    }
94
95
    /**
96
     * {@inheritDoc}
97
     */
98 279
    protected function getDriver()
99
    {
100 279
        return $this->driver;
101
    }
102
103
    /**
104
     * {@inheritDoc}
105
     */
106 729
    protected function wakeupReflection(ClassMetadataInterface $class, ReflectionService $reflService)
107
    {
108 729
    }
109
110
    /**
111
     * {@inheritDoc}
112
     */
113 732
    protected function initializeReflection(ClassMetadataInterface $class, ReflectionService $reflService)
114
    {
115 732
    }
116
117
    /**
118
     * {@inheritDoc}
119
     */
120 729
    protected function isEntity(ClassMetadataInterface $class)
121
    {
122 729
        return ! $class->isMappedSuperclass && ! $class->isEmbeddedDocument;
0 ignored issues
show
Bug introduced by
Accessing isMappedSuperclass on the interface Doctrine\Common\Persistence\Mapping\ClassMetadata suggest that you code against a concrete implementation. How about adding an instanceof check?

If you access a property on an interface, you most likely code against a concrete implementation of the interface.

Available Fixes

  1. Adding an additional type check:

    interface SomeInterface { }
    class SomeClass implements SomeInterface {
        public $a;
    }
    
    function someFunction(SomeInterface $object) {
        if ($object instanceof SomeClass) {
            $a = $object->a;
        }
    }
    
  2. Changing the type hint:

    interface SomeInterface { }
    class SomeClass implements SomeInterface {
        public $a;
    }
    
    function someFunction(SomeClass $object) {
        $a = $object->a;
    }
    
Loading history...
Bug introduced by
Accessing isEmbeddedDocument on the interface Doctrine\Common\Persistence\Mapping\ClassMetadata suggest that you code against a concrete implementation. How about adding an instanceof check?

If you access a property on an interface, you most likely code against a concrete implementation of the interface.

Available Fixes

  1. Adding an additional type check:

    interface SomeInterface { }
    class SomeClass implements SomeInterface {
        public $a;
    }
    
    function someFunction(SomeInterface $object) {
        if ($object instanceof SomeClass) {
            $a = $object->a;
        }
    }
    
  2. Changing the type hint:

    interface SomeInterface { }
    class SomeClass implements SomeInterface {
        public $a;
    }
    
    function someFunction(SomeClass $object) {
        $a = $object->a;
    }
    
Loading history...
123
    }
124
125
    /**
126
     * {@inheritDoc}
127
     */
128 732
    protected function doLoadMetadata($class, $parent, $rootEntityFound, array $nonSuperclassParents = array())
129
    {
130
        /** @var $class ClassMetadata */
131
        /** @var $parent ClassMetadata */
132 732
        if ($parent) {
133 277
            $class->setInheritanceType($parent->inheritanceType);
134 277
            $class->setDiscriminatorField($parent->discriminatorField);
135 277
            $class->setDiscriminatorMap($parent->discriminatorMap);
136 277
            $class->setDefaultDiscriminatorValue($parent->defaultDiscriminatorValue);
137 277
            $class->setIdGeneratorType($parent->generatorType);
138 277
            $this->addInheritedFields($class, $parent);
0 ignored issues
show
Compatibility introduced by
$class of type object<Doctrine\Common\P...\Mapping\ClassMetadata> is not a sub-type of object<Doctrine\ODM\Mong...\Mapping\ClassMetadata>. It seems like you assume a concrete implementation of the interface Doctrine\Common\Persistence\Mapping\ClassMetadata to be always present.

This check looks for parameters that are defined as one type in their type hint or doc comment but seem to be used as a narrower type, i.e an implementation of an interface or a subclass.

Consider changing the type of the parameter or doing an instanceof check before assuming your parameter is of the expected type.

Loading history...
139 277
            $this->addInheritedRelations($class, $parent);
0 ignored issues
show
Compatibility introduced by
$class of type object<Doctrine\Common\P...\Mapping\ClassMetadata> is not a sub-type of object<Doctrine\ODM\Mong...\Mapping\ClassMetadata>. It seems like you assume a concrete implementation of the interface Doctrine\Common\Persistence\Mapping\ClassMetadata to be always present.

This check looks for parameters that are defined as one type in their type hint or doc comment but seem to be used as a narrower type, i.e an implementation of an interface or a subclass.

Consider changing the type of the parameter or doing an instanceof check before assuming your parameter is of the expected type.

Loading history...
140 277
            $this->addInheritedIndexes($class, $parent);
0 ignored issues
show
Compatibility introduced by
$class of type object<Doctrine\Common\P...\Mapping\ClassMetadata> is not a sub-type of object<Doctrine\ODM\Mong...\Mapping\ClassMetadata>. It seems like you assume a concrete implementation of the interface Doctrine\Common\Persistence\Mapping\ClassMetadata to be always present.

This check looks for parameters that are defined as one type in their type hint or doc comment but seem to be used as a narrower type, i.e an implementation of an interface or a subclass.

Consider changing the type of the parameter or doing an instanceof check before assuming your parameter is of the expected type.

Loading history...
141 277
            $class->setIdentifier($parent->identifier);
142 277
            $class->setVersioned($parent->isVersioned);
143 277
            $class->setVersionField($parent->versionField);
144 277
            $class->setLifecycleCallbacks($parent->lifecycleCallbacks);
145 277
            $class->setAlsoLoadMethods($parent->alsoLoadMethods);
146 277
            $class->setChangeTrackingPolicy($parent->changeTrackingPolicy);
147 277
            $class->setFile($parent->getFile());
148 277
            if ($parent->isMappedSuperclass) {
149 230
                $class->setCustomRepositoryClass($parent->customRepositoryClassName);
150 230
            }
151 277
        }
152
153
        // Invoke driver
154
        try {
155 732
            $this->driver->loadMetadataForClass($class->getName(), $class);
156 732
        } catch (\ReflectionException $e) {
157
            throw MappingException::reflectionFailure($class->getName(), $e);
158
        }
159
160 729
        $this->validateIdentifier($class);
0 ignored issues
show
Compatibility introduced by
$class of type object<Doctrine\Common\P...\Mapping\ClassMetadata> is not a sub-type of object<Doctrine\ODM\Mong...\Mapping\ClassMetadata>. It seems like you assume a concrete implementation of the interface Doctrine\Common\Persistence\Mapping\ClassMetadata to be always present.

This check looks for parameters that are defined as one type in their type hint or doc comment but seem to be used as a narrower type, i.e an implementation of an interface or a subclass.

Consider changing the type of the parameter or doing an instanceof check before assuming your parameter is of the expected type.

Loading history...
161
162 729
        if ($parent && $rootEntityFound && $parent->generatorType === $class->generatorType) {
0 ignored issues
show
Bug introduced by
Accessing generatorType on the interface Doctrine\Common\Persistence\Mapping\ClassMetadata suggest that you code against a concrete implementation. How about adding an instanceof check?

If you access a property on an interface, you most likely code against a concrete implementation of the interface.

Available Fixes

  1. Adding an additional type check:

    interface SomeInterface { }
    class SomeClass implements SomeInterface {
        public $a;
    }
    
    function someFunction(SomeInterface $object) {
        if ($object instanceof SomeClass) {
            $a = $object->a;
        }
    }
    
  2. Changing the type hint:

    interface SomeInterface { }
    class SomeClass implements SomeInterface {
        public $a;
    }
    
    function someFunction(SomeClass $object) {
        $a = $object->a;
    }
    
Loading history...
163 59
            if ($parent->generatorType) {
164 59
                $class->setIdGeneratorType($parent->generatorType);
165 59
            }
166 59
            if ($parent->generatorOptions) {
0 ignored issues
show
Bug Best Practice introduced by
The expression $parent->generatorOptions of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

This check marks implicit conversions of arrays to boolean values in a comparison. While in PHP an empty array is considered to be equal (but not identical) to false, this is not always apparent.

Consider making the comparison explicit by using empty(..) or ! empty(...) instead.

Loading history...
167
                $class->setIdGeneratorOptions($parent->generatorOptions);
168
            }
169 59
            if ($parent->idGenerator) {
170 59
                $class->setIdGenerator($parent->idGenerator);
171 59
            }
172 59
        } else {
173 729
            $this->completeIdGeneratorMapping($class);
0 ignored issues
show
Compatibility introduced by
$class of type object<Doctrine\Common\P...\Mapping\ClassMetadata> is not a sub-type of object<Doctrine\ODM\Mong...ping\ClassMetadataInfo>. It seems like you assume a concrete implementation of the interface Doctrine\Common\Persistence\Mapping\ClassMetadata to be always present.

This check looks for parameters that are defined as one type in their type hint or doc comment but seem to be used as a narrower type, i.e an implementation of an interface or a subclass.

Consider changing the type of the parameter or doing an instanceof check before assuming your parameter is of the expected type.

Loading history...
174
        }
175
176 729
        if ($parent && $parent->isInheritanceTypeSingleCollection()) {
177 48
            $class->setDatabase($parent->getDatabase());
178 48
            $class->setCollection($parent->getCollection());
179 48
        }
180
181 729
        $class->setParentClasses($nonSuperclassParents);
182
183 729 View Code Duplication
        if ($this->evm->hasListeners(Events::loadClassMetadata)) {
0 ignored issues
show
Duplication introduced by
This code seems to be duplicated across your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
184 2
            $eventArgs = new \Doctrine\ODM\MongoDB\Event\LoadClassMetadataEventArgs($class, $this->dm);
185 2
            $this->evm->dispatchEvent(Events::loadClassMetadata, $eventArgs);
186 2
        }
187 729
    }
188
189
    /**
190
     * Validates the identifier mapping.
191
     *
192
     * @param ClassMetadata $class
193
     * @throws MappingException
194
     */
195 729
    protected function validateIdentifier($class)
196
    {
197 729
        if ( ! $class->identifier && ! $class->isMappedSuperclass && ! $class->isEmbeddedDocument) {
198
            throw MappingException::identifierRequired($class->name);
199
        }
200 729
    }
201
202
    /**
203
     * Creates a new ClassMetadata instance for the given class name.
204
     *
205
     * @param string $className
206
     * @return \Doctrine\ODM\MongoDB\Mapping\ClassMetadata
207
     */
208 732
    protected function newClassMetadataInstance($className)
209
    {
210 732
        return new ClassMetadata($className);
211
    }
212
213 729
    private function completeIdGeneratorMapping(ClassMetadataInfo $class)
214
    {
215 729
        $idGenOptions = $class->generatorOptions;
216 729
        switch ($class->generatorType) {
217 729
            case ClassMetadata::GENERATOR_TYPE_AUTO:
218 665
                $class->setIdGenerator(new \Doctrine\ODM\MongoDB\Id\AutoGenerator($class));
0 ignored issues
show
Unused Code introduced by
The call to AutoGenerator::__construct() has too many arguments starting with $class.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
219 665
                break;
220 116
            case ClassMetadata::GENERATOR_TYPE_INCREMENT:
221 6
                $incrementGenerator = new \Doctrine\ODM\MongoDB\Id\IncrementGenerator($class);
0 ignored issues
show
Unused Code introduced by
The call to IncrementGenerator::__construct() has too many arguments starting with $class.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
222 6
                if (isset($idGenOptions['key'])) {
223
                    $incrementGenerator->setKey($idGenOptions['key']);
224
                }
225 6
                if (isset($idGenOptions['collection'])) {
226
                    $incrementGenerator->setCollection($idGenOptions['collection']);
227
                }
228 6
                $class->setIdGenerator($incrementGenerator);
229 6
                break;
230 110
            case ClassMetadata::GENERATOR_TYPE_UUID:
231 3
                $uuidGenerator = new \Doctrine\ODM\MongoDB\Id\UuidGenerator($class);
0 ignored issues
show
Unused Code introduced by
The call to UuidGenerator::__construct() has too many arguments starting with $class.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
232 3
                isset($idGenOptions['salt']) && $uuidGenerator->setSalt($idGenOptions['salt']);
233 3
                $class->setIdGenerator($uuidGenerator);
234 3
                break;
235 107
            case ClassMetadata::GENERATOR_TYPE_ALNUM:
236 1
                $alnumGenerator = new \Doctrine\ODM\MongoDB\Id\AlnumGenerator($class);
0 ignored issues
show
Unused Code introduced by
The call to AlnumGenerator::__construct() has too many arguments starting with $class.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
237 1
                if (isset($idGenOptions['pad'])) {
238
                    $alnumGenerator->setPad($idGenOptions['pad']);
239
                }
240 1
                if (isset($idGenOptions['chars'])) {
241 1
                    $alnumGenerator->setChars($idGenOptions['chars']);
242 1
                } elseif (isset($idGenOptions['awkwardSafe'])) {
243
                    $alnumGenerator->setAwkwardSafeMode($idGenOptions['awkwardSafe']);
244
                }
245
246 1
                $class->setIdGenerator($alnumGenerator);
247 1
                break;
248 106
            case ClassMetadata::GENERATOR_TYPE_CUSTOM:
249
                if (empty($idGenOptions['class'])) {
250
                    throw MappingException::missingIdGeneratorClass($class->name);
251
                }
252
253
                $customGenerator = new $idGenOptions['class'];
254
                unset($idGenOptions['class']);
255
                if ( ! $customGenerator instanceof \Doctrine\ODM\MongoDB\Id\AbstractIdGenerator) {
256
                    throw MappingException::classIsNotAValidGenerator(get_class($customGenerator));
257
                }
258
259
                $methods = get_class_methods($customGenerator);
260
                foreach ($idGenOptions as $name => $value) {
261
                    $method = 'set' . ucfirst($name);
262
                    if ( ! in_array($method, $methods)) {
263
                        throw MappingException::missingGeneratorSetter(get_class($customGenerator), $name);
264
                    }
265
266
                    $customGenerator->$method($value);
267
                }
268
                $class->setIdGenerator($customGenerator);
269
                break;
270 106
            case ClassMetadata::GENERATOR_TYPE_NONE;
271 106
                break;
272
            default:
273
                throw new MappingException("Unknown generator type: " . $class->generatorType);
274 729
        }
275 729
    }
276
277
    /**
278
     * Adds inherited fields to the subclass mapping.
279
     *
280
     * @param ClassMetadata $subClass
281
     * @param ClassMetadata $parentClass
282
     */
283 277
    private function addInheritedFields(ClassMetadata $subClass, ClassMetadata $parentClass)
284
    {
285 277
        foreach ($parentClass->fieldMappings as $fieldName => $mapping) {
286 75 View Code Duplication
            if ( ! isset($mapping['inherited']) && ! $parentClass->isMappedSuperclass) {
0 ignored issues
show
Duplication introduced by
This code seems to be duplicated across your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
287 70
                $mapping['inherited'] = $parentClass->name;
288 70
            }
289 75
            if ( ! isset($mapping['declared'])) {
290 75
                $mapping['declared'] = $parentClass->name;
291 75
            }
292 75
            $subClass->addInheritedFieldMapping($mapping);
293 277
        }
294 277
        foreach ($parentClass->reflFields as $name => $field) {
295 75
            $subClass->reflFields[$name] = $field;
296 277
        }
297 277
    }
298
299
300
    /**
301
     * Adds inherited association mappings to the subclass mapping.
302
     *
303
     * @param \Doctrine\ORM\Mapping\ClassMetadata $subClass
304
     * @param \Doctrine\ORM\Mapping\ClassMetadata $parentClass
305
     *
306
     * @return void
307
     *
308
     * @throws MappingException
309
     */
310 277
    private function addInheritedRelations(ClassMetadata $subClass, ClassMetadata $parentClass)
311
    {
312 277
        foreach ($parentClass->associationMappings as $field => $mapping) {
313 43
            if ($parentClass->isMappedSuperclass) {
314 3
                $mapping['sourceDocument'] = $subClass->name;
315 3
            }
316
317 43 View Code Duplication
            if ( ! isset($mapping['inherited']) && ! $parentClass->isMappedSuperclass) {
0 ignored issues
show
Duplication introduced by
This code seems to be duplicated across your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
318 40
                $mapping['inherited'] = $parentClass->name;
319 40
            }
320 43
            if ( ! isset($mapping['declared'])) {
321 43
                $mapping['declared'] = $parentClass->name;
322 43
            }
323 43
            $subClass->addInheritedAssociationMapping($mapping);
324 277
        }
325 277
    }
326
327
    /**
328
     * Adds inherited indexes to the subclass mapping.
329
     *
330
     * @param ClassMetadata $subClass
331
     * @param ClassMetadata $parentClass
332
     */
333 277
    private function addInheritedIndexes(ClassMetadata $subClass, ClassMetadata $parentClass)
334
    {
335 277
        foreach ($parentClass->indexes as $index) {
336 18
            $subClass->addIndex($index['keys'], $index['options']);
337 277
        }
338 277
    }
339
}
340