doctrine / orm

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

Spacing +3 added lines, -3 removed lines

@@ -12,7 +12,7 @@ 
 use Doctrine\Tests\Models\CMS\CmsUser;
 use Doctrine\Tests\OrmFunctionalTestCase;
 
-require_once __DIR__ . '/../../TestInit.php';
+require_once __DIR__.'/../../TestInit.php';
 
 class CustomFunctionsTest extends OrmFunctionalTestCase
 {
@@ -32,10 +32,10 @@ 
         $this->em->flush();
 
         // Instead of defining the function with the class name, we use a callback
-        $this->em->getConfiguration()->addCustomStringFunction('FOO', function ($funcName) {
+        $this->em->getConfiguration()->addCustomStringFunction('FOO', function($funcName) {
             return new NoOp($funcName);
         });
-        $this->em->getConfiguration()->addCustomNumericFunction('BAR', function ($funcName) {
+        $this->em->getConfiguration()->addCustomNumericFunction('BAR', function($funcName) {
             return new NoOp($funcName);
         });
 

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

Spacing +14 added lines, -14 removed lines

@@ -134,7 +134,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)
@@ -150,7 +150,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)
@@ -166,7 +166,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)
@@ -179,7 +179,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)
@@ -194,7 +194,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)
@@ -209,7 +209,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});
     }
 
     /**
@@ -482,8 +482,8 @@ 
     public function testIterateWithFetchJoinOneToManyWithOrderByColumnFromBoth($useOutputWalkers) : void
     {
         $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');
     }
@@ -491,8 +491,8 @@ 
     public function testIterateWithFetchJoinOneToManyWithOrderByColumnFromBothWithLimitWithOutputWalker() : void
     {
         $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');
     }
@@ -503,8 +503,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');
     }
@@ -588,7 +588,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);
         }
     }
 
@@ -738,7 +738,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       = sprintf('name%d%d', $i, $j);
                 $department->company    = $company;

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

Spacing +5 added lines, -5 removed lines

@@ -47,7 +47,7 @@ 
         $entity = $this->em->find('\Doctrine\Tests\Models\CustomType\CustomTypeUpperCase', $id);
 
         self::assertEquals('foo', $entity->lowerCaseString, 'Entity holds lowercase string');
-        self::assertEquals('FOO', $this->em->getConnection()->fetchColumn('select lowerCaseString from customtype_uppercases where id=' . $entity->id . ''), 'Database holds uppercase string');
+        self::assertEquals('FOO', $this->em->getConnection()->fetchColumn('select lowerCaseString from customtype_uppercases where id='.$entity->id.''), 'Database holds uppercase string');
     }
 
     /**
@@ -68,7 +68,7 @@ 
 
         $entity = $this->em->find('\Doctrine\Tests\Models\CustomType\CustomTypeUpperCase', $id);
         self::assertEquals('foo', $entity->namedLowerCaseString, 'Entity holds lowercase string');
-        self::assertEquals('FOO', $this->em->getConnection()->fetchColumn('select named_lower_case_string from customtype_uppercases where id=' . $entity->id . ''), 'Database holds uppercase string');
+        self::assertEquals('FOO', $this->em->getConnection()->fetchColumn('select named_lower_case_string from customtype_uppercases where id='.$entity->id.''), 'Database holds uppercase string');
 
         $entity->namedLowerCaseString = 'bar';
 
@@ -81,7 +81,7 @@ 
 
         $entity = $this->em->find('\Doctrine\Tests\Models\CustomType\CustomTypeUpperCase', $id);
         self::assertEquals('bar', $entity->namedLowerCaseString, 'Entity holds lowercase string');
-        self::assertEquals('BAR', $this->em->getConnection()->fetchColumn('select named_lower_case_string from customtype_uppercases where id=' . $entity->id . ''), 'Database holds uppercase string');
+        self::assertEquals('BAR', $this->em->getConnection()->fetchColumn('select named_lower_case_string from customtype_uppercases where id='.$entity->id.''), 'Database holds uppercase string');
     }
 
     public function testTypeValueSqlWithAssociations() : void
@@ -108,7 +108,7 @@ 
         $entity = $this->em->find(CustomTypeParent::class, $parentId);
 
         self::assertTrue($entity->customInteger < 0, 'Fetched customInteger negative');
-        self::assertEquals(1, $this->em->getConnection()->fetchColumn('select customInteger from customtype_parents where id=' . $entity->id . ''), 'Database has stored customInteger positive');
+        self::assertEquals(1, $this->em->getConnection()->fetchColumn('select customInteger from customtype_parents where id='.$entity->id.''), 'Database has stored customInteger positive');
 
         self::assertNotNull($parent->child, 'Child attached');
         self::assertCount(2, $entity->getMyFriends(), '2 friends attached');
@@ -127,7 +127,7 @@ 
 
         $this->em->clear();
 
-        $query = $this->em->createQuery('SELECT p, p.customInteger, c from Doctrine\Tests\Models\CustomType\CustomTypeParent p JOIN p.child c where p.id = ' . $parentId);
+        $query = $this->em->createQuery('SELECT p, p.customInteger, c from Doctrine\Tests\Models\CustomType\CustomTypeParent p JOIN p.child c where p.id = '.$parentId);
 
         $result = $query->getResult();
 

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

Spacing +2 added lines, -2 removed lines

@@ -62,7 +62,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;
             }
 
@@ -74,7 +74,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/SecondLevelCacheQueryCacheTest.php

Spacing +1 added lines, -1 removed lines

@@ -804,7 +804,7 @@
         $this->secondLevelCacheLogger->clearStats();
         $this->em->clear();
 
-        $getHash = function (AbstractQuery $query) {
+        $getHash = function(AbstractQuery $query) {
             $method = new \ReflectionMethod($query, 'getHash');
             $method->setAccessible(true);
 

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

Spacing +5 added lines, -5 removed lines

@@ -233,7 +233,7 @@ 
 
         $contracts = $this->em->createQuery('SELECT c FROM Doctrine\Tests\Models\Company\CompanyContract c ORDER BY c.id')->getScalarResult();
 
-        $discrValues = array_map(function ($a) {
+        $discrValues = array_map(function($a) {
             return $a['c_discr'];
         }, $contracts);
 
@@ -335,19 +335,19 @@ 
 
         $repos     = $this->em->getRepository(CompanyContract::class);
         $contracts = $repos->findBy(['salesPerson' => $this->salesPerson->getId()]);
-        self::assertCount(3, $contracts, 'There should be 3 entities related to ' . $this->salesPerson->getId() . " for 'Doctrine\Tests\Models\Company\CompanyContract'");
+        self::assertCount(3, $contracts, 'There should be 3 entities related to '.$this->salesPerson->getId()." for 'Doctrine\Tests\Models\Company\CompanyContract'");
 
         $repos     = $this->em->getRepository(CompanyFixContract::class);
         $contracts = $repos->findBy(['salesPerson' => $this->salesPerson->getId()]);
-        self::assertCount(1, $contracts, 'There should be 1 entities related to ' . $this->salesPerson->getId() . " for 'Doctrine\Tests\Models\Company\CompanyFixContract'");
+        self::assertCount(1, $contracts, 'There should be 1 entities related to '.$this->salesPerson->getId()." for 'Doctrine\Tests\Models\Company\CompanyFixContract'");
 
         $repos     = $this->em->getRepository(CompanyFlexContract::class);
         $contracts = $repos->findBy(['salesPerson' => $this->salesPerson->getId()]);
-        self::assertCount(2, $contracts, 'There should be 2 entities related to ' . $this->salesPerson->getId() . " for 'Doctrine\Tests\Models\Company\CompanyFlexContract'");
+        self::assertCount(2, $contracts, 'There should be 2 entities related to '.$this->salesPerson->getId()." for 'Doctrine\Tests\Models\Company\CompanyFlexContract'");
 
         $repos     = $this->em->getRepository(CompanyFlexUltraContract::class);
         $contracts = $repos->findBy(['salesPerson' => $this->salesPerson->getId()]);
-        self::assertCount(1, $contracts, 'There should be 1 entities related to ' . $this->salesPerson->getId() . " for 'Doctrine\Tests\Models\Company\CompanyFlexUltraContract'");
+        self::assertCount(1, $contracts, 'There should be 1 entities related to '.$this->salesPerson->getId()." for 'Doctrine\Tests\Models\Company\CompanyFlexUltraContract'");
     }
 
     /**

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

Spacing +5 added lines, -5 removed lines

@@ -23,7 +23,7 @@ 
 
     protected function setUp() : void
     {
-        if (! class_exists('GearmanClient', false)) {
+        if ( ! class_exists('GearmanClient', false)) {
             $this->markTestSkipped('pecl/gearman is required for this test to run.');
         }
 
@@ -140,13 +140,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/OptimisticTest.php

Spacing +2 added lines, -2 removed lines

@@ -120,7 +120,7 @@ 
         $test = new OptimisticStandard();
 
         for ($i = 0; $i < 5; $i++) {
-            $test->name = 'test' . $i;
+            $test->name = 'test'.$i;
 
             $this->em->persist($test);
             $this->em->flush();
@@ -252,7 +252,7 @@ 
         $caughtException = null;
 
         try {
-            $expectedVersionExpired = DateTime::createFromFormat('U', (string) ($test->version->getTimestamp()-3600));
+            $expectedVersionExpired = DateTime::createFromFormat('U', (string) ($test->version->getTimestamp() - 3600));
 
             $this->em->lock($test, LockMode::OPTIMISTIC, $expectedVersionExpired);
         } catch (OptimisticLockException $e) {

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

Spacing +9 added lines, -9 removed lines

@@ -33,7 +33,7 @@ 
 
         while ($worker->work()) {
             if ($worker->returnCode() !== GEARMAN_SUCCESS) {
-                echo 'return_code: ' . $worker->returnCode() . "\n";
+                echo 'return_code: '.$worker->returnCode()."\n";
                 break;
             }
         }
@@ -58,14 +58,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 Doctrine\ORM\Query $query */
             $query = $em->createQuery($fixture['dql']);
             $query->setLockMode($fixture['lockMode']);
@@ -76,7 +76,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']);
         });
@@ -84,18 +84,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'];
@@ -104,11 +104,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();