doctrine / orm

lib/Doctrine/ORM/Cache/DefaultEntityHydrator.php

Spacing +5 added lines, -5 removed lines

@@ -54,11 +54,11 @@ 
         }
 
         foreach ($metadata->getPropertiesIterator() as $name => $association) {
-            if (! isset($data[$name]) || $association instanceof FieldMetadata) {
+            if ( ! isset($data[$name]) || $association instanceof FieldMetadata) {
                 continue;
             }
 
-            if (! $association instanceof ToOneAssociationMetadata) {
+            if ( ! $association instanceof ToOneAssociationMetadata) {
                 unset($data[$name]);
 
                 continue;
@@ -68,7 +68,7 @@ 
             $targetClassMetadata = $this->em->getClassMetadata($targetEntity);
             $targetPersister     = $this->uow->getEntityPersister($targetEntity);
 
-            if (! $association->getCache()) {
+            if ( ! $association->getCache()) {
                 $owningAssociation = ! $association->isOwningSide()
                     ? $targetClassMetadata->getProperty($association->getMappedBy())
                     : $association;
@@ -116,7 +116,7 @@ 
                 continue;
             }
 
-            if (! $association->isPrimaryKey()) {
+            if ( ! $association->isPrimaryKey()) {
                 $targetClass = StaticClassNameConverter::getClass($data[$name]);
                 $targetId    = $this->uow->getEntityIdentifier($data[$name]);
                 $data[$name] = new AssociationCacheEntry($targetClass, $targetId);
@@ -165,7 +165,7 @@ 
                 ($association instanceof OneToOneAssociationMetadata && ! $association->isOwningSide())
             );
 
-            if (! $isEagerLoad) {
+            if ( ! $isEagerLoad) {
                 $data[$name] = $this->em->getReference($assocClass, $assocId);
 
                 continue;

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

Spacing +4 added lines, -4 removed lines

@@ -82,11 +82,11 @@ 
         $joinTableMetadata->setName($namingStrategy->joinTableName($souceEntity, $this->targetEntity, $this->fieldName));
 
         if ($this->joinTableAnnotation !== null) {
-            if (! empty($this->joinTableAnnotation->name)) {
+            if ( ! empty($this->joinTableAnnotation->name)) {
                 $joinTableMetadata->setName($this->joinTableAnnotation->name);
             }
 
-            if (! empty($this->joinTableAnnotation->schema)) {
+            if ( ! empty($this->joinTableAnnotation->schema)) {
                 $joinTableMetadata->setSchema($this->joinTableAnnotation->schema);
             }
 
@@ -122,7 +122,7 @@ 
 
         $selfReferenceEntity = $souceEntity === $this->targetEntity;
 
-        if (! $joinTableMetadata->getJoinColumns()) {
+        if ( ! $joinTableMetadata->getJoinColumns()) {
             $joinColumnAnnotation = new Annotation\JoinColumn();
 
             $joinColumnAnnotation->onDelete             = 'CASCADE';
@@ -137,7 +137,7 @@ 
             $joinTableMetadata->addJoinColumn($this->joinColumnMetadataBuilder->build());
         }
 
-        if (! $joinTableMetadata->getInverseJoinColumns()) {
+        if ( ! $joinTableMetadata->getInverseJoinColumns()) {
             $joinColumnAnnotation = new Annotation\JoinColumn();
 
             $joinColumnAnnotation->onDelete             = 'CASCADE';

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

Spacing +5 added lines, -5 removed lines

@@ -59,7 +59,7 @@ 
         $namingStrategy     = $this->metadataBuildingContext->getNamingStrategy();
         $joinColumnMetadata = new Mapping\JoinColumnMetadata();
 
-        if (! $this->componentMetadata->isMappedSuperclass) {
+        if ( ! $this->componentMetadata->isMappedSuperclass) {
             $joinColumnMetadata->setTableName($this->componentMetadata->getTableName());
         }
 
@@ -73,19 +73,19 @@ 
         $joinColumnMetadata->setNullable($this->joinColumnAnnotation->nullable);
         $joinColumnMetadata->setUnique($this->joinColumnAnnotation->unique);
 
-        if (! empty($this->joinColumnAnnotation->name)) {
+        if ( ! empty($this->joinColumnAnnotation->name)) {
             $joinColumnMetadata->setColumnName($this->joinColumnAnnotation->name);
         }
 
-        if (! empty($this->joinColumnAnnotation->referencedColumnName)) {
+        if ( ! empty($this->joinColumnAnnotation->referencedColumnName)) {
             $joinColumnMetadata->setReferencedColumnName($this->joinColumnAnnotation->referencedColumnName);
         }
 
-        if (! empty($this->joinColumnAnnotation->fieldName)) {
+        if ( ! empty($this->joinColumnAnnotation->fieldName)) {
             $joinColumnMetadata->setAliasedName($this->joinColumnAnnotation->fieldName);
         }
 
-        if (! empty($this->joinColumnAnnotation->columnDefinition)) {
+        if ( ! empty($this->joinColumnAnnotation->columnDefinition)) {
             $joinColumnMetadata->setColumnDefinition($this->joinColumnAnnotation->columnDefinition);
         }
 

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

Spacing +4 added lines, -4 removed lines

@@ -122,7 +122,7 @@ 
                     break;
 
                 default:
-                    if (! isset($fieldOptions['default'])) {
+                    if ( ! isset($fieldOptions['default'])) {
                         throw Mapping\MappingException::unsupportedOptimisticLockingType($fieldType);
                     }
 
@@ -145,15 +145,15 @@ 
         $fieldMetadata->setNullable($this->columnAnnotation->nullable);
         $fieldMetadata->setUnique($this->columnAnnotation->unique);
 
-        if (! $this->componentMetadata->isMappedSuperclass) {
+        if ( ! $this->componentMetadata->isMappedSuperclass) {
             $fieldMetadata->setTableName($this->componentMetadata->getTableName());
         }
 
-        if (! empty($this->columnAnnotation->columnDefinition)) {
+        if ( ! empty($this->columnAnnotation->columnDefinition)) {
             $fieldMetadata->setColumnDefinition($this->columnAnnotation->columnDefinition);
         }
 
-        if (! empty($this->columnAnnotation->length)) {
+        if ( ! empty($this->columnAnnotation->length)) {
             $fieldMetadata->setLength($this->columnAnnotation->length);
         }
 

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

Spacing +1 added lines, -1 removed lines

@@ -99,7 +99,7 @@
         assert($this->fieldName !== null);
         assert($this->fieldType !== null);
 
-        if (! $this->generatedValueAnnotation) {
+        if ( ! $this->generatedValueAnnotation) {
             return null;
         }
 

lib/Doctrine/ORM/Mapping/ClassMetadataFactory.php

Spacing +14 added lines, -14 removed lines

@@ -130,7 +130,7 @@ 
      */
     public function isTransient($className) : bool
     {
-        if (! $this->initialized) {
+        if ( ! $this->initialized) {
             $this->initialize();
         }
 
@@ -178,7 +178,7 @@ 
      */
     public function getAllMetadata() : array
     {
-        if (! $this->initialized) {
+        if ( ! $this->initialized) {
             $this->initialize();
         }
 
@@ -214,7 +214,7 @@ 
 
     public function getTargetPlatform() : Platforms\AbstractPlatform
     {
-        if (! $this->targetPlatform) {
+        if ( ! $this->targetPlatform) {
             $this->targetPlatform = $this->em->getConnection()->getDatabasePlatform();
         }
 
@@ -256,7 +256,7 @@ 
 
         try {
             if ($this->cacheDriver) {
-                $cached = $this->cacheDriver->fetch($entityClassName . $this->cacheSalt);
+                $cached = $this->cacheDriver->fetch($entityClassName.$this->cacheSalt);
 
                 if ($cached instanceof ClassMetadata) {
                     $this->loadedMetadata[$entityClassName] = $cached;
@@ -266,7 +266,7 @@ 
                     foreach ($this->loadMetadata($entityClassName, $metadataBuildingContext) as $loadedClass) {
                         $loadedClassName = $loadedClass->getClassName();
 
-                        $this->cacheDriver->save($loadedClassName . $this->cacheSalt, $loadedClass, null);
+                        $this->cacheDriver->save($loadedClassName.$this->cacheSalt, $loadedClass, null);
                     }
                 }
             } else {
@@ -275,7 +275,7 @@ 
         } catch (PersistenceMappingException $loadingException) {
             $fallbackMetadataResponse = $this->onNotFoundMetadata($entityClassName, $metadataBuildingContext);
 
-            if (! $fallbackMetadataResponse) {
+            if ( ! $fallbackMetadataResponse) {
                 throw $loadingException;
             }
 
@@ -311,7 +311,7 @@ 
      */
     protected function loadMetadata(string $name, ClassMetadataBuildingContext $metadataBuildingContext) : array
     {
-        if (! $this->initialized) {
+        if ( ! $this->initialized) {
             $this->initialize();
         }
 
@@ -357,7 +357,7 @@ 
         $parentClasses = [];
 
         foreach (array_reverse($this->getReflectionService()->getParentClasses($name)) as $parentClass) {
-            if (! $this->getDriver()->isTransient($parentClass)) {
+            if ( ! $this->getDriver()->isTransient($parentClass)) {
                 $parentClasses[] = $parentClass;
             }
         }
@@ -372,7 +372,7 @@ 
         string $className,
         ClassMetadataBuildingContext $metadataBuildingContext
     ) : ?ClassMetadata {
-        if (! $this->evm->hasListeners(Events::onClassMetadataNotFound)) {
+        if ( ! $this->evm->hasListeners(Events::onClassMetadataNotFound)) {
             return null;
         }
 
@@ -416,7 +416,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;
         }
@@ -426,13 +426,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());
                 }
             }

lib/Doctrine/ORM/Mapping/MappingException.php

Spacing +38 added lines, -38 removed lines

@@ -23,7 +23,7 @@ 
      */
     public static function pathRequired()
     {
-        return new self('Specifying the paths to your entities is required ' .
+        return new self('Specifying the paths to your entities is required '.
             'in the AnnotationDriver to retrieve all class names.');
     }
 
@@ -213,7 +213,7 @@ 
      */
     public static function missingQueryMapping($entity, $queryName)
     {
-        return new self('Query named "' . $queryName . '" in "' . $entity . ' requires a result class or result set mapping.');
+        return new self('Query named "'.$queryName.'" in "'.$entity.' requires a result class or result set mapping.');
     }
 
     /**
@@ -224,7 +224,7 @@ 
      */
     public static function missingResultSetMappingEntity($entity, $resultName)
     {
-        return new self('Result set mapping named "' . $resultName . '" in "' . $entity . ' requires a entity class name.');
+        return new self('Result set mapping named "'.$resultName.'" in "'.$entity.' requires a entity class name.');
     }
 
     /**
@@ -235,7 +235,7 @@ 
      */
     public static function missingResultSetMappingFieldName($entity, $resultName)
     {
-        return new self('Result set mapping named "' . $resultName . '" in "' . $entity . ' requires a field name.');
+        return new self('Result set mapping named "'.$resultName.'" in "'.$entity.' requires a field name.');
     }
 
     /**
@@ -282,8 +282,8 @@ 
     {
         $message = sprintf("The mapping of field '%s' is invalid: The option '%s' is required.", $field, $expectedOption);
 
-        if (! empty($hint)) {
-            $message .= ' (Hint: ' . $hint . ')';
+        if ( ! empty($hint)) {
+            $message .= ' (Hint: '.$hint.')';
         }
 
         return new self($message);
@@ -312,7 +312,7 @@ 
      */
     public static function reflectionFailure($entity, ReflectionException $previousException)
     {
-        return new self('An error occurred in ' . $entity, 0, $previousException);
+        return new self('An error occurred in '.$entity, 0, $previousException);
     }
 
     /**
@@ -323,8 +323,8 @@ 
      */
     public static function joinColumnMustPointToMappedField($className, $joinColumn)
     {
-        return new self('The column ' . $joinColumn . ' must be mapped to a field in class '
-            . $className . ' since it is referenced by a join column of another class.');
+        return new self('The column '.$joinColumn.' must be mapped to a field in class '
+            . $className.' since it is referenced by a join column of another class.');
     }
 
     /**
@@ -358,7 +358,7 @@ 
      */
     public static function propertyTypeIsRequired($className, $propertyName)
     {
-        return new self("The attribute 'type' is required for the column description of property " . $className . '::$' . $propertyName . '.');
+        return new self("The attribute 'type' is required for the column description of property ".$className.'::$'.$propertyName.'.');
     }
 
     /**
@@ -384,7 +384,7 @@ 
      */
     public static function duplicateQueryMapping($entity, $queryName)
     {
-        return new self('Query named "' . $queryName . '" in "' . $entity . '" was already declared, but it must be declared only once');
+        return new self('Query named "'.$queryName.'" in "'.$entity.'" was already declared, but it must be declared only once');
     }
 
     /**
@@ -395,7 +395,7 @@ 
      */
     public static function duplicateResultSetMapping($entity, $resultName)
     {
-        return new self('Result set mapping named "' . $resultName . '" in "' . $entity . '" was already declared, but it must be declared only once');
+        return new self('Result set mapping named "'.$resultName.'" in "'.$entity.'" was already declared, but it must be declared only once');
     }
 
     /**
@@ -405,7 +405,7 @@ 
      */
     public static function singleIdNotAllowedOnCompositePrimaryKey($entity)
     {
-        return new self('Single id is not allowed on composite primary key in entity ' . $entity);
+        return new self('Single id is not allowed on composite primary key in entity '.$entity);
     }
 
     /**
@@ -415,7 +415,7 @@ 
      */
     public static function noIdDefined($entity)
     {
-        return new self('No ID defined for entity ' . $entity);
+        return new self('No ID defined for entity '.$entity);
     }
 
     /**
@@ -423,7 +423,7 @@ 
      */
     public static function unsupportedOptimisticLockingType(Type $unsupportedType)
     {
-        return new self('Locking type "' . $unsupportedType->getName() . '" is not supported by Doctrine.');
+        return new self('Locking type "'.$unsupportedType->getName().'" is not supported by Doctrine.');
     }
 
     /**
@@ -433,13 +433,13 @@ 
      */
     public static function fileMappingDriversRequireConfiguredDirectoryPath($path = null)
     {
-        if (! empty($path)) {
-            $path = '[' . $path . ']';
+        if ( ! empty($path)) {
+            $path = '['.$path.']';
         }
 
         return new self(
-            'File mapping drivers must have a valid directory path, ' .
-            'however the given path ' . $path . ' seems to be incorrect!'
+            'File mapping drivers must have a valid directory path, '.
+            'however the given path '.$path.' seems to be incorrect!'
         );
     }
 
@@ -455,7 +455,7 @@ 
     public static function invalidClassInDiscriminatorMap($className, $owningClass)
     {
         return new self(sprintf(
-            "Entity class '%s' used in the discriminator map of class '%s' " .
+            "Entity class '%s' used in the discriminator map of class '%s' ".
             'does not exist.',
             $className,
             $owningClass
@@ -521,7 +521,7 @@ 
     public static function sqlConversionNotAllowedForPrimaryKeyProperties($className, $fieldName, $type)
     {
         return new self(sprintf(
-            'It is not possible to set id field "%s" to type "%s" in entity class "%s". ' .
+            'It is not possible to set id field "%s" to type "%s" in entity class "%s". '.
             'The type "%s" requires conversion SQL which is not allowed for identifiers.',
             $fieldName,
             $type,
@@ -557,7 +557,7 @@ 
      */
     public static function duplicateColumnName($className, $columnName)
     {
-        return new self("Duplicate definition of column '" . $columnName . "' on entity '" . $className . "' in a field or discriminator column mapping.");
+        return new self("Duplicate definition of column '".$columnName."' on entity '".$className."' in a field or discriminator column mapping.");
     }
 
     /**
@@ -568,7 +568,7 @@ 
      */
     public static function illegalToManyAssociationOnMappedSuperclass($className, $field)
     {
-        return new self("It is illegal to put an inverse side one-to-many or many-to-many association on mapped superclass '" . $className . '#' . $field . "'.");
+        return new self("It is illegal to put an inverse side one-to-many or many-to-many association on mapped superclass '".$className.'#'.$field."'.");
     }
 
     /**
@@ -580,8 +580,8 @@ 
      */
     public static function cannotMapCompositePrimaryKeyEntitiesAsForeignId($className, $targetEntity, $targetField)
     {
-        return new self("It is not possible to map entity '" . $className . "' with a composite primary key " .
-            "as part of the primary key of another entity '" . $targetEntity . '#' . $targetField . "'.");
+        return new self("It is not possible to map entity '".$className."' with a composite primary key ".
+            "as part of the primary key of another entity '".$targetEntity.'#'.$targetField."'.");
     }
 
     /**
@@ -635,8 +635,8 @@ 
      */
     public static function illegalOrphanRemoval($className, $field)
     {
-        return new self('Orphan removal is only allowed on one-to-one and one-to-many ' .
-            'associations, but ' . $className . '#' . $field . ' is not.');
+        return new self('Orphan removal is only allowed on one-to-one and one-to-many '.
+            'associations, but '.$className.'#'.$field.' is not.');
     }
 
     /**
@@ -668,7 +668,7 @@ 
      */
     public static function noInheritanceOnMappedSuperClass($className)
     {
-        return new self("It is not supported to define inheritance information on a mapped superclass '" . $className . "'.");
+        return new self("It is not supported to define inheritance information on a mapped superclass '".$className."'.");
     }
 
     /**
@@ -680,8 +680,8 @@ 
     public static function mappedClassNotPartOfDiscriminatorMap($className, $rootClassName)
     {
         return new self(
-            "Entity '" . $className . "' has to be part of the discriminator map of '" . $rootClassName . "' " .
-            "to be properly mapped in the inheritance hierarchy. Alternatively you can make '" . $className . "' an abstract class " .
+            "Entity '".$className."' has to be part of the discriminator map of '".$rootClassName."' ".
+            "to be properly mapped in the inheritance hierarchy. Alternatively you can make '".$className."' an abstract class ".
             'to avoid this exception from occurring.'
         );
     }
@@ -694,7 +694,7 @@ 
      */
     public static function lifecycleCallbackMethodNotFound($className, $methodName)
     {
-        return new self("Entity '" . $className . "' has no public method '" . $methodName . "' to be registered as lifecycle callback.");
+        return new self("Entity '".$className."' has no public method '".$methodName."' to be registered as lifecycle callback.");
     }
 
     /**
@@ -728,7 +728,7 @@ 
      */
     public static function invalidFetchMode($className, $annotation)
     {
-        return new self("Entity '" . $className . "' has a mapping with invalid fetch mode '" . $annotation . "'");
+        return new self("Entity '".$className."' has a mapping with invalid fetch mode '".$annotation."'");
     }
 
     /**
@@ -738,7 +738,7 @@ 
      */
     public static function compositeKeyAssignedIdGeneratorRequired($className)
     {
-        return new self("Entity '" . $className . "' has a composite identifier but uses an ID generator other than manually assigning (Identity, Sequence). This is not supported.");
+        return new self("Entity '".$className."' has a composite identifier but uses an ID generator other than manually assigning (Identity, Sequence). This is not supported.");
     }
 
     /**
@@ -750,7 +750,7 @@ 
      */
     public static function invalidTargetEntityClass($targetEntity, $sourceEntity, $associationName)
     {
-        return new self("The target-entity '" . $targetEntity . "' cannot be found in '" . $sourceEntity . '#' . $associationName . "'.");
+        return new self("The target-entity '".$targetEntity."' cannot be found in '".$sourceEntity.'#'.$associationName."'.");
     }
 
     /**
@@ -762,8 +762,8 @@ 
      */
     public static function invalidCascadeOption(array $cascades, $className, $propertyName)
     {
-        $cascades = implode(', ', array_map(static function ($e) {
-            return "'" . $e . "'";
+        $cascades = implode(', ', array_map(static function($e) {
+            return "'".$e."'";
         }, $cascades));
 
         return new self(sprintf(
@@ -796,7 +796,7 @@ 
     {
         return new self(
             sprintf(
-                'Infinite nesting detected for embedded property %s::%s. ' .
+                'Infinite nesting detected for embedded property %s::%s. '.
                 'You cannot embed an embeddable from the same type inside an embeddable.',
                 $className,
                 $propertyName
@@ -811,7 +811,7 @@ 
     {
         return new self(
             sprintf(
-                'Class %s not found in namespaces %s.' .
+                'Class %s not found in namespaces %s.'.
                 $className,
                 implode(', ', $namespaces)
             )

lib/Doctrine/ORM/Sequencing/Generator/TableGenerator.php

Spacing +1 added lines, -1 removed lines

@@ -58,7 +58,7 @@
                         $this->allocationSize
                     );
 
-                    if ($conn->executeUpdate($updateSql, [1 => $currentLevel, 2 => $currentLevel+1]) !== 1) {
+                    if ($conn->executeUpdate($updateSql, [1 => $currentLevel, 2 => $currentLevel + 1]) !== 1) {
                         // no affected rows, concurrency issue, throw exception
                     }
                 } else {

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;
         });
     }