doctrine / orm

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

Spacing +7 added lines, -7 removed lines

@@ -29,7 +29,7 @@ 
      */
     public function testIssue2059() : void
     {
-        if (! $this->em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
+        if ( ! $this->em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
             $this->markTestSkipped('Platform does not support foreign keys.');
         }
 
@@ -51,7 +51,7 @@ 
 
     public function testLoadMetadataFromDatabase() : void
     {
-        if (! $this->em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
+        if ( ! $this->em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
             $this->markTestSkipped('Platform does not support foreign keys.');
         }
 
@@ -89,7 +89,7 @@ 
 
     public function testLoadMetadataWithForeignKeyFromDatabase() : void
     {
-        if (! $this->em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
+        if ( ! $this->em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
             $this->markTestSkipped('Platform does not support foreign keys.');
         }
 
@@ -122,7 +122,7 @@ 
 
     public function testDetectManyToManyTables() : void
     {
-        if (! $this->em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
+        if ( ! $this->em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
             $this->markTestSkipped('Platform does not support foreign keys.');
         }
 
@@ -162,7 +162,7 @@ 
 
     public function testLoadMetadataFromDatabaseDetail() : void
     {
-        if (! $this->em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
+        if ( ! $this->em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
             $this->markTestSkipped('Platform does not support foreign keys.');
         }
 
@@ -201,7 +201,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'));
 
@@ -243,7 +243,7 @@ 
 
         self::assertNotEmpty($indexes['index1']['columns']);
         self::assertEquals(
-            ['column_index1','column_index2'],
+            ['column_index1', 'column_index2'],
             $indexes['index1']['columns']
         );
 

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

Spacing +2 added lines, -2 removed lines

@@ -33,8 +33,8 @@
 
         $configuration = $this->em->getConfiguration();
 
-        $configuration->setProxyNamespace(__NAMESPACE__ . '\\ProxyTest');
-        $configuration->setProxyDir(__DIR__ . '/../../Proxies');
+        $configuration->setProxyNamespace(__NAMESPACE__.'\\ProxyTest');
+        $configuration->setProxyDir(__DIR__.'/../../Proxies');
         $configuration->setAutoGenerateProxyClasses(StaticProxyFactory::AUTOGENERATE_ALWAYS);
 
         $this->factory = new StaticProxyFactory($this->em, $configuration->buildGhostObjectFactory());

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

Spacing +13 added lines, -13 removed lines

@@ -112,7 +112,7 @@ 
         $this->em->flush();
         $this->em->clear();
 
-        $dql     = 'SELECT p FROM ' . __NAMESPACE__ . '\DDC93Person p';
+        $dql     = 'SELECT p FROM '.__NAMESPACE__.'\DDC93Person p';
         $persons = $this->em->createQuery($dql)->getResult();
 
         self::assertCount(3, $persons);
@@ -125,7 +125,7 @@ 
             self::assertEquals('United States of America', $person->address->country->name);
         }
 
-        $dql     = 'SELECT p FROM ' . __NAMESPACE__ . '\DDC93Person p';
+        $dql     = 'SELECT p FROM '.__NAMESPACE__.'\DDC93Person p';
         $persons = $this->em->createQuery($dql)->getArrayResult();
 
         foreach ($persons as $person) {
@@ -150,7 +150,7 @@ 
         $this->em->flush();
 
         // SELECT
-        $selectDql    = 'SELECT p FROM ' . __NAMESPACE__ . '\\DDC93Person p WHERE p.address.city = :city AND p.address.country.name = :country';
+        $selectDql    = 'SELECT p FROM '.__NAMESPACE__.'\\DDC93Person p WHERE p.address.city = :city AND p.address.country.name = :country';
         $loadedPerson = $this->em->createQuery($selectDql)
             ->setParameter('city', 'Karlsruhe')
             ->setParameter('country', 'Germany')
@@ -165,7 +165,7 @@ 
         );
 
         // UPDATE
-        $updateDql = 'UPDATE ' . __NAMESPACE__ . '\\DDC93Person p SET p.address.street = :street, p.address.country.name = :country WHERE p.address.city = :city';
+        $updateDql = 'UPDATE '.__NAMESPACE__.'\\DDC93Person p SET p.address.street = :street, p.address.country.name = :country WHERE p.address.city = :city';
         $this->em->createQuery($updateDql)
             ->setParameter('street', 'Boo')
             ->setParameter('country', 'DE')
@@ -177,7 +177,7 @@ 
         self::assertEquals('DE', $person->address->country->name);
 
         // DELETE
-        $this->em->createQuery('DELETE ' . __NAMESPACE__ . '\\DDC93Person p WHERE p.address.city = :city AND p.address.country.name = :country')
+        $this->em->createQuery('DELETE '.__NAMESPACE__.'\\DDC93Person p WHERE p.address.city = :city AND p.address.country.name = :country')
             ->setParameter('city', 'Karlsruhe')
             ->setParameter('country', 'DE')
             ->execute();
@@ -194,7 +194,7 @@ 
         $this->em->clear();
 
         // Prove that the entity was persisted correctly.
-        $dql = 'SELECT p FROM ' . __NAMESPACE__ . '\\DDC93Person p WHERE p.name = :name';
+        $dql = 'SELECT p FROM '.__NAMESPACE__.'\\DDC93Person p WHERE p.name = :name';
 
         $person = $this->em->createQuery($dql)
             ->setParameter('name', 'Karl')
@@ -208,7 +208,7 @@ 
         // Clear the EM and prove that the embeddable can be the subject of a partial query.
         $this->em->clear();
 
-        $dql = 'SELECT PARTIAL p.{id,address.city} FROM ' . __NAMESPACE__ . '\\DDC93Person p WHERE p.name = :name';
+        $dql = 'SELECT PARTIAL p.{id,address.city} FROM '.__NAMESPACE__.'\\DDC93Person p WHERE p.name = :name';
 
         $person = $this->em->createQuery($dql)
             ->setParameter('name', 'Karl')
@@ -224,7 +224,7 @@ 
         // Clear the EM and prove that the embeddable can be the subject of a partial query regardless of attributes positions.
         $this->em->clear();
 
-        $dql = 'SELECT PARTIAL p.{address.city, id} FROM ' . __NAMESPACE__ . '\\DDC93Person p WHERE p.name = :name';
+        $dql = 'SELECT PARTIAL p.{address.city, id} FROM '.__NAMESPACE__.'\\DDC93Person p WHERE p.name = :name';
 
         $person = $this->em->createQuery($dql)
             ->setParameter('name', 'Karl')
@@ -243,7 +243,7 @@ 
         $this->expectException(QueryException::class);
         $this->expectExceptionMessage('no field or association named address.asdfasdf');
 
-        $this->em->createQuery('SELECT p FROM ' . __NAMESPACE__ . '\\DDC93Person p WHERE p.address.asdfasdf IS NULL')
+        $this->em->createQuery('SELECT p FROM '.__NAMESPACE__.'\\DDC93Person p WHERE p.address.asdfasdf IS NULL')
             ->execute();
     }
 
@@ -252,7 +252,7 @@ 
         $this->expectException(QueryException::class);
         $this->expectExceptionMessage("no mapped field named 'address.asdfasdf'");
 
-        $this->em->createQuery('SELECT PARTIAL p.{id,address.asdfasdf} FROM ' . __NAMESPACE__ . '\\DDC93Person p')
+        $this->em->createQuery('SELECT PARTIAL p.{id,address.asdfasdf} FROM '.__NAMESPACE__.'\\DDC93Person p')
             ->execute();
     }
 
@@ -314,15 +314,15 @@ 
         $this->expectException(MappingException::class);
         $this->expectExceptionMessage(
             sprintf(
-                'Infinite nesting detected for embedded property %s::nested. ' .
+                'Infinite nesting detected for embedded property %s::nested. '.
                 'You cannot embed an embeddable from the same type inside an embeddable.',
-                __NAMESPACE__ . '\\' . $declaredEmbeddableClassName
+                __NAMESPACE__.'\\'.$declaredEmbeddableClassName
             )
         );
 
         $this->schemaTool->createSchema(
             [
-            $this->em->getClassMetadata(__NAMESPACE__ . '\\' . $embeddableClassName),
+            $this->em->getClassMetadata(__NAMESPACE__.'\\'.$embeddableClassName),
             ]
         );
     }

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

Spacing +4 added lines, -4 removed lines

@@ -201,7 +201,7 @@ 
     {
         $listener = new ListenerSecondLevelCacheTest(
             [
-                Events::postFlush => function () {
+                Events::postFlush => function() {
                     throw new \RuntimeException('post flush failure');
                 },
             ]
@@ -233,7 +233,7 @@ 
 
         $listener = new ListenerSecondLevelCacheTest(
             [
-                Events::postUpdate => function () {
+                Events::postUpdate => function() {
                     throw new \RuntimeException('post update failure');
                 },
             ]
@@ -252,7 +252,7 @@ 
         self::assertInstanceOf(State::class, $state);
         self::assertEquals($stateName, $state->getName());
 
-        $state->setName($stateName . uniqid());
+        $state->setName($stateName.uniqid());
 
         $this->em->persist($state);
 
@@ -279,7 +279,7 @@ 
         $this->em->clear();
 
         $listener = new ListenerSecondLevelCacheTest([
-            Events::postRemove => function () {
+            Events::postRemove => function() {
                 throw new \RuntimeException('post remove failure');
             },
         ]);

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/PostLoadEventTest.php

Spacing +3 added lines, -3 removed lines

@@ -231,8 +231,8 @@ 
         $eventManager->addEventListener([Events::postLoad], $listener);
 
         $this->em->find(CmsUser::class, $this->userId);
-        self::assertSame(1, $listener->countHandledEvents(CmsUser::class), CmsUser::class . ' should be handled once!');
-        self::assertSame(1, $listener->countHandledEvents(CmsEmail::class), CmsEmail::class . ' should be handled once!');
+        self::assertSame(1, $listener->countHandledEvents(CmsUser::class), CmsUser::class.' should be handled once!');
+        self::assertSame(1, $listener->countHandledEvents(CmsEmail::class), CmsEmail::class.' should be handled once!');
     }
 
     private function loadFixture()
@@ -308,7 +308,7 @@ 
     {
         $object = $event->getObject();
         $class  = StaticClassNameConverter::getClass($object);
-        if (! isset($this->firedByClasses[$class])) {
+        if ( ! isset($this->firedByClasses[$class])) {
             $this->firedByClasses[$class] = 1;
         } else {
             $this->firedByClasses[$class]++;

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