doctrine / orm

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

Spacing +15 added lines, -15 removed lines

@@ -24,7 +24,7 @@ 
         parent::setUp();
         $this->_factory = new ProxyFactory(
                 $this->_em,
-                __DIR__ . '/../../Proxies',
+                __DIR__.'/../../Proxies',
                 'Doctrine\Tests\Proxies',
                 true);
     }
@@ -68,7 +68,7 @@ 
     {
         $id = $this->createProduct();
 
-        $entity = $this->_em->getReference(ECommerceProduct::class , $id);
+        $entity = $this->_em->getReference(ECommerceProduct::class, $id);
         $class = $this->_em->getClassMetadata(get_class($entity));
 
         $this->assertEquals(ECommerceProduct::class, $class->name);
@@ -81,8 +81,8 @@ 
     {
         $id = $this->createProduct();
 
-        $entity = $this->_em->getReference(ECommerceProduct::class , $id);
-        $entity2 = $this->_em->find(ECommerceProduct::class , $id);
+        $entity = $this->_em->getReference(ECommerceProduct::class, $id);
+        $entity2 = $this->_em->find(ECommerceProduct::class, $id);
 
         $this->assertSame($entity, $entity2);
         $this->assertEquals('Doctrine Cookbook', $entity2->getName());
@@ -96,7 +96,7 @@ 
         $id = $this->createProduct();
 
         /* @var $entity ECommerceProduct */
-        $entity = $this->_em->getReference(ECommerceProduct::class , $id);
+        $entity = $this->_em->getReference(ECommerceProduct::class, $id);
 
         /* @var $clone ECommerceProduct */
         $clone = clone $entity;
@@ -121,7 +121,7 @@ 
         $id = $this->createProduct();
 
         /* @var $entity ECommerceProduct */
-        $entity = $this->_em->getReference(ECommerceProduct::class , $id);
+        $entity = $this->_em->getReference(ECommerceProduct::class, $id);
 
         $this->assertFalse($entity->__isInitialized__, "Pre-Condition: Object is unitialized proxy.");
         $this->_em->getUnitOfWork()->initializeObject($entity);
@@ -136,13 +136,13 @@ 
         $id = $this->createProduct();
 
         /* @var $entity ECommerceProduct */
-        $entity = $this->_em->getReference(ECommerceProduct::class , $id);
+        $entity = $this->_em->getReference(ECommerceProduct::class, $id);
         $entity->setName('Doctrine 2 Cookbook');
 
         $this->_em->flush();
         $this->_em->clear();
 
-        $entity = $this->_em->getReference(ECommerceProduct::class , $id);
+        $entity = $this->_em->getReference(ECommerceProduct::class, $id);
         $this->assertEquals('Doctrine 2 Cookbook', $entity->getName());
     }
 
@@ -154,7 +154,7 @@ 
         $id = $this->createProduct();
 
         /* @var $entity ECommerceProduct */
-        $entity = $this->_em->getReference(ECommerceProduct::class , $id);
+        $entity = $this->_em->getReference(ECommerceProduct::class, $id);
 
         $this->assertFalse($entity->wakeUp);
 
@@ -168,7 +168,7 @@ 
         $id = $this->createProduct();
 
         /* @var $entity ECommerceProduct */
-        $entity = $this->_em->getReference(ECommerceProduct::class , $id);
+        $entity = $this->_em->getReference(ECommerceProduct::class, $id);
 
         $this->assertFalse($entity->__isInitialized__, "Pre-Condition: Object is unitialized proxy.");
         $this->assertEquals($id, $entity->getId());
@@ -183,7 +183,7 @@ 
         $id = $this->createAuction();
 
         /* @var $entity CompanyAuction */
-        $entity = $this->_em->getReference(CompanyAuction::class , $id);
+        $entity = $this->_em->getReference(CompanyAuction::class, $id);
 
         $this->assertFalse($entity->__isInitialized__, "Pre-Condition: Object is unitialized proxy.");
         $this->assertEquals($id, $entity->getId());
@@ -218,7 +218,7 @@ 
         $id = $this->createProduct();
 
         /* @var $entity ECommerceProduct */
-        $entity = $this->_em->getReference(ECommerceProduct::class , $id);
+        $entity = $this->_em->getReference(ECommerceProduct::class, $id);
 
         $this->assertFalse($entity->__isInitialized__, "Pre-Condition: Object is unitialized proxy.");
         $this->assertEquals('Doctrine Cookbook', $entity->getName());
@@ -233,7 +233,7 @@ 
         $id = $this->createProduct();
 
         /* @var $entity ECommerceProduct */
-        $entity = $this->_em->getReference(ECommerceProduct::class , $id);
+        $entity = $this->_em->getReference(ECommerceProduct::class, $id);
         $className = ClassUtils::getClass($entity);
 
         $this->assertInstanceOf(Proxy::class, $entity);
@@ -241,8 +241,8 @@ 
         $this->assertEquals(ECommerceProduct::class, $className);
 
         $restName = str_replace($this->_em->getConfiguration()->getProxyNamespace(), "", get_class($entity));
-        $restName = substr(get_class($entity), strlen($this->_em->getConfiguration()->getProxyNamespace()) +1);
-        $proxyFileName = $this->_em->getConfiguration()->getProxyDir() . DIRECTORY_SEPARATOR . str_replace("\\", "", $restName) . ".php";
+        $restName = substr(get_class($entity), strlen($this->_em->getConfiguration()->getProxyNamespace()) + 1);
+        $proxyFileName = $this->_em->getConfiguration()->getProxyDir().DIRECTORY_SEPARATOR.str_replace("\\", "", $restName).".php";
         $this->assertTrue(file_exists($proxyFileName), "Proxy file name cannot be found generically.");
 
         $entity->__load();

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

Spacing +7 added lines, -7 removed lines

@@ -24,9 +24,9 @@ 
         $worker->addFunction("dqlWithLock", [$lockAgent, "dqlWithLock"]);
         $worker->addFunction('lock', [$lockAgent, 'lock']);
 
-        while($worker->work()) {
+        while ($worker->work()) {
             if ($worker->returnCode() != GEARMAN_SUCCESS) {
-                echo "return_code: " . $worker->returnCode() . "\n";
+                echo "return_code: ".$worker->returnCode()."\n";
                 break;
             }
         }
@@ -77,18 +77,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'];
@@ -97,11 +97,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/'], true);
+        $annotDriver = $config->newDefaultAnnotationDriver([__DIR__.'/../../../Models/'], true);
         $config->setMetadataDriverImpl($annotDriver);
 
         $cache = new ArrayCache();

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

Spacing +5 added lines, -5 removed lines

@@ -17,7 +17,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.');
         }
 
@@ -133,11 +133,11 @@ 
         $this->gearman->runTasks();
 
         $this->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.");
         $this->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/LockTest.php

Indentation +2 added lines, -2 removed lines

@@ -206,7 +206,7 @@
         $this->expectExceptionMessage('The optimistic lock on an entity failed.');
 
         $this->_em->createQuery($dql)
-                  ->setHint(Query::HINT_LOCK_MODE, LockMode::OPTIMISTIC)
-                  ->getSQL();
+                    ->setHint(Query::HINT_LOCK_MODE, LockMode::OPTIMISTIC)
+                    ->getSQL();
     }
 }

Spacing +6 added lines, -6 removed lines

@@ -87,7 +87,7 @@ 
         $article = new CmsArticle();
 
         $this->expectException(\InvalidArgumentException::class);
-        $this->expectExceptionMessage('Entity ' . CmsArticle::class);
+        $this->expectExceptionMessage('Entity '.CmsArticle::class);
 
         $this->_em->lock($article, LockMode::OPTIMISTIC, $article->version + 1);
     }
@@ -136,7 +136,7 @@ 
     {
         $writeLockSql = $this->_em->getConnection()->getDatabasePlatform()->getWriteLockSQL();
 
-        if (! $writeLockSql) {
+        if ( ! $writeLockSql) {
             $this->markTestSkipped('Database Driver has no Write Lock support.');
         }
 
@@ -156,8 +156,8 @@ 
             throw $e;
         }
 
-        $query = array_pop( $this->_sqlLoggerStack->queries );
-        $query = array_pop( $this->_sqlLoggerStack->queries );
+        $query = array_pop($this->_sqlLoggerStack->queries);
+        $query = array_pop($this->_sqlLoggerStack->queries);
         $this->assertContains($writeLockSql, $query['sql']);
     }
 
@@ -168,7 +168,7 @@ 
     {
         $readLockSql = $this->_em->getConnection()->getDatabasePlatform()->getReadLockSQL();
 
-        if (! $readLockSql) {
+        if ( ! $readLockSql) {
             $this->markTestSkipped('Database Driver has no Write Lock support.');
         }
 
@@ -200,7 +200,7 @@ 
      */
     public function testLockOptimisticNonVersionedThrowsExceptionInDQL()
     {
-        $dql = "SELECT u FROM " . CmsUser::class . " u WHERE u.username = 'gblanco'";
+        $dql = "SELECT u FROM ".CmsUser::class." u WHERE u.username = 'gblanco'";
 
         $this->expectException(OptimisticLockException::class);
         $this->expectExceptionMessage('The optimistic lock on an entity failed.');

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

Spacing +2 added lines, -2 removed lines

@@ -115,7 +115,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();
@@ -249,7 +249,7 @@ 
         $caughtException = null;
 
         try {
-            $expectedVersionExpired = DateTime::createFromFormat('U', $test->version->getTimestamp()-3600);
+            $expectedVersionExpired = DateTime::createFromFormat('U', $test->version->getTimestamp() - 3600);
 
             $this->_em->lock($test, LockMode::OPTIMISTIC, $expectedVersionExpired);
         } catch (OptimisticLockException $e) {

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

Spacing +1 added lines, -1 removed lines

@@ -232,7 +232,7 @@
         $entity     = $this->_em->find(Travel::class, $entitiId);
 
         $this->assertEquals(0, $entity->getVisitedCities()->count());
-        $this->assertEquals($queryCount+2, $this->getCurrentQueryCount());
+        $this->assertEquals($queryCount + 2, $this->getCurrentQueryCount());
 
         $this->_em->clear();
 

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

Indentation +2 added lines, -2 removed lines

@@ -102,8 +102,8 @@
         $routeId = $this->createPersistedRouteWithLegs();
 
         $route = $this->_em->createQuery("SELECT r, l FROM Doctrine\Tests\Models\Routing\RoutingRoute r JOIN r.legs l WHERE r.id = ?1")
-                           ->setParameter(1, $routeId)
-                           ->getSingleResult();
+                            ->setParameter(1, $routeId)
+                            ->getSingleResult();
 
         $this->assertEquals(2, count($route->legs));
         $this->assertEquals("Berlin", $route->legs[0]->fromLocation->getName());

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

Spacing +15 added lines, -15 removed lines

@@ -27,7 +27,7 @@ 
      */
     public function testIssue2059()
     {
-        if (!$this->_em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
+        if ( ! $this->_em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
             $this->markTestSkipped('Platform does not support foreign keys.');
         }
 
@@ -49,7 +49,7 @@ 
 
     public function testLoadMetadataFromDatabase()
     {
-        if (!$this->_em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
+        if ( ! $this->_em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
             $this->markTestSkipped('Platform does not support foreign keys.');
         }
 
@@ -65,22 +65,22 @@ 
         $this->assertArrayHasKey('DbdriverFoo', $metadatas);
         $metadata = $metadatas['DbdriverFoo'];
 
-        $this->assertArrayHasKey('id',          $metadata->fieldMappings);
-        $this->assertEquals('id',               $metadata->fieldMappings['id']['fieldName']);
-        $this->assertEquals('id',               strtolower($metadata->fieldMappings['id']['columnName']));
-        $this->assertEquals('integer',          (string)$metadata->fieldMappings['id']['type']);
+        $this->assertArrayHasKey('id', $metadata->fieldMappings);
+        $this->assertEquals('id', $metadata->fieldMappings['id']['fieldName']);
+        $this->assertEquals('id', strtolower($metadata->fieldMappings['id']['columnName']));
+        $this->assertEquals('integer', (string) $metadata->fieldMappings['id']['type']);
 
-        $this->assertArrayHasKey('bar',         $metadata->fieldMappings);
-        $this->assertEquals('bar',              $metadata->fieldMappings['bar']['fieldName']);
-        $this->assertEquals('bar',              strtolower($metadata->fieldMappings['bar']['columnName']));
-        $this->assertEquals('string',           (string)$metadata->fieldMappings['bar']['type']);
-        $this->assertEquals(200,                $metadata->fieldMappings['bar']['length']);
+        $this->assertArrayHasKey('bar', $metadata->fieldMappings);
+        $this->assertEquals('bar', $metadata->fieldMappings['bar']['fieldName']);
+        $this->assertEquals('bar', strtolower($metadata->fieldMappings['bar']['columnName']));
+        $this->assertEquals('string', (string) $metadata->fieldMappings['bar']['type']);
+        $this->assertEquals(200, $metadata->fieldMappings['bar']['length']);
         $this->assertTrue($metadata->fieldMappings['bar']['nullable']);
     }
 
     public function testLoadMetadataWithForeignKeyFromDatabase()
     {
-        if (!$this->_em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
+        if ( ! $this->_em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
             $this->markTestSkipped('Platform does not support foreign keys.');
         }
 
@@ -114,7 +114,7 @@ 
 
     public function testDetectManyToManyTables()
     {
-        if (!$this->_em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
+        if ( ! $this->_em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
             $this->markTestSkipped('Platform does not support foreign keys.');
         }
 
@@ -169,7 +169,7 @@ 
 
         $table->addColumn('column_index1', 'string');
         $table->addColumn('column_index2', 'string');
-        $table->addIndex(['column_index1','column_index2'], 'index1');
+        $table->addIndex(['column_index1', 'column_index2'], 'index1');
 
         $table->addColumn('column_unique_index1', 'string');
         $table->addColumn('column_unique_index2', 'string');
@@ -208,7 +208,7 @@ 
 
         $this->assertTrue( ! empty($metadata->table['indexes']['index1']['columns']));
         $this->assertEquals(
-            ['column_index1','column_index2'],
+            ['column_index1', 'column_index2'],
             $metadata->table['indexes']['index1']['columns']
         );
 

tests/Doctrine/Tests/ORM/EntityManagerTest.php

Indentation +1 added lines, -1 removed lines

@@ -103,7 +103,7 @@
     public function testCreateQueryBuilderAliasValid()
     {
         $q = $this->_em->createQueryBuilder()
-             ->select('u')->from(CmsUser::class, 'u');
+                ->select('u')->from(CmsUser::class, 'u');
         $q2 = clone $q;
 
         $this->assertEquals('SELECT u FROM Doctrine\Tests\Models\CMS\CmsUser u', $q->getQuery()->getDql());

Spacing +4 added lines, -4 removed lines

@@ -162,7 +162,7 @@ 
      */
     public function testThrowsExceptionOnNonObjectValues($methodName) {
         $this->expectException(ORMInvalidArgumentException::class);
-        $this->expectExceptionMessage('EntityManager#' . $methodName . '() expects parameter 1 to be an entity object, NULL given.');
+        $this->expectExceptionMessage('EntityManager#'.$methodName.'() expects parameter 1 to be an entity object, NULL given.');
 
         $this->_em->$methodName(null);
     }
@@ -196,7 +196,7 @@ 
      */
     public function testTransactionalAcceptsReturn()
     {
-        $return = $this->_em->transactional(function ($em) {
+        $return = $this->_em->transactional(function($em) {
             return 'foo';
         });
 
@@ -238,8 +238,8 @@ 
     public function testTransactionalReThrowsThrowables()
     {
         try {
-            $this->_em->transactional(function () {
-                (function (array $value) {
+            $this->_em->transactional(function() {
+                (function(array $value) {
                     // this only serves as an IIFE that throws a `TypeError`
                 })(null);
             });