Completed
Pull Request — master (#1298)
by Narcotic
07:22 queued 02:32
created

ClassMetadataFactory::getFqcnFromAlias()   A

Complexity

Conditions 1
Paths 1

Size

Total Lines 4
Code Lines 2

Duplication

Lines 0
Ratio 0 %

Code Coverage

Tests 0
CRAP Score 2
Metric Value
dl 0
loc 4
ccs 0
cts 2
cp 0
rs 10
cc 1
eloc 2
nc 1
nop 2
crap 2
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 928
    public function setDocumentManager(DocumentManager $dm)
62
    {
63 928
        $this->dm = $dm;
64 928
    }
65
66
    /**
67
     * Sets the Configuration instance
68
     *
69
     * @param Configuration $config
70
     */
71 928
    public function setConfiguration(Configuration $config)
72
    {
73 928
        $this->config = $config;
74 928
    }
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 745
    protected function initialize()
81
    {
82 745
        $this->driver = $this->config->getMetadataDriverImpl();
83 745
        $this->evm = $this->dm->getEventManager();
84 745
        $this->initialized = true;
85 745
    }
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 290
    protected function getDriver()
99
    {
100 290
        return $this->driver;
101
    }
102
103
    /**
104
     * {@inheritDoc}
105
     */
106 742
    protected function wakeupReflection(ClassMetadataInterface $class, ReflectionService $reflService)
107
    {
108 742
    }
109
110
    /**
111
     * {@inheritDoc}
112
     */
113 745
    protected function initializeReflection(ClassMetadataInterface $class, ReflectionService $reflService)
114
    {
115 745
    }
116
117
    /**
118
     * {@inheritDoc}
119
     */
120 742
    protected function isEntity(ClassMetadataInterface $class)
121
    {
122 742
        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 745
    protected function doLoadMetadata($class, $parent, $rootEntityFound, array $nonSuperclassParents = array())
129
    {
130
        /** @var $class ClassMetadata */
131
        /** @var $parent ClassMetadata */
132 745
        if ($parent) {
133 288
            $class->setInheritanceType($parent->inheritanceType);
134 288
            $class->setDiscriminatorField($parent->discriminatorField);
135 288
            $class->setDiscriminatorMap($parent->discriminatorMap);
136 288
            $class->setDefaultDiscriminatorValue($parent->defaultDiscriminatorValue);
137 288
            $class->setIdGeneratorType($parent->generatorType);
138 288
            $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 288
            $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 288
            $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 288
            $class->setIdentifier($parent->identifier);
142 288
            $class->setVersioned($parent->isVersioned);
143 288
            $class->setVersionField($parent->versionField);
144 288
            $class->setLifecycleCallbacks($parent->lifecycleCallbacks);
145 288
            $class->setAlsoLoadMethods($parent->alsoLoadMethods);
146 288
            $class->setChangeTrackingPolicy($parent->changeTrackingPolicy);
147 288
            $class->setFile($parent->getFile());
148 288
            if ($parent->isMappedSuperclass) {
149 240
                $class->setCustomRepositoryClass($parent->customRepositoryClassName);
150 240
            }
151 288
        }
152
153
        // Invoke driver
154
        try {
155 745
            $this->driver->loadMetadataForClass($class->getName(), $class);
156 745
        } catch (\ReflectionException $e) {
157
            throw MappingException::reflectionFailure($class->getName(), $e);
158
        }
159
160 742
        $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 742
        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 60
            if ($parent->generatorType) {
164 60
                $class->setIdGeneratorType($parent->generatorType);
165 60
            }
166 60
            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 60
            if ($parent->idGenerator) {
170 60
                $class->setIdGenerator($parent->idGenerator);
171 60
            }
172 60
        } else {
173 742
            $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 742
        if ($parent && $parent->isInheritanceTypeSingleCollection()) {
177 48
            $class->setDatabase($parent->getDatabase());
178 48
            $class->setCollection($parent->getCollection());
179 48
        }
180
181 742
        $class->setParentClasses($nonSuperclassParents);
182
183 742 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 742
    }
188
189
    /**
190
     * Validates the identifier mapping.
191
     *
192
     * @param ClassMetadata $class
193
     * @throws MappingException
194
     */
195 742
    protected function validateIdentifier($class)
196
    {
197 742
        if ( ! $class->identifier && ! $class->isMappedSuperclass && ! $class->isEmbeddedDocument) {
198
            throw MappingException::identifierRequired($class->name);
199
        }
200 742
    }
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 745
    protected function newClassMetadataInstance($className)
209
    {
210 745
        return new ClassMetadata($className);
211
    }
212
213 742
    private function completeIdGeneratorMapping(ClassMetadataInfo $class)
214
    {
215 742
        $idGenOptions = $class->generatorOptions;
216 742
        switch ($class->generatorType) {
217 742
            case ClassMetadata::GENERATOR_TYPE_AUTO:
218 677
                $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 677
                break;
220 117
            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 111
            case ClassMetadata::GENERATOR_TYPE_UUID:
231 4
                $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 4
                isset($idGenOptions['salt']) && $uuidGenerator->setSalt($idGenOptions['salt']);
233 4
                $class->setIdGenerator($uuidGenerator);
234 4
                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 742
        }
275 742
    }
276
277
    /**
278
     * Adds inherited fields to the subclass mapping.
279
     *
280
     * @param ClassMetadata $subClass
281
     * @param ClassMetadata $parentClass
282
     */
283 288
    private function addInheritedFields(ClassMetadata $subClass, ClassMetadata $parentClass)
284
    {
285 288
        foreach ($parentClass->fieldMappings as $fieldName => $mapping) {
286 76 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 71
                $mapping['inherited'] = $parentClass->name;
288 71
            }
289 76
            if ( ! isset($mapping['declared'])) {
290 76
                $mapping['declared'] = $parentClass->name;
291 76
            }
292 76
            $subClass->addInheritedFieldMapping($mapping);
293 288
        }
294 288
        foreach ($parentClass->reflFields as $name => $field) {
295 76
            $subClass->reflFields[$name] = $field;
296 288
        }
297 288
    }
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 288
    private function addInheritedRelations(ClassMetadata $subClass, ClassMetadata $parentClass)
311
    {
312 288
        foreach ($parentClass->associationMappings as $field => $mapping) {
313 44
            if ($parentClass->isMappedSuperclass) {
314 3
                $mapping['sourceDocument'] = $subClass->name;
315 3
            }
316
317 44 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 41
                $mapping['inherited'] = $parentClass->name;
319 41
            }
320 44
            if ( ! isset($mapping['declared'])) {
321 44
                $mapping['declared'] = $parentClass->name;
322 44
            }
323 44
            $subClass->addInheritedAssociationMapping($mapping);
324 288
        }
325 288
    }
326
327
    /**
328
     * Adds inherited indexes to the subclass mapping.
329
     *
330
     * @param ClassMetadata $subClass
331
     * @param ClassMetadata $parentClass
332
     */
333 288
    private function addInheritedIndexes(ClassMetadata $subClass, ClassMetadata $parentClass)
334
    {
335 288
        foreach ($parentClass->indexes as $index) {
336 18
            $subClass->addIndex($index['keys'], $index['options']);
337 288
        }
338 288
    }
339
}
340