doctrine / orm

lib/Doctrine/ORM/Mapping/ClassMetadataFactory.php

Spacing +17 added lines, -17 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\ORM\Mapping;
 
@@ -68,7 +68,7 @@ 
         $this->configuration = $configuration;
         $this->connection = $connection;
         $this->eventManager = $eventManager;
-        $this->driver      = $this->configuration->getMetadataDriverImpl();
+        $this->driver = $this->configuration->getMetadataDriverImpl();
     }
 
     /**
@@ -133,12 +133,12 @@ 
                 $classMetadata->setCache(clone $parent->getCache());
             }
 
-            if (! empty($parent->entityListeners) && empty($classMetadata->entityListeners)) {
+            if ( ! empty($parent->entityListeners) && empty($classMetadata->entityListeners)) {
                 $classMetadata->entityListeners = $parent->entityListeners;
             }
         }
 
-        if (! $classMetadata->discriminatorMap && $classMetadata->inheritanceType !== InheritanceType::NONE && $classMetadata->isRootEntity()) {
+        if ( ! $classMetadata->discriminatorMap && $classMetadata->inheritanceType !== InheritanceType::NONE && $classMetadata->isRootEntity()) {
             $this->addDefaultDiscriminatorMap($classMetadata);
         }
 
@@ -152,7 +152,7 @@ 
 
     protected function completeRuntimeMetadata(ClassMetadata $class, ?ClassMetadata $parent = null) : void
     {
-        if (! $parent || ! $parent->isMappedSuperclass) {
+        if ( ! $parent || ! $parent->isMappedSuperclass) {
             return;
         }
 
@@ -170,7 +170,7 @@ 
                 continue;
             }
 
-            if (! ($property instanceof ToOneAssociationMetadata)) {
+            if ( ! ($property instanceof ToOneAssociationMetadata)) {
                 continue;
             }
 
@@ -189,7 +189,7 @@ 
      */
     protected function validateRuntimeMetadata(ClassMetadata $class, ?ClassMetadata $parent = null) : void
     {
-        if (! $class->getReflectionClass()) {
+        if ( ! $class->getReflectionClass()) {
             // only validate if there is a reflection class instance
             return;
         }
@@ -199,13 +199,13 @@ 
         $class->validateLifecycleCallbacks($this->getReflectionService());
 
         // verify inheritance
-        if (! $class->isMappedSuperclass && $class->inheritanceType !== InheritanceType::NONE) {
-            if (! $parent) {
-                if (! $class->discriminatorMap) {
+        if ( ! $class->isMappedSuperclass && $class->inheritanceType !== InheritanceType::NONE) {
+            if ( ! $parent) {
+                if ( ! $class->discriminatorMap) {
                     throw MappingException::missingDiscriminatorMap($class->getClassName());
                 }
 
-                if (! $class->discriminatorColumn) {
+                if ( ! $class->discriminatorColumn) {
                     throw MappingException::missingDiscriminatorColumn($class->getClassName());
                 }
             }
@@ -326,7 +326,7 @@ 
     private function completeIdentifierGeneratorMappings(ClassMetadata $class) : void
     {
         foreach ($class->getDeclaredPropertiesIterator() as $property) {
-            if (! $property instanceof FieldMetadata /*&& ! $property instanceof AssocationMetadata*/) {
+            if ( ! $property instanceof FieldMetadata /*&& ! $property instanceof AssocationMetadata*/) {
                 continue;
             }
 
@@ -336,7 +336,7 @@ 
 
     private function completeFieldIdentifierGeneratorMapping(FieldMetadata $field)
     {
-        if (! $field->hasValueGenerator()) {
+        if ( ! $field->hasValueGenerator()) {
             return;
         }
 
@@ -388,10 +388,10 @@ 
 
             case GeneratorType::CUSTOM:
                 $definition = $generator->getDefinition();
-                if (! isset($definition['class'])) {
+                if ( ! isset($definition['class'])) {
                     throw new ORMException(sprintf('Cannot instantiate custom generator, no class has been defined'));
                 }
-                if (! class_exists($definition['class'])) {
+                if ( ! class_exists($definition['class'])) {
                     throw new ORMException(sprintf('Cannot instantiate custom generator : %s', var_export($definition, true))); //$definition['class']));
                 }
 
@@ -403,7 +403,7 @@ 
                 break;
 
             default:
-                throw new ORMException('Unknown generator type: ' . $generator->getType());
+                throw new ORMException('Unknown generator type: '.$generator->getType());
         }
     }
 
@@ -425,7 +425,7 @@ 
 
     private function getTargetPlatform() : Platforms\AbstractPlatform
     {
-        if (! $this->targetPlatform) {
+        if ( ! $this->targetPlatform) {
             $this->targetPlatform = $this->connection->getDatabasePlatform();
         }
 

lib/Doctrine/ORM/Mapping/AbstractClassMetadataFactory.php

Spacing +5 added lines, -5 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\ORM\Mapping;
 
@@ -158,7 +158,7 @@ 
 
         try {
             if ($this->cacheDriver) {
-                $cached = $this->cacheDriver->fetch($realClassName . $this->cacheSalt);
+                $cached = $this->cacheDriver->fetch($realClassName.$this->cacheSalt);
 
                 if ($cached instanceof ClassMetadata) {
                     $this->loadedMetadata[$realClassName] = $cached;
@@ -168,7 +168,7 @@ 
                     foreach ($this->loadMetadata($realClassName, $metadataBuildingContext) as $loadedClass) {
                         $loadedClassName = $loadedClass->getClassName();
 
-                        $this->cacheDriver->save($loadedClassName . $this->cacheSalt, $loadedClass, null);
+                        $this->cacheDriver->save($loadedClassName.$this->cacheSalt, $loadedClass, null);
                     }
                 }
             } else {
@@ -177,7 +177,7 @@ 
         } catch (CommonMappingException $loadingException) {
             $fallbackMetadataResponse = $this->onNotFoundMetadata($realClassName, $metadataBuildingContext);
 
-            if (! $fallbackMetadataResponse) {
+            if ( ! $fallbackMetadataResponse) {
                 throw $loadingException;
             }
 
@@ -263,7 +263,7 @@ 
         $parentClasses = [];
 
         foreach (array_reverse($this->getReflectionService()->getParentClasses($name)) as $parentClass) {
-            if (! $this->getDriver()->isTransient($parentClass)) {
+            if ( ! $this->getDriver()->isTransient($parentClass)) {
                 $parentClasses[] = $parentClass;
             }
         }

tests/Doctrine/Tests/ORM/Cache/DefaultCacheFactoryTest.php

Spacing +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\Tests\ORM\Cache;
 
@@ -51,7 +51,7 @@ 
     {
         parent::setUp();
 
-        $this->em            = $this->getTestEntityManager();
+        $this->em = $this->getTestEntityManager();
         /*
         $this->enableSecondLevelCache();
 

tests/Doctrine/Tests/OrmFunctionalTestCase.php

Spacing +72 added lines, -72 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\Tests;
 
@@ -346,7 +346,7 @@ 
         $conn = static::$sharedConn;
 
         // In case test is skipped, tearDown is called, but no setup may have run
-        if (! $conn) {
+        if ( ! $conn) {
             return;
         }
 
@@ -420,7 +420,7 @@ 
         }
 
         if (isset($this->usedModelSets['directorytree'])) {
-            $conn->executeUpdate('DELETE FROM ' . $platform->quoteIdentifier('file'));
+            $conn->executeUpdate('DELETE FROM '.$platform->quoteIdentifier('file'));
             // MySQL doesn't know deferred deletions therefore only executing the second query gives errors.
             $conn->executeUpdate('DELETE FROM Directory WHERE parentDirectory_id IS NOT NULL');
             $conn->executeUpdate('DELETE FROM Directory');
@@ -513,12 +513,12 @@ 
                 )
             );
 
-            $conn->executeUpdate('DELETE FROM ' . $platform->quoteIdentifier('quote-users-groups'));
-            $conn->executeUpdate('DELETE FROM ' . $platform->quoteIdentifier('quote-group'));
-            $conn->executeUpdate('DELETE FROM ' . $platform->quoteIdentifier('quote-phone'));
-            $conn->executeUpdate('DELETE FROM ' . $platform->quoteIdentifier('quote-user'));
-            $conn->executeUpdate('DELETE FROM ' . $platform->quoteIdentifier('quote-address'));
-            $conn->executeUpdate('DELETE FROM ' . $platform->quoteIdentifier('quote-city'));
+            $conn->executeUpdate('DELETE FROM '.$platform->quoteIdentifier('quote-users-groups'));
+            $conn->executeUpdate('DELETE FROM '.$platform->quoteIdentifier('quote-group'));
+            $conn->executeUpdate('DELETE FROM '.$platform->quoteIdentifier('quote-phone'));
+            $conn->executeUpdate('DELETE FROM '.$platform->quoteIdentifier('quote-user'));
+            $conn->executeUpdate('DELETE FROM '.$platform->quoteIdentifier('quote-address'));
+            $conn->executeUpdate('DELETE FROM '.$platform->quoteIdentifier('quote-city'));
         }
 
         if (isset($this->usedModelSets['vct_onetoone'])) {
@@ -631,7 +631,7 @@ 
         $classes = [];
 
         foreach ($classNames as $className) {
-            if (! isset(static::$entityTablesCreated[$className])) {
+            if ( ! isset(static::$entityTablesCreated[$className])) {
                 static::$entityTablesCreated[$className] = true;
                 $classes[]                               = $this->em->getClassMetadata($className);
             }
@@ -651,25 +651,25 @@ 
     {
         $this->setUpDBALTypes();
 
-        if (! isset(static::$sharedConn)) {
+        if ( ! isset(static::$sharedConn)) {
             static::$sharedConn = TestUtil::getConnection();
         }
 
         if (isset($GLOBALS['DOCTRINE_MARK_SQL_LOGS'])) {
             if (in_array(static::$sharedConn->getDatabasePlatform()->getName(), ['mysql', 'postgresql'], true)) {
-                static::$sharedConn->executeQuery('SELECT 1 /*' . get_class($this) . '*/');
+                static::$sharedConn->executeQuery('SELECT 1 /*'.get_class($this).'*/');
             } elseif (static::$sharedConn->getDatabasePlatform()->getName() === 'oracle') {
-                static::$sharedConn->executeQuery('SELECT 1 /*' . get_class($this) . '*/ FROM dual');
+                static::$sharedConn->executeQuery('SELECT 1 /*'.get_class($this).'*/ FROM dual');
             }
         }
 
-        if (! $this->em) {
+        if ( ! $this->em) {
             $this->em         = $this->getEntityManager();
             $this->schemaTool = new SchemaTool($this->em);
         }
 
         foreach ($this->usedModelSets as $setName => $bool) {
-            if (! isset(static::$tablesCreated[$setName])) {
+            if ( ! isset(static::$tablesCreated[$setName])) {
                 $this->setUpEntitySchema(static::$modelSets[$setName]);
 
                 static::$tablesCreated[$setName] = true;
@@ -744,65 +744,65 @@ 
 
         $config->setMetadataDriverImpl(
             $mappingDriver ?? $config->newDefaultAnnotationDriver([
-                realpath(__DIR__ . '/Models/CMS'),
-                realpath(__DIR__ . '/Models/Cache'),
+                realpath(__DIR__.'/Models/CMS'),
+                realpath(__DIR__.'/Models/Cache'),
 
-                realpath(__DIR__ . '/Models/Company'),
-                realpath(__DIR__ . '/Models/CompositeKeyInheritance'),
+                realpath(__DIR__.'/Models/Company'),
+                realpath(__DIR__.'/Models/CompositeKeyInheritance'),
                 //realpath(__DIR__ . '/Models/CustomType'),
-                realpath(__DIR__ . '/Models/DDC117'),
-                realpath(__DIR__ . '/Models/DDC1476'),
-                realpath(__DIR__ . '/Models/DDC1590'),
-                realpath(__DIR__ . '/Models/DDC1872'),
+                realpath(__DIR__.'/Models/DDC117'),
+                realpath(__DIR__.'/Models/DDC1476'),
+                realpath(__DIR__.'/Models/DDC1590'),
+                realpath(__DIR__.'/Models/DDC1872'),
                 //realpath(__DIR__ . '/Models/DDC2372'),
-                realpath(__DIR__ . '/Models/DDC2504'),
-                realpath(__DIR__ . '/Models/DDC2825'),
-                realpath(__DIR__ . '/Models/DDC3231'),
-                realpath(__DIR__ . '/Models/DDC3293'),
-                realpath(__DIR__ . '/Models/DDC3346'),
-                realpath(__DIR__ . '/Models/DDC3579'),
-                realpath(__DIR__ . '/Models/DDC3597'),
-                realpath(__DIR__ . '/Models/DDC3699'),
-                realpath(__DIR__ . '/Models/DDC3711'),
-                realpath(__DIR__ . '/Models/DDC3899'),
+                realpath(__DIR__.'/Models/DDC2504'),
+                realpath(__DIR__.'/Models/DDC2825'),
+                realpath(__DIR__.'/Models/DDC3231'),
+                realpath(__DIR__.'/Models/DDC3293'),
+                realpath(__DIR__.'/Models/DDC3346'),
+                realpath(__DIR__.'/Models/DDC3579'),
+                realpath(__DIR__.'/Models/DDC3597'),
+                realpath(__DIR__.'/Models/DDC3699'),
+                realpath(__DIR__.'/Models/DDC3711'),
+                realpath(__DIR__.'/Models/DDC3899'),
                 //realpath(__DIR__ . '/Models/DDC4006'),
                 //realpath(__DIR__ . '/Models/DDC5934'),
-                realpath(__DIR__ . '/Models/DDC6412'),
-                realpath(__DIR__ . '/Models/DDC753'),
-                realpath(__DIR__ . '/Models/DDC869'),
+                realpath(__DIR__.'/Models/DDC6412'),
+                realpath(__DIR__.'/Models/DDC753'),
+                realpath(__DIR__.'/Models/DDC869'),
                 //realpath(__DIR__ . '/Models/DDC889'),
-                realpath(__DIR__ . '/Models/DDC964'),
-                realpath(__DIR__ . '/Models/DirectoryTree'),
-                realpath(__DIR__ . '/Models/ECommerce'),
-                realpath(__DIR__ . '/Models/Forum'),
-                realpath(__DIR__ . '/Models/FriendObject'),
-                realpath(__DIR__ . '/Models/Generic'),
-                realpath(__DIR__ . '/Models/GeoNames'),
-                realpath(__DIR__ . '/Models/Global'),
-                realpath(__DIR__ . '/Models/Hydration'),
-                realpath(__DIR__ . '/Models/IdentityIsAssociation'),
-                realpath(__DIR__ . '/Models/Issue5989'),
-                realpath(__DIR__ . '/Models/JoinedInheritanceType'),
-                realpath(__DIR__ . '/Models/Legacy'),
-                realpath(__DIR__ . '/Models/ManyToManyPersister'),
-                realpath(__DIR__ . '/Models/MixedToOneIdentity'),
-                realpath(__DIR__ . '/Models/Navigation'),
-                realpath(__DIR__ . '/Models/NonPublicSchemaJoins'),
-                realpath(__DIR__ . '/Models/NullDefault'),
-                realpath(__DIR__ . '/Models/OneToOneInverseSideLoad'),
-                realpath(__DIR__ . '/Models/OneToOneSingleTableInheritance'),
-                realpath(__DIR__ . '/Models/Pagination'),
-                realpath(__DIR__ . '/Models/ProxySpecifics'),
-                realpath(__DIR__ . '/Models/Quote'),
-                realpath(__DIR__ . '/Models/Reflection'),
-                realpath(__DIR__ . '/Models/Routing'),
-                realpath(__DIR__ . '/Models/StockExchange'),
-                realpath(__DIR__ . '/Models/Taxi'),
-                realpath(__DIR__ . '/Models/Tweet'),
-                realpath(__DIR__ . '/Models/ValueConversionType'),
-                realpath(__DIR__ . '/Models/ValueObjects'),
-                realpath(__DIR__ . '/Models/VersionedManyToOne'),
-                realpath(__DIR__ . '/Models/VersionedOneToOne'),
+                realpath(__DIR__.'/Models/DDC964'),
+                realpath(__DIR__.'/Models/DirectoryTree'),
+                realpath(__DIR__.'/Models/ECommerce'),
+                realpath(__DIR__.'/Models/Forum'),
+                realpath(__DIR__.'/Models/FriendObject'),
+                realpath(__DIR__.'/Models/Generic'),
+                realpath(__DIR__.'/Models/GeoNames'),
+                realpath(__DIR__.'/Models/Global'),
+                realpath(__DIR__.'/Models/Hydration'),
+                realpath(__DIR__.'/Models/IdentityIsAssociation'),
+                realpath(__DIR__.'/Models/Issue5989'),
+                realpath(__DIR__.'/Models/JoinedInheritanceType'),
+                realpath(__DIR__.'/Models/Legacy'),
+                realpath(__DIR__.'/Models/ManyToManyPersister'),
+                realpath(__DIR__.'/Models/MixedToOneIdentity'),
+                realpath(__DIR__.'/Models/Navigation'),
+                realpath(__DIR__.'/Models/NonPublicSchemaJoins'),
+                realpath(__DIR__.'/Models/NullDefault'),
+                realpath(__DIR__.'/Models/OneToOneInverseSideLoad'),
+                realpath(__DIR__.'/Models/OneToOneSingleTableInheritance'),
+                realpath(__DIR__.'/Models/Pagination'),
+                realpath(__DIR__.'/Models/ProxySpecifics'),
+                realpath(__DIR__.'/Models/Quote'),
+                realpath(__DIR__.'/Models/Reflection'),
+                realpath(__DIR__.'/Models/Routing'),
+                realpath(__DIR__.'/Models/StockExchange'),
+                realpath(__DIR__.'/Models/Taxi'),
+                realpath(__DIR__.'/Models/Tweet'),
+                realpath(__DIR__.'/Models/ValueConversionType'),
+                realpath(__DIR__.'/Models/ValueObjects'),
+                realpath(__DIR__.'/Models/VersionedManyToOne'),
+                realpath(__DIR__.'/Models/VersionedOneToOne'),
 
             ])
         );
@@ -853,13 +853,13 @@ 
 
             foreach ($last25queries as $i => $query) {
                 $params = array_map(
-                    function ($p) {
+                    function($p) {
                         return is_object($p) ? get_class($p) : var_export($p, true);
                     },
                     $query['params'] ?: []
                 );
 
-                $queries .= $i . ". SQL: '" . $query['sql'] . "' Params: " . implode(', ', $params) . PHP_EOL;
+                $queries .= $i.". SQL: '".$query['sql']."' Params: ".implode(', ', $params).PHP_EOL;
             }
 
             $trace    = $e->getTrace();
@@ -872,11 +872,11 @@ 
                         break;
                     }
 
-                    $traceMsg .= $part['file'] . ':' . $part['line'] . PHP_EOL;
+                    $traceMsg .= $part['file'].':'.$part['line'].PHP_EOL;
                 }
             }
 
-            $message = '[' . get_class($e) . '] ' . $e->getMessage() . PHP_EOL . PHP_EOL . 'With queries:' . PHP_EOL . $queries . PHP_EOL . 'Trace:' . PHP_EOL . $traceMsg;
+            $message = '['.get_class($e).'] '.$e->getMessage().PHP_EOL.PHP_EOL.'With queries:'.PHP_EOL.$queries.PHP_EOL.'Trace:'.PHP_EOL.$traceMsg;
 
             throw new \Exception($message, (int) $e->getCode(), $e);
         }

tests/Doctrine/Tests/OrmTestCase.php

Spacing +53 added lines, -53 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\Tests;
 
@@ -66,7 +66,7 @@ 
     {
         $reader = new Annotations\CachedReader(new Annotations\AnnotationReader(), new ArrayCache());
 
-        Annotations\AnnotationRegistry::registerFile(__DIR__ . '/../../../lib/Doctrine/ORM/Annotation/DoctrineAnnotations.php');
+        Annotations\AnnotationRegistry::registerFile(__DIR__.'/../../../lib/Doctrine/ORM/Annotation/DoctrineAnnotations.php');
 
         return new AnnotationDriver($reader, (array) $paths);
     }
@@ -105,64 +105,64 @@ 
         $config->setAutoGenerateProxyClasses(ProxyFactory::AUTOGENERATE_EVAL);
         $config->setMetadataDriverImpl(
             $config->newDefaultAnnotationDriver([
-                realpath(__DIR__ . '/Models/Cache'),
-                realpath(__DIR__ . '/Models/DDC1476'),
-                realpath(__DIR__ . '/Models/DDC3231'),
-                realpath(__DIR__ . '/Models/DDC3711'),
-                realpath(__DIR__ . '/Models/DDC869'),
-                realpath(__DIR__ . '/Models/FriendObject'),
-                realpath(__DIR__ . '/Models/Issue5989'),
-                realpath(__DIR__ . '/Models/NonPublicSchemaJoins'),
-                realpath(__DIR__ . '/Models/Quote'),
+                realpath(__DIR__.'/Models/Cache'),
+                realpath(__DIR__.'/Models/DDC1476'),
+                realpath(__DIR__.'/Models/DDC3231'),
+                realpath(__DIR__.'/Models/DDC3711'),
+                realpath(__DIR__.'/Models/DDC869'),
+                realpath(__DIR__.'/Models/FriendObject'),
+                realpath(__DIR__.'/Models/Issue5989'),
+                realpath(__DIR__.'/Models/NonPublicSchemaJoins'),
+                realpath(__DIR__.'/Models/Quote'),
                 //realpath(__DIR__ . '/Models/ValueConversionType'),
-                realpath(__DIR__ . '/Models/CMS'),
-                realpath(__DIR__ . '/Models/DDC1590'),
-                realpath(__DIR__ . '/Models/DDC3293'),
-                realpath(__DIR__ . '/Models/DDC3899'),
+                realpath(__DIR__.'/Models/CMS'),
+                realpath(__DIR__.'/Models/DDC1590'),
+                realpath(__DIR__.'/Models/DDC3293'),
+                realpath(__DIR__.'/Models/DDC3899'),
                 //realpath(__DIR__ . '/Models/DDC889'),
-                realpath(__DIR__ . '/Models/Generic'),
-                realpath(__DIR__ . '/Models/JoinedInheritanceType'),
-                realpath(__DIR__ . '/Models/NullDefault'),
-                realpath(__DIR__ . '/Models/Reflection'),
-                realpath(__DIR__ . '/Models/ValueObjects'),
-                realpath(__DIR__ . '/Models/Company'),
-                realpath(__DIR__ . '/Models/DDC1872'),
-                realpath(__DIR__ . '/Models/DDC3346'),
+                realpath(__DIR__.'/Models/Generic'),
+                realpath(__DIR__.'/Models/JoinedInheritanceType'),
+                realpath(__DIR__.'/Models/NullDefault'),
+                realpath(__DIR__.'/Models/Reflection'),
+                realpath(__DIR__.'/Models/ValueObjects'),
+                realpath(__DIR__.'/Models/Company'),
+                realpath(__DIR__.'/Models/DDC1872'),
+                realpath(__DIR__.'/Models/DDC3346'),
                 //realpath(__DIR__ . '/Models/DDC4006'),
-                realpath(__DIR__ . '/Models/DDC964'),
-                realpath(__DIR__ . '/Models/GeoNames'),
-                realpath(__DIR__ . '/Models/Legacy'),
-                realpath(__DIR__ . '/Models/OneToOneInverseSideLoad'),
-                realpath(__DIR__ . '/Models/Routing'),
-                realpath(__DIR__ . '/Models/VersionedManyToOne'),
-                realpath(__DIR__ . '/Models/CompositeKeyInheritance'),
+                realpath(__DIR__.'/Models/DDC964'),
+                realpath(__DIR__.'/Models/GeoNames'),
+                realpath(__DIR__.'/Models/Legacy'),
+                realpath(__DIR__.'/Models/OneToOneInverseSideLoad'),
+                realpath(__DIR__.'/Models/Routing'),
+                realpath(__DIR__.'/Models/VersionedManyToOne'),
+                realpath(__DIR__.'/Models/CompositeKeyInheritance'),
                 //realpath(__DIR__ . '/Models/DDC2372'),
-                realpath(__DIR__ . '/Models/DDC3579'),
+                realpath(__DIR__.'/Models/DDC3579'),
                 //realpath(__DIR__ . '/Models/DDC5934'),
-                realpath(__DIR__ . '/Models/DirectoryTree'),
-                realpath(__DIR__ . '/Models/Global'),
-                realpath(__DIR__ . '/Models/ManyToManyPersister'),
-                realpath(__DIR__ . '/Models/OneToOneSingleTableInheritance'),
-                realpath(__DIR__ . '/Models/StockExchange'),
-                realpath(__DIR__ . '/Models/VersionedOneToOne'),
+                realpath(__DIR__.'/Models/DirectoryTree'),
+                realpath(__DIR__.'/Models/Global'),
+                realpath(__DIR__.'/Models/ManyToManyPersister'),
+                realpath(__DIR__.'/Models/OneToOneSingleTableInheritance'),
+                realpath(__DIR__.'/Models/StockExchange'),
+                realpath(__DIR__.'/Models/VersionedOneToOne'),
                 //realpath(__DIR__ . '/Models/CustomType'),
-                realpath(__DIR__ . '/Models/DDC2504'),
-                realpath(__DIR__ . '/Models/DDC3597'),
+                realpath(__DIR__.'/Models/DDC2504'),
+                realpath(__DIR__.'/Models/DDC3597'),
                 //realpath(__DIR__ . '/Models/DDC6412'),
-                realpath(__DIR__ . '/Models/ECommerce'),
-                realpath(__DIR__ . '/Models/Hydration'),
+                realpath(__DIR__.'/Models/ECommerce'),
+                realpath(__DIR__.'/Models/Hydration'),
                 //realpath(__DIR__ . '/Models/MixedToOneIdentity'),
-                realpath(__DIR__ . '/Models/Pagination'),
-                realpath(__DIR__ . '/Models/Taxi'),
-                realpath(__DIR__ . '/Models/DDC117'),
-                realpath(__DIR__ . '/Models/DDC2825'),
-                realpath(__DIR__ . '/Models/DDC3699'),
-                realpath(__DIR__ . '/Models/DDC753'),
-                realpath(__DIR__ . '/Models/Forum'),
-                realpath(__DIR__ . '/Models/IdentityIsAssociation'),
-                realpath(__DIR__ . '/Models/Navigation'),
-                realpath(__DIR__ . '/Models/ProxySpecifics'),
-                realpath(__DIR__ . '/Models/Tweet'),
+                realpath(__DIR__.'/Models/Pagination'),
+                realpath(__DIR__.'/Models/Taxi'),
+                realpath(__DIR__.'/Models/DDC117'),
+                realpath(__DIR__.'/Models/DDC2825'),
+                realpath(__DIR__.'/Models/DDC3699'),
+                realpath(__DIR__.'/Models/DDC753'),
+                realpath(__DIR__.'/Models/Forum'),
+                realpath(__DIR__.'/Models/IdentityIsAssociation'),
+                realpath(__DIR__.'/Models/Navigation'),
+                realpath(__DIR__.'/Models/ProxySpecifics'),
+                realpath(__DIR__.'/Models/Tweet'),
             ])
         );
 
@@ -195,7 +195,7 @@ 
             $eventManager = new EventManager();
         }
 
-        if (! ($metadatas instanceof MetadataCollection)){
+        if ( ! ($metadatas instanceof MetadataCollection)) {
             $metadataFactory = new ClassMetadataFactory($config, $conn, $eventManager);
             $metadatas = MetadataCollection::fromClassMetadatas(...$metadataFactory->getAllMetadata());
         }

lib/Doctrine/ORM/EntityManager.php

Spacing +20 added lines, -20 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\ORM;
 
@@ -257,7 +257,7 @@ 
     {
         $query = new Query($this);
 
-        if (! empty($dql)) {
+        if ( ! empty($dql)) {
             $query->setDQL($dql);
         }
 
@@ -347,7 +347,7 @@ 
         $class     = $this->mappings->get($entityName);
         $className = $class->getClassName();
 
-        if (! is_array($id)) {
+        if ( ! is_array($id)) {
             if ($class->isIdentifierComposite()) {
                 throw ORMInvalidArgumentException::invalidCompositeIdentifier();
             }
@@ -357,7 +357,7 @@ 
 
         foreach ($id as $i => $value) {
             if (is_object($value)) {
-                try{
+                try {
                     $this->mappings->get($value);
                     $id[$i] = $this->unitOfWork->getSingleIdentifierValue($value);
                 } catch (\Exception $e) {
@@ -373,7 +373,7 @@ 
         $sortedId = [];
 
         foreach ($class->identifier as $identifier) {
-            if (! isset($id[$identifier])) {
+            if ( ! isset($id[$identifier])) {
                 throw ORMException::missingIdentifierField($className, $identifier);
             }
 
@@ -390,7 +390,7 @@ 
         // Check identity map first
         $entity = $unitOfWork->tryGetById($sortedId, $class->getRootClassName());
         if ($entity !== false) {
-            if (! ($entity instanceof $className)) {
+            if ( ! ($entity instanceof $className)) {
                 return null;
             }
 
@@ -414,7 +414,7 @@ 
 
         switch (true) {
             case $lockMode === LockMode::OPTIMISTIC:
-                if (! $class->isVersioned()) {
+                if ( ! $class->isVersioned()) {
                     throw OptimisticLockException::notVersioned($className);
                 }
 
@@ -426,7 +426,7 @@ 
 
             case $lockMode === LockMode::PESSIMISTIC_READ:
             case $lockMode === LockMode::PESSIMISTIC_WRITE:
-                if (! $this->getConnection()->isTransactionActive()) {
+                if ( ! $this->getConnection()->isTransactionActive()) {
                     throw TransactionRequiredException::transactionRequired();
                 }
 
@@ -445,7 +445,7 @@ 
         $class     = $this->mappings->get($entityName);
         $className = $class->getClassName();
 
-        if (! is_array($id)) {
+        if ( ! is_array($id)) {
             if ($class->isIdentifierComposite()) {
                 throw ORMInvalidArgumentException::invalidCompositeIdentifier();
             }
@@ -459,7 +459,7 @@ 
             $scalarId[$i] = $value;
 
             if (is_object($value)) {
-                try{
+                try {
                     $this->mappings->get($value);
                     $scalarId[$i] = $this->unitOfWork->getSingleIdentifierValue($value);
                 } catch (\Exception $e) {
@@ -475,7 +475,7 @@ 
         $sortedId = [];
 
         foreach ($class->identifier as $identifier) {
-            if (! isset($scalarId[$identifier])) {
+            if ( ! isset($scalarId[$identifier])) {
                 throw ORMException::missingIdentifierField($className, $identifier);
             }
 
@@ -516,7 +516,7 @@ 
         $class     = $this->mappings->get($entityName);
         $className = $class->getClassName();
 
-        if (! is_array($id)) {
+        if ( ! is_array($id)) {
             if ($class->isIdentifierComposite()) {
                 throw ORMInvalidArgumentException::invalidCompositeIdentifier();
             }
@@ -526,7 +526,7 @@ 
 
         foreach ($id as $i => $value) {
             if (is_object($value)) {
-                try{
+                try {
                     $this->mappings->get($value);
                     $id[$i] = $this->unitOfWork->getSingleIdentifierValue($value);
                 } catch (\Exception $e) {
@@ -542,7 +542,7 @@ 
         $sortedId = [];
 
         foreach ($class->identifier as $identifier) {
-            if (! isset($id[$identifier])) {
+            if ( ! isset($id[$identifier])) {
                 throw ORMException::missingIdentifierField($className, $identifier);
             }
 
@@ -615,7 +615,7 @@ 
      */
     public function persist($entity)
     {
-        if (! is_object($entity)) {
+        if ( ! is_object($entity)) {
             throw ORMInvalidArgumentException::invalidObject('EntityManager#persist()', $entity);
         }
 
@@ -637,7 +637,7 @@ 
      */
     public function remove($entity)
     {
-        if (! is_object($entity)) {
+        if ( ! is_object($entity)) {
             throw ORMInvalidArgumentException::invalidObject('EntityManager#remove()', $entity);
         }
 
@@ -657,7 +657,7 @@ 
      */
     public function refresh($entity)
     {
-        if (! is_object($entity)) {
+        if ( ! is_object($entity)) {
             throw ORMInvalidArgumentException::invalidObject('EntityManager#refresh()', $entity);
         }
 
@@ -812,7 +812,7 @@ 
      */
     public static function create($connection, Configuration $config, EventManager $eventManager)
     {
-        if (! $config->getMetadataDriverImpl()) {
+        if ( ! $config->getMetadataDriverImpl()) {
             throw ORMException::missingMappingDriverImpl();
         }
 
@@ -855,12 +855,12 @@ 
             return DriverManager::getConnection($connection, $config, $eventManager ?: new EventManager());
         }
 
-        if (! $connection instanceof Connection) {
+        if ( ! $connection instanceof Connection) {
             throw new \InvalidArgumentException(
                 sprintf(
                     'Invalid $connection argument of type %s given%s.',
                     is_object($connection) ? get_class($connection) : gettype($connection),
-                    is_object($connection) ? '' : ': "' . $connection . '"'
+                    is_object($connection) ? '' : ': "'.$connection.'"'
                 )
             );
         }

lib/Doctrine/ORM/Persisters/Entity/BasicEntityPersister.php

Spacing +81 added lines, -81 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\ORM\Persisters\Entity;
 
@@ -300,16 +300,16 @@ 
         $tableName      = $versionedClass->table->getQuotedQualifiedName($this->platform);
         $columnName     = $this->platform->quoteIdentifier($versionProperty->getColumnName());
         $identifier     = array_map(
-            function ($columnName) {
+            function($columnName) {
                 return $this->platform->quoteIdentifier($columnName);
             },
             array_keys($versionedClass->getIdentifierColumns($this->em))
         );
 
         // FIXME: Order with composite keys might not be correct
-        $sql = 'SELECT ' . $columnName
-             . ' FROM ' . $tableName
-             . ' WHERE ' . implode(' = ? AND ', $identifier) . ' = ?';
+        $sql = 'SELECT '.$columnName
+             . ' FROM '.$tableName
+             . ' WHERE '.implode(' = ? AND ', $identifier).' = ?';
 
         $flattenedId = $this->em->getIdentifierFlattener()->flattenIdentifier($versionedClass, $id);
         $versionType = $versionProperty->getType();
@@ -348,13 +348,13 @@ 
         $tableName  = $this->class->getTableName();
         $updateData = $this->prepareUpdateData($entity);
 
-        if (! isset($updateData[$tableName])) {
+        if ( ! isset($updateData[$tableName])) {
             return;
         }
 
         $data = $updateData[$tableName];
 
-        if (! $data) {
+        if ( ! $data) {
             return;
         }
 
@@ -407,7 +407,7 @@ 
 
             if ($value !== null) {
                 // @todo guilhermeblanco Make sure we do not have flat association values.
-                if (! is_array($value)) {
+                if ( ! is_array($value)) {
                     $value = [$targetClass->identifier[0] => $value];
                 }
 
@@ -420,7 +420,7 @@ 
                 $referencedColumnName = $joinColumn->getReferencedColumnName();
                 $targetField          = $targetClass->fieldNames[$referencedColumnName];
 
-                if (! $joinColumn->getType()) {
+                if ( ! $joinColumn->getType()) {
                     $joinColumn->setType(PersisterHelper::getTypeOfColumn($referencedColumnName, $targetClass, $this->em));
                 }
 
@@ -486,7 +486,7 @@ 
                         $quotedColumnName     = $this->platform->quoteIdentifier($joinColumn->getColumnName());
                         $referencedColumnName = $joinColumn->getReferencedColumnName();
 
-                        if (! $joinColumn->getType()) {
+                        if ( ! $joinColumn->getType()) {
                             $joinColumn->setType(PersisterHelper::getTypeOfColumn($referencedColumnName, $targetClass, $this->em));
                         }
 
@@ -513,18 +513,18 @@ 
                 case Type::SMALLINT:
                 case Type::INTEGER:
                 case Type::BIGINT:
-                    $set[] = $versionColumnName . ' = ' . $versionColumnName . ' + 1';
+                    $set[] = $versionColumnName.' = '.$versionColumnName.' + 1';
                     break;
 
                 case Type::DATETIME:
-                    $set[] = $versionColumnName . ' = CURRENT_TIMESTAMP';
+                    $set[] = $versionColumnName.' = CURRENT_TIMESTAMP';
                     break;
             }
         }
 
-        $sql = 'UPDATE ' . $quotedTableName
-             . ' SET ' . implode(', ', $set)
-             . ' WHERE ' . implode(' = ? AND ', $where) . ' = ?';
+        $sql = 'UPDATE '.$quotedTableName
+             . ' SET '.implode(', ', $set)
+             . ' WHERE '.implode(' = ? AND ', $where).' = ?';
 
         $result = $this->conn->executeUpdate($sql, $params, $types);
 
@@ -541,7 +541,7 @@ 
     protected function deleteJoinTableRecords($identifier)
     {
         foreach ($this->class->getDeclaredPropertiesIterator() as $association) {
-            if (! ($association instanceof ManyToManyAssociationMetadata)) {
+            if ( ! ($association instanceof ManyToManyAssociationMetadata)) {
                 continue;
             }
 
@@ -553,7 +553,7 @@ 
             $otherKeys         = [];
             $keys              = [];
 
-            if (! $owningAssociation->isOwningSide()) {
+            if ( ! $owningAssociation->isOwningSide()) {
                 $class             = $this->em->getClassMetadata($association->getTargetEntity());
                 $owningAssociation = $class->getProperty($association->getMappedBy());
             }
@@ -666,7 +666,7 @@ 
             }
 
             // Only owning side of x-1 associations can have a FK column.
-            if (! $property instanceof ToOneAssociationMetadata || ! $property->isOwningSide()) {
+            if ( ! $property instanceof ToOneAssociationMetadata || ! $property->isOwningSide()) {
                 continue;
             }
 
@@ -686,7 +686,7 @@ 
                 /** @var JoinColumnMetadata $joinColumn */
                 $referencedColumnName = $joinColumn->getReferencedColumnName();
 
-                if (! $joinColumn->getType()) {
+                if ( ! $joinColumn->getType()) {
                     $joinColumn->setType(PersisterHelper::getTypeOfColumn($referencedColumnName, $targetClass, $this->em));
                 }
 
@@ -714,7 +714,7 @@ 
         $propertyName = $this->class->fieldNames[$columnName];
         $property     = $this->class->getProperty($propertyName);
 
-        if (! $property) {
+        if ( ! $property) {
             return null;
         }
 
@@ -733,7 +733,7 @@ 
             /** @var JoinColumnMetadata $joinColumn */
             $referencedColumnName = $joinColumn->getReferencedColumnName();
 
-            if (! $joinColumn->getType()) {
+            if ( ! $joinColumn->getType()) {
                 $joinColumn->setType(PersisterHelper::getTypeOfColumn($referencedColumnName, $targetClass, $this->em));
             }
 
@@ -838,7 +838,7 @@ 
             $sourceKeyColumn = $joinColumn->getReferencedColumnName();
             $targetKeyColumn = $joinColumn->getColumnName();
 
-            if (! isset($sourceClass->fieldNames[$sourceKeyColumn])) {
+            if ( ! isset($sourceClass->fieldNames[$sourceKeyColumn])) {
                 throw MappingException::joinColumnMustPointToMappedField(
                     $sourceClass->getClassName(),
                     $sourceKeyColumn
@@ -1065,7 +1065,7 @@ 
         $criteria    = [];
         $parameters  = [];
 
-        if (! $association->isOwningSide()) {
+        if ( ! $association->isOwningSide()) {
             $class       = $this->em->getClassMetadata($association->getTargetEntity());
             $owningAssoc = $class->getProperty($association->getMappedBy());
         }
@@ -1094,7 +1094,7 @@ 
                 $value = $value[$targetClass->identifier[0]];
             }
 
-            $criteria[$joinTableName . '.' . $quotedColumnName] = $value;
+            $criteria[$joinTableName.'.'.$quotedColumnName] = $value;
             $parameters[]                                       = [
                 'value' => $value,
                 'field' => $fieldName,
@@ -1144,11 +1144,11 @@ 
 
         switch ($lockMode) {
             case LockMode::PESSIMISTIC_READ:
-                $lockSql = ' ' . $this->platform->getReadLockSQL();
+                $lockSql = ' '.$this->platform->getReadLockSQL();
                 break;
 
             case LockMode::PESSIMISTIC_WRITE:
-                $lockSql = ' ' . $this->platform->getWriteLockSQL();
+                $lockSql = ' '.$this->platform->getWriteLockSQL();
                 break;
         }
 
@@ -1159,14 +1159,14 @@ 
 
         if ($filterSql !== '') {
             $conditionSql = $conditionSql
-                ? $conditionSql . ' AND ' . $filterSql
+                ? $conditionSql.' AND '.$filterSql
                 : $filterSql;
         }
 
-        $select = 'SELECT ' . $columnList;
-        $from   = ' FROM ' . $tableName . ' ' . $tableAlias;
-        $join   = $this->currentPersisterContext->selectJoinSql . $joinSql;
-        $where  = ($conditionSql ? ' WHERE ' . $conditionSql : '');
+        $select = 'SELECT '.$columnList;
+        $from   = ' FROM '.$tableName.' '.$tableAlias;
+        $join   = $this->currentPersisterContext->selectJoinSql.$joinSql;
+        $where  = ($conditionSql ? ' WHERE '.$conditionSql : '');
         $lock   = $this->platform->appendLockHint($from, $lockMode);
         $query  = $select
             . $lock
@@ -1174,7 +1174,7 @@ 
             . $where
             . $orderBySql;
 
-        return $this->platform->modifyLimitQuery($query, $limit, $offset) . $lockSql;
+        return $this->platform->modifyLimitQuery($query, $limit, $offset).$lockSql;
     }
 
     /**
@@ -1193,13 +1193,13 @@ 
 
         if ($filterSql !== '') {
             $conditionSql = $conditionSql
-                ? $conditionSql . ' AND ' . $filterSql
+                ? $conditionSql.' AND '.$filterSql
                 : $filterSql;
         }
 
         $sql = 'SELECT COUNT(*) '
-            . 'FROM ' . $tableName . ' ' . $tableAlias
-            . (empty($conditionSql) ? '' : ' WHERE ' . $conditionSql);
+            . 'FROM '.$tableName.' '.$tableAlias
+            . (empty($conditionSql) ? '' : ' WHERE '.$conditionSql);
 
         return $sql;
     }
@@ -1216,7 +1216,7 @@ 
      */
     final protected function getOrderBySQL(array $orderBy, $baseTableAlias)
     {
-        if (! $orderBy) {
+        if ( ! $orderBy) {
             return '';
         }
 
@@ -1225,7 +1225,7 @@ 
         foreach ($orderBy as $fieldName => $orientation) {
             $orientation = strtoupper(trim($orientation));
 
-            if (! in_array($orientation, ['ASC', 'DESC'], true)) {
+            if ( ! in_array($orientation, ['ASC', 'DESC'], true)) {
                 throw ORMException::invalidOrientation($this->class->getClassName(), $fieldName);
             }
 
@@ -1235,11 +1235,11 @@ 
                 $tableAlias = $this->getSQLTableAlias($property->getTableName());
                 $columnName = $this->platform->quoteIdentifier($property->getColumnName());
 
-                $orderByList[] = $tableAlias . '.' . $columnName . ' ' . $orientation;
+                $orderByList[] = $tableAlias.'.'.$columnName.' '.$orientation;
 
                 continue;
             } elseif ($property instanceof AssociationMetadata) {
-                if (! $property->isOwningSide()) {
+                if ( ! $property->isOwningSide()) {
                     throw ORMException::invalidFindByInverseAssociation($this->class->getClassName(), $fieldName);
                 }
 
@@ -1252,7 +1252,7 @@ 
                     /** @var JoinColumnMetadata $joinColumn */
                     $quotedColumnName = $this->platform->quoteIdentifier($joinColumn->getColumnName());
 
-                    $orderByList[] = $tableAlias . '.' . $quotedColumnName . ' ' . $orientation;
+                    $orderByList[] = $tableAlias.'.'.$quotedColumnName.' '.$orientation;
                 }
 
                 continue;
@@ -1261,7 +1261,7 @@ 
             throw ORMException::unrecognizedField($fieldName);
         }
 
-        return ' ORDER BY ' . implode(', ', $orderByList);
+        return ' ORDER BY '.implode(', ', $orderByList);
     }
 
     /**
@@ -1303,7 +1303,7 @@ 
                     $isAssocToOneInverseSide = $property instanceof ToOneAssociationMetadata && ! $property->isOwningSide();
                     $isAssocFromOneEager     = ! $property instanceof ManyToManyAssociationMetadata && $property->getFetchMode() === FetchMode::EAGER;
 
-                    if (! ($isAssocFromOneEager || $isAssocToOneInverseSide)) {
+                    if ( ! ($isAssocFromOneEager || $isAssocToOneInverseSide)) {
                         break;
                     }
 
@@ -1318,7 +1318,7 @@ 
                         break; // now this is why you shouldn't use inheritance
                     }
 
-                    $assocAlias = 'e' . ($eagerAliasCounter++);
+                    $assocAlias = 'e'.($eagerAliasCounter++);
 
                     $this->currentPersisterContext->rsm->addJoinedEntityResult($targetEntity, $assocAlias, 'r', $fieldName);
 
@@ -1346,14 +1346,14 @@ 
                         $this->currentPersisterContext->rsm->addIndexBy($assocAlias, $property->getIndexedBy());
                     }
 
-                    if (! $property->isOwningSide()) {
+                    if ( ! $property->isOwningSide()) {
                         $owningAssociation = $eagerEntity->getProperty($property->getMappedBy());
                     }
 
                     $joinTableAlias = $this->getSQLTableAlias($eagerEntity->getTableName(), $assocAlias);
                     $joinTableName  = $eagerEntity->table->getQuotedQualifiedName($this->platform);
 
-                    $this->currentPersisterContext->selectJoinSql .= ' ' . $this->getJoinSQLForAssociation($property);
+                    $this->currentPersisterContext->selectJoinSql .= ' '.$this->getJoinSQLForAssociation($property);
 
                     $sourceClass      = $this->em->getClassMetadata($owningAssociation->getSourceEntity());
                     $targetClass      = $this->em->getClassMetadata($owningAssociation->getTargetEntity());
@@ -1377,7 +1377,7 @@ 
                         $joinCondition[] = $filterSql;
                     }
 
-                    $this->currentPersisterContext->selectJoinSql .= ' ' . $joinTableName . ' ' . $joinTableAlias . ' ON ';
+                    $this->currentPersisterContext->selectJoinSql .= ' '.$joinTableName.' '.$joinTableAlias.' ON ';
                     $this->currentPersisterContext->selectJoinSql .= implode(' AND ', $joinCondition);
 
                     break;
@@ -1399,7 +1399,7 @@ 
      */
     protected function getSelectColumnAssociationSQL($field, AssociationMetadata $association, ClassMetadata $class, $alias = 'r')
     {
-        if (! ($association->isOwningSide() && $association instanceof ToOneAssociationMetadata)) {
+        if ( ! ($association->isOwningSide() && $association instanceof ToOneAssociationMetadata)) {
             return '';
         }
 
@@ -1414,7 +1414,7 @@ 
             $referencedColumnName = $joinColumn->getReferencedColumnName();
             $resultColumnName     = $this->getSQLColumnAlias();
 
-            if (! $joinColumn->getType()) {
+            if ( ! $joinColumn->getType()) {
                 $joinColumn->setType(PersisterHelper::getTypeOfColumn($referencedColumnName, $targetClass, $this->em));
             }
 
@@ -1444,7 +1444,7 @@ 
         $owningAssociation = $association;
         $sourceTableAlias  = $this->getSQLTableAlias($this->class->getTableName());
 
-        if (! $association->isOwningSide()) {
+        if ( ! $association->isOwningSide()) {
             $targetEntity      = $this->em->getClassMetadata($association->getTargetEntity());
             $owningAssociation = $targetEntity->getProperty($association->getMappedBy());
         }
@@ -1466,7 +1466,7 @@ 
             );
         }
 
-        return ' INNER JOIN ' . $joinTableName . ' ON ' . implode(' AND ', $conditions);
+        return ' INNER JOIN '.$joinTableName.' ON '.implode(' AND ', $conditions);
     }
 
     /**
@@ -1561,7 +1561,7 @@ 
                             $columnName           = $joinColumn->getColumnName();
                             $referencedColumnName = $joinColumn->getReferencedColumnName();
 
-                            if (! $joinColumn->getType()) {
+                            if ( ! $joinColumn->getType()) {
                                 $joinColumn->setType(PersisterHelper::getTypeOfColumn($referencedColumnName, $targetClass, $this->em));
                             }
 
@@ -1600,7 +1600,7 @@ 
 
         $this->currentPersisterContext->rsm->addFieldResult($alias, $columnAlias, $field, $class->getClassName());
 
-        return $property->getType()->convertToPHPValueSQL($sql, $this->platform) . ' AS ' . $columnAlias;
+        return $property->getType()->convertToPHPValueSQL($sql, $this->platform).' AS '.$columnAlias;
     }
 
     /**
@@ -1614,14 +1614,14 @@ 
     protected function getSQLTableAlias($tableName, $assocName = '')
     {
         if ($tableName) {
-            $tableName .= '#' . $assocName;
+            $tableName .= '#'.$assocName;
         }
 
         if (isset($this->currentPersisterContext->sqlTableAliases[$tableName])) {
             return $this->currentPersisterContext->sqlTableAliases[$tableName];
         }
 
-        $tableAlias = 't' . $this->currentPersisterContext->sqlAliasCounter++;
+        $tableAlias = 't'.$this->currentPersisterContext->sqlAliasCounter++;
 
         $this->currentPersisterContext->sqlTableAliases[$tableName] = $tableAlias;
 
@@ -1647,7 +1647,7 @@ 
         }
 
         $lock  = $this->getLockTablesSql($lockMode);
-        $where = ($conditionSql ? ' WHERE ' . $conditionSql : '') . ' ';
+        $where = ($conditionSql ? ' WHERE '.$conditionSql : '').' ';
         $sql   = 'SELECT 1 '
              . $lock
              . $where
@@ -1670,7 +1670,7 @@ 
         $tableName = $this->class->table->getQuotedQualifiedName($this->platform);
 
         return $this->platform->appendLockHint(
-            'FROM ' . $tableName . ' ' . $this->getSQLTableAlias($this->class->getTableName()),
+            'FROM '.$tableName.' '.$this->getSQLTableAlias($this->class->getTableName()),
             $lockMode
         );
     }
@@ -1720,19 +1720,19 @@ 
 
             if ($comparison !== null) {
                 // special case null value handling
-                if (($comparison === Comparison::EQ || $comparison === Comparison::IS) && $value ===null) {
-                    $selectedColumns[] = $column . ' IS NULL';
+                if (($comparison === Comparison::EQ || $comparison === Comparison::IS) && $value === null) {
+                    $selectedColumns[] = $column.' IS NULL';
 
                     continue;
                 }
 
                 if ($comparison === Comparison::NEQ && $value === null) {
-                    $selectedColumns[] = $column . ' IS NOT NULL';
+                    $selectedColumns[] = $column.' IS NOT NULL';
 
                     continue;
                 }
 
-                $selectedColumns[] = $column . ' ' . sprintf(self::$comparisonMap[$comparison], $placeholder);
+                $selectedColumns[] = $column.' '.sprintf(self::$comparisonMap[$comparison], $placeholder);
 
                 continue;
             }
@@ -1780,7 +1780,7 @@ 
             $tableAlias = $this->getSQLTableAlias($property->getTableName());
             $columnName = $this->platform->quoteIdentifier($property->getColumnName());
 
-            return [$tableAlias . '.' . $columnName];
+            return [$tableAlias.'.'.$columnName];
         }
 
         if ($property instanceof AssociationMetadata) {
@@ -1789,7 +1789,7 @@ 
 
             // Many-To-Many requires join table check for joinColumn
             if ($owningAssociation instanceof ManyToManyAssociationMetadata) {
-                if (! $owningAssociation->isOwningSide()) {
+                if ( ! $owningAssociation->isOwningSide()) {
                     $owningAssociation = $association;
                 }
 
@@ -1803,14 +1803,14 @@ 
                 foreach ($joinColumns as $joinColumn) {
                     $quotedColumnName = $this->platform->quoteIdentifier($joinColumn->getColumnName());
 
-                    $columns[] = $joinTableName . '.' . $quotedColumnName;
+                    $columns[] = $joinTableName.'.'.$quotedColumnName;
                 }
             } else {
-                if (! $owningAssociation->isOwningSide()) {
+                if ( ! $owningAssociation->isOwningSide()) {
                     throw ORMException::invalidFindByInverseAssociation($this->class->getClassName(), $field);
                 }
 
-                $class      = $this->class->isInheritedProperty($field)
+                $class = $this->class->isInheritedProperty($field)
                     ? $owningAssociation->getDeclaringClass()
                     : $this->class
                 ;
@@ -1819,7 +1819,7 @@ 
                 foreach ($owningAssociation->getJoinColumns() as $joinColumn) {
                     $quotedColumnName = $this->platform->quoteIdentifier($joinColumn->getColumnName());
 
-                    $columns[] = $tableAlias . '.' . $quotedColumnName;
+                    $columns[] = $tableAlias.'.'.$quotedColumnName;
                 }
             }
 
@@ -1928,7 +1928,7 @@ 
                 $value = $value[$targetClass->identifier[0]];
             }
 
-            $criteria[$tableAlias . '.' . $quotedColumnName] = $value;
+            $criteria[$tableAlias.'.'.$quotedColumnName] = $value;
             $parameters[]                                    = [
                 'value' => $value,
                 'field' => $fieldName,
@@ -2013,7 +2013,7 @@ 
             case ($property instanceof AssociationMetadata):
                 $class = $this->em->getClassMetadata($property->getTargetEntity());
 
-                if (! $property->isOwningSide()) {
+                if ( ! $property->isOwningSide()) {
                     $property = $class->getProperty($property->getMappedBy());
                     $class    = $this->em->getClassMetadata($property->getTargetEntity());
                 }
@@ -2027,7 +2027,7 @@ 
                     /** @var JoinColumnMetadata $joinColumn */
                     $referencedColumnName = $joinColumn->getReferencedColumnName();
 
-                    if (! $joinColumn->getType()) {
+                    if ( ! $joinColumn->getType()) {
                         $joinColumn->setType(PersisterHelper::getTypeOfColumn($referencedColumnName, $class, $this->em));
                     }
 
@@ -2042,7 +2042,7 @@ 
         }
 
         if (is_array($value)) {
-            return array_map(function ($type) {
+            return array_map(function($type) {
                 return $type->getBindingType() + Connection::ARRAY_PARAM_OFFSET;
             }, $types);
         }
@@ -2069,7 +2069,7 @@ 
             return [$newValue];
         }
 
-        $unitOfWork      = $this->em->getUnitOfWork();
+        $unitOfWork = $this->em->getUnitOfWork();
 
         if (is_object($value) && $this->mappings->has($value)) {
             $class     = $this->mappings->get($value);
@@ -2098,7 +2098,7 @@ 
      */
     private function getIndividualValue($value)
     {
-        if (! is_object($value) || ! $this->em->getMetadataFactory()->hasMetadataFor(StaticClassNameConverter::getClass($value))) {
+        if ( ! is_object($value) || ! $this->em->getMetadataFactory()->hasMetadataFor(StaticClassNameConverter::getClass($value))) {
             return $value;
         }
 
@@ -2112,7 +2112,7 @@ 
     {
         $criteria = $this->getIdentifier($entity);
 
-        if (! $criteria) {
+        if ( ! $criteria) {
             return false;
         }
 
@@ -2120,12 +2120,12 @@ 
 
         $sql = 'SELECT 1 '
              . $this->getLockTablesSql(null)
-             . ' WHERE ' . $this->getSelectConditionSQL($criteria);
+             . ' WHERE '.$this->getSelectConditionSQL($criteria);
 
         list($params, $types) = $this->expandParameters($criteria);
 
         if ($extraConditions !== null) {
-            $sql                                 .= ' AND ' . $this->getSelectConditionCriteriaSQL($extraConditions);
+            $sql                                 .= ' AND '.$this->getSelectConditionCriteriaSQL($extraConditions);
             list($criteriaParams, $criteriaTypes) = $this->expandCriteriaParameters($extraConditions);
 
             $params = array_merge($params, $criteriaParams);
@@ -2135,7 +2135,7 @@ 
         $filterSql = $this->generateFilterConditionSQL($this->class, $alias);
 
         if ($filterSql) {
-            $sql .= ' AND ' . $filterSql;
+            $sql .= ' AND '.$filterSql;
         }
 
         return (bool) $this->conn->fetchColumn($sql, $params, 0, $types);
@@ -2148,13 +2148,13 @@ 
      */
     protected function getJoinSQLForAssociation(AssociationMetadata $association)
     {
-        if (! $association->isOwningSide()) {
+        if ( ! $association->isOwningSide()) {
             return 'LEFT JOIN';
         }
 
         // if one of the join columns is nullable, return left join
         foreach ($association->getJoinColumns() as $joinColumn) {
-            if (! $joinColumn->isNullable()) {
+            if ( ! $joinColumn->isNullable()) {
                 continue;
             }
 
@@ -2171,7 +2171,7 @@ 
      */
     public function getSQLColumnAlias()
     {
-        return $this->platform->getSQLResultCasing('c' . $this->currentPersisterContext->sqlAliasCounter++);
+        return $this->platform->getSQLResultCasing('c'.$this->currentPersisterContext->sqlAliasCounter++);
     }
 
     /**
@@ -2190,13 +2190,13 @@ 
             $filterExpr = $filter->addFilterConstraint($targetEntity, $targetTableAlias);
 
             if ($filterExpr !== '') {
-                $filterClauses[] = '(' . $filterExpr . ')';
+                $filterClauses[] = '('.$filterExpr.')';
             }
         }
 
         $sql = implode(' AND ', $filterClauses);
 
-        return $sql ? '(' . $sql . ')' : ''; // Wrap again to avoid "X or Y and FilterConditionSQL"
+        return $sql ? '('.$sql.')' : ''; // Wrap again to avoid "X or Y and FilterConditionSQL"
     }
 
     /**

lib/Doctrine/ORM/Mapping/MetadataCollection.php

Spacing +4 added lines, -4 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\ORM\Mapping;
 
@@ -18,7 +18,7 @@ 
     private function __construct(array $metadata)
     {
         $metadataByClassName = array_combine(
-            array_map(function (ClassMetadata $metadata) { return $metadata->getClassName(); }, $metadata),
+            array_map(function(ClassMetadata $metadata) { return $metadata->getClassName(); }, $metadata),
             $metadata
         );
         $this->metadata = $metadataByClassName;
@@ -27,8 +27,8 @@ 
     public function get(string $name) : ClassMetadata
     {
         $name = StaticClassNameConverter::getRealClass($name);
-        if(!isset($this->metadata[$name])){
-            throw new \Exception('No metadata found for ' . $name);
+        if ( ! isset($this->metadata[$name])) {
+            throw new \Exception('No metadata found for '.$name);
         }
         return $this->metadata[$name];
     }