doctrine / orm

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

Spacing +2 added lines, -2 removed lines

@@ -156,7 +156,7 @@ 
     {
         $this->addresses = $addresses;
 
-        $addresses->map(function ($address) {
+        $addresses->map(function($address) {
             $address->user = $this;
         });
     }
@@ -165,7 +165,7 @@ 
     {
         $this->pets = $pets;
 
-        $pets->map(function ($pet) {
+        $pets->map(function($pet) {
             $pet->owner = $this;
         });
     }

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

Spacing +2 added lines, -2 removed lines

@@ -83,7 +83,7 @@ 
                 );
             }
 
-            if (! $associationMetadata->isOwningSide()) {
+            if ( ! $associationMetadata->isOwningSide()) {
                 throw Mapping\MappingException::illegalInverseIdentifierAssociation(
                     $this->componentMetadata->getClassName(),
                     $this->fieldName
@@ -100,7 +100,7 @@ 
             // @todo guilhermeblanco The below block of code modifies component metadata properties, and it should be moved
             //                       to the component metadata builder instead of here.
 
-            if (! in_array($this->fieldName, $this->componentMetadata->identifier, true)) {
+            if ( ! in_array($this->fieldName, $this->componentMetadata->identifier, true)) {
                 $this->componentMetadata->identifier[] = $this->fieldName;
             }
 

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

Spacing +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
         $associationMetadata->setOwningSide(false);
         $associationMetadata->setMappedBy($this->oneToManyAnnotation->mappedBy);
 
-        if (! empty($this->oneToManyAnnotation->indexBy)) {
+        if ( ! empty($this->oneToManyAnnotation->indexBy)) {
             $associationMetadata->setIndexedBy($this->oneToManyAnnotation->indexBy);
         }
 

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

Spacing +4 added lines, -4 removed lines

@@ -86,16 +86,16 @@ 
         $associationMetadata->setFetchMode($this->getFetchMode($this->manyToManyAnnotation->fetch));
         $associationMetadata->setOwningSide(true);
 
-        if (! empty($this->manyToManyAnnotation->mappedBy)) {
+        if ( ! empty($this->manyToManyAnnotation->mappedBy)) {
             $associationMetadata->setMappedBy($this->manyToManyAnnotation->mappedBy);
             $associationMetadata->setOwningSide(false);
         }
 
-        if (! empty($this->manyToManyAnnotation->inversedBy)) {
+        if ( ! empty($this->manyToManyAnnotation->inversedBy)) {
             $associationMetadata->setInversedBy($this->manyToManyAnnotation->inversedBy);
         }
 
-        if (! empty($this->manyToManyAnnotation->indexBy)) {
+        if ( ! empty($this->manyToManyAnnotation->indexBy)) {
             $associationMetadata->setIndexedBy($this->manyToManyAnnotation->indexBy);
         }
 
@@ -113,7 +113,7 @@ 
         $this->buildCache($associationMetadata);
 
         // Check for owning side to consider join column
-        if (! $associationMetadata->isOwningSide()) {
+        if ( ! $associationMetadata->isOwningSide()) {
             return $associationMetadata;
         }
 

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

Spacing +3 added lines, -3 removed lines

@@ -90,12 +90,12 @@ 
     protected function getTargetEntity(string $targetEntity) : string
     {
         // Validate if target entity is defined
-        if (! $targetEntity) {
+        if ( ! $targetEntity) {
             throw Mapping\MappingException::missingTargetEntity($this->fieldName);
         }
 
         // Validate that target entity exists
-        if (! (class_exists($targetEntity) || interface_exists($targetEntity))) {
+        if ( ! (class_exists($targetEntity) || interface_exists($targetEntity))) {
             throw Mapping\MappingException::invalidTargetEntityClass(
                 $targetEntity,
                 $this->componentMetadata->getClassName(),
@@ -150,7 +150,7 @@ 
     {
         $fetchModeConstant = sprintf('%s::%s', Mapping\FetchMode::class, $fetchMode);
 
-        if (! defined($fetchModeConstant)) {
+        if ( ! defined($fetchModeConstant)) {
             throw Mapping\MappingException::invalidFetchMode($this->componentMetadata->getClassName(), $fetchMode);
         }
 

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

Spacing +1 added lines, -1 removed lines

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

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

Spacing +5 added lines, -5 removed lines

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

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

Spacing +8 added lines, -8 removed lines

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

lib/Doctrine/ORM/Mapping/ClassMetadata.php

Spacing +19 added lines, -19 removed lines

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