doctrine / orm

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

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
      */
     protected function setUp()
     {
-        $this->visitor = new QueryExpressionVisitor(['o','p']);
+        $this->visitor = new QueryExpressionVisitor(['o', 'p']);
     }
 
     /**

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

Spacing +2 added lines, -2 removed lines

@@ -29,9 +29,9 @@
      */
     public function testAbstractSchemaNameSupportsClassnamesWithLeadingBackslash()
     {
-        $parser = $this->createParser('\\' . CmsUser::class);
+        $parser = $this->createParser('\\'.CmsUser::class);
 
-        $this->assertEquals('\\' . CmsUser::class, $parser->AbstractSchemaName());
+        $this->assertEquals('\\'.CmsUser::class, $parser->AbstractSchemaName());
     }
 
     /**

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

Indentation +36 added lines, -36 removed lines

@@ -49,7 +49,7 @@ 
             }
 
             $query->setHint(ORMQuery::HINT_FORCE_PARTIAL_LOAD, true)
-                  ->useQueryCache(false);
+                    ->useQueryCache(false);
 
             foreach ($queryHints AS $name => $value) {
                 $query->setHint($name, $value);
@@ -88,7 +88,7 @@ 
         }
 
         $query->setHint(ORMQuery::HINT_FORCE_PARTIAL_LOAD, true)
-              ->useQueryCache(false);
+                ->useQueryCache(false);
 
         foreach ($queryHints AS $name => $value) {
             $query->setHint($name, $value);
@@ -1640,9 +1640,9 @@ 
         );
     }
 
-     /**
-     * @group DDC-1430
-     */
+        /**
+         * @group DDC-1430
+         */
     public function testGroupByAllFieldsWhenObjectHasForeignKeys()
     {
         $this->assertSqlGeneration(
@@ -1982,9 +1982,9 @@ 
         );
     }
 
-   /**
-    * @group DDC-1845
-    */
+    /**
+     * @group DDC-1845
+     */
     public function testQuotedWalkJoinVariableDeclaration()
     {
         $this->assertSqlGeneration(
@@ -2018,9 +2018,9 @@ 
         );
     }
 
-   /**
-    * @group DDC-2208
-    */
+    /**
+     * @group DDC-2208
+     */
     public function testCaseThenParameterArithmeticExpression()
     {
         $this->assertSqlGeneration(
@@ -2040,8 +2040,8 @@ 
     }
 
     /**
-    * @group DDC-2268
-    */
+     * @group DDC-2268
+     */
     public function testCaseThenFunction()
     {
         $this->assertSqlGeneration(
@@ -2065,45 +2065,45 @@ 
      */
     public function testSupportsMoreThanTwoParametersInConcatFunction()
     {
-    	$connMock    = $this->_em->getConnection();
-    	$orgPlatform = $connMock->getDatabasePlatform();
+        $connMock    = $this->_em->getConnection();
+        $orgPlatform = $connMock->getDatabasePlatform();
 
-    	$connMock->setDatabasePlatform(new MySqlPlatform());
-    	$this->assertSqlGeneration(
+        $connMock->setDatabasePlatform(new MySqlPlatform());
+        $this->assertSqlGeneration(
             "SELECT u.id FROM Doctrine\Tests\Models\CMS\CmsUser u WHERE CONCAT(u.name, u.status, 's') = ?1",
             "SELECT c0_.id AS id_0 FROM cms_users c0_ WHERE CONCAT(c0_.name, c0_.status, 's') = ?"
-    	);
-    	$this->assertSqlGeneration(
+        );
+        $this->assertSqlGeneration(
             "SELECT CONCAT(u.id, u.name, u.status) FROM Doctrine\Tests\Models\CMS\CmsUser u WHERE u.id = ?1",
             "SELECT CONCAT(c0_.id, c0_.name, c0_.status) AS sclr_0 FROM cms_users c0_ WHERE c0_.id = ?"
-    	);
+        );
 
-    	$connMock->setDatabasePlatform(new PostgreSqlPlatform());
-    	$this->assertSqlGeneration(
+        $connMock->setDatabasePlatform(new PostgreSqlPlatform());
+        $this->assertSqlGeneration(
             "SELECT u.id FROM Doctrine\Tests\Models\CMS\CmsUser u WHERE CONCAT(u.name, u.status, 's') = ?1",
             "SELECT c0_.id AS id_0 FROM cms_users c0_ WHERE c0_.name || c0_.status || 's' = ?"
-    	);
-    	$this->assertSqlGeneration(
+        );
+        $this->assertSqlGeneration(
             "SELECT CONCAT(u.id, u.name, u.status) FROM Doctrine\Tests\Models\CMS\CmsUser u WHERE u.id = ?1",
             "SELECT c0_.id || c0_.name || c0_.status AS sclr_0 FROM cms_users c0_ WHERE c0_.id = ?"
-    	);
+        );
 
-    	$connMock->setDatabasePlatform(new SQLServerPlatform());
-    	$this->assertSqlGeneration(
+        $connMock->setDatabasePlatform(new SQLServerPlatform());
+        $this->assertSqlGeneration(
             "SELECT u.id FROM Doctrine\Tests\Models\CMS\CmsUser u WHERE CONCAT(u.name, u.status, 's') = ?1",
             "SELECT c0_.id AS id_0 FROM cms_users c0_ WHERE (c0_.name + c0_.status + 's') = ?"
-    	);
-    	$this->assertSqlGeneration(
+        );
+        $this->assertSqlGeneration(
             "SELECT CONCAT(u.id, u.name, u.status) FROM Doctrine\Tests\Models\CMS\CmsUser u WHERE u.id = ?1",
             "SELECT (c0_.id + c0_.name + c0_.status) AS sclr_0 FROM cms_users c0_ WHERE c0_.id = ?"
-    	);
+        );
 
-    	$connMock->setDatabasePlatform($orgPlatform);
+        $connMock->setDatabasePlatform($orgPlatform);
     }
 
-     /**
-     * @group DDC-2188
-     */
+        /**
+         * @group DDC-2188
+         */
     public function testArithmeticPriority()
     {
         $this->assertSqlGeneration(
@@ -2123,8 +2123,8 @@ 
     }
 
     /**
-    * @group DDC-2475
-    */
+     * @group DDC-2475
+     */
     public function testOrderByClauseShouldReplaceOrderByRelationMapping()
     {
         $this->assertSqlGeneration(

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());
         }
     }
 
@@ -1597,7 +1597,7 @@ 
     public function testAliasDoesNotExceedPlatformDefinedLength()
     {
         $this->assertSqlGeneration(
-            'SELECT m FROM ' . __NAMESPACE__ .  '\\DDC1384Model m',
+            'SELECT m FROM '.__NAMESPACE__.'\\DDC1384Model m',
             "SELECT d0_.aVeryLongIdentifierThatShouldBeShortenedByTheSQLWalker_fooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooo AS ooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooo_0 FROM DDC1384Model d0_"
         );
     }
@@ -1630,12 +1630,12 @@ 
     public function testSelectWithArithmeticExpressionBeforeField()
     {
         $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 -d0_.value AS sclr_0, d0_.id AS id_1 FROM DDC1474Entity d0_'
         );
 
         $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 d0_.id AS id_0, +d0_.value AS sclr_1 FROM DDC1474Entity d0_'
         );
     }
@@ -2297,8 +2297,8 @@ 
     public function testHavingRegressionUsingVariableWithMathOperatorsExpression($operator)
     {
         $this->assertSqlGeneration(
-            'SELECT COUNT(u.name) AS countName FROM Doctrine\Tests\Models\CMS\CmsUser u HAVING 1 ' . $operator . ' countName > 0',
-            'SELECT COUNT(c0_.name) AS sclr_0 FROM cms_users c0_ HAVING 1 ' . $operator . ' sclr_0 > 0'
+            'SELECT COUNT(u.name) AS countName FROM Doctrine\Tests\Models\CMS\CmsUser u HAVING 1 '.$operator.' countName > 0',
+            'SELECT COUNT(c0_.name) AS sclr_0 FROM cms_users c0_ HAVING 1 '.$operator.' sclr_0 > 0'
         );
     }
 
@@ -2320,7 +2320,7 @@ 
      */
     public function getSql(SqlWalker $sqlWalker)
     {
-        return 'ABS(' . $sqlWalker->walkSimpleArithmeticExpression($this->simpleArithmeticExpression) . ')';
+        return 'ABS('.$sqlWalker->walkSimpleArithmeticExpression($this->simpleArithmeticExpression).')';
     }
 
     /**

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

Indentation +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
         try {
             $query = $this->em->createQuery($dqlToBeTested);
             $query->setHint(Query::HINT_CUSTOM_TREE_WALKERS, [CustomTreeWalkerJoin::class])
-                  ->useQueryCache(false);
+                    ->useQueryCache(false);
 
             $this->assertEquals($sqlToBeConfirmed, $query->getSql());
             $query->free();

Spacing +4 added lines, -4 removed lines

@@ -33,7 +33,7 @@ 
             $this->assertEquals($sqlToBeConfirmed, $query->getSql());
             $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());
 
         }
     }
@@ -74,9 +74,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;
@@ -85,7 +85,7 @@ 
         $userMetadata    = $entityManager->getClassMetadata(CmsUser::class);
         $addressMetadata = $entityManager->getClassMetadata(CmsAddress::class);
 
-        $this->setQueryComponent($rangeVariableDecl->aliasIdentificationVariable . 'a',
+        $this->setQueryComponent($rangeVariableDecl->aliasIdentificationVariable.'a',
             [
                 'metadata'     => $addressMetadata,
                 'parent'       => $rangeVariableDecl->aliasIdentificationVariable,

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

Indentation +1 added lines, -1 removed lines

@@ -231,7 +231,7 @@
      */
     public function testLiteralExprProperlyQuotesStrings()
     {
-       $this->assertEquals("'00010001'", (string) $this->_expr->literal('00010001'));
+        $this->assertEquals("'00010001'", (string) $this->_expr->literal('00010001'));
     }
 
     public function testGreaterThanOrEqualToExpr()

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

Spacing +4 added lines, -4 removed lines

@@ -43,7 +43,7 @@ 
         try {
             $this->assertEquals($sqlToBeConfirmed, $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());
         }
     }
 
@@ -201,9 +201,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;
@@ -212,7 +212,7 @@ 
         $userMetadata    = $entityManager->getClassMetadata(CmsUser::class);
         $addressMetadata = $entityManager->getClassMetadata(CmsAddress::class);
 
-        $this->setQueryComponent($rangeVariableDecl->aliasIdentificationVariable . 'a',
+        $this->setQueryComponent($rangeVariableDecl->aliasIdentificationVariable.'a',
             [
                 'metadata'     => $addressMetadata,
                 'parent'       => $rangeVariableDecl->aliasIdentificationVariable,

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

Spacing +1 added lines, -1 removed lines

@@ -86,6 +86,6 @@
     public function addFilterConstraint(ClassMetadata $targetEntity, $targetTableAlias)
     {
         // getParameter applies quoting automatically
-        return $targetTableAlias . '.id = ' . $this->getParameter('id');
+        return $targetTableAlias.'.id = '.$this->getParameter('id');
     }
 }

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

Indentation +1 added lines, -1 removed lines

@@ -56,7 +56,7 @@
         $query->setDQL($dql);
 
         foreach ($hints as $key => $value) {
-        	$query->setHint($key, $value);
+            $query->setHint($key, $value);
         }
 
         $parser = new Query\Parser($query);

Spacing +8 added lines, -8 removed lines

@@ -27,7 +27,7 @@ 
             $this->addToAssertionCount(1);
         } catch (QueryException $e) {
             if ($debug) {
-                echo $e->getTraceAsString() . PHP_EOL;
+                echo $e->getTraceAsString().PHP_EOL;
             }
 
             $this->fail($e->getMessage());
@@ -42,8 +42,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);
         }
@@ -112,7 +112,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 with namespace
             ['SELECT u FROM \Unknown\Class u'], // unknown, leading backslash
             ['SELECT u FROM Unknown\\\\Class u'], // unknown, syntactically bogus (duplicate \\)
@@ -585,7 +585,7 @@ 
      */
     public function testDQLKeywordInJoinIsAllowed()
     {
-        $this->assertValidDQL('SELECT u FROM ' . __NAMESPACE__ . '\DQLKeywordsModelUser u JOIN u.group g');
+        $this->assertValidDQL('SELECT u FROM '.__NAMESPACE__.'\DQLKeywordsModelUser u JOIN u.group g');
     }
 
     /**
@@ -593,7 +593,7 @@ 
      */
     public function testDQLKeywordInConditionIsAllowed()
     {
-        $this->assertValidDQL('SELECT g FROM ' . __NAMESPACE__ . '\DQLKeywordsModelGroup g WHERE g.from=0');
+        $this->assertValidDQL('SELECT g FROM '.__NAMESPACE__.'\DQLKeywordsModelGroup g WHERE g.from=0');
     }
 
     /* The exception is currently thrown in the SQLWalker, not earlier.
@@ -698,7 +698,7 @@ 
      */
     public function testNewLiteralExpression()
     {
-        $this->assertValidDQL("SELECT new " . __NAMESPACE__ . "\\DummyStruct(u.id, 'foo', 1, true) FROM Doctrine\Tests\Models\CMS\CmsUser u");
+        $this->assertValidDQL("SELECT new ".__NAMESPACE__."\\DummyStruct(u.id, 'foo', 1, true) FROM Doctrine\Tests\Models\CMS\CmsUser u");
     }
 
     /**
@@ -706,7 +706,7 @@ 
      */
     public function testNewLiteralWithSubselectExpression()
     {
-        $this->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");
+        $this->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");
     }
 
     public function testStringPrimaryAcceptsAggregateExpression() : void

tests/Doctrine/Tests/ORM/UnitOfWorkTest.php

Spacing +5 added lines, -5 removed lines

@@ -425,7 +425,7 @@ 
             'empty string, single field'     => [$emptyString, ''],
             'non-empty string, single field' => [$nonEmptyString, $nonEmptyString->id],
             'empty strings, two fields'      => [$emptyStrings, ' '],
-            'non-empty strings, two fields'  => [$nonEmptyStrings, $nonEmptyStrings->id1 . ' ' . $nonEmptyStrings->id2],
+            'non-empty strings, two fields'  => [$nonEmptyStrings, $nonEmptyStrings->id1.' '.$nonEmptyStrings->id2],
             'boolean true'                   => [$booleanTrue, '1'],
             'boolean false'                  => [$booleanFalse, ''],
         ];
@@ -456,7 +456,7 @@ 
 
     public function entitiesWithInvalidIdentifiersProvider()
     {
-        $firstNullString  = new EntityWithCompositeStringIdentifier();
+        $firstNullString = new EntityWithCompositeStringIdentifier();
 
         $firstNullString->id2 = uniqid('id2', true);
 
@@ -515,7 +515,7 @@ 
             ->eventManager
             ->expects(self::any())
             ->method('hasListeners')
-            ->willReturnCallback(function ($eventName) {
+            ->willReturnCallback(function($eventName) {
                 return $eventName === Events::prePersist;
             });
         $this
@@ -524,7 +524,7 @@ 
             ->method('dispatchEvent')
             ->with(
                 self::anything(),
-                self::callback(function (LifecycleEventArgs $args) use ($entity, $generatedFieldValue) {
+                self::callback(function(LifecycleEventArgs $args) use ($entity, $generatedFieldValue) {
                     /* @var $object EntityWithRandomlyGeneratedField */
                     $object = $args->getObject();
 
@@ -564,7 +564,7 @@ 
             ->eventManager
             ->expects(self::any())
             ->method('hasListeners')
-            ->willReturnCallback(function ($eventName) {
+            ->willReturnCallback(function($eventName) {
                 return $eventName === Events::prePersist;
             });
         $this->eventManager->expects(self::never())->method('dispatchEvent');