doctrine / orm

lib/Doctrine/ORM/Mapping/Builder/ManyToOneAssociationMetadataBuilder.php

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
         $associationMetadata->setFetchMode($this->getFetchMode($this->manyToOneAnnotation->fetch));
         $associationMetadata->setOwningSide(true);
 
-        if (! empty($this->manyToOneAnnotation->inversedBy)) {
+        if ( ! empty($this->manyToOneAnnotation->inversedBy)) {
             $associationMetadata->setInversedBy($this->manyToOneAnnotation->inversedBy);
         }
 

lib/Doctrine/ORM/Mapping/Builder/OneToOneAssociationMetadataBuilder.php

Spacing +5 added lines, -5 removed lines

@@ -51,12 +51,12 @@ 
             $associationMetadata->setCascade(array_unique(array_merge($associationMetadata->getCascade(), ['remove'])));
         }
 
-        if (! empty($this->oneToOneAnnotation->mappedBy)) {
+        if ( ! empty($this->oneToOneAnnotation->mappedBy)) {
             $associationMetadata->setMappedBy($this->oneToOneAnnotation->mappedBy);
             $associationMetadata->setOwningSide(false);
         }
 
-        if (! empty($this->oneToOneAnnotation->inversedBy)) {
+        if ( ! empty($this->oneToOneAnnotation->inversedBy)) {
             $associationMetadata->setInversedBy($this->oneToOneAnnotation->inversedBy);
             $associationMetadata->setOwningSide(true);
         }
@@ -65,7 +65,7 @@ 
         $this->buildPrimaryKey($associationMetadata);
 
         // Check for owning side to consider join column
-        if (! $associationMetadata->isOwningSide()) {
+        if ( ! $associationMetadata->isOwningSide()) {
             return $associationMetadata;
         }
 
@@ -89,7 +89,7 @@ 
         if (count($joinColumns) === 1) {
             $joinColumn = reset($joinColumns);
 
-            if (! $associationMetadata->isPrimaryKey()) {
+            if ( ! $associationMetadata->isPrimaryKey()) {
                 $joinColumn->setUnique(true);
             }
 
@@ -98,7 +98,7 @@ 
 
         $tableMetadata = $this->componentMetadata->table;
 
-        if (! $tableMetadata) {
+        if ( ! $tableMetadata) {
             $exception = 'ClassMetadata::setTable() has to be called before defining a one to one relationship.';
 
             throw new RuntimeException($exception);

tests/Doctrine/Tests/ORM/Mapping/ClassMetadataTest.php

Spacing +8 added lines, -8 removed lines

@@ -36,7 +36,7 @@ 
 use function strtoupper;
 use function unserialize;
 
-require_once __DIR__ . '/../../Models/Global/GlobalNamespaceModel.php';
+require_once __DIR__.'/../../Models/Global/GlobalNamespaceModel.php';
 
 class ClassMetadataTest extends OrmTestCase
 {
@@ -165,7 +165,7 @@ 
      */
     public function testMapAssociationInGlobalNamespace() : void
     {
-        require_once __DIR__ . '/../../Models/Global/GlobalNamespaceModel.php';
+        require_once __DIR__.'/../../Models/Global/GlobalNamespaceModel.php';
 
         $cm = new ClassMetadata(DoctrineGlobalArticle::class, null);
         $cm->setTable(new Mapping\TableMetadata('doctrine_global_article'));
@@ -265,7 +265,7 @@ 
      */
     public function testSetDiscriminatorMapInGlobalNamespace() : void
     {
-        require_once __DIR__ . '/../../Models/Global/GlobalNamespaceModel.php';
+        require_once __DIR__.'/../../Models/Global/GlobalNamespaceModel.php';
 
         $cm = new ClassMetadata('DoctrineGlobalUser', null);
         $cm->setTable(new Mapping\TableMetadata('doctrine_global_user'));
@@ -281,7 +281,7 @@ 
      */
     public function testSetSubClassesInGlobalNamespace() : void
     {
-        require_once __DIR__ . '/../../Models/Global/GlobalNamespaceModel.php';
+        require_once __DIR__.'/../../Models/Global/GlobalNamespaceModel.php';
 
         $cm = new ClassMetadata('DoctrineGlobalUser', null);
         $cm->setTable(new Mapping\TableMetadata('doctrine_global_user'));
@@ -767,7 +767,7 @@ 
     public function testEmptyFieldNameThrowsException() : void
     {
         $this->expectException(MappingException::class);
-        $this->expectExceptionMessage("The field or association mapping misses the 'fieldName' attribute in entity '" . CMS\CmsUser::class . "'.");
+        $this->expectExceptionMessage("The field or association mapping misses the 'fieldName' attribute in entity '".CMS\CmsUser::class."'.");
 
         $cm = new ClassMetadata(CMS\CmsUser::class, null);
         $cm->setTable(new Mapping\TableMetadata('cms_users'));
@@ -819,7 +819,7 @@ 
         $cm->addLifecycleCallback('postLoad', 'notfound');
 
         $this->expectException(MappingException::class);
-        $this->expectExceptionMessage("Entity '" . CMS\CmsUser::class . "' has no public method 'notfound' to be registered as lifecycle callback.");
+        $this->expectExceptionMessage("Entity '".CMS\CmsUser::class."' has no public method 'notfound' to be registered as lifecycle callback.");
 
         $cm->validateLifecycleCallbacks(new RuntimeReflectionService());
     }
@@ -841,7 +841,7 @@ 
         $cm->addProperty($association);
 
         $this->expectException(MappingException::class);
-        $this->expectExceptionMessage("The target-entity 'UnknownClass' cannot be found in '" . CMS\CmsUser::class . "#address'.");
+        $this->expectExceptionMessage("The target-entity 'UnknownClass' cannot be found in '".CMS\CmsUser::class."#address'.");
 
         $cm->validateAssociations();
     }
@@ -1089,6 +1089,6 @@ 
      */
     public function propertyToColumnName(string $propertyName, ?string $className = null) : string
     {
-        return strtolower($this->classToTableName($className)) . '_' . $propertyName;
+        return strtolower($this->classToTableName($className)).'_'.$propertyName;
     }
 }

lib/Doctrine/ORM/Mapping/ClassMetadata.php

Spacing +19 added lines, -19 removed lines

@@ -220,11 +220,11 @@ 
             $this->setCache(clone $parent->cache);
         }
 
-        if (! empty($parent->lifecycleCallbacks)) {
+        if ( ! empty($parent->lifecycleCallbacks)) {
             $this->lifecycleCallbacks = $parent->lifecycleCallbacks;
         }
 
-        if (! empty($parent->entityListeners)) {
+        if ( ! empty($parent->entityListeners)) {
             $this->entityListeners = $parent->entityListeners;
         }
     }
@@ -291,7 +291,7 @@ 
      */
     public function __toString()
     {
-        return self::class . '@' . spl_object_id($this);
+        return self::class.'@'.spl_object_id($this);
     }
 
     /**
@@ -387,11 +387,11 @@ 
      */
     public function isIdentifier(string $fieldName) : bool
     {
-        if (! $this->identifier) {
+        if ( ! $this->identifier) {
             return false;
         }
 
-        if (! $this->isIdentifierComposite()) {
+        if ( ! $this->isIdentifierComposite()) {
             return $fieldName === $this->identifier[0];
         }
 
@@ -415,11 +415,11 @@ 
         }
 
         // Verify & complete identifier mapping
-        if (! $this->identifier) {
+        if ( ! $this->identifier) {
             throw MappingException::identifierRequired($this->className);
         }
 
-        $explicitlyGeneratedProperties = array_filter($this->properties, static function (Property $property) : bool {
+        $explicitlyGeneratedProperties = array_filter($this->properties, static function(Property $property) : bool {
             return $property instanceof FieldMetadata
                 && $property->isPrimaryKey()
                 && $property->hasValueGenerator();
@@ -440,7 +440,7 @@ 
         foreach ($this->lifecycleCallbacks as $callbacks) {
             /** @var array $callbacks */
             foreach ($callbacks as $callbackFuncName) {
-                if (! $reflectionService->hasPublicMethod($this->className, $callbackFuncName)) {
+                if ( ! $reflectionService->hasPublicMethod($this->className, $callbackFuncName)) {
                     throw MappingException::lifecycleCallbackMethodNotFound($this->className, $callbackFuncName);
                 }
             }
@@ -469,7 +469,7 @@ 
             throw MappingException::singleIdNotAllowedOnCompositePrimaryKey($this->className);
         }
 
-        if (! isset($this->identifier[0])) {
+        if ( ! isset($this->identifier[0])) {
             throw MappingException::noIdDefined($this->className);
         }
 
@@ -528,7 +528,7 @@ 
             // Association defined as Id field
             $targetClass = $em->getClassMetadata($property->getTargetEntity());
 
-            if (! $property->isOwningSide()) {
+            if ( ! $property->isOwningSide()) {
                 $property    = $targetClass->getProperty($property->getMappedBy());
                 $targetClass = $em->getClassMetadata($property->getTargetEntity());
             }
@@ -542,7 +542,7 @@ 
                 $columnName           = $joinColumn->getColumnName();
                 $referencedColumnName = $joinColumn->getReferencedColumnName();
 
-                if (! $joinColumn->getType()) {
+                if ( ! $joinColumn->getType()) {
                     $joinColumn->setType(PersisterHelper::getTypeOfColumn($referencedColumnName, $targetClass, $em));
                 }
 
@@ -576,10 +576,10 @@ 
     {
         $schema = $this->getSchemaName() === null
             ? ''
-            : $this->getSchemaName() . '_';
+            : $this->getSchemaName().'_';
 
         // replace dots with underscores because PostgreSQL creates temporary tables in a special schema
-        return $schema . $this->getTableName() . '_id_tmp';
+        return $schema.$this->getTableName().'_id_tmp';
     }
 
     /**
@@ -613,7 +613,7 @@ 
      */
     public function setInheritanceType($type) : void
     {
-        if (! $this->isInheritanceType($type)) {
+        if ( ! $this->isInheritanceType($type)) {
             throw MappingException::invalidInheritanceType($this->className, $type);
         }
 
@@ -631,7 +631,7 @@ 
     {
         $fieldName = $property->getName();
 
-        if (! isset($this->properties[$fieldName])) {
+        if ( ! isset($this->properties[$fieldName])) {
             throw MappingException::invalidOverrideFieldName($this->className, $fieldName);
         }
 
@@ -825,7 +825,7 @@ 
         $inheritedProperty = $declaringClass->isMappedSuperclass ? clone $property : $property;
 
         if ($inheritedProperty instanceof FieldMetadata) {
-            if (! $declaringClass->isMappedSuperclass) {
+            if ( ! $declaringClass->isMappedSuperclass) {
                 $inheritedProperty->setTableName($property->getTableName());
             }
 
@@ -918,11 +918,11 @@ 
             'method' => $methodName,
         ];
 
-        if (! class_exists($class)) {
+        if ( ! class_exists($class)) {
             throw MappingException::entityListenerClassNotFound($class, $this->className);
         }
 
-        if (! method_exists($class, $methodName)) {
+        if ( ! method_exists($class, $methodName)) {
             throw MappingException::entityListenerMethodNotFound($class, $methodName, $this->className);
         }
 
@@ -990,7 +990,7 @@ 
             return;
         }
 
-        if (! (class_exists($className) || interface_exists($className))) {
+        if ( ! (class_exists($className) || interface_exists($className))) {
             throw MappingException::invalidClassInDiscriminatorMap($className, $this->className);
         }
 

lib/Doctrine/ORM/Mapping/Driver/DatabaseDriver.php

Spacing +7 added lines, -7 removed lines

@@ -145,8 +145,8 @@ 
     ) : Mapping\ComponentMetadata {
         $this->reverseEngineerMappingFromDatabase();
 
-        if (! isset($this->classToTableNames[$className])) {
-            throw new InvalidArgumentException('Unknown class ' . $className);
+        if ( ! isset($this->classToTableNames[$className])) {
+            throw new InvalidArgumentException('Unknown class '.$className);
         }
 
         $metadata = new Mapping\ClassMetadata($className, $parent);
@@ -175,7 +175,7 @@ 
                     }
                 }
 
-                if (! $otherFk) {
+                if ( ! $otherFk) {
                     // the definition of this many to many table does not contain
                     // enough foreign key information to continue reverse engineering.
                     continue;
@@ -258,9 +258,9 @@ 
                 $allForeignKeyColumns = array_merge($allForeignKeyColumns, $foreignKey->getLocalColumns());
             }
 
-            if (! $table->hasPrimaryKey()) {
+            if ( ! $table->hasPrimaryKey()) {
                 throw new Mapping\MappingException(
-                    'Table ' . $table->getName() . ' has no primary key. Doctrine does not ' .
+                    'Table '.$table->getName().' has no primary key. Doctrine does not '.
                     "support reverse engineering from tables that don't have a primary key."
                 );
             }
@@ -457,7 +457,7 @@ 
             }
 
             // Here we need to check if $fkColumns are the same as $primaryKeys
-            if (! array_diff($fkColumns, $primaryKeys)) {
+            if ( ! array_diff($fkColumns, $primaryKeys)) {
                 $metadata->addProperty($associationMapping);
             } else {
                 $metadata->addProperty($associationMapping);
@@ -502,7 +502,7 @@ 
      */
     private function getClassNameForTable($tableName)
     {
-        return $this->namespace . (
+        return $this->namespace.(
             $this->classNamesForTables[$tableName]
                 ?? Inflector::classify(strtolower($tableName))
         );

lib/Doctrine/ORM/Mapping/ComponentMetadata.php

Spacing +1 added lines, -1 removed lines

@@ -58,7 +58,7 @@
         // Restore ReflectionClass and properties
         $this->reflectionClass = $reflectionService->getClass($this->className);
 
-        if (! $this->reflectionClass) {
+        if ( ! $this->reflectionClass) {
             return;
         }
 

lib/Doctrine/ORM/Mapping/Driver/XmlDriver.php

Spacing +5 added lines, -5 removed lines

@@ -301,7 +301,7 @@ 
                 $fieldName = (string) $overrideElement['name'];
                 $property  = $classMetadata->getProperty($fieldName);
 
-                if (! $property) {
+                if ( ! $property) {
                     throw Mapping\MappingException::invalidOverrideFieldName($classMetadata->getClassName(), $fieldName);
                 }
 
@@ -335,7 +335,7 @@ 
                 $fieldName = (string) $overrideElement['name'];
                 $property  = $classMetadata->getProperty($fieldName);
 
-                if (! $property) {
+                if ( ! $property) {
                     throw Mapping\MappingException::invalidOverrideFieldName($classMetadata->getClassName(), $fieldName);
                 }
 
@@ -393,7 +393,7 @@ 
                 // Check for fetch
                 if (isset($overrideElement['fetch'])) {
                     $override->setFetchMode(
-                        constant('Doctrine\ORM\Mapping\FetchMode::' . (string) $overrideElement['fetch'])
+                        constant('Doctrine\ORM\Mapping\FetchMode::'.(string) $overrideElement['fetch'])
                     );
                 }
 
@@ -404,7 +404,7 @@ 
         // Evaluate <lifecycle-callbacks...>
         if (isset($xmlRoot->{'lifecycle-callbacks'})) {
             foreach ($xmlRoot->{'lifecycle-callbacks'}->{'lifecycle-callback'} as $lifecycleCallback) {
-                $eventName  = constant(Events::class . '::' . (string) $lifecycleCallback['type']);
+                $eventName  = constant(Events::class.'::'.(string) $lifecycleCallback['type']);
                 $methodName = (string) $lifecycleCallback['method'];
 
                 $classMetadata->addLifecycleCallback($eventName, $methodName);
@@ -416,7 +416,7 @@ 
             foreach ($xmlRoot->{'entity-listeners'}->{'entity-listener'} as $listenerElement) {
                 $listenerClassName = (string) $listenerElement['class'];
 
-                if (! class_exists($listenerClassName)) {
+                if ( ! class_exists($listenerClassName)) {
                     throw Mapping\MappingException::entityListenerClassNotFound(
                         $listenerClassName,
                         $classMetadata->getClassName()

lib/Doctrine/ORM/Mapping/Driver/AnnotationDriver.php

Spacing +12 added lines, -12 removed lines

@@ -204,7 +204,7 @@ 
             return $this->classNames;
         }
 
-        if (! $this->paths) {
+        if ( ! $this->paths) {
             throw Mapping\MappingException::pathRequired();
         }
 
@@ -212,7 +212,7 @@ 
         $includedFiles = [];
 
         foreach ($this->paths as $path) {
-            if (! is_dir($path)) {
+            if ( ! is_dir($path)) {
                 throw Mapping\MappingException::fileMappingDriversRequireConfiguredDirectoryPath($path);
             }
 
@@ -221,14 +221,14 @@ 
                     new RecursiveDirectoryIterator($path, FilesystemIterator::SKIP_DOTS),
                     RecursiveIteratorIterator::LEAVES_ONLY
                 ),
-                '/^.+' . preg_quote($this->fileExtension) . '$/i',
+                '/^.+'.preg_quote($this->fileExtension).'$/i',
                 RecursiveRegexIterator::GET_MATCH
             );
 
             foreach ($iterator as $file) {
                 $sourceFile = $file[0];
 
-                if (! preg_match('(^phar:)i', $sourceFile)) {
+                if ( ! preg_match('(^phar:)i', $sourceFile)) {
                     $sourceFile = realpath($sourceFile);
                 }
 
@@ -294,7 +294,7 @@ 
             ->withCacheAnnotation($classAnnotations[Annotation\Cache::class] ?? null)
             ->build();
 
-        if (! $classMetadata->isEmbeddedClass) {
+        if ( ! $classMetadata->isEmbeddedClass) {
             $this->attachLifecycleCallbacks($classAnnotations, $reflectionClass, $classMetadata);
             $this->attachEntityListeners($classAnnotations, $classMetadata);
         }
@@ -365,7 +365,7 @@ 
             ];
 
             foreach ($entityListenersAnnot->value as $listenerClassName) {
-                if (! class_exists($listenerClassName)) {
+                if ( ! class_exists($listenerClassName)) {
                     throw Mapping\MappingException::entityListenerClassNotFound(
                         $listenerClassName,
                         $metadata->getClassName()
@@ -453,7 +453,7 @@ 
                 $fieldName = $associationOverrideAnnotation->name;
                 $property  = $metadata->getProperty($fieldName);
 
-                if (! $property) {
+                if ( ! $property) {
                     throw Mapping\MappingException::invalidOverrideFieldName($metadata->getClassName(), $fieldName);
                 }
 
@@ -506,7 +506,7 @@ 
 
                 // Check for fetch
                 if ($associationOverrideAnnotation->fetch) {
-                    $override->setFetchMode(constant(Mapping\FetchMode::class . '::' . $associationOverrideAnnotation->fetch));
+                    $override->setFetchMode(constant(Mapping\FetchMode::class.'::'.$associationOverrideAnnotation->fetch));
                 }
 
                 $metadata->setPropertyOverride($override);
@@ -527,7 +527,7 @@ 
                 $fieldName = $attributeOverrideAnnotation->name;
                 $property  = $metadata->getProperty($fieldName);
 
-                if (! $property) {
+                if ( ! $property) {
                     throw Mapping\MappingException::invalidOverrideFieldName($metadata->getClassName(), $fieldName);
                 }
 
@@ -556,7 +556,7 @@ 
         $classAnnotations = $this->reader->getClassAnnotations($reflectionClass);
 
         foreach ($classAnnotations as $key => $annot) {
-            if (! is_numeric($key)) {
+            if ( ! is_numeric($key)) {
                 continue;
             }
 
@@ -574,7 +574,7 @@ 
         $propertyAnnotations = $this->reader->getPropertyAnnotations($reflectionProperty);
 
         foreach ($propertyAnnotations as $key => $annot) {
-            if (! is_numeric($key)) {
+            if ( ! is_numeric($key)) {
                 continue;
             }
 
@@ -592,7 +592,7 @@ 
         $methodAnnotations = $this->reader->getMethodAnnotations($reflectionMethod);
 
         foreach ($methodAnnotations as $key => $annot) {
-            if (! is_numeric($key)) {
+            if ( ! is_numeric($key)) {
                 continue;
             }
 

lib/Doctrine/ORM/Mapping/Builder/ClassMetadataBuilder.php

Spacing +2 added lines, -2 removed lines

@@ -66,7 +66,7 @@ 
         $this->metadataBuildingContext            = $metadataBuildingContext;
         $this->tableMetadataBuilder               = $tableMetadataBuilder ?: new TableMetadataBuilder($metadataBuildingContext);
         $this->cacheMetadataBuilder               = $cacheMetadataBuilder ?: new CacheMetadataBuilder($metadataBuildingContext);
-        $this->discriminatorColumnMetadataBuilder = $discriminatorColumnMetadataBuilder?: new DiscriminatorColumnMetadataBuilder($metadataBuildingContext);
+        $this->discriminatorColumnMetadataBuilder = $discriminatorColumnMetadataBuilder ?: new DiscriminatorColumnMetadataBuilder($metadataBuildingContext);
     }
 
     public function withClassName(string $className) : ClassMetadataBuilder
@@ -233,7 +233,7 @@ 
             && $parentMetadata->inheritanceType === Mapping\InheritanceType::SINGLE_TABLE) {
             // Handle the case where a middle mapped super class inherits from a single table inheritance tree.
             do {
-                if (! $parentMetadata->isMappedSuperclass) {
+                if ( ! $parentMetadata->isMappedSuperclass) {
                     $tableMetadata = $parentMetadata->table;
                     break;
                 }