doctrine / orm

tests/Doctrine/Tests/ORM/Persisters/BasicEntityPersisterCompositeTypeParametersTest.php

Indentation +2 added lines, -2 removed lines

@@ -42,7 +42,7 @@ 
         $country = new Country('IT', 'Italy');
         $admin1  = new Admin1(10, 'Rome', $country);
 
-         [$values, $types] = $this->persister->expandParameters(['admin1' => $admin1]);
+            [$values, $types] = $this->persister->expandParameters(['admin1' => $admin1]);
 
         self::assertEquals([Type::getType('integer'), Type::getType('string')], $types);
         self::assertEquals([10, 'IT'], $values);
@@ -56,7 +56,7 @@ 
         $criteria = Criteria::create();
         $criteria->andWhere(Criteria::expr()->eq('admin1', $admin1));
 
-         [$values, $types] = $this->persister->expandCriteriaParameters($criteria);
+            [$values, $types] = $this->persister->expandCriteriaParameters($criteria);
 
         self::assertEquals([Type::getType('integer'), Type::getType('string')], $types);
         self::assertEquals([10, 'IT'], $values);

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

Spacing +7 added lines, -7 removed lines

@@ -65,7 +65,7 @@ 
 
             $query->free();
         } catch (Exception $e) {
-            $this->fail($e->getMessage() . "\n" . $e->getTraceAsString());
+            $this->fail($e->getMessage()."\n".$e->getTraceAsString());
         }
 
         self::assertEquals($sqlToBeConfirmed, $sqlGenerated);
@@ -1625,7 +1625,7 @@ 
     public function testAliasDoesNotExceedPlatformDefinedLength() : void
     {
         $this->assertSqlGeneration(
-            'SELECT m FROM ' . __NAMESPACE__ . '\\DDC1384Model m',
+            'SELECT m FROM '.__NAMESPACE__.'\\DDC1384Model m',
             'SELECT t0."aVeryLongIdentifierThatShouldBeShortenedByTheSQLWalker_fooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooo" AS c0 FROM "DDC1384Model" t0'
         );
     }
@@ -1658,12 +1658,12 @@ 
     public function testSelectWithArithmeticExpressionBeforeField() : void
     {
         $this->assertSqlGeneration(
-            'SELECT - e.value AS value, e.id FROM ' . __NAMESPACE__ . '\DDC1474Entity e',
+            'SELECT - e.value AS value, e.id FROM '.__NAMESPACE__.'\DDC1474Entity e',
             'SELECT -t0."value" AS c0, t0."id" AS c1 FROM "DDC1474Entity" t0'
         );
 
         $this->assertSqlGeneration(
-            'SELECT e.id, + e.value AS value FROM ' . __NAMESPACE__ . '\DDC1474Entity e',
+            'SELECT e.id, + e.value AS value FROM '.__NAMESPACE__.'\DDC1474Entity e',
             'SELECT t0."id" AS c0, +t0."value" AS c1 FROM "DDC1474Entity" t0'
         );
     }
@@ -2340,8 +2340,8 @@ 
     public function testHavingRegressionUsingVariableWithMathOperatorsExpression($operator) : void
     {
         $this->assertSqlGeneration(
-            'SELECT COUNT(u.name) AS countName FROM Doctrine\Tests\Models\CMS\CmsUser u HAVING 1 ' . $operator . ' countName > 0',
-            'SELECT COUNT(t0."name") AS c0 FROM "cms_users" t0 HAVING 1 ' . $operator . ' c0 > 0'
+            'SELECT COUNT(u.name) AS countName FROM Doctrine\Tests\Models\CMS\CmsUser u HAVING 1 '.$operator.' countName > 0',
+            'SELECT COUNT(t0."name") AS c0 FROM "cms_users" t0 HAVING 1 '.$operator.' c0 > 0'
         );
     }
 
@@ -2360,7 +2360,7 @@ 
 
     public function getSql(SqlWalker $sqlWalker)
     {
-        return 'ABS(' . $sqlWalker->walkSimpleArithmeticExpression($this->simpleArithmeticExpression) . ')';
+        return 'ABS('.$sqlWalker->walkSimpleArithmeticExpression($this->simpleArithmeticExpression).')';
     }
 
     public function parse(Parser $parser)

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

Spacing +6 added lines, -6 removed lines

@@ -45,7 +45,7 @@ 
         try {
             $sqlGenerated = $this->generateSql($dqlToBeTested, $treeWalkers, $outputWalker);
         } catch (Exception $e) {
-            $this->fail($e->getMessage() . ' at "' . $e->getFile() . '" on line ' . $e->getLine());
+            $this->fail($e->getMessage().' at "'.$e->getFile().'" on line '.$e->getLine());
         }
 
         self::assertEquals($sqlToBeConfirmed, $sqlGenerated);
@@ -145,7 +145,7 @@ 
             $condExpr = $whereClause->conditionalExpression;
 
             // Since Phase 1 AST optimizations were included, we need to re-add the ConditionalExpression
-            if (! ($condExpr instanceof Query\AST\ConditionalExpression)) {
+            if ( ! ($condExpr instanceof Query\AST\ConditionalExpression)) {
                 $condExpr = new Query\AST\ConditionalExpression([$condExpr]);
 
                 $whereClause->conditionalExpression = $condExpr;
@@ -168,7 +168,7 @@ 
                 $singleTerm = $selectStatement->whereClause->conditionalExpression->conditionalTerms[0];
 
                 // Since Phase 1 AST optimizations were included, we need to re-add the ConditionalExpression
-                if (! ($singleTerm instanceof Query\AST\ConditionalTerm)) {
+                if ( ! ($singleTerm instanceof Query\AST\ConditionalTerm)) {
                     $singleTerm = new Query\AST\ConditionalTerm([$singleTerm]);
 
                     $selectStatement->whereClause->conditionalExpression->conditionalTerms[0] = $singleTerm;
@@ -206,9 +206,9 @@ 
     {
         $rangeVariableDecl       = $identificationVariableDecl->rangeVariableDeclaration;
         $joinAssocPathExpression = new Query\AST\JoinAssociationPathExpression($rangeVariableDecl->aliasIdentificationVariable, 'address');
-        $joinAssocDeclaration    = new Query\AST\JoinAssociationDeclaration($joinAssocPathExpression, $rangeVariableDecl->aliasIdentificationVariable . 'a', null);
+        $joinAssocDeclaration    = new Query\AST\JoinAssociationDeclaration($joinAssocPathExpression, $rangeVariableDecl->aliasIdentificationVariable.'a', null);
         $join                    = new Query\AST\Join(Query\AST\Join::JOIN_TYPE_LEFT, $joinAssocDeclaration);
-        $selectExpression        = new Query\AST\SelectExpression($rangeVariableDecl->aliasIdentificationVariable . 'a', null, false);
+        $selectExpression        = new Query\AST\SelectExpression($rangeVariableDecl->aliasIdentificationVariable.'a', null, false);
 
         $identificationVariableDecl->joins[]                = $join;
         $selectStatement->selectClause->selectExpressions[] = $selectExpression;
@@ -218,7 +218,7 @@ 
         $addressMetadata = $entityManager->getClassMetadata(CmsAddress::class);
 
         $this->setQueryComponent(
-            $rangeVariableDecl->aliasIdentificationVariable . 'a',
+            $rangeVariableDecl->aliasIdentificationVariable.'a',
             [
                 'metadata'     => $addressMetadata,
                 'parent'       => $rangeVariableDecl->aliasIdentificationVariable,

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

Spacing +4 added lines, -4 removed lines

@@ -35,7 +35,7 @@ 
 
             $query->free();
         } catch (Exception $e) {
-            $this->fail($e->getMessage() . ' at "' . $e->getFile() . '" on line ' . $e->getLine());
+            $this->fail($e->getMessage().' at "'.$e->getFile().'" on line '.$e->getLine());
         }
 
         self::assertEquals($sqlToBeConfirmed, $sqlGenerated);
@@ -77,9 +77,9 @@ 
     {
         $rangeVariableDecl       = $identificationVariableDecl->rangeVariableDeclaration;
         $joinAssocPathExpression = new Query\AST\JoinAssociationPathExpression($rangeVariableDecl->aliasIdentificationVariable, 'address');
-        $joinAssocDeclaration    = new Query\AST\JoinAssociationDeclaration($joinAssocPathExpression, $rangeVariableDecl->aliasIdentificationVariable . 'a', null);
+        $joinAssocDeclaration    = new Query\AST\JoinAssociationDeclaration($joinAssocPathExpression, $rangeVariableDecl->aliasIdentificationVariable.'a', null);
         $join                    = new Query\AST\Join(Query\AST\Join::JOIN_TYPE_LEFT, $joinAssocDeclaration);
-        $selectExpression        = new Query\AST\SelectExpression($rangeVariableDecl->aliasIdentificationVariable . 'a', null, false);
+        $selectExpression        = new Query\AST\SelectExpression($rangeVariableDecl->aliasIdentificationVariable.'a', null, false);
 
         $identificationVariableDecl->joins[]                = $join;
         $selectStatement->selectClause->selectExpressions[] = $selectExpression;
@@ -89,7 +89,7 @@ 
         $addressMetadata = $entityManager->getClassMetadata(CmsAddress::class);
 
         $this->setQueryComponent(
-            $rangeVariableDecl->aliasIdentificationVariable . 'a',
+            $rangeVariableDecl->aliasIdentificationVariable.'a',
             [
                 'metadata'     => $addressMetadata,
                 'parent'       => $rangeVariableDecl->aliasIdentificationVariable,

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

Spacing +9 added lines, -9 removed lines

@@ -19,16 +19,16 @@
     public function providerParameterTypeInferer()
     {
         $data = [
-            [1,                 Type::INTEGER],
-            ['bar',             ParameterType::STRING],
-            ['1',               ParameterType::STRING],
-            [new DateTime(),     Type::DATETIME],
+            [1, Type::INTEGER],
+            ['bar', ParameterType::STRING],
+            ['1', ParameterType::STRING],
+            [new DateTime(), Type::DATETIME],
             [new DateInterval('P1D'), Type::DATEINTERVAL],
-            [[2],          Connection::PARAM_INT_ARRAY],
-            [['foo'],      Connection::PARAM_STR_ARRAY],
-            [['1','2'],    Connection::PARAM_STR_ARRAY],
-            [[],           Connection::PARAM_STR_ARRAY],
-            [true,              Type::BOOLEAN],
+            [[2], Connection::PARAM_INT_ARRAY],
+            [['foo'], Connection::PARAM_STR_ARRAY],
+            [['1', '2'], Connection::PARAM_STR_ARRAY],
+            [[], Connection::PARAM_STR_ARRAY],
+            [true, Type::BOOLEAN],
         ];
 
         if (PHP_VERSION_ID >= 50500) {

tests/Doctrine/Tests/ORM/EntityManagerTest.php

Spacing +4 added lines, -4 removed lines

@@ -141,7 +141,7 @@ 
     public function testThrowsExceptionOnNonObjectValues($methodName) : void
     {
         $this->expectException(ORMInvalidArgumentException::class);
-        $this->expectExceptionMessage('EntityManager#' . $methodName . '() expects parameter 1 to be an entity object, NULL given.');
+        $this->expectExceptionMessage('EntityManager#'.$methodName.'() expects parameter 1 to be an entity object, NULL given.');
 
         $this->em->{$methodName}(null);
     }
@@ -186,7 +186,7 @@ 
     {
         self::assertSame(
             $value,
-            $this->em->transactional(static function ($em) use ($value) {
+            $this->em->transactional(static function($em) use ($value) {
                 return $value;
             })
         );
@@ -219,8 +219,8 @@ 
     public function testTransactionalReThrowsThrowables() : void
     {
         try {
-            $this->em->transactional(static function () {
-                (static function (array $value) {
+            $this->em->transactional(static function() {
+                (static function(array $value) {
                     // this only serves as an IIFE that throws a `TypeError`
                 })(null);
             });

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

Spacing +3 added lines, -3 removed lines

@@ -59,7 +59,7 @@ 
 
     protected function createRegion()
     {
-        $this->directory = sys_get_temp_dir() . '/doctrine_lock_' . uniqid();
+        $this->directory = sys_get_temp_dir().'/doctrine_lock_'.uniqid();
 
         $region = new DefaultRegion('concurren_region_test', $this->cache);
 
@@ -262,7 +262,7 @@ 
         $reflectionDirectory->setAccessible(true);
         $reflectionDirectory->setValue($region, str_repeat('a', 10000));
 
-        set_error_handler(static function () {
+        set_error_handler(static function() {
         }, E_WARNING);
         self::assertTrue($region->evictAll());
         restore_error_handler();
@@ -275,7 +275,7 @@ 
     {
         $path = $path ?: $this->directory;
 
-        if (! is_dir($path)) {
+        if ( ! is_dir($path)) {
             return;
         }
 

tests/Doctrine/Tests/ORM/PersistentCollectionTest.php

Spacing +3 added lines, -3 removed lines

@@ -173,7 +173,7 @@ 
             ->expects(self::once())
             ->method('loadCollection')
             ->with($this->collection)
-            ->willReturnCallback(static function (PersistentCollection $persistentCollection) use ($persistedElement) : void {
+            ->willReturnCallback(static function(PersistentCollection $persistentCollection) use ($persistedElement) : void {
                 $persistentCollection->unwrap()->add($persistedElement);
             });
 
@@ -210,7 +210,7 @@ 
             ->expects(self::once())
             ->method('loadCollection')
             ->with($this->collection)
-            ->willReturnCallback(static function (PersistentCollection $persistentCollection) use (
+            ->willReturnCallback(static function(PersistentCollection $persistentCollection) use (
                 $persistedElement,
                 $newElementThatIsAlsoPersisted
             ) : void {
@@ -252,7 +252,7 @@ 
             ->expects(self::once())
             ->method('loadCollection')
             ->with($this->collection)
-            ->willReturnCallback(static function (PersistentCollection $persistentCollection) use (
+            ->willReturnCallback(static function(PersistentCollection $persistentCollection) use (
                 $persistedElement,
                 $newElementThatIsAlsoPersisted
             ) : void {

tests/Doctrine/Tests/OrmFunctionalTestCase.php

Spacing +20 added lines, -20 removed lines

@@ -346,7 +346,7 @@ 
         $conn = static::$sharedConn;
 
         // In case test is skipped, tearDown is called, but no setup may have run
-        if (! $conn) {
+        if ( ! $conn) {
             return;
         }
 
@@ -420,7 +420,7 @@ 
         }
 
         if (isset($this->usedModelSets['directorytree'])) {
-            $conn->executeUpdate('DELETE FROM ' . $platform->quoteIdentifier('file'));
+            $conn->executeUpdate('DELETE FROM '.$platform->quoteIdentifier('file'));
             // MySQL doesn't know deferred deletions therefore only executing the second query gives errors.
             $conn->executeUpdate('DELETE FROM Directory WHERE parentDirectory_id IS NOT NULL');
             $conn->executeUpdate('DELETE FROM Directory');
@@ -513,12 +513,12 @@ 
                 )
             );
 
-            $conn->executeUpdate('DELETE FROM ' . $platform->quoteIdentifier('quote-users-groups'));
-            $conn->executeUpdate('DELETE FROM ' . $platform->quoteIdentifier('quote-group'));
-            $conn->executeUpdate('DELETE FROM ' . $platform->quoteIdentifier('quote-phone'));
-            $conn->executeUpdate('DELETE FROM ' . $platform->quoteIdentifier('quote-user'));
-            $conn->executeUpdate('DELETE FROM ' . $platform->quoteIdentifier('quote-address'));
-            $conn->executeUpdate('DELETE FROM ' . $platform->quoteIdentifier('quote-city'));
+            $conn->executeUpdate('DELETE FROM '.$platform->quoteIdentifier('quote-users-groups'));
+            $conn->executeUpdate('DELETE FROM '.$platform->quoteIdentifier('quote-group'));
+            $conn->executeUpdate('DELETE FROM '.$platform->quoteIdentifier('quote-phone'));
+            $conn->executeUpdate('DELETE FROM '.$platform->quoteIdentifier('quote-user'));
+            $conn->executeUpdate('DELETE FROM '.$platform->quoteIdentifier('quote-address'));
+            $conn->executeUpdate('DELETE FROM '.$platform->quoteIdentifier('quote-city'));
         }
 
         if (isset($this->usedModelSets['vct_onetoone'])) {
@@ -630,7 +630,7 @@ 
         $classes = [];
 
         foreach ($classNames as $className) {
-            if (! isset(static::$entityTablesCreated[$className])) {
+            if ( ! isset(static::$entityTablesCreated[$className])) {
                 static::$entityTablesCreated[$className] = true;
                 $classes[]                               = $this->em->getClassMetadata($className);
             }
@@ -649,25 +649,25 @@ 
     {
         $this->setUpDBALTypes();
 
-        if (! isset(static::$sharedConn)) {
+        if ( ! isset(static::$sharedConn)) {
             static::$sharedConn = TestUtil::getConnection();
         }
 
         if (isset($GLOBALS['DOCTRINE_MARK_SQL_LOGS'])) {
             if (in_array(static::$sharedConn->getDatabasePlatform()->getName(), ['mysql', 'postgresql'], true)) {
-                static::$sharedConn->executeQuery('SELECT 1 /*' . get_class($this) . '*/');
+                static::$sharedConn->executeQuery('SELECT 1 /*'.get_class($this).'*/');
             } elseif (static::$sharedConn->getDatabasePlatform()->getName() === 'oracle') {
-                static::$sharedConn->executeQuery('SELECT 1 /*' . get_class($this) . '*/ FROM dual');
+                static::$sharedConn->executeQuery('SELECT 1 /*'.get_class($this).'*/ FROM dual');
             }
         }
 
-        if (! $this->em) {
+        if ( ! $this->em) {
             $this->em         = $this->getEntityManager();
             $this->schemaTool = new SchemaTool($this->em);
         }
 
         foreach ($this->usedModelSets as $setName => $bool) {
-            if (! isset(static::$tablesCreated[$setName])) {
+            if ( ! isset(static::$tablesCreated[$setName])) {
                 $this->setUpEntitySchema(static::$modelSets[$setName]);
 
                 static::$tablesCreated[$setName] = true;
@@ -742,8 +742,8 @@ 
 
         $config->setMetadataDriverImpl(
             $mappingDriver ?? $config->newDefaultAnnotationDriver([
-                realpath(__DIR__ . '/Models/Cache'),
-                realpath(__DIR__ . '/Models/GeoNames'),
+                realpath(__DIR__.'/Models/Cache'),
+                realpath(__DIR__.'/Models/GeoNames'),
             ])
         );
 
@@ -791,13 +791,13 @@ 
 
             foreach ($last25queries as $i => $query) {
                 $params = array_map(
-                    static function ($p) {
+                    static function($p) {
                         return is_object($p) ? get_class($p) : var_export($p, true);
                     },
                     $query['params'] ?: []
                 );
 
-                $queries .= $i . ". SQL: '" . $query['sql'] . "' Params: " . implode(', ', $params) . PHP_EOL;
+                $queries .= $i.". SQL: '".$query['sql']."' Params: ".implode(', ', $params).PHP_EOL;
             }
 
             $trace    = $e->getTrace();
@@ -810,11 +810,11 @@ 
                         break;
                     }
 
-                    $traceMsg .= $part['file'] . ':' . $part['line'] . PHP_EOL;
+                    $traceMsg .= $part['file'].':'.$part['line'].PHP_EOL;
                 }
             }
 
-            $message = '[' . get_class($e) . '] ' . $e->getMessage() . PHP_EOL . PHP_EOL . 'With queries:' . PHP_EOL . $queries . PHP_EOL . 'Trace:' . PHP_EOL . $traceMsg;
+            $message = '['.get_class($e).'] '.$e->getMessage().PHP_EOL.PHP_EOL.'With queries:'.PHP_EOL.$queries.PHP_EOL.'Trace:'.PHP_EOL.$traceMsg;
 
             throw new Exception($message, (int) $e->getCode(), $e);
         }