doctrine / orm

lib/Doctrine/ORM/Cache/Region/DefaultMultiGetRegion.php

Spacing +1 added lines, -1 removed lines

@@ -70,7 +70,7 @@
         $returnableItems = [];
 
         foreach ($keysToRetrieve as $index => $key) {
-            if (! $items[$key] instanceof CacheEntry) {
+            if ( ! $items[$key] instanceof CacheEntry) {
                 return null;
             }
 

lib/Doctrine/ORM/Cache/Region/DefaultRegion.php

Spacing +4 added lines, -4 removed lines

@@ -96,7 +96,7 @@ 
     {
         $entry = $this->cache->fetch($this->getCacheEntryKey($key));
 
-        if (! $entry instanceof CacheEntry) {
+        if ( ! $entry instanceof CacheEntry) {
             return null;
         }
 
@@ -114,7 +114,7 @@ 
             $entryKey   = $this->getCacheEntryKey($key);
             $entryValue = $this->cache->fetch($entryKey);
 
-            if (! $entryValue instanceof CacheEntry) {
+            if ( ! $entryValue instanceof CacheEntry) {
                 return null;
             }
 
@@ -130,7 +130,7 @@ 
      */
     protected function getCacheEntryKey(CacheKey $key)
     {
-        return $this->name . self::REGION_KEY_SEPARATOR . $key->hash;
+        return $this->name.self::REGION_KEY_SEPARATOR.$key->hash;
     }
 
     /**
@@ -154,7 +154,7 @@ 
      */
     public function evictAll()
     {
-        if (! $this->cache instanceof ClearableCache) {
+        if ( ! $this->cache instanceof ClearableCache) {
             throw new \BadMethodCallException(sprintf(
                 'Clearing all cache entries is not supported by the supplied cache adapter of type %s',
                 get_class($this->cache)

tests/Doctrine/Tests/ORM/Cache/DefaultRegionTest.php

Spacing +2 added lines, -2 removed lines

@@ -103,7 +103,7 @@ 
     public function corruptedDataDoesNotLeakIntoApplicationWhenGettingSingleEntry() : void
     {
         $key1 = new CacheKeyMock('key.1');
-        $this->cache->save($this->region->getName() . '_' . $key1->hash, 'a-very-invalid-value');
+        $this->cache->save($this->region->getName().'_'.$key1->hash, 'a-very-invalid-value');
 
         self::assertTrue($this->region->contains($key1));
         self::assertNull($this->region->get($key1));
@@ -116,7 +116,7 @@ 
     public function corruptedDataDoesNotLeakIntoApplicationWhenGettingMultipleEntries() : void
     {
         $key1 = new CacheKeyMock('key.1');
-        $this->cache->save($this->region->getName() . '_' . $key1->hash, 'a-very-invalid-value');
+        $this->cache->save($this->region->getName().'_'.$key1->hash, 'a-very-invalid-value');
 
         self::assertTrue($this->region->contains($key1));
         self::assertNull($this->region->getMultiple(new CollectionCacheEntry([$key1])));

tests/Doctrine/Tests/ORM/Cache/MultiGetRegionTest.php

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
     public function corruptedDataDoesNotLeakIntoApplication() : void
     {
         $key1 = new CacheKeyMock('key.1');
-        $this->cache->save($this->region->getName() . '_' . $key1->hash, 'a-very-invalid-value');
+        $this->cache->save($this->region->getName().'_'.$key1->hash, 'a-very-invalid-value');
 
         self::assertTrue($this->region->contains($key1));
         self::assertNull($this->region->getMultiple(new CollectionCacheEntry([$key1])));

lib/Doctrine/ORM/Query/Parser.php

Spacing +12 added lines, -12 removed lines

@@ -487,7 +487,7 @@ 
         $tokenStr = substr($dql, $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, QueryException::dqlError($this->query->getDQL()));
     }
@@ -634,7 +634,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;
@@ -686,13 +686,13 @@ 
                 }
 
                 $this->semanticalError(
-                    "There is no mapped field named '$field' on class " . $class->name . ".", $deferredItem['token']
+                    "There is no mapped field named '$field' on class ".$class->name.".", $deferredItem['token']
                 );
             }
 
             if (array_intersect($class->identifier, $expr->partialFieldSet) != $class->identifier) {
                 $this->semanticalError(
-                    "The partial field selection of class " . $class->name . " must contain the identifier.",
+                    "The partial field selection of class ".$class->name." must contain the identifier.",
                     $deferredItem['token']
                 );
             }
@@ -761,7 +761,7 @@ 
             // Check if field or association exists
             if ( ! isset($class->associationMappings[$field]) && ! isset($class->fieldMappings[$field])) {
                 $this->semanticalError(
-                    'Class ' . $class->name . ' has no field or association named ' . $field,
+                    'Class '.$class->name.' has no field or association named '.$field,
                     $deferredItem['token']
                 );
             }
@@ -801,8 +801,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']);
             }
@@ -976,7 +976,7 @@ 
 
         [$namespaceAlias, $simpleClassName] = explode(':', $this->lexer->token['value']);
 
-        return $this->em->getConfiguration()->getEntityNamespace($namespaceAlias) . '\\' . $simpleClassName;
+        return $this->em->getConfiguration()->getEntityNamespace($namespaceAlias).'\\'.$simpleClassName;
     }
 
     /**
@@ -988,7 +988,7 @@ 
      */
     private function validateAbstractSchemaName($schemaName)
     {
-        if (! (class_exists($schemaName, true) || interface_exists($schemaName, true))) {
+        if ( ! (class_exists($schemaName, true) || interface_exists($schemaName, true))) {
             $this->semanticalError("Class '$schemaName' is not defined.", $this->lexer->token);
         }
     }
@@ -1044,7 +1044,7 @@ 
 
         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.'
             );
         }
 
@@ -1058,7 +1058,7 @@ 
         $class = $qComp['metadata'];
 
         if ( ! $class->hasAssociation($field)) {
-            $this->semanticalError('Class ' . $class->name . ' has no association named ' . $field);
+            $this->semanticalError('Class '.$class->name.' has no association named '.$field);
         }
 
         return new AST\JoinAssociationPathExpression($identVariable, $field);
@@ -2587,7 +2587,7 @@ 
             return $this->NullComparisonExpression();
         }
 
-        if ($token['type'] === Lexer::T_IS  && $lookahead['type'] === Lexer::T_EMPTY) {
+        if ($token['type'] === Lexer::T_IS && $lookahead['type'] === Lexer::T_EMPTY) {
             return $this->EmptyCollectionComparisonExpression();
         }
 

tests/Doctrine/Tests/ORM/Tools/Pagination/WhereInWalkerTest.php

Spacing +4 added lines, -4 removed lines

@@ -18,7 +18,7 @@ 
     {
         parent::setUp();
 
-        if (! Type::hasType('rot13')) {
+        if ( ! Type::hasType('rot13')) {
             Type::addType('rot13', Rot13Type::class);
         }
     }
@@ -188,7 +188,7 @@ 
      */
     public function testWhereInQueryWithArbitraryJoin_NoWhere()
     {
-        $whereInQuery  = $this->entityManager->createQuery(
+        $whereInQuery = $this->entityManager->createQuery(
             'SELECT p FROM Doctrine\Tests\ORM\Tools\Pagination\BlogPost p JOIN Doctrine\Tests\ORM\Tools\Pagination\Category c WITH p.category = c'
         );
         $whereInQuery->setHint(Query::HINT_CUSTOM_TREE_WALKERS, [WhereInWalker::class]);
@@ -227,7 +227,7 @@ 
     public function testWillReplaceBoundQueryIdentifiersWithConvertedTypesAsPerIdentifierMapping() : void
     {
         $whereInQuery = $this->entityManager->createQuery(
-            'SELECT e.id4 FROM ' . AuxiliaryEntity::class . ' e'
+            'SELECT e.id4 FROM '.AuxiliaryEntity::class.' e'
         );
         $whereInQuery->setHint(Query::HINT_CUSTOM_TREE_WALKERS, [WhereInWalker::class]);
         $whereInQuery->setHint(WhereInWalker::HINT_PAGINATOR_ID_COUNT, 3);
@@ -242,7 +242,7 @@ 
     public function testWillReplaceBoundQueryIdentifiersWithConvertedTypesAsPerAssociatedEntityIdentifierMapping() : void
     {
         $whereInQuery = $this->entityManager->createQuery(
-            'SELECT e FROM ' . OwningManyToOneIdForeignKeyEntity::class . ' e'
+            'SELECT e FROM '.OwningManyToOneIdForeignKeyEntity::class.' e'
         );
         $whereInQuery->setHint(Query::HINT_CUSTOM_TREE_WALKERS, [WhereInWalker::class]);
         $whereInQuery->setHint(WhereInWalker::HINT_PAGINATOR_ID_COUNT, 3);

lib/Doctrine/ORM/Tools/Pagination/WhereInWalker.php

Spacing +2 added lines, -2 removed lines

@@ -108,7 +108,7 @@ 
                 [$pathExpression]
             );
             $expression = new InExpression($arithmeticExpression);
-            $expression->literals[] = new InputParameter(":" . self::PAGINATOR_ID_ALIAS);
+            $expression->literals[] = new InputParameter(":".self::PAGINATOR_ID_ALIAS);
 
             $this->convertWhereInIdentifiersToDatabaseValue(
                 PersisterHelper::getTypeOfField(
@@ -177,7 +177,7 @@ 
             ->getEntityManager()
             ->getConnection();
 
-        $query->setParameter(self::PAGINATOR_ID_ALIAS, array_map(static function ($id) use ($connection, $type) {
+        $query->setParameter(self::PAGINATOR_ID_ALIAS, array_map(static function($id) use ($connection, $type) {
             return $connection->convertToDatabaseValue($id, $type);
         }, $identifiers));
     }

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

Spacing +2 added lines, -2 removed lines

@@ -21,7 +21,7 @@ 
     {
         parent::setUp();
 
-        if (! DBALType::hasType(GH5988CustomIdObjectHashType::class)) {
+        if ( ! DBALType::hasType(GH5988CustomIdObjectHashType::class)) {
             DBALType::addType(GH5988CustomIdObjectHashType::class, GH5988CustomIdObjectHashType::class);
         }
 
@@ -54,7 +54,7 @@ 
      */
     public function convertToDatabaseValue($value, AbstractPlatform $platform)
     {
-        return $value->id . '_test';
+        return $value->id.'_test';
     }
     /**
      * {@inheritdoc}

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

Spacing +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
     public function testForeignKeyOnSTIWithMultipleMapping()
     {
         $em = $this->_em;
-        if (! $em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
+        if ( ! $em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
             $this->markTestSkipped('Platform does not support foreign keys.');
         }
         $schemaTool = new SchemaTool($em);