doctrine / orm

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

tests/Doctrine/Tests/ORM/Query/QueryTest.php

Indentation +14 added lines, -14 removed lines

@@ -85,18 +85,18 @@ 
     {
         $q = $this->_em->createQuery("select a from Doctrine\Tests\Models\CMS\CmsArticle a");
         $q2 = $q->expireQueryCache(true)
-          ->setQueryCacheLifetime(3600)
-          ->setQueryCacheDriver(null)
-          ->expireResultCache(true)
-          ->setHint('foo', 'bar')
-          ->setHint('bar', 'baz')
-          ->setParameter(1, 'bar')
-          ->setParameters(new ArrayCollection([new Parameter(2, 'baz')]))
-          ->setResultCacheDriver(null)
-          ->setResultCacheId('foo')
-          ->setDQL('foo')
-          ->setFirstResult(10)
-          ->setMaxResults(10);
+            ->setQueryCacheLifetime(3600)
+            ->setQueryCacheDriver(null)
+            ->expireResultCache(true)
+            ->setHint('foo', 'bar')
+            ->setHint('bar', 'baz')
+            ->setParameter(1, 'bar')
+            ->setParameters(new ArrayCollection([new Parameter(2, 'baz')]))
+            ->setResultCacheDriver(null)
+            ->setResultCacheId('foo')
+            ->setDQL('foo')
+            ->setFirstResult(10)
+            ->setMaxResults(10);
 
         $this->assertSame($q2, $q);
     }
@@ -254,11 +254,11 @@ 
         $this->_em->getConfiguration()->setResultCacheImpl(new ArrayCache());
 
         $query = $this->_em->createQuery("SELECT u FROM Doctrine\Tests\Models\CMS\CmsUser u")
-                           ->useResultCache(true);
+                            ->useResultCache(true);
 
         /** @var DriverConnectionMock $driverConnectionMock */
         $driverConnectionMock = $this->_em->getConnection()
-                                          ->getWrappedConnection();
+                                            ->getWrappedConnection();
 
         $driverConnectionMock->setStatementMock(new StatementArrayMock([['id_0' => 1]]));
 

Spacing +2 added lines, -2 removed lines

@@ -163,7 +163,7 @@ 
             9 => "St Julien"
         ];
 
-        $query  = $this->_em
+        $query = $this->_em
                 ->createQuery("SELECT a FROM Doctrine\Tests\Models\CMS\CmsAddress a WHERE a.city IN (:cities)")
                 ->setParameter('cities', $cities);
 
@@ -179,7 +179,7 @@ 
      */
     public function testProcessParameterValueClassMetadata()
     {
-        $query  = $this->_em->createQuery("SELECT a FROM Doctrine\Tests\Models\CMS\CmsAddress a WHERE a.city IN (:cities)");
+        $query = $this->_em->createQuery("SELECT a FROM Doctrine\Tests\Models\CMS\CmsAddress a WHERE a.city IN (:cities)");
         $this->assertEquals(
             CmsAddress::class,
             $query->processParameterValue($this->_em->getClassMetadata(CmsAddress::class))

tests/Doctrine/Tests/ORM/Query/QueryExpressionVisitorTest.php

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
      */
     protected function setUp()
     {
-        $this->visitor = new QueryExpressionVisitor(['o','p']);
+        $this->visitor = new QueryExpressionVisitor(['o', 'p']);
     }
 
     /**

tests/Doctrine/Tests/ORM/Query/ParserTest.php

Spacing +2 added lines, -2 removed lines

@@ -29,9 +29,9 @@
      */
     public function testAbstractSchemaNameSupportsClassnamesWithLeadingBackslash()
     {
-        $parser = $this->createParser('\\' . CmsUser::class);
+        $parser = $this->createParser('\\'.CmsUser::class);
 
-        $this->assertEquals('\\' . CmsUser::class, $parser->AbstractSchemaName());
+        $this->assertEquals('\\'.CmsUser::class, $parser->AbstractSchemaName());
     }
 
     /**

tests/Doctrine/Tests/ORM/Query/SelectSqlGenerationTest.php

Indentation +36 added lines, -36 removed lines

@@ -49,7 +49,7 @@ 
             }
 
             $query->setHint(ORMQuery::HINT_FORCE_PARTIAL_LOAD, true)
-                  ->useQueryCache(false);
+                    ->useQueryCache(false);
 
             foreach ($queryHints AS $name => $value) {
                 $query->setHint($name, $value);
@@ -88,7 +88,7 @@ 
         }
 
         $query->setHint(ORMQuery::HINT_FORCE_PARTIAL_LOAD, true)
-              ->useQueryCache(false);
+                ->useQueryCache(false);
 
         foreach ($queryHints AS $name => $value) {
             $query->setHint($name, $value);
@@ -1640,9 +1640,9 @@ 
         );
     }
 
-     /**
-     * @group DDC-1430
-     */
+        /**
+         * @group DDC-1430
+         */
     public function testGroupByAllFieldsWhenObjectHasForeignKeys()
     {
         $this->assertSqlGeneration(
@@ -1982,9 +1982,9 @@ 
         );
     }
 
-   /**
-    * @group DDC-1845
-    */
+    /**
+     * @group DDC-1845
+     */
     public function testQuotedWalkJoinVariableDeclaration()
     {
         $this->assertSqlGeneration(
@@ -2018,9 +2018,9 @@ 
         );
     }
 
-   /**
-    * @group DDC-2208
-    */
+    /**
+     * @group DDC-2208
+     */
     public function testCaseThenParameterArithmeticExpression()
     {
         $this->assertSqlGeneration(
@@ -2040,8 +2040,8 @@ 
     }
 
     /**
-    * @group DDC-2268
-    */
+     * @group DDC-2268
+     */
     public function testCaseThenFunction()
     {
         $this->assertSqlGeneration(
@@ -2065,45 +2065,45 @@ 
      */
     public function testSupportsMoreThanTwoParametersInConcatFunction()
     {
-    	$connMock    = $this->_em->getConnection();
-    	$orgPlatform = $connMock->getDatabasePlatform();
+        $connMock    = $this->_em->getConnection();
+        $orgPlatform = $connMock->getDatabasePlatform();
 
-    	$connMock->setDatabasePlatform(new MySqlPlatform());
-    	$this->assertSqlGeneration(
+        $connMock->setDatabasePlatform(new MySqlPlatform());
+        $this->assertSqlGeneration(
             "SELECT u.id FROM Doctrine\Tests\Models\CMS\CmsUser u WHERE CONCAT(u.name, u.status, 's') = ?1",
             "SELECT c0_.id AS id_0 FROM cms_users c0_ WHERE CONCAT(c0_.name, c0_.status, 's') = ?"
-    	);
-    	$this->assertSqlGeneration(
+        );
+        $this->assertSqlGeneration(
             "SELECT CONCAT(u.id, u.name, u.status) FROM Doctrine\Tests\Models\CMS\CmsUser u WHERE u.id = ?1",
             "SELECT CONCAT(c0_.id, c0_.name, c0_.status) AS sclr_0 FROM cms_users c0_ WHERE c0_.id = ?"
-    	);
+        );
 
-    	$connMock->setDatabasePlatform(new PostgreSqlPlatform());
-    	$this->assertSqlGeneration(
+        $connMock->setDatabasePlatform(new PostgreSqlPlatform());
+        $this->assertSqlGeneration(
             "SELECT u.id FROM Doctrine\Tests\Models\CMS\CmsUser u WHERE CONCAT(u.name, u.status, 's') = ?1",
             "SELECT c0_.id AS id_0 FROM cms_users c0_ WHERE c0_.name || c0_.status || 's' = ?"
-    	);
-    	$this->assertSqlGeneration(
+        );
+        $this->assertSqlGeneration(
             "SELECT CONCAT(u.id, u.name, u.status) FROM Doctrine\Tests\Models\CMS\CmsUser u WHERE u.id = ?1",
             "SELECT c0_.id || c0_.name || c0_.status AS sclr_0 FROM cms_users c0_ WHERE c0_.id = ?"
-    	);
+        );
 
-    	$connMock->setDatabasePlatform(new SQLServerPlatform());
-    	$this->assertSqlGeneration(
+        $connMock->setDatabasePlatform(new SQLServerPlatform());
+        $this->assertSqlGeneration(
             "SELECT u.id FROM Doctrine\Tests\Models\CMS\CmsUser u WHERE CONCAT(u.name, u.status, 's') = ?1",
             "SELECT c0_.id AS id_0 FROM cms_users c0_ WHERE (c0_.name + c0_.status + 's') = ?"
-    	);
-    	$this->assertSqlGeneration(
+        );
+        $this->assertSqlGeneration(
             "SELECT CONCAT(u.id, u.name, u.status) FROM Doctrine\Tests\Models\CMS\CmsUser u WHERE u.id = ?1",
             "SELECT (c0_.id + c0_.name + c0_.status) AS sclr_0 FROM cms_users c0_ WHERE c0_.id = ?"
-    	);
+        );
 
-    	$connMock->setDatabasePlatform($orgPlatform);
+        $connMock->setDatabasePlatform($orgPlatform);
     }
 
-     /**
-     * @group DDC-2188
-     */
+        /**
+         * @group DDC-2188
+         */
     public function testArithmeticPriority()
     {
         $this->assertSqlGeneration(
@@ -2123,8 +2123,8 @@ 
     }
 
     /**
-    * @group DDC-2475
-    */
+     * @group DDC-2475
+     */
     public function testOrderByClauseShouldReplaceOrderByRelationMapping()
     {
         $this->assertSqlGeneration(

Spacing +7 added lines, -7 removed lines

@@ -65,7 +65,7 @@ 
 
             $query->free();
         } catch (\Exception $e) {
-            $this->fail($e->getMessage() ."\n".$e->getTraceAsString());
+            $this->fail($e->getMessage()."\n".$e->getTraceAsString());
         }
     }
 
@@ -1597,7 +1597,7 @@ 
     public function testAliasDoesNotExceedPlatformDefinedLength()
     {
         $this->assertSqlGeneration(
-            'SELECT m FROM ' . __NAMESPACE__ .  '\\DDC1384Model m',
+            'SELECT m FROM '.__NAMESPACE__.'\\DDC1384Model m',
             "SELECT d0_.aVeryLongIdentifierThatShouldBeShortenedByTheSQLWalker_fooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooo AS ooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooo_0 FROM DDC1384Model d0_"
         );
     }
@@ -1630,12 +1630,12 @@ 
     public function testSelectWithArithmeticExpressionBeforeField()
     {
         $this->assertSqlGeneration(
-            'SELECT - e.value AS value, e.id FROM ' . __NAMESPACE__ . '\DDC1474Entity e',
+            'SELECT - e.value AS value, e.id FROM '.__NAMESPACE__.'\DDC1474Entity e',
             'SELECT -d0_.value AS sclr_0, d0_.id AS id_1 FROM DDC1474Entity d0_'
         );
 
         $this->assertSqlGeneration(
-            'SELECT e.id, + e.value AS value FROM ' . __NAMESPACE__ . '\DDC1474Entity e',
+            'SELECT e.id, + e.value AS value FROM '.__NAMESPACE__.'\DDC1474Entity e',
             'SELECT d0_.id AS id_0, +d0_.value AS sclr_1 FROM DDC1474Entity d0_'
         );
     }
@@ -2297,8 +2297,8 @@ 
     public function testHavingRegressionUsingVariableWithMathOperatorsExpression($operator)
     {
         $this->assertSqlGeneration(
-            'SELECT COUNT(u.name) AS countName FROM Doctrine\Tests\Models\CMS\CmsUser u HAVING 1 ' . $operator . ' countName > 0',
-            'SELECT COUNT(c0_.name) AS sclr_0 FROM cms_users c0_ HAVING 1 ' . $operator . ' sclr_0 > 0'
+            'SELECT COUNT(u.name) AS countName FROM Doctrine\Tests\Models\CMS\CmsUser u HAVING 1 '.$operator.' countName > 0',
+            'SELECT COUNT(c0_.name) AS sclr_0 FROM cms_users c0_ HAVING 1 '.$operator.' sclr_0 > 0'
         );
     }
 
@@ -2320,7 +2320,7 @@ 
      */
     public function getSql(SqlWalker $sqlWalker)
     {
-        return 'ABS(' . $sqlWalker->walkSimpleArithmeticExpression($this->simpleArithmeticExpression) . ')';
+        return 'ABS('.$sqlWalker->walkSimpleArithmeticExpression($this->simpleArithmeticExpression).')';
     }
 
     /**