doctrine / orm

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');

tests/Doctrine/Tests/ORM/Tools/Console/Command/ConvertDoctrine1SchemaCommandTest.php

Indentation +2 added lines, -2 removed lines

@@ -17,8 +17,8 @@
 
         $output = $this->createMock(OutputInterface::class);
         $output->expects($this->once())
-               ->method('writeln')
-               ->with($this->equalTo('No Metadata Classes to process.'));
+                ->method('writeln')
+                ->with($this->equalTo('No Metadata Classes to process.'));
 
         $command->convertDoctrine1Schema([], sys_get_temp_dir(), 'annotation', 4, null, $output);
     }

tests/Doctrine/Tests/ORM/Tools/SchemaValidatorTest.php

Indentation +2 added lines, -2 removed lines

@@ -30,8 +30,8 @@
     public function testCmsModelSet(string $path)
     {
         $this->em->getConfiguration()
-                 ->getMetadataDriverImpl()
-                 ->addPaths([$path]);
+                    ->getMetadataDriverImpl()
+                    ->addPaths([$path]);
 
         self::assertEmpty($this->validator->validateMapping());
     }

Spacing +11 added lines, -11 removed lines

@@ -39,12 +39,12 @@ 
     public function modelSetProvider(): array
     {
         return [
-            'cms'        => [__DIR__ . '/../../Models/CMS'],
-            'company'    => [__DIR__ . '/../../Models/Company'],
-            'ecommerce'  => [__DIR__ . '/../../Models/ECommerce'],
-            'forum'      => [__DIR__ . '/../../Models/Forum'],
-            'navigation' => [__DIR__ . '/../../Models/Navigation'],
-            'routing'    => [__DIR__ . '/../../Models/Routing'],
+            'cms'        => [__DIR__.'/../../Models/CMS'],
+            'company'    => [__DIR__.'/../../Models/Company'],
+            'ecommerce'  => [__DIR__.'/../../Models/ECommerce'],
+            'forum'      => [__DIR__.'/../../Models/Forum'],
+            'navigation' => [__DIR__.'/../../Models/Navigation'],
+            'routing'    => [__DIR__.'/../../Models/Routing'],
         ];
     }
 
@@ -124,8 +124,8 @@ 
 
         $this->assertEquals(
             [
-                "The field Doctrine\Tests\ORM\Tools\DDC3274One#two is on the inverse side of a bi-directional " .
-                "relationship, but the specified mappedBy association on the target-entity " .
+                "The field Doctrine\Tests\ORM\Tools\DDC3274One#two is on the inverse side of a bi-directional ".
+                "relationship, but the specified mappedBy association on the target-entity ".
                 "Doctrine\Tests\ORM\Tools\DDC3274Two#one does not contain the required 'inversedBy=\"two\"' attribute."
             ],
             $ce
@@ -142,7 +142,7 @@ 
 
         $this->assertEquals(
             [
-                "The association Doctrine\Tests\ORM\Tools\DDC3322One#invalidAssoc is ordered by a foreign field " .
+                "The association Doctrine\Tests\ORM\Tools\DDC3322One#invalidAssoc is ordered by a foreign field ".
                 "invalidField that is not a field on the target entity Doctrine\Tests\ORM\Tools\DDC3322ValidEntity1."
             ],
             $ce
@@ -159,7 +159,7 @@ 
 
         $this->assertEquals(
             [
-                "The association Doctrine\Tests\ORM\Tools\DDC3322Two#invalidAssoc is ordered by a field oneToMany " .
+                "The association Doctrine\Tests\ORM\Tools\DDC3322Two#invalidAssoc is ordered by a field oneToMany ".
                 "on Doctrine\Tests\ORM\Tools\DDC3322ValidEntity1 that is a collection-valued association."
             ],
             $ce
@@ -176,7 +176,7 @@ 
 
         $this->assertEquals(
             [
-                "The association Doctrine\Tests\ORM\Tools\DDC3322Three#invalidAssoc is ordered by a field oneToOneInverse " .
+                "The association Doctrine\Tests\ORM\Tools\DDC3322Three#invalidAssoc is ordered by a field oneToOneInverse ".
                 "on Doctrine\Tests\ORM\Tools\DDC3322ValidEntity1 that is the inverse side of an association."
             ],
             $ce

tests/Doctrine/Tests/ORM/Tools/SetupTest.php

Spacing +2 added lines, -2 removed lines

@@ -37,7 +37,7 @@ 
         $loaders = spl_autoload_functions();
         $numberOfLoaders = count($loaders);
         for ($i = 0; $i < $numberOfLoaders; $i++) {
-            if ($i > $this->originalAutoloaderCount+1) {
+            if ($i > $this->originalAutoloaderCount + 1) {
                 spl_autoload_unregister($loaders[$i]);
             }
         }
@@ -45,7 +45,7 @@ 
 
     public function testDirectoryAutoload()
     {
-        Setup::registerAutoloadDirectory(__DIR__ . "/../../../../../vendor/doctrine/common/lib");
+        Setup::registerAutoloadDirectory(__DIR__."/../../../../../vendor/doctrine/common/lib");
 
         $this->assertEquals($this->originalAutoloaderCount + 2, count(spl_autoload_functions()));
     }

tests/Doctrine/Tests/ORM/Tools/EntityRepositoryGeneratorTest.php

Spacing +13 added lines, -13 removed lines

@@ -32,7 +32,7 @@ 
     public function setUp()
     {
         $this->_namespace   = uniqid('doctrine_');
-        $this->_tmpDir      = \sys_get_temp_dir() . DIRECTORY_SEPARATOR . $this->_namespace;
+        $this->_tmpDir      = \sys_get_temp_dir().DIRECTORY_SEPARATOR.$this->_namespace;
         \mkdir($this->_tmpDir);
 
         $this->_generator = new EntityGenerator();
@@ -78,7 +78,7 @@ 
         $em = $this->_getTestEntityManager();
         $ns = $this->_namespace;
 
-        $className = $ns . '\DDC3231User1Tmp';
+        $className = $ns.'\DDC3231User1Tmp';
         $this->writeEntityClass(DDC3231User1::class, $className);
 
         $rpath = $this->writeRepositoryClass($className);
@@ -90,10 +90,10 @@ 
         $repo = new \ReflectionClass($em->getRepository($className));
 
         $this->assertTrue($repo->inNamespace());
-        $this->assertSame($className . 'Repository', $repo->getName());
+        $this->assertSame($className.'Repository', $repo->getName());
         $this->assertSame(EntityRepository::class, $repo->getParentClass()->getName());
 
-        require_once __DIR__ . '/../../Models/DDC3231/DDC3231User1NoNamespace.php';
+        require_once __DIR__.'/../../Models/DDC3231/DDC3231User1NoNamespace.php';
 
         $className2 = 'DDC3231User1NoNamespaceTmp';
         $this->writeEntityClass(\DDC3231User1NoNamespace::class, $className2);
@@ -107,7 +107,7 @@ 
         $repo2 = new \ReflectionClass($em->getRepository($className2));
 
         $this->assertFalse($repo2->inNamespace());
-        $this->assertSame($className2 . 'Repository', $repo2->getName());
+        $this->assertSame($className2.'Repository', $repo2->getName());
         $this->assertSame(EntityRepository::class, $repo2->getParentClass()->getName());
     }
 
@@ -119,7 +119,7 @@ 
         $em = $this->_getTestEntityManager();
         $ns = $this->_namespace;
 
-        $className = $ns . '\DDC3231User2Tmp';
+        $className = $ns.'\DDC3231User2Tmp';
         $this->writeEntityClass(DDC3231User2::class, $className);
 
         $rpath = $this->writeRepositoryClass($className, DDC3231EntityRepository::class);
@@ -132,11 +132,11 @@ 
         $repo = new \ReflectionClass($em->getRepository($className));
 
         $this->assertTrue($repo->inNamespace());
-        $this->assertSame($className . 'Repository', $repo->getName());
+        $this->assertSame($className.'Repository', $repo->getName());
         $this->assertSame(DDC3231EntityRepository::class, $repo->getParentClass()->getName());
 
 
-        require_once __DIR__ . '/../../Models/DDC3231/DDC3231User2NoNamespace.php';
+        require_once __DIR__.'/../../Models/DDC3231/DDC3231User2NoNamespace.php';
 
         $className2 = 'DDC3231User2NoNamespaceTmp';
         $this->writeEntityClass('DDC3231User2NoNamespace', $className2);
@@ -151,7 +151,7 @@ 
         $repo2 = new \ReflectionClass($em->getRepository($className2));
 
         $this->assertFalse($repo2->inNamespace());
-        $this->assertSame($className2 . 'Repository', $repo2->getName());
+        $this->assertSame($className2.'Repository', $repo2->getName());
         $this->assertSame(DDC3231EntityRepository::class, $repo2->getParentClass()->getName());
     }
 
@@ -169,11 +169,11 @@ 
         $metadata               = $cmf->getMetadataFor($className);
         $metadata->namespace    = $this->_namespace;
         $metadata->name         = $newClassName;
-        $metadata->customRepositoryClassName = $newClassName . "Repository";
+        $metadata->customRepositoryClassName = $newClassName."Repository";
 
         $this->_generator->writeEntityClass($metadata, $this->_tmpDir);
 
-        require $this->_tmpDir . DIRECTORY_SEPARATOR . str_replace('\\', DIRECTORY_SEPARATOR, $newClassName) . ".php";
+        require $this->_tmpDir.DIRECTORY_SEPARATOR.str_replace('\\', DIRECTORY_SEPARATOR, $newClassName).".php";
     }
 
     /**
@@ -185,9 +185,9 @@ 
     {
         $this->_repositoryGenerator->setDefaultRepositoryName($defaultRepository);
 
-        $this->_repositoryGenerator->writeEntityRepositoryClass($className . 'Repository', $this->_tmpDir);
+        $this->_repositoryGenerator->writeEntityRepositoryClass($className.'Repository', $this->_tmpDir);
 
-        return $this->_tmpDir . DIRECTORY_SEPARATOR . str_replace('\\', DIRECTORY_SEPARATOR, $className) . 'Repository.php';
+        return $this->_tmpDir.DIRECTORY_SEPARATOR.str_replace('\\', DIRECTORY_SEPARATOR, $className).'Repository.php';
     }
 
 }