doctrine / orm

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
@@ -126,7 +126,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(
@@ -158,7 +158,7 @@ 
         $transientFieldsFqns = [];
 
         foreach ($metadata->getDeclaredPropertiesIterator() as $name => $property) {
-            if (! $property instanceof TransientMetadata) {
+            if ( ! $property instanceof TransientMetadata) {
                 continue;
             }
 
@@ -196,11 +196,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/ORMInvalidArgumentException.php

Spacing +17 added lines, -17 removed lines

@@ -30,7 +30,7 @@ 
      */
     public static function scheduleInsertForManagedEntity($entity)
     {
-        return new self('A managed+dirty entity ' . self::objToStr($entity) . ' can not be scheduled for insertion.');
+        return new self('A managed+dirty entity '.self::objToStr($entity).' can not be scheduled for insertion.');
     }
 
     /**
@@ -40,7 +40,7 @@ 
      */
     public static function scheduleInsertForRemovedEntity($entity)
     {
-        return new self('Removed entity ' . self::objToStr($entity) . ' can not be scheduled for insertion.');
+        return new self('Removed entity '.self::objToStr($entity).' can not be scheduled for insertion.');
     }
 
     /**
@@ -50,7 +50,7 @@ 
      */
     public static function scheduleInsertTwice($entity)
     {
-        return new self('Entity ' . self::objToStr($entity) . ' can not be scheduled for insertion twice.');
+        return new self('Entity '.self::objToStr($entity).' can not be scheduled for insertion twice.');
     }
 
     /**
@@ -62,7 +62,7 @@ 
     public static function entityWithoutIdentity($className, $entity)
     {
         return new self(
-            "The given entity of type '" . $className . "' (" . self::objToStr($entity) . ') has no identity/no ' .
+            "The given entity of type '".$className."' (".self::objToStr($entity).') has no identity/no '.
             'id values set. It cannot be added to the identity map.'
         );
     }
@@ -74,7 +74,7 @@ 
      */
     public static function readOnlyRequiresManagedEntity($entity)
     {
-        return new self('Only managed entities can be marked or checked as read only. But ' . self::objToStr($entity) . ' is not');
+        return new self('Only managed entities can be marked or checked as read only. But '.self::objToStr($entity).' is not');
     }
 
     /**
@@ -85,7 +85,7 @@ 
     public static function newEntitiesFoundThroughRelationships($newEntitiesWithAssociations)
     {
         $errorMessages = array_map(
-            static function (array $newEntityWithAssociation) : string {
+            static function(array $newEntityWithAssociation) : string {
                 [$associationMetadata, $entity] = $newEntityWithAssociation;
 
                 return self::newEntityFoundThroughRelationshipMessage($associationMetadata, $entity);
@@ -154,7 +154,7 @@ 
      */
     public static function entityNotManaged($entity)
     {
-        return new self('Entity ' . self::objToStr($entity) . ' is not managed. An entity is managed if its fetched ' .
+        return new self('Entity '.self::objToStr($entity).' is not managed. An entity is managed if its fetched '.
             'from the database or registered as new through EntityManager#persist');
     }
 
@@ -166,7 +166,7 @@ 
      */
     public static function entityHasNoIdentity($entity, $operation)
     {
-        return new self('Entity has no identity, therefore ' . $operation . ' cannot be performed. ' . self::objToStr($entity));
+        return new self('Entity has no identity, therefore '.$operation.' cannot be performed. '.self::objToStr($entity));
     }
 
     /**
@@ -177,7 +177,7 @@ 
      */
     public static function entityIsRemoved($entity, $operation)
     {
-        return new self('Entity is removed, therefore ' . $operation . ' cannot be performed. ' . self::objToStr($entity));
+        return new self('Entity is removed, therefore '.$operation.' cannot be performed. '.self::objToStr($entity));
     }
 
     /**
@@ -188,7 +188,7 @@ 
      */
     public static function detachedEntityCannot($entity, $operation)
     {
-        return new self('Detached entity ' . self::objToStr($entity) . ' cannot be ' . $operation);
+        return new self('Detached entity '.self::objToStr($entity).' cannot be '.$operation);
     }
 
     /**
@@ -200,8 +200,8 @@ 
      */
     public static function invalidObject($context, $given, $parameterIndex = 1)
     {
-        return new self($context . ' expects parameter ' . $parameterIndex .
-            ' to be an entity object, ' . gettype($given) . ' given.');
+        return new self($context.' expects parameter '.$parameterIndex.
+            ' to be an entity object, '.gettype($given).' given.');
     }
 
     /**
@@ -209,7 +209,7 @@ 
      */
     public static function invalidCompositeIdentifier()
     {
-        return new self('Binding an entity with a composite primary key to a query is not supported. ' .
+        return new self('Binding an entity with a composite primary key to a query is not supported. '.
             'You should split the parameter into the explicit fields and bind them separately.');
     }
 
@@ -246,7 +246,7 @@ 
      */
     private static function objToStr($obj) : string
     {
-        return method_exists($obj, '__toString') ? (string) $obj : get_class($obj) . '@' . spl_object_id($obj);
+        return method_exists($obj, '__toString') ? (string) $obj : get_class($obj).'@'.spl_object_id($obj);
     }
 
     /**
@@ -255,15 +255,15 @@ 
     private static function newEntityFoundThroughRelationshipMessage(AssociationMetadata $association, $entity) : string
     {
         return 'A new entity was found through the relationship \''
-            . $association->getSourceEntity() . '#' . $association->getName() . '\' that was not'
-            . ' configured to cascade persist operations for entity: ' . self::objToStr($entity) . '.'
+            . $association->getSourceEntity().'#'.$association->getName().'\' that was not'
+            . ' configured to cascade persist operations for entity: '.self::objToStr($entity).'.'
             . ' To solve this issue: Either explicitly call EntityManager#persist()'
             . ' on this unknown entity or configure cascade persist'
             . ' this association in the mapping for example @ManyToOne(..,cascade={"persist"}).'
             . (method_exists($entity, '__toString')
                 ? ''
                 : ' If you cannot find out which entity causes the problem implement \''
-                . $association->getTargetEntity() . '#__toString()\' to get a clue.'
+                . $association->getTargetEntity().'#__toString()\' to get a clue.'
             );
     }
 }

lib/Doctrine/ORM/Mapping/Factory/Autoloader.php

Spacing +3 added lines, -3 removed lines

@@ -46,7 +46,7 @@ 
         // remove namespace separators from remaining class name
         $fileName = str_replace('\\', '', $classNameRelativeToMetadataNamespace);
 
-        return $metadataDir . DIRECTORY_SEPARATOR . $fileName . '.php';
+        return $metadataDir.DIRECTORY_SEPARATOR.$fileName.'.php';
     }
 
     /**
@@ -63,7 +63,7 @@ 
     ) : Closure {
         $metadataNamespace = ltrim($metadataNamespace, '\\');
 
-        if (! ($notFoundCallback === null || is_callable($notFoundCallback))) {
+        if ( ! ($notFoundCallback === null || is_callable($notFoundCallback))) {
             $type = is_object($notFoundCallback) ? get_class($notFoundCallback) : gettype($notFoundCallback);
 
             throw new InvalidArgumentException(
@@ -71,7 +71,7 @@ 
             );
         }
 
-        $autoloader = static function ($className) use ($metadataDir, $metadataNamespace, $notFoundCallback) {
+        $autoloader = static function($className) use ($metadataDir, $metadataNamespace, $notFoundCallback) {
             if (strpos($className, $metadataNamespace) === 0) {
                 $file = Autoloader::resolveFile($metadataDir, $metadataNamespace, $className);
 

lib/Doctrine/ORM/Mapping/Factory/Strategy/FileWriterClassMetadataGeneratorStrategy.php

Spacing +3 added lines, -3 removed lines

@@ -36,7 +36,7 @@ 
 
         $this->ensureDirectoryIsReady(dirname($filePath));
 
-        $tmpFileName = $filePath . '.' . uniqid('', true);
+        $tmpFileName = $filePath.'.'.uniqid('', true);
 
         file_put_contents($tmpFileName, $sourceCode);
         @chmod($tmpFileName, 0664);
@@ -50,11 +50,11 @@ 
      */
     private function ensureDirectoryIsReady(string $directory)
     {
-        if (! is_dir($directory) && (@mkdir($directory, 0775, true) === false)) {
+        if ( ! is_dir($directory) && (@mkdir($directory, 0775, true) === false)) {
             throw new RuntimeException(sprintf('Your metadata directory "%s" must be writable', $directory));
         }
 
-        if (! is_writable($directory)) {
+        if ( ! is_writable($directory)) {
             throw new RuntimeException(sprintf('Your proxy directory "%s" must be writable', $directory));
         }
     }

lib/Doctrine/ORM/Mapping/DefaultEntityListenerResolver.php

Spacing +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
      */
     public function register($object)
     {
-        if (! is_object($object)) {
+        if ( ! is_object($object)) {
             throw new InvalidArgumentException(sprintf('An object was expected, but got "%s".', gettype($object)));
         }
 

lib/Doctrine/ORM/Mapping/Exporter/VariableExporter.php

Spacing +5 added lines, -5 removed lines

@@ -24,30 +24,30 @@
      */
     public function export($value, int $indentationLevel = 0) : string
     {
-        if (! is_array($value)) {
+        if ( ! is_array($value)) {
             return var_export($value, true);
         }
 
         $indentation  = str_repeat(self::INDENTATION, $indentationLevel);
-        $longestKey   = array_reduce(array_keys($value), static function ($k, $v) {
+        $longestKey   = array_reduce(array_keys($value), static function($k, $v) {
             return (string) (strlen((string) $k) > strlen((string) $v) ? $k : $v);
         });
         $maxKeyLength = strlen($longestKey) + (is_numeric($longestKey) ? 0 : 2);
 
         $lines = [];
 
-        $lines[] = $indentation . '[';
+        $lines[] = $indentation.'[';
 
         foreach ($value as $entryKey => $entryValue) {
             $lines[] = sprintf(
                 '%s%s => %s,',
-                $indentation . self::INDENTATION,
+                $indentation.self::INDENTATION,
                 str_pad(var_export($entryKey, true), $maxKeyLength),
                 ltrim($this->export($entryValue, $indentationLevel + 1))
             );
         }
 
-        $lines[] = $indentation . ']';
+        $lines[] = $indentation.']';
 
         return implode(PHP_EOL, $lines);
     }

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

Spacing +7 added lines, -7 removed lines

@@ -416,7 +416,7 @@ 
 
                 if (isset($manyToOneElement['fetch'])) {
                     $association->setFetchMode(
-                        constant('Doctrine\ORM\Mapping\FetchMode::' . (string) $manyToOneElement['fetch'])
+                        constant('Doctrine\ORM\Mapping\FetchMode::'.(string) $manyToOneElement['fetch'])
                     );
                 }
 
@@ -569,7 +569,7 @@ 
                 $fieldName = (string) $overrideElement['name'];
                 $property  = $metadata->getProperty($fieldName);
 
-                if (! $property) {
+                if ( ! $property) {
                     throw Mapping\MappingException::invalidOverrideFieldName($metadata->getClassName(), $fieldName);
                 }
 
@@ -627,7 +627,7 @@ 
                 // Check for fetch
                 if (isset($overrideElement['fetch'])) {
                     $override->setFetchMode(
-                        constant('Doctrine\ORM\Mapping\FetchMode::' . (string) $overrideElement['fetch'])
+                        constant('Doctrine\ORM\Mapping\FetchMode::'.(string) $overrideElement['fetch'])
                     );
                 }
 
@@ -638,7 +638,7 @@ 
         // Evaluate <lifecycle-callbacks...>
         if (isset($xmlRoot->{'lifecycle-callbacks'})) {
             foreach ($xmlRoot->{'lifecycle-callbacks'}->{'lifecycle-callback'} as $lifecycleCallback) {
-                $eventName  = constant(Events::class . '::' . (string) $lifecycleCallback['type']);
+                $eventName  = constant(Events::class.'::'.(string) $lifecycleCallback['type']);
                 $methodName = (string) $lifecycleCallback['method'];
 
                 $metadata->addLifecycleCallback($methodName, $eventName);
@@ -650,7 +650,7 @@ 
             foreach ($xmlRoot->{'entity-listeners'}->{'entity-listener'} as $listenerElement) {
                 $listenerClassName = (string) $listenerElement['class'];
 
-                if (! class_exists($listenerClassName)) {
+                if ( ! class_exists($listenerClassName)) {
                     throw Mapping\MappingException::entityListenerClassNotFound(
                         $listenerClassName,
                         $metadata->getClassName()
@@ -816,7 +816,7 @@ 
         $fieldName = null
     ) {
         $baseRegion    = strtolower(str_replace('\\', '_', $metadata->getRootClassName()));
-        $defaultRegion = $baseRegion . ($fieldName ? '__' . $fieldName : '');
+        $defaultRegion = $baseRegion.($fieldName ? '__'.$fieldName : '');
 
         $region = (string) ($cacheMapping['region'] ?? $defaultRegion);
         $usage  = isset($cacheMapping['usage'])
@@ -844,7 +844,7 @@ 
             Events::preFlush,
         ];
 
-        return array_filter($events, static function ($eventName) use ($method) {
+        return array_filter($events, static function($eventName) use ($method) {
             return $eventName === $method->getName();
         });
     }

lib/Doctrine/ORM/Mapping/ClassMetadata.php

Spacing +40 added lines, -40 removed lines

@@ -262,7 +262,7 @@ 
      */
     public function __toString()
     {
-        return self::class . '@' . spl_object_id($this);
+        return self::class.'@'.spl_object_id($this);
     }
 
     /**
@@ -349,7 +349,7 @@ 
         // Restore ReflectionClass and properties
         $this->reflectionClass = $reflectionService->getClass($this->className);
 
-        if (! $this->reflectionClass) {
+        if ( ! $this->reflectionClass) {
             return;
         }
 
@@ -373,11 +373,11 @@ 
         }
 
         // Verify & complete identifier mapping
-        if (! $this->identifier) {
+        if ( ! $this->identifier) {
             throw MappingException::identifierRequired($this->className);
         }
 
-        $explicitlyGeneratedProperties = array_filter($this->declaredProperties, static function (Property $property) : bool {
+        $explicitlyGeneratedProperties = array_filter($this->declaredProperties, static function(Property $property) : bool {
             return $property instanceof FieldMetadata
                 && $property->isPrimaryKey()
                 && $property->hasValueGenerator();
@@ -396,14 +396,14 @@ 
     public function validateAssociations() : void
     {
         array_map(
-            function (Property $property) {
-                if (! ($property instanceof AssociationMetadata)) {
+            function(Property $property) {
+                if ( ! ($property instanceof AssociationMetadata)) {
                     return;
                 }
 
                 $targetEntity = $property->getTargetEntity();
 
-                if (! class_exists($targetEntity)) {
+                if ( ! class_exists($targetEntity)) {
                     throw MappingException::invalidTargetEntityClass($targetEntity, $this->className, $property->getName());
                 }
             },
@@ -421,7 +421,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);
                 }
             }
@@ -445,11 +445,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];
         }
 
@@ -477,7 +477,7 @@ 
             $property->setColumnName($columnName);
         }
 
-        if (! $this->isMappedSuperclass) {
+        if ( ! $this->isMappedSuperclass) {
             $property->setTableName($this->getTableName());
         }
 
@@ -497,7 +497,7 @@ 
                 throw MappingException::sqlConversionNotAllowedForPrimaryKeyProperties($this->className, $property);
             }
 
-            if (! in_array($fieldName, $this->identifier, true)) {
+            if ( ! in_array($fieldName, $this->identifier, true)) {
                 $this->identifier[] = $fieldName;
             }
         }
@@ -547,7 +547,7 @@ 
         $fieldName    = $property->getName();
         $targetEntity = $property->getTargetEntity();
 
-        if (! $targetEntity) {
+        if ( ! $targetEntity) {
             throw MappingException::missingTargetEntity($fieldName);
         }
 
@@ -560,7 +560,7 @@ 
                 throw MappingException::illegalOrphanRemovalOnIdentifierAssociation($this->className, $fieldName);
             }
 
-            if (! in_array($property->getName(), $this->identifier, true)) {
+            if ( ! in_array($property->getName(), $this->identifier, true)) {
                 if ($property instanceof ToOneAssociationMetadata && count($property->getJoinColumns()) >= 2) {
                     throw MappingException::cannotMapCompositePrimaryKeyEntitiesAsForeignId(
                         $property->getTargetEntity(),
@@ -622,7 +622,7 @@ 
                 /** @var JoinColumnMetadata $joinColumn */
                 if ($property instanceof OneToOneAssociationMetadata && $this->inheritanceType !== InheritanceType::SINGLE_TABLE) {
                     if (count($property->getJoinColumns()) === 1) {
-                        if (! $property->isPrimaryKey()) {
+                        if ( ! $property->isPrimaryKey()) {
                             $joinColumn->setUnique(true);
                         }
                     } else {
@@ -630,13 +630,13 @@ 
                     }
                 }
 
-                $joinColumn->setTableName(! $this->isMappedSuperclass ? $this->getTableName() : null);
+                $joinColumn->setTableName( ! $this->isMappedSuperclass ? $this->getTableName() : null);
 
-                if (! $joinColumn->getColumnName()) {
+                if ( ! $joinColumn->getColumnName()) {
                     $joinColumn->setColumnName($this->namingStrategy->joinColumnName($fieldName, $this->className));
                 }
 
-                if (! $joinColumn->getReferencedColumnName()) {
+                if ( ! $joinColumn->getReferencedColumnName()) {
                     $joinColumn->setReferencedColumnName($this->namingStrategy->referenceColumnName());
                 }
 
@@ -644,7 +644,7 @@ 
             }
 
             if ($uniqueConstraintColumns) {
-                if (! $this->table) {
+                if ( ! $this->table) {
                     throw new RuntimeException(
                         'ClassMetadata::setTable() has to be called before defining a one to one relationship.'
                     );
@@ -664,7 +664,7 @@ 
         if ($property->isOrphanRemoval()) {
             $cascades = $property->getCascade();
 
-            if (! in_array('remove', $cascades, true)) {
+            if ( ! in_array('remove', $cascades, true)) {
                 $cascades[] = 'remove';
 
                 $property->setCascade($cascades);
@@ -727,14 +727,14 @@ 
     protected function validateAndCompleteOneToManyMapping(OneToManyAssociationMetadata $property)
     {
         // OneToMany MUST have mappedBy
-        if (! $property->getMappedBy()) {
+        if ( ! $property->getMappedBy()) {
             throw MappingException::oneToManyRequiresMappedBy($property->getName());
         }
 
         if ($property->isOrphanRemoval()) {
             $cascades = $property->getCascade();
 
-            if (! in_array('remove', $cascades, true)) {
+            if ( ! in_array('remove', $cascades, true)) {
                 $cascades[] = 'remove';
 
                 $property->setCascade($cascades);
@@ -757,7 +757,7 @@ 
 
             $property->setJoinTable($joinTable);
 
-            if (! $joinTable->getName()) {
+            if ( ! $joinTable->getName()) {
                 $joinTableName = $this->namingStrategy->joinTableName(
                     $property->getSourceEntity(),
                     $property->getTargetEntity(),
@@ -769,7 +769,7 @@ 
 
             $selfReferencingEntityWithoutJoinColumns = $property->getSourceEntity() === $property->getTargetEntity() && ! $joinTable->hasColumns();
 
-            if (! $joinTable->getJoinColumns()) {
+            if ( ! $joinTable->getJoinColumns()) {
                 $referencedColumnName = $this->namingStrategy->referenceColumnName();
                 $sourceReferenceName  = $selfReferencingEntityWithoutJoinColumns ? 'source' : $referencedColumnName;
                 $columnName           = $this->namingStrategy->joinKeyColumnName($property->getSourceEntity(), $sourceReferenceName);
@@ -782,7 +782,7 @@ 
                 $joinTable->addJoinColumn($joinColumn);
             }
 
-            if (! $joinTable->getInverseJoinColumns()) {
+            if ( ! $joinTable->getInverseJoinColumns()) {
                 $referencedColumnName = $this->namingStrategy->referenceColumnName();
                 $targetReferenceName  = $selfReferencingEntityWithoutJoinColumns ? 'target' : $referencedColumnName;
                 $columnName           = $this->namingStrategy->joinKeyColumnName($property->getTargetEntity(), $targetReferenceName);
@@ -797,13 +797,13 @@ 
 
             foreach ($joinTable->getJoinColumns() as $joinColumn) {
                 /** @var JoinColumnMetadata $joinColumn */
-                if (! $joinColumn->getReferencedColumnName()) {
+                if ( ! $joinColumn->getReferencedColumnName()) {
                     $joinColumn->setReferencedColumnName($this->namingStrategy->referenceColumnName());
                 }
 
                 $referencedColumnName = $joinColumn->getReferencedColumnName();
 
-                if (! $joinColumn->getColumnName()) {
+                if ( ! $joinColumn->getColumnName()) {
                     $columnName = $this->namingStrategy->joinKeyColumnName(
                         $property->getSourceEntity(),
                         $referencedColumnName
@@ -815,13 +815,13 @@ 
 
             foreach ($joinTable->getInverseJoinColumns() as $inverseJoinColumn) {
                 /** @var JoinColumnMetadata $inverseJoinColumn */
-                if (! $inverseJoinColumn->getReferencedColumnName()) {
+                if ( ! $inverseJoinColumn->getReferencedColumnName()) {
                     $inverseJoinColumn->setReferencedColumnName($this->namingStrategy->referenceColumnName());
                 }
 
                 $referencedColumnName = $inverseJoinColumn->getReferencedColumnName();
 
-                if (! $inverseJoinColumn->getColumnName()) {
+                if ( ! $inverseJoinColumn->getColumnName()) {
                     $columnName = $this->namingStrategy->joinKeyColumnName(
                         $property->getTargetEntity(),
                         $referencedColumnName
@@ -855,7 +855,7 @@ 
             throw MappingException::singleIdNotAllowedOnCompositePrimaryKey($this->className);
         }
 
-        if (! isset($this->identifier[0])) {
+        if ( ! isset($this->identifier[0])) {
             throw MappingException::noIdDefined($this->className);
         }
 
@@ -914,7 +914,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());
             }
@@ -928,7 +928,7 @@ 
                 $columnName           = $joinColumn->getColumnName();
                 $referencedColumnName = $joinColumn->getReferencedColumnName();
 
-                if (! $joinColumn->getType()) {
+                if ( ! $joinColumn->getType()) {
                     $joinColumn->setType(PersisterHelper::getTypeOfColumn($referencedColumnName, $targetClass, $em));
                 }
 
@@ -962,10 +962,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';
     }
 
     /**
@@ -999,7 +999,7 @@ 
      */
     public function setInheritanceType($type) : void
     {
-        if (! $this->isInheritanceType($type)) {
+        if ( ! $this->isInheritanceType($type)) {
             throw MappingException::invalidInheritanceType($this->className, $type);
         }
 
@@ -1017,7 +1017,7 @@ 
     {
         $fieldName = $property->getName();
 
-        if (! isset($this->declaredProperties[$fieldName])) {
+        if ( ! isset($this->declaredProperties[$fieldName])) {
             throw MappingException::invalidOverrideFieldName($this->className, $fieldName);
         }
 
@@ -1214,7 +1214,7 @@ 
         $declaringClass    = $property->getDeclaringClass();
 
         if ($inheritedProperty instanceof FieldMetadata) {
-            if (! $declaringClass->isMappedSuperclass) {
+            if ( ! $declaringClass->isMappedSuperclass) {
                 $inheritedProperty->setTableName($property->getTableName());
             }
 
@@ -1325,11 +1325,11 @@ 
             'method' => $method,
         ];
 
-        if (! class_exists($class)) {
+        if ( ! class_exists($class)) {
             throw MappingException::entityListenerClassNotFound($class, $this->className);
         }
 
-        if (! method_exists($class, $method)) {
+        if ( ! method_exists($class, $method)) {
             throw MappingException::entityListenerMethodNotFound($class, $method, $this->className);
         }
 
@@ -1396,7 +1396,7 @@ 
             return;
         }
 
-        if (! (class_exists($className) || interface_exists($className))) {
+        if ( ! (class_exists($className) || interface_exists($className))) {
             throw MappingException::invalidClassInDiscriminatorMap($className, $this->className);
         }
 

lib/Doctrine/ORM/Persisters/SqlExpressionVisitor.php

Spacing +3 added lines, -3 removed lines

@@ -74,13 +74,13 @@
 
         switch ($expr->getType()) {
             case CompositeExpression::TYPE_AND:
-                return '(' . implode(' AND ', $expressionList) . ')';
+                return '('.implode(' AND ', $expressionList).')';
 
             case CompositeExpression::TYPE_OR:
-                return '(' . implode(' OR ', $expressionList) . ')';
+                return '('.implode(' OR ', $expressionList).')';
 
             default:
-                throw new RuntimeException('Unknown composite ' . $expr->getType());
+                throw new RuntimeException('Unknown composite '.$expr->getType());
         }
     }