doctrine / orm

tests/Doctrine/Tests/TestUtil.php

Spacing +1 added lines, -1 removed lines

@@ -90,7 +90,7 @@
     {
         $realDbParams = self::getParamsForMainConnection();
 
-        if (! self::$initialized) {
+        if ( ! self::$initialized) {
             $tmpDbParams = self::getParamsForTemporaryConnection();
 
             $realConn = DriverManager::getConnection($realDbParams);

tests/Doctrine/Tests/ORM/Cache/DefaultQueryCacheTest.php

Spacing +2 added lines, -2 removed lines

@@ -390,7 +390,7 @@ 
         $uow->registerManaged($state, ['id' => $state->getId()], ['name' => $city->getName()]);
         $uow->registerManaged($city, ['id' => $city->getId()], ['name' => $city->getName(), 'state' => $state]);
 
-        $this->region->addReturn('put', true);  // put root entity
+        $this->region->addReturn('put', true); // put root entity
         $this->region->addReturn('put', false); // association fails
 
         self::assertFalse($this->queryCache->put($key, $rsm, $result));
@@ -423,7 +423,7 @@ 
         $uow->registerManaged($city2, ['id' => $city2->getId()], ['name' => $city2->getName(), 'state' => $state]);
         $uow->registerManaged($state, ['id' => $state->getId()], ['name' => $state->getName(), 'cities' => $state->getCities()]);
 
-        $this->region->addReturn('put', true);  // put root entity
+        $this->region->addReturn('put', true); // put root entity
         $this->region->addReturn('put', false); // collection association fails
 
         self::assertFalse($this->queryCache->put($key, $rsm, $result));

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

Spacing +7 added lines, -7 removed lines

@@ -139,7 +139,7 @@ 
         $size         = 500;
         $startPersist = microtime(true);
 
-        echo PHP_EOL . $label;
+        echo PHP_EOL.$label;
 
         for ($i = 0; $i < $size; $i++) {
             $em->persist(new Country(sprintf('Country %d', $i)));
@@ -173,13 +173,13 @@ 
         $startPersist = microtime(true);
         $country      = new Country('Country');
 
-        echo PHP_EOL . $label;
+        echo PHP_EOL.$label;
 
         $em->persist($country);
         $em->flush();
 
         for ($i = 0; $i < $size / 2; $i++) {
-            $state = new State('State ' . $i, $country);
+            $state = new State('State '.$i, $country);
 
             $em->persist($state);
 
@@ -228,10 +228,10 @@ 
         $countries    = [];
         $startPersist = microtime(true);
 
-        echo PHP_EOL . $label;
+        echo PHP_EOL.$label;
 
         for ($i = 0; $i < $size; $i++) {
-            $country = new Country('Country ' . $i);
+            $country = new Country('Country '.$i);
 
             $em->persist($country);
 
@@ -263,10 +263,10 @@ 
         $startPersist = microtime(true);
         $rep          = $em->getRepository(Country::class);
 
-        echo PHP_EOL . $label;
+        echo PHP_EOL.$label;
 
         for ($i = 0; $i < $size; $i++) {
-            $em->persist(new Country('Country ' . $i));
+            $em->persist(new Country('Country '.$i));
         }
 
         $em->flush();

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

Spacing +2 added lines, -2 removed lines

@@ -306,7 +306,7 @@ 
 
         $queryCount = $this->getCurrentQueryCount();
         self::assertFalse($user->articles->contains($article));
-        self::assertEquals($queryCount+1, $this->getCurrentQueryCount(), 'Checking for contains of persisted entity should cause one query to be executed.');
+        self::assertEquals($queryCount + 1, $this->getCurrentQueryCount(), 'Checking for contains of persisted entity should cause one query to be executed.');
         self::assertFalse($user->articles->isInitialized(), 'Post-Condition: Collection is not initialized.');
 
         // Test One to Many existence with state managed
@@ -473,7 +473,7 @@ 
 
         $queryCount = $this->getCurrentQueryCount();
         self::assertTrue($group->users->contains($user));
-        self::assertEquals($queryCount+1, $this->getCurrentQueryCount(), 'Checking for contains of managed entity should cause one query to be executed.');
+        self::assertEquals($queryCount + 1, $this->getCurrentQueryCount(), 'Checking for contains of managed entity should cause one query to be executed.');
         self::assertFalse($user->groups->isInitialized(), 'Post-Condition: Collection is not initialized.');
 
         $newUser       = new CmsUser();

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

Spacing +5 added lines, -5 removed lines

@@ -30,7 +30,7 @@ 
         $this->em->flush();
         $this->em->clear();
 
-        $dql     = 'SELECT d FROM ' . DecimalModel::class . ' d';
+        $dql     = 'SELECT d FROM '.DecimalModel::class.' d';
         $decimal = $this->em->createQuery($dql)->getSingleResult();
 
         self::assertSame('0.15', $decimal->decimal);
@@ -49,7 +49,7 @@ 
         $this->em->flush();
         $this->em->clear();
 
-        $dql  = 'SELECT b FROM ' . BooleanModel::class . ' b WHERE b.booleanField = true';
+        $dql  = 'SELECT b FROM '.BooleanModel::class.' b WHERE b.booleanField = true';
         $bool = $this->em->createQuery($dql)->getSingleResult();
 
         self::assertTrue($bool->booleanField);
@@ -59,7 +59,7 @@ 
         $this->em->flush();
         $this->em->clear();
 
-        $dql  = 'SELECT b FROM ' . BooleanModel::class . ' b WHERE b.booleanField = false';
+        $dql  = 'SELECT b FROM '.BooleanModel::class.' b WHERE b.booleanField = false';
         $bool = $this->em->createQuery($dql)->getSingleResult();
 
         self::assertFalse($bool->booleanField);
@@ -75,7 +75,7 @@ 
         $this->em->flush();
         $this->em->clear();
 
-        $dql       = 'SELECT s FROM ' . SerializationModel::class . ' s';
+        $dql       = 'SELECT s FROM '.SerializationModel::class.' s';
         $serialize = $this->em->createQuery($dql)->getSingleResult();
 
         self::assertSame(['foo' => 'bar', 'bar' => 'baz'], $serialize->array);
@@ -90,7 +90,7 @@ 
         $this->em->flush();
         $this->em->clear();
 
-        $dql       = 'SELECT s FROM ' . SerializationModel::class . ' s';
+        $dql       = 'SELECT s FROM '.SerializationModel::class.' s';
         $serialize = $this->em->createQuery($dql)->getSingleResult();
 
         self::assertInstanceOf('stdClass', $serialize->object);

Indentation +6 added lines, -6 removed lines

@@ -167,12 +167,12 @@
         $this->em->clear();
 
         $dateTimeDb = $this->em->createQueryBuilder()
-             ->select('d')
-             ->from(DateTimeModel::class, 'd')
-             ->where('d.datetime = ?1')
-             ->setParameter(1, $date, DBALType::DATETIME)
-             ->getQuery()
-             ->getSingleResult();
+                ->select('d')
+                ->from(DateTimeModel::class, 'd')
+                ->where('d.datetime = ?1')
+                ->setParameter(1, $date, DBALType::DATETIME)
+                ->getQuery()
+                ->getSingleResult();
 
         self::assertInstanceOf(DateTime::class, $dateTimeDb->datetime);
         self::assertSame('2009-10-02 20:10:52', $dateTimeDb->datetime->format('Y-m-d H:i:s'));

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/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]++;