doctrine / orm

lib/Doctrine/ORM/Cache/DefaultCache.php

Spacing +15 added lines, -16 removed lines

@@ -55,7 +55,7 @@ 
         $metadata  = $this->em->getClassMetadata($className);
         $persister = $this->uow->getEntityPersister($metadata->getRootClassName());
 
-        if (! ($persister instanceof CachedPersister)) {
+        if ( ! ($persister instanceof CachedPersister)) {
             return null;
         }
 
@@ -70,7 +70,7 @@ 
         $metadata  = $this->em->getClassMetadata($className);
         $persister = $this->uow->getCollectionPersister($metadata->getProperty($association));
 
-        if (! ($persister instanceof CachedPersister)) {
+        if ( ! ($persister instanceof CachedPersister)) {
             return null;
         }
 
@@ -85,7 +85,7 @@ 
         $metadata  = $this->em->getClassMetadata($className);
         $persister = $this->uow->getEntityPersister($metadata->getRootClassName());
 
-        if (! ($persister instanceof CachedPersister)) {
+        if ( ! ($persister instanceof CachedPersister)) {
             return false;
         }
 
@@ -100,7 +100,7 @@ 
         $metadata  = $this->em->getClassMetadata($className);
         $persister = $this->uow->getEntityPersister($metadata->getRootClassName());
 
-        if (! ($persister instanceof CachedPersister)) {
+        if ( ! ($persister instanceof CachedPersister)) {
             return;
         }
 
@@ -115,7 +115,7 @@ 
         $metadata  = $this->em->getClassMetadata($className);
         $persister = $this->uow->getEntityPersister($metadata->getRootClassName());
 
-        if (! ($persister instanceof CachedPersister)) {
+        if ( ! ($persister instanceof CachedPersister)) {
             return;
         }
 
@@ -132,7 +132,7 @@ 
         foreach ($metadatas as $metadata) {
             $persister = $this->uow->getEntityPersister($metadata->getRootClassName());
 
-            if (! ($persister instanceof CachedPersister)) {
+            if ( ! ($persister instanceof CachedPersister)) {
                 continue;
             }
 
@@ -148,7 +148,7 @@ 
         $metadata  = $this->em->getClassMetadata($className);
         $persister = $this->uow->getCollectionPersister($metadata->getProperty($association));
 
-        if (! ($persister instanceof CachedPersister)) {
+        if ( ! ($persister instanceof CachedPersister)) {
             return false;
         }
 
@@ -163,7 +163,7 @@ 
         $metadata  = $this->em->getClassMetadata($className);
         $persister = $this->uow->getCollectionPersister($metadata->getProperty($association));
 
-        if (! ($persister instanceof CachedPersister)) {
+        if ( ! ($persister instanceof CachedPersister)) {
             return;
         }
 
@@ -178,7 +178,7 @@ 
         $metadata  = $this->em->getClassMetadata($className);
         $persister = $this->uow->getCollectionPersister($metadata->getProperty($association));
 
-        if (! ($persister instanceof CachedPersister)) {
+        if ( ! ($persister instanceof CachedPersister)) {
             return;
         }
 
@@ -194,13 +194,13 @@ 
 
         foreach ($metadatas as $metadata) {
             foreach ($metadata->getPropertiesIterator() as $association) {
-                if (! $association instanceof ToManyAssociationMetadata) {
+                if ( ! $association instanceof ToManyAssociationMetadata) {
                     continue;
                 }
 
                 $persister = $this->uow->getCollectionPersister($association);
 
-                if (! ($persister instanceof CachedPersister)) {
+                if ( ! ($persister instanceof CachedPersister)) {
                     continue;
                 }
 
@@ -251,11 +251,10 @@ 
     public function getQueryCache($regionName = null)
     {
         if ($regionName === null) {
-            return $this->defaultQueryCache ?:
-                $this->defaultQueryCache = $this->cacheFactory->buildQueryCache($this->em);
+            return $this->defaultQueryCache ?: $this->defaultQueryCache = $this->cacheFactory->buildQueryCache($this->em);
         }
 
-        if (! isset($this->queryCaches[$regionName])) {
+        if ( ! isset($this->queryCaches[$regionName])) {
             $this->queryCaches[$regionName] = $this->cacheFactory->buildQueryCache($this->em, $regionName);
         }
 
@@ -270,7 +269,7 @@ 
      */
     private function buildEntityCacheKey(ClassMetadata $metadata, $identifier)
     {
-        if (! is_array($identifier)) {
+        if ( ! is_array($identifier)) {
             $identifier = $this->toIdentifierArray($metadata, $identifier);
         }
 
@@ -286,7 +285,7 @@ 
      */
     private function buildCollectionCacheKey(ClassMetadata $metadata, $association, $ownerIdentifier)
     {
-        if (! is_array($ownerIdentifier)) {
+        if ( ! is_array($ownerIdentifier)) {
             $ownerIdentifier = $this->toIdentifierArray($metadata, $ownerIdentifier);
         }
 

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/Tools/SchemaTool.php

Spacing +21 added lines, -21 removed lines

@@ -180,11 +180,11 @@ 
                     $pkColumns = [];
 
                     foreach ($class->getPropertiesIterator() as $fieldName => $property) {
-                        if (! ($property instanceof FieldMetadata)) {
+                        if ( ! ($property instanceof FieldMetadata)) {
                             continue;
                         }
 
-                        if (! $class->isInheritedProperty($fieldName)) {
+                        if ( ! $class->isInheritedProperty($fieldName)) {
                             $columnName = $this->platform->quoteIdentifier($property->getColumnName());
 
                             $this->gatherColumn($class, $property, $table);
@@ -219,7 +219,7 @@ 
                             }
                         }
 
-                        if (! empty($inheritedKeyColumns)) {
+                        if ( ! empty($inheritedKeyColumns)) {
                             // Add a FK constraint on the ID column
                             $rootClass = $this->em->getClassMetadata($class->getRootClassName());
 
@@ -264,7 +264,7 @@ 
                 }
             }
 
-            if (! $table->hasIndex('primary')) {
+            if ( ! $table->hasIndex('primary')) {
                 $table->setPrimaryKey($pkColumns);
             }
 
@@ -324,7 +324,7 @@ 
             $processedClasses[$class->getClassName()] = true;
 
             foreach ($class->getPropertiesIterator() as $property) {
-                if (! $property instanceof FieldMetadata
+                if ( ! $property instanceof FieldMetadata
                     || ! $property->hasValueGenerator()
                     || $property->getValueGenerator()->getType() !== GeneratorType::SEQUENCE
                     || $class->getClassName() !== $class->getRootClassName()) {
@@ -334,7 +334,7 @@ 
                 $generator  = $property->getValueGenerator()->getGenerator();
                 $quotedName = $generator->getSequenceName();
 
-                if (! $schema->hasSequence($quotedName)) {
+                if ( ! $schema->hasSequence($quotedName)) {
                     $schema->createSequence($quotedName, $generator->getAllocationSize());
                 }
             }
@@ -347,7 +347,7 @@ 
             }
         }
 
-        if (! $this->platform->supportsSchemas() && ! $this->platform->canEmulateSchemas()) {
+        if ( ! $this->platform->supportsSchemas() && ! $this->platform->canEmulateSchemas()) {
             $schema->visit(new RemoveNamespacedAssets());
         }
 
@@ -386,7 +386,7 @@ 
                 break;
         }
 
-        if (! empty($discrColumn->getColumnDefinition())) {
+        if ( ! empty($discrColumn->getColumnDefinition())) {
             $options['columnDefinition'] = $discrColumn->getColumnDefinition();
         }
 
@@ -404,7 +404,7 @@ 
         $pkColumns = [];
 
         foreach ($class->getPropertiesIterator() as $fieldName => $property) {
-            if (! ($property instanceof FieldMetadata)) {
+            if ( ! ($property instanceof FieldMetadata)) {
                 continue;
             }
 
@@ -508,7 +508,7 @@ 
     private function gatherRelationsSql($class, $table, $schema, &$addedFks, &$blacklistedFks)
     {
         foreach ($class->getPropertiesIterator() as $fieldName => $property) {
-            if (! ($property instanceof AssociationMetadata)) {
+            if ( ! ($property instanceof AssociationMetadata)) {
                 continue;
             }
 
@@ -516,7 +516,7 @@ 
                 continue;
             }
 
-            if (! $property->isOwningSide()) {
+            if ( ! $property->isOwningSide()) {
                 continue;
             }
 
@@ -606,7 +606,7 @@ 
         $idColumns        = $class->getIdentifierColumns($this->em);
         $idColumnNameList = array_keys($idColumns);
 
-        if (! in_array($referencedColumnName, $idColumnNameList, true)) {
+        if ( ! in_array($referencedColumnName, $idColumnNameList, true)) {
             return null;
         }
 
@@ -614,7 +614,7 @@ 
         foreach ($class->getIdentifierFieldNames() as $fieldName) {
             $property = $class->getProperty($fieldName);
 
-            if (! ($property instanceof AssociationMetadata)) {
+            if ( ! ($property instanceof AssociationMetadata)) {
                 continue;
             }
 
@@ -670,7 +670,7 @@ 
                 $joinColumn->getReferencedColumnName()
             );
 
-            if (! $definingClass) {
+            if ( ! $definingClass) {
                 throw MissingColumnException::fromColumnSourceAndTarget(
                     $joinColumn->getReferencedColumnName(),
                     $mapping->getSourceEntity(),
@@ -685,14 +685,14 @@ 
             $localColumns[]      = $quotedColumnName;
             $foreignColumns[]    = $quotedReferencedColumnName;
 
-            if (! $theJoinTable->hasColumn($quotedColumnName)) {
+            if ( ! $theJoinTable->hasColumn($quotedColumnName)) {
                 // Only add the column to the table if it does not exist already.
                 // It might exist already if the foreign key is mapped into a regular
                 // property as well.
                 $property  = $definingClass->getProperty($referencedFieldName);
                 $columnDef = null;
 
-                if (! empty($joinColumn->getColumnDefinition())) {
+                if ( ! empty($joinColumn->getColumnDefinition())) {
                     $columnDef = $joinColumn->getColumnDefinition();
                 } elseif ($property->getColumnDefinition()) {
                     $columnDef = $property->getColumnDefinition();
@@ -724,7 +724,7 @@ 
                 $uniqueConstraints[] = ['columns' => [$quotedColumnName]];
             }
 
-            if (! empty($joinColumn->getOnDelete())) {
+            if ( ! empty($joinColumn->getOnDelete())) {
                 $fkOptions['onDelete'] = $joinColumn->getOnDelete();
             }
         }
@@ -735,7 +735,7 @@ 
             $theJoinTable->addUniqueIndex($unique['columns'], is_numeric($indexName) ? null : $indexName);
         }
 
-        $compositeName = $theJoinTable->getName() . '.' . implode('', $localColumns);
+        $compositeName = $theJoinTable->getName().'.'.implode('', $localColumns);
 
         if (isset($addedFks[$compositeName])
             && ($foreignTableName !== $addedFks[$compositeName]['foreignTableName']
@@ -752,7 +752,7 @@ 
             }
 
             $blacklistedFks[$compositeName] = true;
-        } elseif (! isset($blacklistedFks[$compositeName])) {
+        } elseif ( ! isset($blacklistedFks[$compositeName])) {
             $addedFks[$compositeName] = [
                 'foreignTableName' => $foreignTableName,
                 'foreignColumns'   => $foreignColumns,
@@ -851,7 +851,7 @@ 
         $fullSchema = $sm->createSchema();
 
         foreach ($fullSchema->getTables() as $table) {
-            if (! $schema->hasTable($table->getName())) {
+            if ( ! $schema->hasTable($table->getName())) {
                 foreach ($table->getForeignKeys() as $foreignKey) {
                     /** @var $foreignKey ForeignKeyConstraint */
                     if ($schema->hasTable($foreignKey->getForeignTableName())) {
@@ -876,7 +876,7 @@ 
                 if ($table->hasPrimaryKey()) {
                     $columns = $table->getPrimaryKey()->getColumns();
                     if (count($columns) === 1) {
-                        $checkSequence = $table->getName() . '_' . $columns[0] . '_seq';
+                        $checkSequence = $table->getName().'_'.$columns[0].'_seq';
                         if ($fullSchema->hasSequence($checkSequence)) {
                             $visitor->acceptSequence($fullSchema->getSequence($checkSequence));
                         }