doctrine / orm

tests/Doctrine/Tests/ORM/Query/LanguageRecognitionTest.php

Spacing +9 added lines, -9 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\Tests\ORM\Query;
 
@@ -30,7 +30,7 @@ 
             $this->addToAssertionCount(1);
         } catch (QueryException $e) {
             if ($debug) {
-                echo $e->getTraceAsString() . PHP_EOL;
+                echo $e->getTraceAsString().PHP_EOL;
             }
 
             $this->fail($e->getMessage());
@@ -45,8 +45,8 @@ 
             $this->fail('No syntax errors were detected, when syntax errors were expected');
         } catch (QueryException $e) {
             if ($debug) {
-                echo $e->getMessage() . PHP_EOL;
-                echo $e->getTraceAsString() . PHP_EOL;
+                echo $e->getMessage().PHP_EOL;
+                echo $e->getTraceAsString().PHP_EOL;
             }
             $this->addToAssertionCount(1);
         }
@@ -108,7 +108,7 @@ 
             ['SELECT foo: FROM Doctrine\Tests\Models\CMS\CmsUser foo:'],
 
             /* Checks for invalid AbstractSchemaName */
-            ['SELECT u FROM UnknownClass u'],  // unknown
+            ['SELECT u FROM UnknownClass u'], // unknown
             ['SELECT u FROM \Unknown\Class u'], // unknown, leading backslash
             ['SELECT u FROM Unknown\\\\Class u'], // unknown, syntactically bogus (duplicate \\)
             ['SELECT u FROM Unknown\Class\ u'], // unknown, syntactically bogus (trailing \)
@@ -574,7 +574,7 @@ 
      */
     public function testDQLKeywordInJoinIsAllowed()
     {
-        self::assertValidDQL('SELECT u FROM ' . __NAMESPACE__ . '\DQLKeywordsModelUser u JOIN u.group g');
+        self::assertValidDQL('SELECT u FROM '.__NAMESPACE__.'\DQLKeywordsModelUser u JOIN u.group g');
     }
 
     /**
@@ -582,7 +582,7 @@ 
      */
     public function testDQLKeywordInConditionIsAllowed()
     {
-        self::assertValidDQL('SELECT g FROM ' . __NAMESPACE__ . '\DQLKeywordsModelGroup g WHERE g.from=0');
+        self::assertValidDQL('SELECT g FROM '.__NAMESPACE__.'\DQLKeywordsModelGroup g WHERE g.from=0');
     }
 
     /* The exception is currently thrown in the SQLWalker, not earlier.
@@ -687,7 +687,7 @@ 
      */
     public function testNewLiteralExpression()
     {
-        self::assertValidDQL("SELECT new " . __NAMESPACE__ . "\\DummyStruct(u.id, 'foo', 1, true) FROM Doctrine\Tests\Models\CMS\CmsUser u");
+        self::assertValidDQL("SELECT new ".__NAMESPACE__."\\DummyStruct(u.id, 'foo', 1, true) FROM Doctrine\Tests\Models\CMS\CmsUser u");
     }
 
     /**
@@ -695,7 +695,7 @@ 
      */
     public function testNewLiteralWithSubselectExpression()
     {
-        self::assertValidDQL("SELECT new " . __NAMESPACE__ . "\\DummyStruct(u.id, 'foo', (SELECT 1 FROM Doctrine\Tests\Models\CMS\CmsUser su), true) FROM Doctrine\Tests\Models\CMS\CmsUser u");
+        self::assertValidDQL("SELECT new ".__NAMESPACE__."\\DummyStruct(u.id, 'foo', (SELECT 1 FROM Doctrine\Tests\Models\CMS\CmsUser su), true) FROM Doctrine\Tests\Models\CMS\CmsUser u");
     }
 }
 

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

Indentation +6 added lines, -6 removed lines

@@ -135,13 +135,13 @@
         $cmf = new ClassMetadataFactory();
         $driver = $this->createMock(MappingDriver::class);
         $driver->expects($this->at(0))
-               ->method('isTransient')
-               ->with($this->equalTo(CmsUser::class))
-               ->will($this->returnValue(true));
+                ->method('isTransient')
+                ->with($this->equalTo(CmsUser::class))
+                ->will($this->returnValue(true));
         $driver->expects($this->at(1))
-               ->method('isTransient')
-               ->with($this->equalTo(CmsArticle::class))
-               ->will($this->returnValue(false));
+                ->method('isTransient')
+                ->with($this->equalTo(CmsArticle::class))
+                ->will($this->returnValue(false));
 
         $em = $this->createEntityManager($driver);
 

tests/Doctrine/Tests/ORM/Functional/QueryTest.php

Indentation +22 added lines, -22 removed lines

@@ -150,7 +150,7 @@ 
         $this->expectExceptionMessage('Too few parameters: the query defines 1 parameters but you only bound 0');
 
         $this->em->createQuery('SELECT u FROM ' . CmsUser::class . ' u WHERE u.name = ?1')
-                  ->getSingleResult();
+                    ->getSingleResult();
     }
 
     public function testInvalidInputParameterThrowsException()
@@ -158,8 +158,8 @@ 
         $this->expectException(QueryException::class);
 
         $this->em->createQuery('SELECT u FROM ' . CmsUser::class . ' u WHERE u.name = ?')
-                  ->setParameter(1, 'jwage')
-                  ->getSingleResult();
+                    ->setParameter(1, 'jwage')
+                    ->getSingleResult();
     }
 
     public function testSetParameters()
@@ -169,8 +169,8 @@ 
         $parameters->add(new Parameter(2, 'active'));
 
         $this->em->createQuery('SELECT u FROM ' . CmsUser::class . ' u WHERE u.name = ?1 AND u.status = ?2')
-                  ->setParameters($parameters)
-                  ->getResult();
+                    ->setParameters($parameters)
+                    ->getResult();
 
         $extractValue = function (Parameter $parameter) {
             return $parameter->getValue();
@@ -187,8 +187,8 @@ 
         $parameters = [1 => 'jwage', 2 => 'active'];
 
         $this->em->createQuery('SELECT u FROM ' . CmsUser::class . ' u WHERE u.name = ?1 AND u.status = ?2')
-                  ->setParameters($parameters)
-                  ->getResult();
+                    ->setParameters($parameters)
+                    ->getResult();
 
         self::assertSame(
             array_values($parameters),
@@ -332,7 +332,7 @@ 
     public function testGetSingleResultThrowsExceptionOnNoResult()
     {
         $this->em->createQuery("select a from Doctrine\Tests\Models\CMS\CmsArticle a")
-             ->getSingleResult();
+                ->getSingleResult();
     }
 
     /**
@@ -341,7 +341,7 @@ 
     public function testGetSingleScalarResultThrowsExceptionOnNoResult()
     {
         $this->em->createQuery("select a from Doctrine\Tests\Models\CMS\CmsArticle a")
-             ->getSingleScalarResult();
+                ->getSingleScalarResult();
     }
 
     /**
@@ -372,7 +372,7 @@ 
         $this->em->clear();
 
         $this->em->createQuery("select a from Doctrine\Tests\Models\CMS\CmsArticle a")
-             ->getSingleScalarResult();
+                ->getSingleScalarResult();
     }
 
     public function testModifiedLimitQuery()
@@ -389,27 +389,27 @@ 
         $this->em->clear();
 
         $data = $this->em->createQuery('SELECT u FROM ' . CmsUser::class . ' u')
-                  ->setFirstResult(1)
-                  ->setMaxResults(2)
-                  ->getResult();
+                    ->setFirstResult(1)
+                    ->setMaxResults(2)
+                    ->getResult();
 
         self::assertCount(2, $data);
         self::assertEquals('gblanco1', $data[0]->username);
         self::assertEquals('gblanco2', $data[1]->username);
 
         $data = $this->em->createQuery('SELECT u FROM ' . CmsUser::class . ' u')
-                  ->setFirstResult(3)
-                  ->setMaxResults(2)
-                  ->getResult();
+                    ->setFirstResult(3)
+                    ->setMaxResults(2)
+                    ->getResult();
 
         self::assertCount(2, $data);
         self::assertEquals('gblanco3', $data[0]->username);
         self::assertEquals('gblanco4', $data[1]->username);
 
         $data = $this->em->createQuery('SELECT u FROM ' . CmsUser::class . ' u')
-                  ->setFirstResult(3)
-                  ->setMaxResults(2)
-                  ->getScalarResult();
+                    ->setFirstResult(3)
+                    ->setMaxResults(2)
+                    ->getScalarResult();
     }
 
     /**
@@ -620,9 +620,9 @@ 
     {
         $qb = $this->em->createQueryBuilder();
         $qb->select('u')
-           ->from(CmsUser::class, 'u')
-           ->innerJoin('u.articles', 'a')
-           ->where('(u.id = 0) OR (u.id IS NULL)');
+            ->from(CmsUser::class, 'u')
+            ->innerJoin('u.articles', 'a')
+            ->where('(u.id = 0) OR (u.id IS NULL)');
 
         $query = $qb->getQuery();
         $users = $query->execute();

lib/Doctrine/ORM/AbstractQuery.php

Spacing +11 added lines, -11 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\ORM;
 
@@ -293,7 +293,7 @@ 
     public function getParameter($key)
     {
         $filteredParameters = $this->parameters->filter(
-            function (Query\Parameter $parameter) use ($key) : bool {
+            function(Query\Parameter $parameter) use ($key) : bool {
                 $parameterName = $parameter->getName();
 
                 return $key === $parameterName || (string) $key === (string) $parameterName;
@@ -712,7 +712,7 @@ 
             return null;
         }
 
-        if (! is_array($result)) {
+        if ( ! is_array($result)) {
             return $result;
         }
 
@@ -746,7 +746,7 @@ 
             throw new NoResultException;
         }
 
-        if (! is_array($result)) {
+        if ( ! is_array($result)) {
             return $result;
         }
 
@@ -835,7 +835,7 @@ 
             $this->setHydrationMode($hydrationMode);
         }
 
-        if (! empty($parameters)) {
+        if ( ! empty($parameters)) {
             $this->setParameters($parameters);
         }
 
@@ -876,11 +876,11 @@ 
             $this->setHydrationMode($hydrationMode);
         }
 
-        if (! empty($parameters)) {
+        if ( ! empty($parameters)) {
             $this->setParameters($parameters);
         }
 
-        $setCacheEntry = function () {
+        $setCacheEntry = function() {
         };
 
         if ($this->hydrationCacheProfile !== null) {
@@ -894,11 +894,11 @@ 
                 return $result[$realCacheKey];
             }
 
-            if (! $result) {
+            if ( ! $result) {
                 $result = [];
             }
 
-            $setCacheEntry = function ($data) use ($cache, $result, $cacheKey, $realCacheKey, $queryCacheProfile) {
+            $setCacheEntry = function($data) use ($cache, $result, $cacheKey, $realCacheKey, $queryCacheProfile) {
                 $result[$realCacheKey] = $data;
 
                 $cache->save($cacheKey, $result, $queryCacheProfile->getLifetime());
@@ -1060,7 +1060,7 @@ 
     {
         $query  = $this->getSQL();
         $hints  = $this->getHints();
-        $params = array_map(function (Parameter $parameter) {
+        $params = array_map(function(Parameter $parameter) {
             $value = $parameter->getValue();
 
             // Small optimization
@@ -1074,6 +1074,6 @@ 
 
         ksort($hints);
 
-        return sha1($query . '-' . serialize($params) . '-' . serialize($hints));
+        return sha1($query.'-'.serialize($params).'-'.serialize($hints));
     }
 }

lib/Doctrine/ORM/Mapping/ClassMetadataFactory.php

Spacing +20 added lines, -20 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\ORM\Mapping;
 
@@ -80,7 +80,7 @@ 
         string $className,
         ClassMetadataBuildingContext $metadataBuildingContext
     ) : ?ClassMetadata {
-        if (! $this->evm->hasListeners(Events::onClassMetadataNotFound)) {
+        if ( ! $this->evm->hasListeners(Events::onClassMetadataNotFound)) {
             return null;
         }
 
@@ -141,20 +141,20 @@ 
                 $classMetadata->setCache(clone $parent->getCache());
             }
 
-            if (! empty($parent->namedNativeQueries)) {
+            if ( ! empty($parent->namedNativeQueries)) {
                 $this->addInheritedNamedNativeQueries($classMetadata, $parent);
             }
 
-            if (! empty($parent->sqlResultSetMappings)) {
+            if ( ! empty($parent->sqlResultSetMappings)) {
                 $this->addInheritedSqlResultSetMappings($classMetadata, $parent);
             }
 
-            if (! empty($parent->entityListeners) && empty($classMetadata->entityListeners)) {
+            if ( ! empty($parent->entityListeners) && empty($classMetadata->entityListeners)) {
                 $classMetadata->entityListeners = $parent->entityListeners;
             }
         }
 
-        if (! $classMetadata->discriminatorMap && $classMetadata->inheritanceType !== InheritanceType::NONE && $classMetadata->isRootEntity()) {
+        if ( ! $classMetadata->discriminatorMap && $classMetadata->inheritanceType !== InheritanceType::NONE && $classMetadata->isRootEntity()) {
             $this->addDefaultDiscriminatorMap($classMetadata);
         }
 
@@ -174,7 +174,7 @@ 
 
     protected function completeRuntimeMetadata(ClassMetadata $class, ?ClassMetadata $parent = null) : void
     {
-        if (! $parent || ! $parent->isMappedSuperclass) {
+        if ( ! $parent || ! $parent->isMappedSuperclass) {
             return;
         }
 
@@ -192,7 +192,7 @@ 
                 continue;
             }
 
-            if (! ($property instanceof ToOneAssociationMetadata)) {
+            if ( ! ($property instanceof ToOneAssociationMetadata)) {
                 continue;
             }
 
@@ -211,7 +211,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;
         }
@@ -221,13 +221,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());
                 }
             }
@@ -437,7 +437,7 @@ 
     private function completeIdentifierGeneratorMappings(ClassMetadata $class) : void
     {
         foreach ($class->getDeclaredPropertiesIterator() as $property) {
-            if (! $property instanceof FieldMetadata /*&& ! $property instanceof AssocationMetadata*/) {
+            if ( ! $property instanceof FieldMetadata /*&& ! $property instanceof AssocationMetadata*/) {
                 continue;
             }
 
@@ -447,7 +447,7 @@ 
 
     private function completeFieldIdentifierGeneratorMapping(FieldMetadata $field)
     {
-        if (! $field->hasValueGenerator()) {
+        if ( ! $field->hasValueGenerator()) {
             return;
         }
 
@@ -499,10 +499,10 @@ 
 
             case GeneratorType::CUSTOM:
                 $definition = $generator->getDefinition();
-                if (! isset($definition['class'])) {
+                if ( ! isset($definition['class'])) {
                     throw new ORMException(sprintf('Cannot instantiate custom generator, no class has been defined'));
                 }
-                if (! class_exists($definition['class'])) {
+                if ( ! class_exists($definition['class'])) {
                     throw new ORMException(sprintf('Cannot instantiate custom generator : %s', var_export($definition, true))); //$definition['class']));
                 }
 
@@ -514,7 +514,7 @@ 
                 break;
 
             default:
-                throw new ORMException('Unknown generator type: ' . $generator->getType());
+                throw new ORMException('Unknown generator type: '.$generator->getType());
         }
     }
 
@@ -536,7 +536,7 @@ 
 
     private function getTargetPlatform() : Platforms\AbstractPlatform
     {
-        if (! $this->targetPlatform) {
+        if ( ! $this->targetPlatform) {
             $this->targetPlatform = $this->em->getConnection()->getDatabasePlatform();
         }
 
@@ -549,7 +549,7 @@ 
         $generatedProperties = [];
 
         foreach ($class->getDeclaredPropertiesIterator() as $property) {
-            if (! ($property instanceof LocalColumnMetadata && $property->hasValueGenerator())) {
+            if ( ! ($property instanceof LocalColumnMetadata && $property->hasValueGenerator())) {
                 continue;
             }
 

lib/Doctrine/ORM/Configuration.php

Spacing +7 added lines, -7 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\ORM;
 
@@ -199,7 +199,7 @@ 
      */
     public function newDefaultAnnotationDriver(array $paths = []) : AnnotationDriver
     {
-        AnnotationRegistry::registerFile(__DIR__ . '/Annotation/DoctrineAnnotations.php');
+        AnnotationRegistry::registerFile(__DIR__.'/Annotation/DoctrineAnnotations.php');
 
         $reader = new CachedReader(new AnnotationReader(), new ArrayCache());
 
@@ -277,7 +277,7 @@ 
      */
     public function getNamedQuery(string $queryName) : string
     {
-        if (! isset($this->namedQueries[$queryName])) {
+        if ( ! isset($this->namedQueries[$queryName])) {
             throw ORMException::namedQueryNotFound($queryName);
         }
 
@@ -301,7 +301,7 @@ 
      */
     public function getNamedNativeQuery(string $queryName) : array
     {
-        if (! isset($this->namedNativeQueries[$queryName])) {
+        if ( ! isset($this->namedNativeQueries[$queryName])) {
             throw ORMException::namedNativeQueryNotFound($queryName);
         }
 
@@ -319,7 +319,7 @@ 
     {
         $queryCacheImpl = $this->getQueryCacheImpl();
 
-        if (! $queryCacheImpl) {
+        if ( ! $queryCacheImpl) {
             throw ORMException::queryCacheNotConfigured();
         }
 
@@ -329,7 +329,7 @@ 
 
         $metadataCacheImpl = $this->getMetadataCacheImpl();
 
-        if (! $metadataCacheImpl) {
+        if ( ! $metadataCacheImpl) {
             throw ORMException::metadataCacheNotConfigured();
         }
 
@@ -537,7 +537,7 @@ 
     {
         $reflectionClass = new \ReflectionClass($repositoryClassName);
 
-        if (! $reflectionClass->implementsInterface(ObjectRepository::class)) {
+        if ( ! $reflectionClass->implementsInterface(ObjectRepository::class)) {
             throw ORMException::invalidEntityRepository($repositoryClassName);
         }
 

lib/Doctrine/ORM/Query/Parser.php

Spacing +31 added lines, -31 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\ORM\Query;
 
@@ -389,7 +389,7 @@ 
         }
 
         foreach ($this->queryComponents as $dqlAlias => $qComp) {
-            if (! isset($this->identVariableExpressions[$dqlAlias])) {
+            if ( ! isset($this->identVariableExpressions[$dqlAlias])) {
                 continue;
             }
 
@@ -453,7 +453,7 @@ 
         $tokenStr = substr($dql, (int) $token['position'], $length);
 
         // Building informative message
-        $message = 'line 0, col ' . $tokenPos . " near '" . $tokenStr . "': Error: " . $message;
+        $message = 'line 0, col '.$tokenPos." near '".$tokenStr."': Error: ".$message;
 
         throw QueryException::semanticalError(
             $message,
@@ -556,7 +556,7 @@ 
             $identVariable = $deferredItem['expression'];
 
             // Check if IdentificationVariable exists in queryComponents
-            if (! isset($this->queryComponents[$identVariable])) {
+            if ( ! isset($this->queryComponents[$identVariable])) {
                 $this->semanticalError(
                     sprintf("'%s' is not defined.", $identVariable),
                     $deferredItem['token']
@@ -566,7 +566,7 @@ 
             $qComp = $this->queryComponents[$identVariable];
 
             // Check if queryComponent points to an AbstractSchemaName or a ResultVariable
-            if (! isset($qComp['metadata'])) {
+            if ( ! isset($qComp['metadata'])) {
                 $this->semanticalError(
                     sprintf("'%s' does not point to a Class.", $identVariable),
                     $deferredItem['token']
@@ -600,7 +600,7 @@ 
             // If the namespace is not given then assumes the first FROM entity namespace
             if (strpos($className, '\\') === false && ! class_exists($className) && strpos($fromClassName, '\\') !== false) {
                 $namespace = substr($fromClassName, 0, strrpos($fromClassName, '\\'));
-                $fqcn      = $namespace . '\\' . $className;
+                $fqcn      = $namespace.'\\'.$className;
 
                 if (class_exists($fqcn)) {
                     $expression->className = $fqcn;
@@ -608,13 +608,13 @@ 
                 }
             }
 
-            if (! class_exists($className)) {
+            if ( ! class_exists($className)) {
                 $this->semanticalError(sprintf('Class "%s" is not defined.', $className), $token);
             }
 
             $class = new \ReflectionClass($className);
 
-            if (! $class->isInstantiable()) {
+            if ( ! $class->isInstantiable()) {
                 $this->semanticalError(sprintf('Class "%s" can not be instantiated.', $className), $token);
             }
 
@@ -671,7 +671,7 @@ 
             $resultVariable = $deferredItem['expression'];
 
             // Check if ResultVariable exists in queryComponents
-            if (! isset($this->queryComponents[$resultVariable])) {
+            if ( ! isset($this->queryComponents[$resultVariable])) {
                 $this->semanticalError(
                     sprintf("'%s' is not defined.", $resultVariable),
                     $deferredItem['token']
@@ -681,7 +681,7 @@ 
             $qComp = $this->queryComponents[$resultVariable];
 
             // Check if queryComponent points to an AbstractSchemaName or a ResultVariable
-            if (! isset($qComp['resultVariable'])) {
+            if ( ! isset($qComp['resultVariable'])) {
                 $this->semanticalError(
                     sprintf("'%s' does not point to a ResultVariable.", $resultVariable),
                     $deferredItem['token']
@@ -723,9 +723,9 @@ 
             $property = $class->getProperty($field);
 
             // Check if field or association exists
-            if (! $property) {
+            if ( ! $property) {
                 $this->semanticalError(
-                    'Class ' . $class->getClassName() . ' has no field or association named ' . $field,
+                    'Class '.$class->getClassName().' has no field or association named '.$field,
                     $deferredItem['token']
                 );
             }
@@ -742,7 +742,7 @@ 
             // Validate if PathExpression is one of the expected types
             $expectedType = $pathExpression->expectedType;
 
-            if (! ($expectedType & $fieldType)) {
+            if ( ! ($expectedType & $fieldType)) {
                 // We need to recognize which was expected type(s)
                 $expectedStringTypes = [];
 
@@ -764,8 +764,8 @@ 
                 // Build the error message
                 $semanticalError  = 'Invalid PathExpression. ';
                 $semanticalError .= \count($expectedStringTypes) === 1
-                    ? 'Must be a ' . $expectedStringTypes[0] . '.'
-                    : implode(' or ', $expectedStringTypes) . ' expected.';
+                    ? 'Must be a '.$expectedStringTypes[0].'.'
+                    : implode(' or ', $expectedStringTypes).' expected.';
 
                 $this->semanticalError($semanticalError, $deferredItem['token']);
             }
@@ -777,7 +777,7 @@ 
 
     private function processRootEntityAliasSelected()
     {
-        if (! $this->identVariableExpressions) {
+        if ( ! $this->identVariableExpressions) {
             return;
         }
 
@@ -1007,9 +1007,9 @@ 
     {
         $identVariable = $this->IdentificationVariable();
 
-        if (! isset($this->queryComponents[$identVariable])) {
+        if ( ! isset($this->queryComponents[$identVariable])) {
             $this->semanticalError(
-                'Identification Variable ' . $identVariable . ' used in join path expression but was not defined before.'
+                'Identification Variable '.$identVariable.' used in join path expression but was not defined before.'
             );
         }
 
@@ -1023,8 +1023,8 @@ 
         $class    = $qComp['metadata'];
         $property = $class->getProperty($field);
 
-        if (! ($property !== null && $property instanceof AssociationMetadata)) {
-            $this->semanticalError('Class ' . $class->getClassName() . ' has no association named ' . $field);
+        if ( ! ($property !== null && $property instanceof AssociationMetadata)) {
+            $this->semanticalError('Class '.$class->getClassName().' has no association named '.$field);
         }
 
         return new AST\JoinAssociationPathExpression($identVariable, $field);
@@ -1054,7 +1054,7 @@ 
             while ($this->lexer->isNextToken(Lexer::T_DOT)) {
                 $this->match(Lexer::T_DOT);
                 $this->match(Lexer::T_IDENTIFIER);
-                $field .= '.' . $this->lexer->token['value'];
+                $field .= '.'.$this->lexer->token['value'];
             }
         }
 
@@ -1436,7 +1436,7 @@ 
         // Still need to decide between IdentificationVariable or ResultVariable
         $lookaheadValue = $this->lexer->lookahead['value'];
 
-        if (! isset($this->queryComponents[$lookaheadValue])) {
+        if ( ! isset($this->queryComponents[$lookaheadValue])) {
             $this->semanticalError('Cannot group by undefined identification or result variable.');
         }
 
@@ -1785,7 +1785,7 @@ 
         while ($this->lexer->isNextToken(Lexer::T_DOT)) {
             $this->match(Lexer::T_DOT);
             $this->match(Lexer::T_IDENTIFIER);
-            $field .= '.' . $this->lexer->token['value'];
+            $field .= '.'.$this->lexer->token['value'];
         }
 
         $partialFieldSet[] = $field;
@@ -1799,7 +1799,7 @@ 
             while ($this->lexer->isNextToken(Lexer::T_DOT)) {
                 $this->match(Lexer::T_DOT);
                 $this->match(Lexer::T_IDENTIFIER);
-                $field .= '.' . $this->lexer->token['value'];
+                $field .= '.'.$this->lexer->token['value'];
             }
 
             $partialFieldSet[] = $field;
@@ -2420,7 +2420,7 @@ 
 
         // Phase 1 AST optimization: Prevent AST\ConditionalFactor
         // if only one AST\ConditionalPrimary is defined
-        if (! $not) {
+        if ( ! $not) {
             return $conditionalPrimary;
         }
 
@@ -2439,7 +2439,7 @@ 
     {
         $condPrimary = new AST\ConditionalPrimary;
 
-        if (! $this->lexer->isNextToken(Lexer::T_OPEN_PARENTHESIS)) {
+        if ( ! $this->lexer->isNextToken(Lexer::T_OPEN_PARENTHESIS)) {
             $condPrimary->simpleConditionalExpression = $this->SimpleConditionalExpression();
 
             return $condPrimary;
@@ -2607,7 +2607,7 @@ 
             $this->match(Lexer::T_OF);
         }
 
-        $collMemberExpr      = new AST\CollectionMemberExpression(
+        $collMemberExpr = new AST\CollectionMemberExpression(
             $entityExpr,
             $this->CollectionValuedPathExpression()
         );
@@ -2944,7 +2944,7 @@ 
         $lookaheadType = $this->lexer->lookahead['type'];
         $isDistinct    = false;
 
-        if (! in_array($lookaheadType, [Lexer::T_COUNT, Lexer::T_AVG, Lexer::T_MAX, Lexer::T_MIN, Lexer::T_SUM], true)) {
+        if ( ! in_array($lookaheadType, [Lexer::T_COUNT, Lexer::T_AVG, Lexer::T_MAX, Lexer::T_MIN, Lexer::T_SUM], true)) {
             $this->syntaxError('One of: MAX, MIN, AVG, SUM, COUNT');
         }
 
@@ -2974,7 +2974,7 @@ 
         $lookaheadType = $this->lexer->lookahead['type'];
         $value         = $this->lexer->lookahead['value'];
 
-        if (! in_array($lookaheadType, [Lexer::T_ALL, Lexer::T_ANY, Lexer::T_SOME], true)) {
+        if ( ! in_array($lookaheadType, [Lexer::T_ALL, Lexer::T_ANY, Lexer::T_SOME], true)) {
             $this->syntaxError('ALL, ANY or SOME');
         }
 
@@ -3212,7 +3212,7 @@ 
                 $lookaheadValue = $this->lexer->lookahead['value'];
 
                 // Validate existing component
-                if (! isset($this->queryComponents[$lookaheadValue])) {
+                if ( ! isset($this->queryComponents[$lookaheadValue])) {
                     $this->semanticalError('Cannot add having condition on undefined result variable.');
                 }
 
@@ -3223,7 +3223,7 @@ 
                 }
 
                 // Validating ResultVariable
-                if (! isset($this->queryComponents[$lookaheadValue]['resultVariable'])) {
+                if ( ! isset($this->queryComponents[$lookaheadValue]['resultVariable'])) {
                     $this->semanticalError('Cannot add having condition on a non result variable.');
                 }