doctrine / orm

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

Indentation +10 added lines, -10 removed lines

@@ -136,24 +136,24 @@
                     ->will($this->returnValue( 10 ));
 
         $parserResultMock = $this->getMockBuilder(ParserResult::class)
-                                 ->setMethods(['getSqlExecutor'])
-                                 ->getMock();
+                                    ->setMethods(['getSqlExecutor'])
+                                    ->getMock();
         $parserResultMock->expects($this->once())
-                         ->method('getSqlExecutor')
-                         ->will($this->returnValue($sqlExecMock));
+                            ->method('getSqlExecutor')
+                            ->will($this->returnValue($sqlExecMock));
 
         $cache = $this->getMockBuilder(CacheProvider::class)
-                      ->setMethods(['doFetch', 'doContains', 'doSave', 'doDelete', 'doFlush', 'doGetStats'])
-                      ->getMock();
+                        ->setMethods(['doFetch', 'doContains', 'doSave', 'doDelete', 'doFlush', 'doGetStats'])
+                        ->getMock();
 
         $cache->expects($this->at(0))->method('doFetch')->will($this->returnValue(1));
         $cache->expects($this->at(1))
-              ->method('doFetch')
-              ->with($this->isType('string'))
-              ->will($this->returnValue($parserResultMock));
+                ->method('doFetch')
+                ->with($this->isType('string'))
+                ->will($this->returnValue($parserResultMock));
 
         $cache->expects($this->never())
-              ->method('doSave');
+                ->method('doSave');
 
         $query->setQueryCacheDriver($cache);
 

Spacing +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\Tests\ORM\Functional;
 
@@ -135,7 +135,7 @@ 
 
         $sqlExecMock->expects($this->once())
                     ->method('execute')
-                    ->will($this->returnValue( 10 ));
+                    ->will($this->returnValue(10));
 
         $parserResultMock = $this->getMockBuilder(ParserResult::class)
                                  ->setMethods(['getSqlExecutor'])

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

Indentation +2 added lines, -2 removed lines

@@ -302,8 +302,8 @@
     }
 
     /**
-    * @group DDC-1955
-    */
+     * @group DDC-1955
+     */
     public function testLifecycleCallbackEventArgs()
     {
         $e = new LifecycleCallbackEventArgEntity;

Doc Comments +8 added lines

@@ -396,8 +396,16 @@
     private $name;
     public function getId() {return $this->id;}
     public function getValue() {return $this->value;}
+
+    /**
+     * @param string $value
+     */
     public function setValue($value) {$this->value = $value;}
     public function getName() {return $this->name;}
+
+    /**
+     * @param string $name
+     */
     public function setName($name) {$this->name = $name;}
     /** @ORM\PreUpdate */
     public function testCallback() {$this->value = 'Hello World';}

Spacing +7 added lines, -7 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\Tests\ORM\Functional;
 
@@ -394,13 +394,13 @@ 
     private $value;
     /** @ORM\Column(type="string") */
     private $name;
-    public function getId() {return $this->id;}
-    public function getValue() {return $this->value;}
-    public function setValue($value) {$this->value = $value;}
-    public function getName() {return $this->name;}
-    public function setName($name) {$this->name = $name;}
+    public function getId() {return $this->id; }
+    public function getValue() {return $this->value; }
+    public function setValue($value) {$this->value = $value; }
+    public function getName() {return $this->name; }
+    public function setName($name) {$this->name = $name; }
     /** @ORM\PreUpdate */
-    public function testCallback() {$this->value = 'Hello World';}
+    public function testCallback() {$this->value = 'Hello World'; }
 }
 
 /**

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

Indentation +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
  */
 class ResultCacheTest extends OrmFunctionalTestCase
 {
-   /**
+    /**
      * @var \ReflectionProperty
      */
     private $cacheDataReflection;

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\Tests\ORM\Functional;
 

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

Indentation +2 added lines, -2 removed lines

@@ -13,7 +13,7 @@ 
 
     public function providerParameterTypeInferer()
     {
-         $data = [
+            $data = [
             [1,                 Type::INTEGER],
             ["bar",             PDO::PARAM_STR],
             ["1",               PDO::PARAM_STR],
@@ -23,7 +23,7 @@ 
             [["1","2"],    Connection::PARAM_STR_ARRAY],
             [[],           Connection::PARAM_STR_ARRAY],
             [true,              Type::BOOLEAN],
-         ];
+            ];
 
         if (PHP_VERSION_ID >= 50500) {
             $data[] = [new \DateTimeImmutable(), Type::DATETIME];

Spacing +10 added lines, -10 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\Tests\ORM\Query;
 
@@ -16,15 +16,15 @@ 
     public function providerParameterTypeInferer()
     {
          $data = [
-            [1,                 Type::INTEGER],
-            ["bar",             PDO::PARAM_STR],
-            ["1",               PDO::PARAM_STR],
-            [new \DateTime,     Type::DATETIME],
-            [[2],          Connection::PARAM_INT_ARRAY],
-            [["foo"],      Connection::PARAM_STR_ARRAY],
-            [["1","2"],    Connection::PARAM_STR_ARRAY],
-            [[],           Connection::PARAM_STR_ARRAY],
-            [true,              Type::BOOLEAN],
+            [1, Type::INTEGER],
+            ["bar", PDO::PARAM_STR],
+            ["1", PDO::PARAM_STR],
+            [new \DateTime, Type::DATETIME],
+            [[2], Connection::PARAM_INT_ARRAY],
+            [["foo"], Connection::PARAM_STR_ARRAY],
+            [["1", "2"], Connection::PARAM_STR_ARRAY],
+            [[], Connection::PARAM_STR_ARRAY],
+            [true, Type::BOOLEAN],
          ];
 
         if (PHP_VERSION_ID >= 50500) {

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

Indentation +1 added lines, -1 removed lines

@@ -56,7 +56,7 @@
         $query->setDQL($dql);
 
         foreach ($hints as $key => $value) {
-        	$query->setHint($key, $value);
+            $query->setHint($key, $value);
         }
 
         $parser = new Query\Parser($query);

Doc Comments +6 added lines

@@ -23,6 +23,9 @@ 
         $this->em = $this->getTestEntityManager();
     }
 
+    /**
+     * @param string $dql
+     */
     public function assertValidDQL($dql, $debug = false)
     {
         try {
@@ -37,6 +40,9 @@ 
         }
     }
 
+    /**
+     * @param string $dql
+     */
     public function assertInvalidDQL($dql, $debug = false)
     {
         try {

Spacing +9 added lines, -9 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\Tests\ORM\Query;
 
@@ -30,7 +30,7 @@ 
             $this->addToAssertionCount(1);
         } catch (QueryException $e) {
             if ($debug) {
-                echo $e->getTraceAsString() . PHP_EOL;
+                echo $e->getTraceAsString().PHP_EOL;
             }
 
             $this->fail($e->getMessage());
@@ -45,8 +45,8 @@ 
             $this->fail('No syntax errors were detected, when syntax errors were expected');
         } catch (QueryException $e) {
             if ($debug) {
-                echo $e->getMessage() . PHP_EOL;
-                echo $e->getTraceAsString() . PHP_EOL;
+                echo $e->getMessage().PHP_EOL;
+                echo $e->getTraceAsString().PHP_EOL;
             }
             $this->addToAssertionCount(1);
         }
@@ -115,7 +115,7 @@ 
             ['SELECT foo: FROM Doctrine\Tests\Models\CMS\CmsUser foo:'],
 
             /* Checks for invalid AbstractSchemaName */
-            ['SELECT u FROM UnknownClass u'],  // unknown
+            ['SELECT u FROM UnknownClass u'], // unknown
             ['SELECT u FROM Unknown\Class u'], // unknown with namespace
             ['SELECT u FROM \Unknown\Class u'], // unknown, leading backslash
             ['SELECT u FROM Unknown\\\\Class u'], // unknown, syntactically bogus (duplicate \\)
@@ -588,7 +588,7 @@ 
      */
     public function testDQLKeywordInJoinIsAllowed()
     {
-        self::assertValidDQL('SELECT u FROM ' . __NAMESPACE__ . '\DQLKeywordsModelUser u JOIN u.group g');
+        self::assertValidDQL('SELECT u FROM '.__NAMESPACE__.'\DQLKeywordsModelUser u JOIN u.group g');
     }
 
     /**
@@ -596,7 +596,7 @@ 
      */
     public function testDQLKeywordInConditionIsAllowed()
     {
-        self::assertValidDQL('SELECT g FROM ' . __NAMESPACE__ . '\DQLKeywordsModelGroup g WHERE g.from=0');
+        self::assertValidDQL('SELECT g FROM '.__NAMESPACE__.'\DQLKeywordsModelGroup g WHERE g.from=0');
     }
 
     /* The exception is currently thrown in the SQLWalker, not earlier.
@@ -701,7 +701,7 @@ 
      */
     public function testNewLiteralExpression()
     {
-        self::assertValidDQL("SELECT new " . __NAMESPACE__ . "\\DummyStruct(u.id, 'foo', 1, true) FROM Doctrine\Tests\Models\CMS\CmsUser u");
+        self::assertValidDQL("SELECT new ".__NAMESPACE__."\\DummyStruct(u.id, 'foo', 1, true) FROM Doctrine\Tests\Models\CMS\CmsUser u");
     }
 
     /**
@@ -709,7 +709,7 @@ 
      */
     public function testNewLiteralWithSubselectExpression()
     {
-        self::assertValidDQL("SELECT new " . __NAMESPACE__ . "\\DummyStruct(u.id, 'foo', (SELECT 1 FROM Doctrine\Tests\Models\CMS\CmsUser su), true) FROM Doctrine\Tests\Models\CMS\CmsUser u");
+        self::assertValidDQL("SELECT new ".__NAMESPACE__."\\DummyStruct(u.id, 'foo', (SELECT 1 FROM Doctrine\Tests\Models\CMS\CmsUser su), true) FROM Doctrine\Tests\Models\CMS\CmsUser u");
     }
 }
 

tests/Doctrine/Tests/ORM/Tools/SchemaValidatorTest.php

Indentation +2 added lines, -2 removed lines

@@ -30,8 +30,8 @@
     public function testCmsModelSet(string $path)
     {
         $this->em->getConfiguration()
-                 ->getMetadataDriverImpl()
-                 ->addPaths([$path]);
+                    ->getMetadataDriverImpl()
+                    ->addPaths([$path]);
 
         self::assertEmpty($this->validator->validateMapping());
     }

Spacing +7 added lines, -7 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\Tests\ORM\Tools;
 
@@ -42,12 +42,12 @@ 
     public function modelSetProvider(): array
     {
         return [
-            'cms'        => [__DIR__ . '/../../Models/CMS'],
-            'company'    => [__DIR__ . '/../../Models/Company'],
-            'ecommerce'  => [__DIR__ . '/../../Models/ECommerce'],
-            'forum'      => [__DIR__ . '/../../Models/Forum'],
-            'navigation' => [__DIR__ . '/../../Models/Navigation'],
-            'routing'    => [__DIR__ . '/../../Models/Routing'],
+            'cms'        => [__DIR__.'/../../Models/CMS'],
+            'company'    => [__DIR__.'/../../Models/Company'],
+            'ecommerce'  => [__DIR__.'/../../Models/ECommerce'],
+            'forum'      => [__DIR__.'/../../Models/Forum'],
+            'navigation' => [__DIR__.'/../../Models/Navigation'],
+            'routing'    => [__DIR__.'/../../Models/Routing'],
         ];
     }
 

tests/Doctrine/Tests/ORM/Tools/Pagination/LimitSubqueryWalkerTest.php

Indentation +1 added lines, -1 removed lines

@@ -72,7 +72,7 @@
     /**
      * Arbitrary Join
      */
-     public function testLimitSubqueryWithArbitraryJoin()
+        public function testLimitSubqueryWithArbitraryJoin()
     {
         $dql        = 'SELECT p, c FROM Doctrine\Tests\ORM\Tools\Pagination\MyBlogPost p JOIN Doctrine\Tests\ORM\Tools\Pagination\Category c WITH p.category = c';
         $query      = $this->entityManager->createQuery($dql);

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\Tests\ORM\Tools\Pagination;
 

tests/Doctrine/Tests/ORM/Tools/Pagination/LimitSubqueryOutputWalkerTest.php

Indentation +1 added lines, -1 removed lines

@@ -84,7 +84,7 @@
         $this->entityManager->getConnection()->setDatabasePlatform(new PostgreSqlPlatform);
 
         $query = $this->entityManager->createQuery(
-           'SELECT u, g, COUNT(g.id) AS hidden g_quantity FROM Doctrine\Tests\ORM\Tools\Pagination\User u JOIN u.groups g ORDER BY g_quantity, u.id DESC'
+            'SELECT u, g, COUNT(g.id) AS hidden g_quantity FROM Doctrine\Tests\ORM\Tools\Pagination\User u JOIN u.groups g ORDER BY g_quantity, u.id DESC'
         );
         $limitQuery = clone $query;
         $limitQuery->setHint(Query::HINT_CUSTOM_OUTPUT_WALKER, LimitSubqueryOutputWalker::class);

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\Tests\ORM\Tools\Pagination;
 

tests/Doctrine/Tests/ORM/Mapping/BasicInheritanceMappingTest.php

Indentation +1 added lines, -1 removed lines

@@ -130,7 +130,7 @@
     {
         $this->expectException(MappingException::class);
         $this->expectExceptionMessage(
-              'Entity \'Doctrine\Tests\ORM\Mapping\HierarchyBEntity\' has to be part of the discriminator map'
+                'Entity \'Doctrine\Tests\ORM\Mapping\HierarchyBEntity\' has to be part of the discriminator map'
             . ' of \'Doctrine\Tests\ORM\Mapping\HierarchyBase\' to be properly mapped in the inheritance hierarchy.'
             . ' Alternatively you can make \'Doctrine\Tests\ORM\Mapping\HierarchyBEntity\' an abstract class to'
             . ' avoid this exception from occurring.'

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\Tests\ORM\Mapping;