doctrine / orm

tests/Doctrine/Tests/ORM/Functional/Ticket/GH7820Test.php

Spacing +8 added lines, -8 removed lines

@@ -49,13 +49,13 @@ 
     {
         parent::setUp();
 
-        if (! Type::hasType(GH7820LineTextType::class)) {
+        if ( ! Type::hasType(GH7820LineTextType::class)) {
             Type::addType(GH7820LineTextType::class, GH7820LineTextType::class);
         }
 
         $this->setUpEntitySchema([GH7820Line::class]);
 
-        $this->_em->createQuery('DELETE FROM ' . GH7820Line::class . ' l')
+        $this->_em->createQuery('DELETE FROM '.GH7820Line::class.' l')
             ->execute();
 
         foreach (self::SONG as $index => $line) {
@@ -73,7 +73,7 @@ 
 
         self::assertSame(
             self::SONG,
-            array_map(static function (GH7820Line $line) : string {
+            array_map(static function(GH7820Line $line) : string {
                 return $line->toString();
             }, iterator_to_array(new Paginator($query)))
         );
@@ -95,7 +95,7 @@ 
 
         self::assertSame(
             self::SONG,
-            array_map(static function (GH7820Line $line) : string {
+            array_map(static function(GH7820Line $line) : string {
                 return $line->toString();
             }, iterator_to_array(new Paginator($query))),
             'Expected to return expected data before query cache is populated with DQL -> SQL translation. Were SQL parameters translated?'
@@ -107,7 +107,7 @@ 
 
         self::assertSame(
             self::SONG,
-            array_map(static function (GH7820Line $line) : string {
+            array_map(static function(GH7820Line $line) : string {
                 return $line->toString();
             }, iterator_to_array(new Paginator($query))),
             'Expected to return expected data even when DQL -> SQL translation is present in cache. Were SQL parameters translated again?'
@@ -165,7 +165,7 @@ 
 
     public function __toString() : string
     {
-        return 'Line: ' . $this->text;
+        return 'Line: '.$this->text;
     }
 }
 
@@ -175,7 +175,7 @@ 
     {
         $text = parent::convertToPHPValue($value, $platform);
 
-        if (! is_string($text)) {
+        if ( ! is_string($text)) {
             return $text;
         }
 
@@ -184,7 +184,7 @@ 
 
     public function convertToDatabaseValue($value, AbstractPlatform $platform)
     {
-        if (! $value instanceof GH7820LineText) {
+        if ( ! $value instanceof GH7820LineText) {
             return parent::convertToDatabaseValue($value, $platform);
         }
 

tests/Doctrine/Tests/ORM/Tools/Export/AbstractClassMetadataExporterTest.php

Spacing +13 added lines, -13 removed lines

@@ -44,7 +44,7 @@ 
     {
         $driverMock = new DriverMock();
         $config = new Configuration();
-        $config->setProxyDir(__DIR__ . '/../../Proxies');
+        $config->setProxyDir(__DIR__.'/../../Proxies');
         $config->setProxyNamespace('Doctrine\Tests\Proxies');
         $eventManager = new EventManager();
         $conn = new ConnectionMock([], $driverMock, $config, $eventManager);
@@ -62,7 +62,7 @@ 
             'yaml'       => YamlDriver::class,
         ];
 
-        $this->assertArrayHasKey($type, $mappingDriver, "There is no metadata driver for the type '" . $type . "'.");
+        $this->assertArrayHasKey($type, $mappingDriver, "There is no metadata driver for the type '".$type."'.");
 
         $class  = $mappingDriver[$type];
         $driver = ($type === 'annotation')
@@ -85,10 +85,10 @@ 
 
     public function testExportDirectoryAndFilesAreCreated()
     {
-        $this->_deleteDirectory(__DIR__ . '/export/'.$this->_getType());
+        $this->_deleteDirectory(__DIR__.'/export/'.$this->_getType());
 
         $type = $this->_getType();
-        $metadataDriver = $this->_createMetadataDriver($type, __DIR__ . '/' . $type);
+        $metadataDriver = $this->_createMetadataDriver($type, __DIR__.'/'.$type);
         $em = $this->_createEntityManager($metadataDriver);
         $cmf = $this->_createClassMetadataFactory($em, $type);
         $metadata = $cmf->getAllMetadata();
@@ -99,7 +99,7 @@ 
 
         $type = $this->_getType();
         $cme = new ClassMetadataExporter();
-        $exporter = $cme->getExporter($type, __DIR__ . '/export/' . $type);
+        $exporter = $cme->getExporter($type, __DIR__.'/export/'.$type);
 
         if ($type === 'annotation') {
             $entityGenerator = new EntityGenerator();
@@ -114,9 +114,9 @@ 
         $exporter->export();
 
         if ($type == 'annotation') {
-            $this->assertTrue(file_exists(__DIR__ . '/export/' . $type . '/'.str_replace('\\', '/', ExportedUser::class).$this->_extension));
+            $this->assertTrue(file_exists(__DIR__.'/export/'.$type.'/'.str_replace('\\', '/', ExportedUser::class).$this->_extension));
         } else {
-            $this->assertTrue(file_exists(__DIR__ . '/export/' . $type . '/Doctrine.Tests.ORM.Tools.Export.ExportedUser'.$this->_extension));
+            $this->assertTrue(file_exists(__DIR__.'/export/'.$type.'/Doctrine.Tests.ORM.Tools.Export.ExportedUser'.$this->_extension));
         }
 
         $this->assertHasDeprecationMessages();
@@ -129,7 +129,7 @@ 
     {
         $type = $this->_getType();
 
-        $metadataDriver = $this->_createMetadataDriver($type, __DIR__ . '/export/' . $type);
+        $metadataDriver = $this->_createMetadataDriver($type, __DIR__.'/export/'.$type);
         $em = $this->_createEntityManager($metadataDriver);
         $cmf = $this->_createClassMetadataFactory($em, $type);
         $metadata = $cmf->getAllMetadata();
@@ -215,7 +215,7 @@ 
         $type = $this->_getType();
 
         if ($type == 'xml') {
-            $xml = simplexml_load_file(__DIR__ . '/export/'.$type.'/Doctrine.Tests.ORM.Tools.Export.ExportedUser.dcm.xml');
+            $xml = simplexml_load_file(__DIR__.'/export/'.$type.'/Doctrine.Tests.ORM.Tools.Export.ExportedUser.dcm.xml');
 
             $xml->registerXPathNamespace("d", "http://doctrine-project.org/schemas/orm/doctrine-mapping");
             $nodes = $xml->xpath("/d:doctrine-mapping/d:entity/d:field[@name='name' and @type='string' and @nullable='true']");
@@ -359,7 +359,7 @@ 
         $type = $this->_getType();
 
         if ($type == 'xml') {
-            $xml = simplexml_load_file(__DIR__ . '/export/'.$type.'/Doctrine.Tests.ORM.Tools.Export.ExportedUser.dcm.xml');
+            $xml = simplexml_load_file(__DIR__.'/export/'.$type.'/Doctrine.Tests.ORM.Tools.Export.ExportedUser.dcm.xml');
 
             $xml->registerXPathNamespace("d", "http://doctrine-project.org/schemas/orm/doctrine-mapping");
             $nodes = $xml->xpath("/d:doctrine-mapping/d:entity/d:one-to-many[@field='interests']/d:cascade/d:*");
@@ -368,7 +368,7 @@ 
             $this->assertEquals('cascade-all', $nodes[0]->getName());
         } else if ($type == 'yaml') {
             $yaml = new Parser();
-            $value = $yaml->parse(file_get_contents(__DIR__ . '/export/'.$type.'/Doctrine.Tests.ORM.Tools.Export.ExportedUser.dcm.yml'));
+            $value = $yaml->parse(file_get_contents(__DIR__.'/export/'.$type.'/Doctrine.Tests.ORM.Tools.Export.ExportedUser.dcm.yml'));
 
             $this->assertTrue(isset($value[ExportedUser::class]['oneToMany']['interests']['cascade']));
             $this->assertEquals(1, count($value[ExportedUser::class]['oneToMany']['interests']['cascade']));
@@ -408,10 +408,10 @@ 
         if (is_file($path)) {
             return unlink($path);
         } else if (is_dir($path)) {
-            $files = glob(rtrim($path,'/').'/*');
+            $files = glob(rtrim($path, '/').'/*');
 
             if (is_array($files)) {
-                foreach ($files as $file){
+                foreach ($files as $file) {
                     $this->_deleteDirectory($file);
                 }
             }

tests/Doctrine/Tests/ORM/Functional/Ticket/DDC1383Test.php

Indentation +47 added lines, -47 removed lines

@@ -27,30 +27,30 @@ 
 
     public function testFailingCase()
     {
-		$parent = new DDC1383Entity();
-		$child = new DDC1383Entity();
+        $parent = new DDC1383Entity();
+        $child = new DDC1383Entity();
 
-		$child->setReference($parent);
+        $child->setReference($parent);
 
-		$this->_em->persist($parent);
-		$this->_em->persist($child);
+        $this->_em->persist($parent);
+        $this->_em->persist($child);
 
-		$id = $child->getId();
+        $id = $child->getId();
 
-		$this->_em->flush();
-		$this->_em->clear();
+        $this->_em->flush();
+        $this->_em->clear();
 
-		// Try merging the parent entity
-		$child = $this->_em->merge($child);
-		$parent = $child->getReference();
+        // Try merging the parent entity
+        $child = $this->_em->merge($child);
+        $parent = $child->getReference();
 
-		// Parent is not instance of the abstract class
-		self::assertTrue($parent instanceof DDC1383AbstractEntity,
-				"Entity class is " . get_class($parent) . ', "DDC1383AbstractEntity" was expected');
+        // Parent is not instance of the abstract class
+        self::assertTrue($parent instanceof DDC1383AbstractEntity,
+                "Entity class is " . get_class($parent) . ', "DDC1383AbstractEntity" was expected');
 
-		// Parent is NOT instance of entity
-		self::assertTrue($parent instanceof DDC1383Entity,
-				"Entity class is " . get_class($parent) . ', "DDC1383Entity" was expected');
+        // Parent is NOT instance of entity
+        self::assertTrue($parent instanceof DDC1383Entity,
+                "Entity class is " . get_class($parent) . ', "DDC1383Entity" was expected');
         $this->assertHasDeprecationMessages();
     }
 }
@@ -63,22 +63,22 @@ 
  */
 abstract class DDC1383AbstractEntity
 {
-	/**
-	 * @Id
-	 * @Column(type="integer")
-	 * @GeneratedValue
-	 */
-	protected $id;
-
-	public function getId()
-	{
-		return $this->id;
-	}
-
-	public function setId($id)
-	{
-		$this->id = $id;
-	}
+    /**
+     * @Id
+     * @Column(type="integer")
+     * @GeneratedValue
+     */
+    protected $id;
+
+    public function getId()
+    {
+        return $this->id;
+    }
+
+    public function setId($id)
+    {
+        $this->id = $id;
+    }
 }
 
 /**
@@ -86,18 +86,18 @@ 
  */
 class DDC1383Entity extends DDC1383AbstractEntity
 {
-	/**
-	 * @ManyToOne(targetEntity="DDC1383AbstractEntity")
-	 */
-	protected $reference;
-
-	public function getReference()
-	{
-		return $this->reference;
-	}
-
-	public function setReference(DDC1383AbstractEntity $reference)
-	{
-		$this->reference = $reference;
-	}
+    /**
+     * @ManyToOne(targetEntity="DDC1383AbstractEntity")
+     */
+    protected $reference;
+
+    public function getReference()
+    {
+        return $this->reference;
+    }
+
+    public function setReference(DDC1383AbstractEntity $reference)
+    {
+        $this->reference = $reference;
+    }
 }

Spacing +3 added lines, -3 removed lines

@@ -22,7 +22,7 @@ 
                 $this->_em->getClassMetadata(DDC1383Entity::class),
                 ]
             );
-        } catch(\Exception $ignored) {}
+        } catch (\Exception $ignored) {}
     }
 
     public function testFailingCase()
@@ -46,11 +46,11 @@ 
 
 		// Parent is not instance of the abstract class
 		self::assertTrue($parent instanceof DDC1383AbstractEntity,
-				"Entity class is " . get_class($parent) . ', "DDC1383AbstractEntity" was expected');
+				"Entity class is ".get_class($parent).', "DDC1383AbstractEntity" was expected');
 
 		// Parent is NOT instance of entity
 		self::assertTrue($parent instanceof DDC1383Entity,
-				"Entity class is " . get_class($parent) . ', "DDC1383Entity" was expected');
+				"Entity class is ".get_class($parent).', "DDC1383Entity" was expected');
         $this->assertHasDeprecationMessages();
     }
 }

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

Spacing +3 added lines, -3 removed lines

@@ -17,11 +17,11 @@ 
 
     protected function _loadDriver()
     {
-        if (!class_exists(Yaml::class, true)) {
+        if ( ! class_exists(Yaml::class, true)) {
             $this->markTestSkipped('Please install Symfony YAML Component into the include path of your PHP installation.');
         }
 
-        return new YamlDriver(__DIR__ . DIRECTORY_SEPARATOR . 'yaml');
+        return new YamlDriver(__DIR__.DIRECTORY_SEPARATOR.'yaml');
     }
 
     /**
@@ -32,7 +32,7 @@ 
     public function testJoinTablesWithMappedSuperclassForYamlDriver()
     {
         $yamlDriver = $this->_loadDriver();
-        $yamlDriver->getLocator()->addPaths([__DIR__ . DIRECTORY_SEPARATOR . 'yaml']);
+        $yamlDriver->getLocator()->addPaths([__DIR__.DIRECTORY_SEPARATOR.'yaml']);
 
         $em = $this->_getTestEntityManager();
         $em->getConfiguration()->setMetadataDriverImpl($yamlDriver);

lib/Doctrine/ORM/PersistentCollection.php

Spacing +3 added lines, -3 removed lines

@@ -400,7 +400,7 @@ 
      */
     public function containsKey($key)
     {
-        if (! $this->initialized && $this->association['fetch'] === ClassMetadata::FETCH_EXTRA_LAZY
+        if ( ! $this->initialized && $this->association['fetch'] === ClassMetadata::FETCH_EXTRA_LAZY
             && isset($this->association['indexBy'])) {
             $persister = $this->em->getUnitOfWork()->getCollectionPersister($this->association);
 
@@ -433,7 +433,7 @@ 
             && $this->association['fetch'] === ClassMetadata::FETCH_EXTRA_LAZY
             && isset($this->association['indexBy'])
         ) {
-            if (!$this->typeClass->isIdentifierComposite && $this->typeClass->isIdentifier($this->association['indexBy'])) {
+            if ( ! $this->typeClass->isIdentifierComposite && $this->typeClass->isIdentifier($this->association['indexBy'])) {
                 return $this->em->find($this->typeClass->name, $key);
             }
 
@@ -448,7 +448,7 @@ 
      */
     public function count()
     {
-        if (! $this->initialized && $this->association !== null && $this->association['fetch'] === ClassMetadata::FETCH_EXTRA_LAZY) {
+        if ( ! $this->initialized && $this->association !== null && $this->association['fetch'] === ClassMetadata::FETCH_EXTRA_LAZY) {
             $persister = $this->em->getUnitOfWork()->getCollectionPersister($this->association);
 
             return $persister->count($this) + ($this->isDirty ? $this->collection->count() : 0);

lib/Doctrine/ORM/UnitOfWork.php

Spacing +29 added lines, -29 removed lines

@@ -381,7 +381,7 @@ 
         try {
             // Collection deletions (deletions of complete collections)
             foreach ($this->collectionDeletions as $collectionToDelete) {
-                if (! $collectionToDelete instanceof PersistentCollection) {
+                if ( ! $collectionToDelete instanceof PersistentCollection) {
                     $this->getCollectionPersister($collectionToDelete->getMapping())->delete($collectionToDelete);
 
                     continue;
@@ -513,7 +513,7 @@ 
         $state = $this->getEntityState($entity);
 
         if ($state !== self::STATE_MANAGED && $state !== self::STATE_REMOVED) {
-            throw new \InvalidArgumentException("Entity has to be managed or scheduled for removal for single computation " . self::objToStr($entity));
+            throw new \InvalidArgumentException("Entity has to be managed or scheduled for removal for single computation ".self::objToStr($entity));
         }
 
         $class = $this->em->getClassMetadata(get_class($entity));
@@ -569,7 +569,7 @@ 
         $oid  = spl_object_hash($entity);
         $data = [];
 
-        if (!isset($this->entityChangeSets[$oid])) {
+        if ( ! isset($this->entityChangeSets[$oid])) {
             return $data;
         }
 
@@ -732,7 +732,7 @@ 
                     if ($owner === null) { // cloned
                         $actualValue->setOwner($entity, $assoc);
                     } else if ($owner !== $entity) { // no clone, we have to fix
-                        if (!$actualValue->isInitialized()) {
+                        if ( ! $actualValue->isInitialized()) {
                             $actualValue->initialize(); // we have to do this otherwise the cols share state
                         }
                         $newValue = clone $actualValue;
@@ -878,7 +878,7 @@ 
         $targetClass    = $this->em->getClassMetadata($assoc['targetEntity']);
 
         foreach ($unwrappedValue as $key => $entry) {
-            if (! ($entry instanceof $targetClass->name)) {
+            if ( ! ($entry instanceof $targetClass->name)) {
                 throw ORMInvalidArgumentException::invalidAssociation($targetClass, $assoc, $entry);
             }
 
@@ -956,7 +956,7 @@ 
 
             // Some identifiers may be foreign keys to new entities.
             // In this case, we don't have the value yet and should treat it as if we have a post-insert generator
-            if (! $this->hasMissingIdsWhichAreForeignKeys($class, $idValue)) {
+            if ( ! $this->hasMissingIdsWhichAreForeignKeys($class, $idValue)) {
                 $this->entityIdentifiers[$oid] = $idValue;
             }
         }
@@ -1105,7 +1105,7 @@ 
             }
         } else {
             foreach ($insertionsForClass as $oid => $entity) {
-                if (! isset($this->entityIdentifiers[$oid])) {
+                if ( ! isset($this->entityIdentifiers[$oid])) {
                     //entity was not added to identity map because some identifiers are foreign keys to new entities.
                     //add it now
                     $this->addToEntityIdentifiersAndEntityMap($class, $oid, $entity);
@@ -1270,7 +1270,7 @@ 
 
                 $joinColumns = reset($assoc['joinColumns']);
 
-                $calc->addDependency($targetClass->name, $class->name, (int)empty($joinColumns['nullable']));
+                $calc->addDependency($targetClass->name, $class->name, (int) empty($joinColumns['nullable']));
 
                 // If the target class has mapped subclasses, these share the same dependency.
                 if ( ! $targetClass->subClasses) {
@@ -1793,7 +1793,7 @@ 
                 throw ORMInvalidArgumentException::detachedEntityCannot($entity, "persisted");
 
             default:
-                throw new UnexpectedValueException("Unexpected entity state: $entityState." . self::objToStr($entity));
+                throw new UnexpectedValueException("Unexpected entity state: $entityState.".self::objToStr($entity));
         }
 
         $this->cascadePersist($entity, $visited);
@@ -1863,7 +1863,7 @@ 
             case self::STATE_DETACHED:
                 throw ORMInvalidArgumentException::detachedEntityCannot($entity, "removed");
             default:
-                throw new UnexpectedValueException("Unexpected entity state: $entityState." . self::objToStr($entity));
+                throw new UnexpectedValueException("Unexpected entity state: $entityState.".self::objToStr($entity));
         }
 
     }
@@ -2002,7 +2002,7 @@ 
      */
     private function ensureVersionMatch(ClassMetadata $class, $entity, $managedCopy)
     {
-        if (! ($class->isVersioned && $this->isLoaded($managedCopy) && $this->isLoaded($entity))) {
+        if ( ! ($class->isVersioned && $this->isLoaded($managedCopy) && $this->isLoaded($entity))) {
             return;
         }
 
@@ -2027,7 +2027,7 @@ 
      */
     private function isLoaded($entity)
     {
-        return !($entity instanceof Proxy) || $entity->__isInitialized();
+        return ! ($entity instanceof Proxy) || $entity->__isInitialized();
     }
 
     /**
@@ -2185,7 +2185,7 @@ 
 
         $associationMappings = array_filter(
             $class->associationMappings,
-            function ($assoc) { return $assoc['isCascadeRefresh']; }
+            function($assoc) { return $assoc['isCascadeRefresh']; }
         );
 
         foreach ($associationMappings as $assoc) {
@@ -2228,7 +2228,7 @@ 
 
         $associationMappings = array_filter(
             $class->associationMappings,
-            function ($assoc) { return $assoc['isCascadeDetach']; }
+            function($assoc) { return $assoc['isCascadeDetach']; }
         );
 
         foreach ($associationMappings as $assoc) {
@@ -2272,7 +2272,7 @@ 
 
         $associationMappings = array_filter(
             $class->associationMappings,
-            function ($assoc) { return $assoc['isCascadeMerge']; }
+            function($assoc) { return $assoc['isCascadeMerge']; }
         );
 
         foreach ($associationMappings as $assoc) {
@@ -2311,7 +2311,7 @@ 
 
         $associationMappings = array_filter(
             $class->associationMappings,
-            function ($assoc) { return $assoc['isCascadePersist']; }
+            function($assoc) { return $assoc['isCascadePersist']; }
         );
 
         foreach ($associationMappings as $assoc) {
@@ -2340,7 +2340,7 @@ 
                     break;
 
                 case ($relatedEntities !== null):
-                    if (! $relatedEntities instanceof $assoc['targetEntity']) {
+                    if ( ! $relatedEntities instanceof $assoc['targetEntity']) {
                         throw ORMInvalidArgumentException::invalidAssociation(
                             $this->em->getClassMetadata($assoc['targetEntity']),
                             $assoc,
@@ -2371,13 +2371,13 @@ 
 
         $associationMappings = array_filter(
             $class->associationMappings,
-            function ($assoc) { return $assoc['isCascadeRemove']; }
+            function($assoc) { return $assoc['isCascadeRemove']; }
         );
 
         $entitiesToCascade = [];
 
         foreach ($associationMappings as $assoc) {
-            if ($entity instanceof Proxy && !$entity->__isInitialized__) {
+            if ($entity instanceof Proxy && ! $entity->__isInitialized__) {
                 $entity->__load();
             }
 
@@ -2441,7 +2441,7 @@ 
                     return;
                 }
 
-                if ($entity instanceof Proxy && !$entity->__isInitialized__) {
+                if ($entity instanceof Proxy && ! $entity->__isInitialized__) {
                     $entity->__load();
                 }
 
@@ -2456,7 +2456,7 @@ 
             case LockMode::NONE === $lockMode:
             case LockMode::PESSIMISTIC_READ === $lockMode:
             case LockMode::PESSIMISTIC_WRITE === $lockMode:
-                if (!$this->em->getConnection()->isTransactionActive()) {
+                if ( ! $this->em->getConnection()->isTransactionActive()) {
                     throw TransactionRequiredException::transactionRequired();
                 }
 
@@ -2781,7 +2781,7 @@ 
                             // then we can append this entity for eager loading!
                             if ($hints['fetchMode'][$class->name][$field] == ClassMetadata::FETCH_EAGER &&
                                 isset($hints[self::HINT_DEFEREAGERLOAD]) &&
-                                !$targetClass->isIdentifierComposite &&
+                                ! $targetClass->isIdentifierComposite &&
                                 $newValue instanceof Proxy &&
                                 $newValue->__isInitialized__ === false) {
 
@@ -3146,7 +3146,7 @@ 
     public function getCollectionPersister(array $association)
     {
         $role = isset($association['cache'])
-            ? $association['sourceEntity'] . '::' . $association['fieldName']
+            ? $association['sourceEntity'].'::'.$association['fieldName']
             : $association['type'];
 
         if (isset($this->collectionPersisters[$role])) {
@@ -3364,7 +3364,7 @@ 
      */
     private function afterTransactionComplete()
     {
-        $this->performCallbackOnCachedPersister(function (CachedPersister $persister) {
+        $this->performCallbackOnCachedPersister(function(CachedPersister $persister) {
             $persister->afterTransactionComplete();
         });
     }
@@ -3374,7 +3374,7 @@ 
      */
     private function afterTransactionRolledBack()
     {
-        $this->performCallbackOnCachedPersister(function (CachedPersister $persister) {
+        $this->performCallbackOnCachedPersister(function(CachedPersister $persister) {
             $persister->afterTransactionRolledBack();
         });
     }
@@ -3470,11 +3470,11 @@ 
      */
     private function mergeEntityStateIntoManagedCopy($entity, $managedCopy)
     {
-        if (! $this->isLoaded($entity)) {
+        if ( ! $this->isLoaded($entity)) {
             return;
         }
 
-        if (! $this->isLoaded($managedCopy)) {
+        if ( ! $this->isLoaded($managedCopy)) {
             $managedCopy->__load();
         }
 
@@ -3497,7 +3497,7 @@ 
                     if ($other === null) {
                         $prop->setValue($managedCopy, null);
                     } else {
-                        if ($other instanceof Proxy && !$other->__isInitialized()) {
+                        if ($other instanceof Proxy && ! $other->__isInitialized()) {
                             // do not merge fields marked lazy that have not been fetched.
                             continue;
                         }
@@ -3584,7 +3584,7 @@ 
      */
     private function clearIdentityMapForEntityName($entityName)
     {
-        if (! isset($this->identityMap[$entityName])) {
+        if ( ! isset($this->identityMap[$entityName])) {
             return;
         }
 

lib/Doctrine/ORM/Internal/Hydration/AbstractHydrator.php

Spacing +4 added lines, -4 removed lines

@@ -317,7 +317,7 @@ 
                         : $value;
 
                     if ($cacheKeyInfo['isIdentifier'] && $value !== null) {
-                        $id[$dqlAlias] .= '|' . $value;
+                        $id[$dqlAlias] .= '|'.$value;
                         $nonemptyComponents[$dqlAlias] = true;
                     }
                     break;
@@ -352,11 +352,11 @@ 
 
             // WARNING: BC break! We know this is the desired behavior to type convert values, but this
             // erroneous behavior exists since 2.0 and we're forced to keep compatibility.
-            if (! isset($cacheKeyInfo['isScalar'])) {
+            if ( ! isset($cacheKeyInfo['isScalar'])) {
                 $type  = $cacheKeyInfo['type'];
                 $value = $type ? $type->convertToPHPValue($value, $this->_platform) : $value;
 
-                $fieldName = $cacheKeyInfo['dqlAlias'] . '_' . $fieldName;
+                $fieldName = $cacheKeyInfo['dqlAlias'].'_'.$fieldName;
             }
 
             $rowData[$fieldName] = $value;
@@ -465,7 +465,7 @@ 
     private function getDiscriminatorValues(ClassMetadata $classMetadata) : array
     {
         $values = array_map(
-            function (string $subClass) : string {
+            function(string $subClass) : string {
                 return (string) $this->getClassMetadata($subClass)->discriminatorValue;
             },
             $classMetadata->subClasses

tests/Doctrine/Tests/ORM/Functional/Ticket/GH7079Test.php

Spacing +2 added lines, -2 removed lines

@@ -69,11 +69,11 @@
     private function getTableFullName(array $table) : string
     {
         $join = '.';
-        if (! $this->platform->supportsSchemas() && $this->platform->canEmulateSchemas()) {
+        if ( ! $this->platform->supportsSchemas() && $this->platform->canEmulateSchemas()) {
             $join = '__';
         }
 
-        return $table['schema'] . $join . $table['name'];
+        return $table['schema'].$join.$table['name'];
     }
 
     private function createClassMetadata(string $className) : ClassMetadata

lib/Doctrine/ORM/Tools/Export/ClassMetadataExporter.php

Spacing +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
 
     public function __construct()
     {
-        @trigger_error(self::class . ' is deprecated and will be removed in Doctrine ORM 3.0', E_USER_DEPRECATED);
+        @trigger_error(self::class.' is deprecated and will be removed in Doctrine ORM 3.0', E_USER_DEPRECATED);
     }
 
     /**