doctrine / orm

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

Spacing +3 added lines, -3 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\Tests\ORM\Functional\Ticket;
 
@@ -29,7 +29,7 @@ 
             $entity = new DDC1526Menu;
 
             if (isset($parents[($i % 3)])) {
-                $entity->parent = $parents[($i%3)];
+                $entity->parent = $parents[($i % 3)];
             }
 
             $this->em->persist($entity);
@@ -39,7 +39,7 @@ 
         $this->em->clear();
 
         $dql   = 'SELECT m, c
-            FROM ' . __NAMESPACE__ . '\DDC1526Menu m
+            FROM ' . __NAMESPACE__.'\DDC1526Menu m
             LEFT JOIN m.children c';
         $menus = $this->em->createQuery($dql)->getResult();
 

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

Spacing +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\Tests\ORM\Functional\Ticket;
 
@@ -25,7 +25,7 @@ 
 
         // Show difference between expected and actual queries on error
         self::assertEquals(
-            'SELECT _a FROM ' . __NAMESPACE__ . '\DDC1757A _a, ' . __NAMESPACE__ . '\DDC1757B _b INNER JOIN _b.c _c INNER JOIN _c.d _d',
+            'SELECT _a FROM '.__NAMESPACE__.'\DDC1757A _a, '.__NAMESPACE__.'\DDC1757B _b INNER JOIN _b.c _c INNER JOIN _c.d _d',
             $dql,
             'Wrong DQL query'
         );

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

Spacing +3 added lines, -3 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\Tests\ORM\Functional;
 
@@ -56,7 +56,7 @@ 
         $driver = new DatabaseDriver($sm);
 
         foreach ($driver->getAllClassNames() as $className) {
-            if (! in_array(strtolower($className), $classNames, true)) {
+            if ( ! in_array(strtolower($className), $classNames, true)) {
                 continue;
             }
 
@@ -68,7 +68,7 @@ 
         }
 
         if (count($metadatas) !== count($classNames)) {
-            $this->fail("Have not found all classes matching the names '" . implode(', ', $classNames) . "' only tables " . implode(', ', array_keys($metadatas)));
+            $this->fail("Have not found all classes matching the names '".implode(', ', $classNames)."' only tables ".implode(', ', array_keys($metadatas)));
         }
         return $metadatas;
     }

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

Spacing +8 added lines, -8 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\Tests\ORM\Functional;
 
@@ -30,7 +30,7 @@ 
      */
     public function testIssue2059()
     {
-        if (! $this->em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
+        if ( ! $this->em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
             $this->markTestSkipped('Platform does not support foreign keys.');
         }
 
@@ -52,7 +52,7 @@ 
 
     public function testLoadMetadataFromDatabase()
     {
-        if (! $this->em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
+        if ( ! $this->em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
             $this->markTestSkipped('Platform does not support foreign keys.');
         }
 
@@ -90,7 +90,7 @@ 
 
     public function testLoadMetadataWithForeignKeyFromDatabase()
     {
-        if (! $this->em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
+        if ( ! $this->em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
             $this->markTestSkipped('Platform does not support foreign keys.');
         }
 
@@ -123,7 +123,7 @@ 
 
     public function testDetectManyToManyTables()
     {
-        if (! $this->em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
+        if ( ! $this->em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
             $this->markTestSkipped('Platform does not support foreign keys.');
         }
 
@@ -163,7 +163,7 @@ 
 
     public function testLoadMetadataFromDatabaseDetail()
     {
-        if (! $this->em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
+        if ( ! $this->em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
             $this->markTestSkipped('Platform does not support foreign keys.');
         }
 
@@ -202,7 +202,7 @@ 
 
         // FIXME: Condition here is fugly.
         // NOTE: PostgreSQL and SQL SERVER do not support UNSIGNED integer
-        if (! $this->em->getConnection()->getDatabasePlatform() instanceof PostgreSqlPlatform &&
+        if ( ! $this->em->getConnection()->getDatabasePlatform() instanceof PostgreSqlPlatform &&
              ! $this->em->getConnection()->getDatabasePlatform() instanceof SQLServerPlatform) {
             self::assertNotNull($metadata->getProperty('columnUnsigned'));
 
@@ -244,7 +244,7 @@ 
 
         self::assertNotEmpty($indexes['index1']['columns']);
         self::assertEquals(
-            ['column_index1','column_index2'],
+            ['column_index1', 'column_index2'],
             $indexes['index1']['columns']
         );
 

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

Spacing +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\Tests\ORM\Functional;
 
@@ -38,7 +38,7 @@ 
 
     public function assertCollectionEquals(Collection $first, Collection $second)
     {
-        return $first->forAll(function ($k, $e) use ($second) {
+        return $first->forAll(function($k, $e) use ($second) {
             return $second->contains($e);
         });
     }

tests/Doctrine/Tests/ORM/Functional/Locking/GearmanLockTest.php

Spacing +6 added lines, -6 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\Tests\ORM\Functional\Locking;
 
@@ -19,7 +19,7 @@ 
 
     protected function setUp()
     {
-        if (! class_exists('GearmanClient', false)) {
+        if ( ! class_exists('GearmanClient', false)) {
             $this->markTestSkipped('pecl/gearman is required for this test to run.');
         }
 
@@ -136,13 +136,13 @@ 
 
         self::assertTrue(
             $this->maxRunTime > $forTime,
-            'Because of locking this tests should have run at least ' . $forTime . ' seconds, ' .
-            'but only did for ' . $this->maxRunTime . ' seconds.'
+            'Because of locking this tests should have run at least '.$forTime.' seconds, '.
+            'but only did for '.$this->maxRunTime.' seconds.'
         );
         self::assertTrue(
             $this->maxRunTime < $notLongerThan,
-            'The longest task should not run longer than ' . $notLongerThan . ' seconds, ' .
-            'but did for ' . $this->maxRunTime . ' seconds.'
+            'The longest task should not run longer than '.$notLongerThan.' seconds, '.
+            'but did for '.$this->maxRunTime.' seconds.'
         );
     }
 

tests/Doctrine/Tests/ORM/Functional/Locking/LockAgentWorker.php

Spacing +10 added lines, -10 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\Tests\ORM\Functional\Locking;
 
@@ -28,7 +28,7 @@ 
 
         while ($worker->work()) {
             if ($worker->returnCode() !== GEARMAN_SUCCESS) {
-                echo 'return_code: ' . $worker->returnCode() . "\n";
+                echo 'return_code: '.$worker->returnCode()."\n";
                 break;
             }
         }
@@ -53,14 +53,14 @@ 
 
     public function findWithLock($job)
     {
-        return $this->process($job, function ($fixture, $em) {
+        return $this->process($job, function($fixture, $em) {
             $entity = $em->find($fixture['entityName'], $fixture['entityId'], $fixture['lockMode']);
         });
     }
 
     public function dqlWithLock($job)
     {
-        return $this->process($job, function ($fixture, $em) {
+        return $this->process($job, function($fixture, $em) {
             /* @var $query Doctrine\ORM\Query */
             $query = $em->createQuery($fixture['dql']);
             $query->setLockMode($fixture['lockMode']);
@@ -71,7 +71,7 @@ 
 
     public function lock($job)
     {
-        return $this->process($job, function ($fixture, $em) {
+        return $this->process($job, function($fixture, $em) {
             $entity = $em->find($fixture['entityName'], $fixture['entityId']);
             $em->lock($entity, $fixture['lockMode']);
         });
@@ -79,18 +79,18 @@ 
 
     protected function processWorkload($job)
     {
-        echo 'Received job: ' . $job->handle() . ' for function ' . $job->functionName() . "\n";
+        echo 'Received job: '.$job->handle().' for function '.$job->functionName()."\n";
 
         $workload = $job->workload();
         $workload = unserialize($workload);
 
-        if (! isset($workload['conn']) || ! is_array($workload['conn'])) {
+        if ( ! isset($workload['conn']) || ! is_array($workload['conn'])) {
             throw new \InvalidArgumentException('Missing Database parameters');
         }
 
         $this->em = $this->createEntityManager($workload['conn']);
 
-        if (! isset($workload['fixture'])) {
+        if ( ! isset($workload['fixture'])) {
             throw new \InvalidArgumentException('Missing Fixture parameters');
         }
         return $workload['fixture'];
@@ -99,11 +99,11 @@ 
     protected function createEntityManager($conn)
     {
         $config = new Configuration();
-        $config->setProxyDir(__DIR__ . '/../../../Proxies');
+        $config->setProxyDir(__DIR__.'/../../../Proxies');
         $config->setProxyNamespace('MyProject\Proxies');
         $config->setAutoGenerateProxyClasses(true);
 
-        $annotDriver = $config->newDefaultAnnotationDriver([__DIR__ . '/../../../Models/']);
+        $annotDriver = $config->newDefaultAnnotationDriver([__DIR__.'/../../../Models/']);
         $config->setMetadataDriverImpl($annotDriver);
 
         $cache = new ArrayCache();

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

Indentation +10 added lines, -10 removed lines

@@ -136,24 +136,24 @@
                     ->will($this->returnValue(10));
 
         $parserResultMock = $this->getMockBuilder(ParserResult::class)
-                                 ->setMethods(['getSqlExecutor'])
-                                 ->getMock();
+                                    ->setMethods(['getSqlExecutor'])
+                                    ->getMock();
         $parserResultMock->expects($this->once())
-                         ->method('getSqlExecutor')
-                         ->will($this->returnValue($sqlExecMock));
+                            ->method('getSqlExecutor')
+                            ->will($this->returnValue($sqlExecMock));
 
         $cache = $this->getMockBuilder(CacheProvider::class)
-                      ->setMethods(['doFetch', 'doContains', 'doSave', 'doDelete', 'doFlush', 'doGetStats'])
-                      ->getMock();
+                        ->setMethods(['doFetch', 'doContains', 'doSave', 'doDelete', 'doFlush', 'doGetStats'])
+                        ->getMock();
 
         $cache->expects($this->at(0))->method('doFetch')->will($this->returnValue(1));
         $cache->expects($this->at(1))
-              ->method('doFetch')
-              ->with($this->isType('string'))
-              ->will($this->returnValue($parserResultMock));
+                ->method('doFetch')
+                ->with($this->isType('string'))
+                ->will($this->returnValue($parserResultMock));
 
         $cache->expects($this->never())
-              ->method('doSave');
+                ->method('doSave');
 
         $query->setQueryCacheDriver($cache);
 

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

Spacing +15 added lines, -15 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\Tests\ORM\Functional;
 
@@ -131,7 +131,7 @@ 
         $iter = $paginator->getIterator();
         self::assertCount(9, $iter);
         $result = iterator_to_array($iter);
-        self::assertEquals($checkField . '0', $result[0]->{$checkField});
+        self::assertEquals($checkField.'0', $result[0]->{$checkField});
     }
 
     private function iterateWithOrderAscWithLimit($useOutputWalkers, $fetchJoinCollection, $baseDql, $checkField)
@@ -147,7 +147,7 @@ 
         $iter = $paginator->getIterator();
         self::assertCount(3, $iter);
         $result = iterator_to_array($iter);
-        self::assertEquals($checkField . '0', $result[0]->{$checkField});
+        self::assertEquals($checkField.'0', $result[0]->{$checkField});
     }
 
     private function iterateWithOrderAscWithLimitAndOffset($useOutputWalkers, $fetchJoinCollection, $baseDql, $checkField)
@@ -163,7 +163,7 @@ 
         $iter = $paginator->getIterator();
         self::assertCount(3, $iter);
         $result = iterator_to_array($iter);
-        self::assertEquals($checkField . '3', $result[0]->{$checkField});
+        self::assertEquals($checkField.'3', $result[0]->{$checkField});
     }
 
     private function iterateWithOrderDesc($useOutputWalkers, $fetchJoinCollection, $baseDql, $checkField)
@@ -176,7 +176,7 @@ 
         $iter = $paginator->getIterator();
         self::assertCount(9, $iter);
         $result = iterator_to_array($iter);
-        self::assertEquals($checkField . '8', $result[0]->{$checkField});
+        self::assertEquals($checkField.'8', $result[0]->{$checkField});
     }
 
     private function iterateWithOrderDescWithLimit($useOutputWalkers, $fetchJoinCollection, $baseDql, $checkField)
@@ -191,7 +191,7 @@ 
         $iter = $paginator->getIterator();
         self::assertCount(3, $iter);
         $result = iterator_to_array($iter);
-        self::assertEquals($checkField . '8', $result[0]->{$checkField});
+        self::assertEquals($checkField.'8', $result[0]->{$checkField});
     }
 
     private function iterateWithOrderDescWithLimitAndOffset($useOutputWalkers, $fetchJoinCollection, $baseDql, $checkField)
@@ -206,7 +206,7 @@ 
         $iter = $paginator->getIterator();
         self::assertCount(3, $iter);
         $result = iterator_to_array($iter);
-        self::assertEquals($checkField . '5', $result[0]->{$checkField});
+        self::assertEquals($checkField.'5', $result[0]->{$checkField});
     }
 
     /**
@@ -479,8 +479,8 @@ 
     public function testIterateWithFetchJoinOneToManyWithOrderByColumnFromBoth($useOutputWalkers)
     {
         $dql     = 'SELECT c, d FROM Doctrine\Tests\Models\Pagination\Company c JOIN c.departments d ORDER BY c.name';
-        $dqlAsc  = $dql . ' ASC, d.name';
-        $dqlDesc = $dql . ' DESC, d.name';
+        $dqlAsc  = $dql.' ASC, d.name';
+        $dqlDesc = $dql.' DESC, d.name';
         $this->iterateWithOrderAsc($useOutputWalkers, true, $dqlAsc, 'name');
         $this->iterateWithOrderDesc($useOutputWalkers, true, $dqlDesc, 'name');
     }
@@ -488,8 +488,8 @@ 
     public function testIterateWithFetchJoinOneToManyWithOrderByColumnFromBothWithLimitWithOutputWalker()
     {
         $dql     = 'SELECT c, d FROM Doctrine\Tests\Models\Pagination\Company c JOIN c.departments d ORDER BY c.name';
-        $dqlAsc  = $dql . ' ASC, d.name';
-        $dqlDesc = $dql . ' DESC, d.name';
+        $dqlAsc  = $dql.' ASC, d.name';
+        $dqlDesc = $dql.' DESC, d.name';
         $this->iterateWithOrderAscWithLimit(true, true, $dqlAsc, 'name');
         $this->iterateWithOrderDescWithLimit(true, true, $dqlDesc, 'name');
     }
@@ -500,8 +500,8 @@ 
         $this->expectExceptionMessage('Cannot select distinct identifiers from query with LIMIT and ORDER BY on a column from a fetch joined to-many association. Use output walkers.');
 
         $dql     = 'SELECT c, d FROM Doctrine\Tests\Models\Pagination\Company c JOIN c.departments d ORDER BY c.name';
-        $dqlAsc  = $dql . ' ASC, d.name';
-        $dqlDesc = $dql . ' DESC, d.name';
+        $dqlAsc  = $dql.' ASC, d.name';
+        $dqlDesc = $dql.' DESC, d.name';
         $this->iterateWithOrderAscWithLimit(false, true, $dqlAsc, 'name');
         $this->iterateWithOrderDescWithLimit(false, true, $dqlDesc, 'name');
     }
@@ -585,7 +585,7 @@ 
         $users = iterator_to_array($paginator->getIterator());
         self::assertCount(9, $users);
         foreach ($users as $i => $user) {
-            self::assertEquals('username' . (8 - $i), $user->username);
+            self::assertEquals('username'.(8 - $i), $user->username);
         }
     }
 
@@ -735,7 +735,7 @@ 
             $company->logo->image_width  = 100 + $i;
             $company->logo->image_height = 100 + $i;
             $company->logo->company      = $company;
-            for ($j=0; $j<3; $j++) {
+            for ($j = 0; $j < 3; $j++) {
                 $department             = new Department();
                 $department->name       = "name$i$j";
                 $department->company    = $company;