doctrine / orm

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

tests/Doctrine/Tests/ORM/Decorator/EntityManagerDecoratorTest.php

Spacing +2 added lines, -2 removed lines

@@ -35,7 +35,7 @@ 
         $methods = [];
 
         foreach ($class->getMethods() as $method) {
-            if ($method->isConstructor() || $method->isStatic() || !$method->isPublic()) {
+            if ($method->isConstructor() || $method->isStatic() || ! $method->isPublic()) {
                 continue;
             }
 
@@ -72,7 +72,7 @@ 
      */
     public function testAllMethodCallsAreDelegatedToTheWrappedInstance($method, array $parameters)
     {
-        $return = !in_array($method, self::VOID_METHODS) ? 'INNER VALUE FROM ' . $method : null;
+        $return = ! in_array($method, self::VOID_METHODS) ? 'INNER VALUE FROM '.$method : null;
 
         $this->wrapped->expects($this->once())
             ->method($method)

tests/Doctrine/Tests/ORM/Internal/HydrationCompleteHandlerTest.php

Spacing +2 added lines, -2 removed lines

@@ -89,7 +89,7 @@ 
                 $metadata,
                 Events::postLoad,
                 $entity,
-                $this->callback(function (LifecycleEventArgs $args) use ($entityManager, $entity) {
+                $this->callback(function(LifecycleEventArgs $args) use ($entityManager, $entity) {
                     return $entity === $args->getEntity() && $entityManager === $args->getObjectManager();
                 }),
                 $listenersFlag
@@ -157,7 +157,7 @@ 
                 $this->logicalOr($metadata1, $metadata2),
                 Events::postLoad,
                 $this->logicalOr($entity1, $entity2),
-                $this->callback(function (LifecycleEventArgs $args) use ($entityManager, $entity1, $entity2) {
+                $this->callback(function(LifecycleEventArgs $args) use ($entityManager, $entity1, $entity2) {
                     return in_array($args->getEntity(), [$entity1, $entity2], true)
                         && $entityManager === $args->getObjectManager();
                 }),

tests/Doctrine/Tests/ORM/Hydration/SimpleObjectHydratorTest.php

Indentation +9 added lines, -9 removed lines

@@ -27,10 +27,10 @@ 
         $rsm->addMetaResult('p ', 'discr', 'discr', false, 'string');
         $rsm->setDiscriminatorColumn('p', 'discr');
         $resultSet = [
-              [
-                  'u__id'   => '1',
-                  'u__name' => 'Fabio B. Silva'
-              ],
+                [
+                    'u__id'   => '1',
+                    'u__name' => 'Fabio B. Silva'
+                ],
         ];
 
         $stmt       = new HydratorMockStatement($resultSet);
@@ -80,11 +80,11 @@ 
         $rsm->setDiscriminatorColumn('p', 'discr');
 
         $resultSet = [
-              [
-                  'p__id'   => '1',
-                  'p__name' => 'Fabio B. Silva',
-                  'discr'   => 'subworker'
-              ],
+                [
+                    'p__id'   => '1',
+                    'p__name' => 'Fabio B. Silva',
+                    'discr'   => 'subworker'
+                ],
         ];
 
         $stmt       = new HydratorMockStatement($resultSet);

tests/Doctrine/Tests/ORM/Hydration/ResultSetMappingTest.php

Indentation +2 added lines, -2 removed lines

@@ -189,8 +189,8 @@
     }
 
         /**
-     * @group DDC-1663
-     */
+         * @group DDC-1663
+         */
     public function testAddNamedNativeQueryResultSetMappingWithoutFields()
     {
         $cm = new ClassMetadata(CmsUser::class);

Spacing +4 added lines, -4 removed lines

@@ -76,8 +76,8 @@ 
     {
         $rms = $this->_rsm;
 
-        $this->_rsm->addEntityResult(CmsUser::class,'u');
-        $this->_rsm->addJoinedEntityResult(CmsPhonenumber::class,'p','u','phonenumbers');
+        $this->_rsm->addEntityResult(CmsUser::class, 'u');
+        $this->_rsm->addJoinedEntityResult(CmsPhonenumber::class, 'p', 'u', 'phonenumbers');
         $this->_rsm->addFieldResult('u', 'id', 'id');
         $this->_rsm->addFieldResult('u', 'name', 'name');
         $this->_rsm->setDiscriminatorColumn('name', 'name');
@@ -276,8 +276,8 @@ 
     {
         $this->_rsm->addEntityResult(LegacyUser::class, 'u');
         $this->_rsm->addJoinedEntityResult(LegacyUserReference::class, 'lu', 'u', '_references');
-        $this->_rsm->addMetaResult('lu', '_source',  '_source', true, 'integer');
-        $this->_rsm->addMetaResult('lu', '_target',  '_target', true, 'integer');
+        $this->_rsm->addMetaResult('lu', '_source', '_source', true, 'integer');
+        $this->_rsm->addMetaResult('lu', '_target', '_target', true, 'integer');
         $this->_rsm->addIndexBy('lu', '_source');
 
         $this->assertTrue($this->_rsm->hasIndexBy('lu'));

tests/Doctrine/Tests/ORM/Hydration/ObjectHydratorTest.php

Indentation +27 added lines, -27 removed lines

@@ -1020,14 +1020,14 @@ 
 
         // mocking the proxy factory
         $proxyFactory = $this->getMockBuilder(ProxyFactory::class)
-                             ->setMethods(['getProxy'])
-                             ->disableOriginalConstructor()
-                             ->getMock();
+                                ->setMethods(['getProxy'])
+                                ->disableOriginalConstructor()
+                                ->getMock();
 
         $proxyFactory->expects($this->once())
-                     ->method('getProxy')
-                     ->with($this->equalTo(ECommerceShipping::class), ['id' => 42])
-                     ->will($this->returnValue($proxyInstance));
+                        ->method('getProxy')
+                        ->with($this->equalTo(ECommerceShipping::class), ['id' => 42])
+                        ->will($this->returnValue($proxyInstance));
 
         $this->_em->setProxyFactory($proxyFactory);
 
@@ -1069,14 +1069,14 @@ 
 
         // mocking the proxy factory
         $proxyFactory = $this->getMockBuilder(ProxyFactory::class)
-                             ->setMethods(['getProxy'])
-                             ->disableOriginalConstructor()
-                             ->getMock();
+                                ->setMethods(['getProxy'])
+                                ->disableOriginalConstructor()
+                                ->getMock();
 
         $proxyFactory->expects($this->once())
-                     ->method('getProxy')
-                     ->with($this->equalTo(ECommerceShipping::class), ['id' => 42])
-                     ->will($this->returnValue($proxyInstance));
+                        ->method('getProxy')
+                        ->with($this->equalTo(ECommerceShipping::class), ['id' => 42])
+                        ->will($this->returnValue($proxyInstance));
 
         $this->_em->setProxyFactory($proxyFactory);
 
@@ -1836,10 +1836,10 @@ 
         $rsm->setDiscriminatorColumn('c', 'c_discr');
 
         $resultSet = [
-              [
-                  'c__id'   => '1',
-                  'c_discr' => 'fix',
-              ],
+                [
+                    'c__id'   => '1',
+                    'c_discr' => 'fix',
+                ],
         ];
 
         $stmt     = new HydratorMockStatement($resultSet);
@@ -1868,12 +1868,12 @@ 
         $rsm->setDiscriminatorColumn('e', 'e_discr');
 
         $resultSet = [
-              [
-                  'c__id'   => '1',
-                  'c_discr' => 'fix',
-                  'e__id'   => '1',
-                  'e__name' => 'Fabio B. Silva'
-              ],
+                [
+                    'c__id'   => '1',
+                    'c_discr' => 'fix',
+                    'e__id'   => '1',
+                    'e__name' => 'Fabio B. Silva'
+                ],
         ];
 
         $stmt     = new HydratorMockStatement($resultSet);
@@ -1898,11 +1898,11 @@ 
         $rsm->setDiscriminatorColumn('p', 'discr');
 
         $resultSet = [
-              [
-                  'p__id'   => '1',
-                  'p__name' => 'Fabio B. Silva',
-                  'discr'   => 'subworker'
-              ],
+                [
+                    'p__id'   => '1',
+                    'p__name' => 'Fabio B. Silva',
+                    'discr'   => 'subworker'
+                ],
         ];
 
         $stmt       = new HydratorMockStatement($resultSet);