doctrine / orm

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");
     }
 }
 

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

tests/Doctrine/Tests/ORM/Tools/Export/YamlClassMetadataExporterTest.php

Spacing +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
 {
     protected function _getType()
     {
-        if (!class_exists('Symfony\Component\Yaml\Yaml', true)) {
+        if ( ! class_exists('Symfony\Component\Yaml\Yaml', true)) {
             $this->markTestSkipped('Please install Symfony YAML Component into the include path of your PHP installation.');
         }
 

tests/Doctrine/Tests/ORM/Tools/Export/AbstractClassMetadataExporterTest.php

Spacing +13 added lines, -13 removed lines

@@ -39,7 +39,7 @@ 
     {
         $driverMock = new DriverMock();
         $config = new Configuration();
-        $config->setProxyDir(__DIR__ . '/../../Proxies');
+        $config->setProxyDir(__DIR__.'/../../Proxies');
         $config->setProxyNamespace('Doctrine\Tests\Proxies');
         $eventManager = new EventManager();
         $conn = new ConnectionMock([], $driverMock, $config, $eventManager);
@@ -57,7 +57,7 @@ 
             'yaml'       => YamlDriver::class,
         ];
 
-        $this->assertArrayHasKey($type, $mappingDriver, "There is no metadata driver for the type '" . $type . "'.");
+        $this->assertArrayHasKey($type, $mappingDriver, "There is no metadata driver for the type '".$type."'.");
 
         $class  = $mappingDriver[$type];
         $driver = ($type === 'annotation')
@@ -80,10 +80,10 @@ 
 
     public function testExportDirectoryAndFilesAreCreated()
     {
-        $this->_deleteDirectory(__DIR__ . '/export/'.$this->_getType());
+        $this->_deleteDirectory(__DIR__.'/export/'.$this->_getType());
 
         $type = $this->_getType();
-        $metadataDriver = $this->_createMetadataDriver($type, __DIR__ . '/' . $type);
+        $metadataDriver = $this->_createMetadataDriver($type, __DIR__.'/'.$type);
         $em = $this->_createEntityManager($metadataDriver);
         $cmf = $this->_createClassMetadataFactory($em, $type);
         $metadata = $cmf->getAllMetadata();
@@ -94,7 +94,7 @@ 
 
         $type = $this->_getType();
         $cme = new ClassMetadataExporter();
-        $exporter = $cme->getExporter($type, __DIR__ . '/export/' . $type);
+        $exporter = $cme->getExporter($type, __DIR__.'/export/'.$type);
 
         if ($type === 'annotation') {
             $entityGenerator = new EntityGenerator();
@@ -109,9 +109,9 @@ 
         $exporter->export();
 
         if ($type == 'annotation') {
-            $this->assertTrue(file_exists(__DIR__ . '/export/' . $type . '/'.str_replace('\\', '/', ExportedUser::class).$this->_extension));
+            $this->assertTrue(file_exists(__DIR__.'/export/'.$type.'/'.str_replace('\\', '/', ExportedUser::class).$this->_extension));
         } else {
-            $this->assertTrue(file_exists(__DIR__ . '/export/' . $type . '/Doctrine.Tests.ORM.Tools.Export.ExportedUser'.$this->_extension));
+            $this->assertTrue(file_exists(__DIR__.'/export/'.$type.'/Doctrine.Tests.ORM.Tools.Export.ExportedUser'.$this->_extension));
         }
     }
 
@@ -122,7 +122,7 @@ 
     {
         $type = $this->_getType();
 
-        $metadataDriver = $this->_createMetadataDriver($type, __DIR__ . '/export/' . $type);
+        $metadataDriver = $this->_createMetadataDriver($type, __DIR__.'/export/'.$type);
         $em = $this->_createEntityManager($metadataDriver);
         $cmf = $this->_createClassMetadataFactory($em, $type);
         $metadata = $cmf->getAllMetadata();
@@ -208,7 +208,7 @@ 
         $type = $this->_getType();
 
         if ($type == 'xml') {
-            $xml = simplexml_load_file(__DIR__ . '/export/'.$type.'/Doctrine.Tests.ORM.Tools.Export.ExportedUser.dcm.xml');
+            $xml = simplexml_load_file(__DIR__.'/export/'.$type.'/Doctrine.Tests.ORM.Tools.Export.ExportedUser.dcm.xml');
 
             $xml->registerXPathNamespace("d", "http://doctrine-project.org/schemas/orm/doctrine-mapping");
             $nodes = $xml->xpath("/d:doctrine-mapping/d:entity/d:field[@name='name' and @type='string' and @nullable='true']");
@@ -352,7 +352,7 @@ 
         $type = $this->_getType();
 
         if ($type == 'xml') {
-            $xml = simplexml_load_file(__DIR__ . '/export/'.$type.'/Doctrine.Tests.ORM.Tools.Export.ExportedUser.dcm.xml');
+            $xml = simplexml_load_file(__DIR__.'/export/'.$type.'/Doctrine.Tests.ORM.Tools.Export.ExportedUser.dcm.xml');
 
             $xml->registerXPathNamespace("d", "http://doctrine-project.org/schemas/orm/doctrine-mapping");
             $nodes = $xml->xpath("/d:doctrine-mapping/d:entity/d:one-to-many[@field='interests']/d:cascade/d:*");
@@ -361,7 +361,7 @@ 
             $this->assertEquals('cascade-all', $nodes[0]->getName());
         } else if ($type == 'yaml') {
             $yaml = new Parser();
-            $value = $yaml->parse(file_get_contents(__DIR__ . '/export/'.$type.'/Doctrine.Tests.ORM.Tools.Export.ExportedUser.dcm.yml'));
+            $value = $yaml->parse(file_get_contents(__DIR__.'/export/'.$type.'/Doctrine.Tests.ORM.Tools.Export.ExportedUser.dcm.yml'));
 
             $this->assertTrue(isset($value[ExportedUser::class]['oneToMany']['interests']['cascade']));
             $this->assertEquals(1, count($value[ExportedUser::class]['oneToMany']['interests']['cascade']));
@@ -381,10 +381,10 @@ 
         if (is_file($path)) {
             return unlink($path);
         } else if (is_dir($path)) {
-            $files = glob(rtrim($path,'/').'/*');
+            $files = glob(rtrim($path, '/').'/*');
 
             if (is_array($files)) {
-                foreach ($files as $file){
+                foreach ($files as $file) {
                     $this->_deleteDirectory($file);
                 }
             }