doctrine / orm

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

Spacing +8 added lines, -8 removed lines

@@ -39,7 +39,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
 {
@@ -182,7 +182,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, $this->metadataBuildingContext);
         $cm->setTable(new Mapping\TableMetadata('doctrine_global_article'));
@@ -278,7 +278,7 @@ 
      */
     public function testSetDiscriminatorMapInGlobalNamespace() : void
     {
-        require_once __DIR__ . '/../../Models/Global/GlobalNamespaceModel.php';
+        require_once __DIR__.'/../../Models/Global/GlobalNamespaceModel.php';
 
         $cm = new ClassMetadata('DoctrineGlobalUser', null, $this->metadataBuildingContext);
         $cm->setTable(new Mapping\TableMetadata('doctrine_global_user'));
@@ -294,7 +294,7 @@ 
      */
     public function testSetSubClassesInGlobalNamespace() : void
     {
-        require_once __DIR__ . '/../../Models/Global/GlobalNamespaceModel.php';
+        require_once __DIR__.'/../../Models/Global/GlobalNamespaceModel.php';
 
         $cm = new ClassMetadata('DoctrineGlobalUser', null, $this->metadataBuildingContext);
         $cm->setTable(new Mapping\TableMetadata('doctrine_global_user'));
@@ -775,7 +775,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, $this->metadataBuildingContext);
         $cm->setTable(new Mapping\TableMetadata('cms_users'));
@@ -825,7 +825,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());
     }
@@ -845,7 +845,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/Tools/AttachEntityListenersListener.php

Spacing +1 added lines, -1 removed lines

@@ -45,7 +45,7 @@
         /** @var Mapping\ClassMetadata $metadata */
         $metadata = $event->getClassMetadata();
 
-        if (! isset($this->entityListeners[$metadata->getClassName()])) {
+        if ( ! isset($this->entityListeners[$metadata->getClassName()])) {
             return;
         }
 

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

Spacing +27 added lines, -27 removed lines

@@ -143,7 +143,7 @@ 
 
         $classNames = array_filter(
             $this->locator->getAllClassNames(null),
-            function ($className) {
+            function($className) {
                 return ! $this->isTransient($className);
             }
         );
@@ -275,7 +275,7 @@ 
             ];
 
             foreach ($entityListenersAnnot->value as $listenerClassName) {
-                if (! class_exists($listenerClassName)) {
+                if ( ! class_exists($listenerClassName)) {
                     throw Mapping\MappingException::entityListenerClassNotFound(
                         $listenerClassName,
                         $classMetadata->getClassName()
@@ -341,15 +341,15 @@ 
 
                     $discriminatorColumn->setColumnName($discriminatorColumnAnnot->name);
 
-                    if (! empty($discriminatorColumnAnnot->columnDefinition)) {
+                    if ( ! empty($discriminatorColumnAnnot->columnDefinition)) {
                         $discriminatorColumn->setColumnDefinition($discriminatorColumnAnnot->columnDefinition);
                     }
 
-                    if (! empty($discriminatorColumnAnnot->type)) {
+                    if ( ! empty($discriminatorColumnAnnot->type)) {
                         $discriminatorColumn->setType(Type::getType($discriminatorColumnAnnot->type));
                     }
 
-                    if (! empty($discriminatorColumnAnnot->length)) {
+                    if ( ! empty($discriminatorColumnAnnot->length)) {
                         $discriminatorColumn->setLength($discriminatorColumnAnnot->length);
                     }
                 }
@@ -405,11 +405,11 @@ 
     {
         $table = new Mapping\TableMetadata();
 
-        if (! empty($tableAnnot->name)) {
+        if ( ! empty($tableAnnot->name)) {
             $table->setName($tableAnnot->name);
         }
 
-        if (! empty($tableAnnot->schema)) {
+        if ( ! empty($tableAnnot->schema)) {
             $table->setSchema($tableAnnot->schema);
         }
 
@@ -453,7 +453,7 @@ 
     ) {
         $usage         = constant(sprintf('%s::%s', Mapping\CacheUsage::class, $cacheAnnot->usage));
         $baseRegion    = strtolower(str_replace('\\', '_', $metadata->getRootClassName()));
-        $defaultRegion = $baseRegion . ($fieldName ? '__' . $fieldName : '');
+        $defaultRegion = $baseRegion.($fieldName ? '__'.$fieldName : '');
 
         return new Mapping\CacheMetadata($usage, $cacheAnnot->region ?: $defaultRegion);
     }
@@ -562,11 +562,11 @@ 
             $fieldMetadata->setPrimaryKey(true);
         }
 
-        if (! empty($columnAnnot->columnDefinition)) {
+        if ( ! empty($columnAnnot->columnDefinition)) {
             $fieldMetadata->setColumnDefinition($columnAnnot->columnDefinition);
         }
 
-        if (! empty($columnAnnot->length)) {
+        if ( ! empty($columnAnnot->length)) {
             $fieldMetadata->setLength($columnAnnot->length);
         }
 
@@ -587,7 +587,7 @@ 
                     break;
 
                 default:
-                    if (! isset($customOptions['default'])) {
+                    if ( ! isset($customOptions['default'])) {
                         throw Mapping\MappingException::unsupportedOptimisticLockingType($fieldMetadata->getType());
                     }
             }
@@ -625,11 +625,11 @@ 
         $assocMetadata->setOrphanRemoval($oneToOneAnnot->orphanRemoval);
         $assocMetadata->setFetchMode($this->getFetchMode($className, $oneToOneAnnot->fetch));
 
-        if (! empty($oneToOneAnnot->mappedBy)) {
+        if ( ! empty($oneToOneAnnot->mappedBy)) {
             $assocMetadata->setMappedBy($oneToOneAnnot->mappedBy);
         }
 
-        if (! empty($oneToOneAnnot->inversedBy)) {
+        if ( ! empty($oneToOneAnnot->inversedBy)) {
             $assocMetadata->setInversedBy($oneToOneAnnot->inversedBy);
         }
 
@@ -705,7 +705,7 @@ 
         $assocMetadata->setCascade($this->getCascade($className, $fieldName, $manyToOneAnnot->cascade));
         $assocMetadata->setFetchMode($this->getFetchMode($className, $manyToOneAnnot->fetch));
 
-        if (! empty($manyToOneAnnot->inversedBy)) {
+        if ( ! empty($manyToOneAnnot->inversedBy)) {
             $assocMetadata->setInversedBy($manyToOneAnnot->inversedBy);
         }
 
@@ -782,11 +782,11 @@ 
         $assocMetadata->setOrphanRemoval($oneToManyAnnot->orphanRemoval);
         $assocMetadata->setFetchMode($this->getFetchMode($className, $oneToManyAnnot->fetch));
 
-        if (! empty($oneToManyAnnot->mappedBy)) {
+        if ( ! empty($oneToManyAnnot->mappedBy)) {
             $assocMetadata->setMappedBy($oneToManyAnnot->mappedBy);
         }
 
-        if (! empty($oneToManyAnnot->indexBy)) {
+        if ( ! empty($oneToManyAnnot->indexBy)) {
             $assocMetadata->setIndexedBy($oneToManyAnnot->indexBy);
         }
 
@@ -840,15 +840,15 @@ 
         $assocMetadata->setOrphanRemoval($manyToManyAnnot->orphanRemoval);
         $assocMetadata->setFetchMode($this->getFetchMode($className, $manyToManyAnnot->fetch));
 
-        if (! empty($manyToManyAnnot->mappedBy)) {
+        if ( ! empty($manyToManyAnnot->mappedBy)) {
             $assocMetadata->setMappedBy($manyToManyAnnot->mappedBy);
         }
 
-        if (! empty($manyToManyAnnot->inversedBy)) {
+        if ( ! empty($manyToManyAnnot->inversedBy)) {
             $assocMetadata->setInversedBy($manyToManyAnnot->inversedBy);
         }
 
-        if (! empty($manyToManyAnnot->indexBy)) {
+        if ( ! empty($manyToManyAnnot->indexBy)) {
             $assocMetadata->setIndexedBy($manyToManyAnnot->indexBy);
         }
 
@@ -899,11 +899,11 @@ 
     ) {
         $joinTable = new Mapping\JoinTableMetadata();
 
-        if (! empty($joinTableAnnot->name)) {
+        if ( ! empty($joinTableAnnot->name)) {
             $joinTable->setName($joinTableAnnot->name);
         }
 
-        if (! empty($joinTableAnnot->schema)) {
+        if ( ! empty($joinTableAnnot->schema)) {
             $joinTable->setSchema($joinTableAnnot->schema);
         }
 
@@ -954,11 +954,11 @@ 
         $joinColumn->setNullable($joinColumnAnnot->nullable);
         $joinColumn->setUnique($joinColumnAnnot->unique);
 
-        if (! empty($joinColumnAnnot->fieldName)) {
+        if ( ! empty($joinColumnAnnot->fieldName)) {
             $joinColumn->setAliasedName($joinColumnAnnot->fieldName);
         }
 
-        if (! empty($joinColumnAnnot->columnDefinition)) {
+        if ( ! empty($joinColumnAnnot->columnDefinition)) {
             $joinColumn->setColumnDefinition($joinColumnAnnot->columnDefinition);
         }
 
@@ -1014,7 +1014,7 @@ 
     {
         $fetchModeConstant = sprintf('%s::%s', Mapping\FetchMode::class, $fetchMode);
 
-        if (! defined($fetchModeConstant)) {
+        if ( ! defined($fetchModeConstant)) {
             throw Mapping\MappingException::invalidFetchMode($className, $fetchMode);
         }
 
@@ -1056,7 +1056,7 @@ 
         $classAnnotations = $this->reader->getClassAnnotations($reflectionClass);
 
         foreach ($classAnnotations as $key => $annot) {
-            if (! is_numeric($key)) {
+            if ( ! is_numeric($key)) {
                 continue;
             }
 
@@ -1074,7 +1074,7 @@ 
         $propertyAnnotations = $this->reader->getPropertyAnnotations($reflectionProperty);
 
         foreach ($propertyAnnotations as $key => $annot) {
-            if (! is_numeric($key)) {
+            if ( ! is_numeric($key)) {
                 continue;
             }
 
@@ -1092,7 +1092,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/TableMetadataBuilder.php

Spacing +2 added lines, -2 removed lines

@@ -54,11 +54,11 @@
             return $tableMetadata;
         }
 
-        if (! empty($this->tableAnnotation->name)) {
+        if ( ! empty($this->tableAnnotation->name)) {
             $tableMetadata->setName($this->tableAnnotation->name);
         }
 
-        if (! empty($this->tableAnnotation->schema)) {
+        if ( ! empty($this->tableAnnotation->schema)) {
             $tableMetadata->setSchema($this->tableAnnotation->schema);
         }
 

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

Spacing +1 added lines, -1 removed lines

@@ -61,7 +61,7 @@
 
         $componentName = $this->componentMetadata->getRootClassName();
         $baseRegion    = strtolower(str_replace('\\', '_', $componentName));
-        $defaultRegion = $baseRegion . ($this->fieldName ? '__' . $this->fieldName : '');
+        $defaultRegion = $baseRegion.($this->fieldName ? '__'.$this->fieldName : '');
 
         $usage  = constant(sprintf('%s::%s', Mapping\CacheUsage::class, strtoupper($this->cacheAnnotation->usage)));
         $region = $this->cacheAnnotation->region ?: $defaultRegion;

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

Spacing +2 added lines, -2 removed lines

@@ -65,11 +65,11 @@
         $discriminatorColumn->setType(Type::getType($annotation->type ?? 'string'));
         $discriminatorColumn->setColumnName($annotation->name);
 
-        if (! empty($annotation->columnDefinition)) {
+        if ( ! empty($annotation->columnDefinition)) {
             $discriminatorColumn->setColumnDefinition($annotation->columnDefinition);
         }
 
-        if (! empty($annotation->length)) {
+        if ( ! empty($annotation->length)) {
             $discriminatorColumn->setLength($annotation->length);
         }
 

lib/Doctrine/ORM/Proxy/Factory/StaticProxyFactory.php

Spacing +8 added lines, -8 removed lines

@@ -56,7 +56,7 @@ 
      */
     public function generateProxyClasses(array $classes) : int
     {
-        $concreteClasses = array_filter($classes, static function (ClassMetadata $metadata) : bool {
+        $concreteClasses = array_filter($classes, static function(ClassMetadata $metadata) : bool {
             return ! ($metadata->isMappedSuperclass || $metadata->getReflectionClass()->isAbstract());
         });
 
@@ -65,7 +65,7 @@ 
                 ->proxyFactory
                 ->createProxy(
                     $metadata->getClassName(),
-                    static function () {
+                    static function() {
                         // empty closure, serves its purpose, for now
                     },
                     $this->skippedFieldsFqns($metadata)
@@ -89,7 +89,7 @@ 
                 ->getUnitOfWork()
                 ->getEntityPersister($metadata->getClassName());
 
-        $proxyInstance                                            = $this
+        $proxyInstance = $this
             ->proxyFactory
             ->createProxy(
                 $metadata->getClassName(),
@@ -108,7 +108,7 @@ 
 
     private function makeInitializer(ClassMetadata $metadata, EntityPersister $persister) : Closure
     {
-        return static function (
+        return static function(
             GhostObjectInterface $ghostObject,
             string $method,
             // we don't care
@@ -127,7 +127,7 @@ 
             $identifier = $persister->getIdentifier($ghostObject);
 
             // @TODO how do we use `$properties` in the persister? That would be a massive optimisation
-            if (! $persister->loadById($identifier, $ghostObject)) {
+            if ( ! $persister->loadById($identifier, $ghostObject)) {
                 $initializer = $originalInitializer;
 
                 throw EntityNotFoundException::fromClassNameAndIdentifier(
@@ -159,7 +159,7 @@ 
         $transientFieldsFqns = [];
 
         foreach ($metadata->getPropertiesIterator() as $name => $property) {
-            if (! $property instanceof TransientMetadata) {
+            if ( ! $property instanceof TransientMetadata) {
                 continue;
             }
 
@@ -197,11 +197,11 @@ 
     private function propertyFqcn(ReflectionProperty $property) : string
     {
         if ($property->isPrivate()) {
-            return "\0" . $property->getDeclaringClass()->getName() . "\0" . $property->getName();
+            return "\0".$property->getDeclaringClass()->getName()."\0".$property->getName();
         }
 
         if ($property->isProtected()) {
-            return "\0*\0" . $property->getName();
+            return "\0*\0".$property->getName();
         }
 
         return $property->getName();

lib/Doctrine/ORM/Utility/NormalizeIdentifier.php

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
         $normalizedAssociatedId = [];
 
         foreach ($targetClass->getPropertiesIterator() as $name => $property) {
-            if (! array_key_exists($name, $flatIdentifier)) {
+            if ( ! array_key_exists($name, $flatIdentifier)) {
                 continue;
             }
 

lib/Doctrine/ORM/Utility/PersisterHelper.php

Spacing +2 added lines, -2 removed lines

@@ -58,14 +58,14 @@
 
         // iterate over association mappings
         foreach ($class->getPropertiesIterator() as $association) {
-            if (! ($association instanceof AssociationMetadata)) {
+            if ( ! ($association instanceof AssociationMetadata)) {
                 continue;
             }
 
             // resolve join columns over to-one or to-many
             $targetClass = $em->getClassMetadata($association->getTargetEntity());
 
-            if (! $association->isOwningSide()) {
+            if ( ! $association->isOwningSide()) {
                 $association = $targetClass->getProperty($association->getMappedBy());
                 $targetClass = $em->getClassMetadata($association->getTargetEntity());
             }