doctrine / orm

tests/Doctrine/Tests/ORM/Functional/Ticket/DDC656Test.php

Doc Comments +6 added lines

@@ -66,6 +66,9 @@ 
         return $this->name;
     }
 
+    /**
+     * @param string $name
+     */
     public function setName($name) {
         $this->name = $name;
     }
@@ -74,6 +77,9 @@ 
         return $this->type;
     }
 
+    /**
+     * @param string $type
+     */
     public function setType($type) {
         $this->type = $type;
     }

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\Ticket;
 
@@ -17,7 +17,7 @@ 
                 $this->em->getClassMetadata(DDC656Entity::class)
                 ]
             );
-        } catch(\Exception $e) {
+        } catch (\Exception $e) {
 
         }
     }

tests/Doctrine/Tests/ORM/Functional/Ticket/DDC881Test.php

Doc Comments +9 added lines

@@ -118,6 +118,9 @@ 
         return $this->name;
     }
 
+    /**
+     * @param string $name
+     */
     public function setName($name)
     {
         $this->name = $name;
@@ -155,6 +158,9 @@ 
         $this->calls = new \Doctrine\Common\Collections\ArrayCollection();
     }
 
+    /**
+     * @param integer $id
+     */
     public function setId($id)
     {
         $this->id = $id;
@@ -165,6 +171,9 @@ 
         $this->user = $user;
     }
 
+    /**
+     * @param string $phoneNumber
+     */
     public function setPhoneNumber($phoneNumber)
     {
         $this->phonenumber = $phoneNumber;

Spacing +3 added lines, -3 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\Tests\ORM\Functional\Ticket;
 
@@ -83,14 +83,14 @@ 
         $this->em->clear();
 
         // fetch-join that foreign-key/primary-key entity association
-        $dql = "SELECT c, p FROM " . DDC881PhoneCall::class . " c JOIN c.phonenumber p";
+        $dql = "SELECT c, p FROM ".DDC881PhoneCall::class." c JOIN c.phonenumber p";
         $calls = $this->em->createQuery($dql)->getResult();
 
         self::assertCount(2, $calls);
         self::assertNotInstanceOf(GhostObjectInterface::class, $calls[0]->getPhoneNumber());
         self::assertNotInstanceOf(GhostObjectInterface::class, $calls[1]->getPhoneNumber());
 
-        $dql = "SELECT p, c FROM " . DDC881PhoneNumber::class . " p JOIN p.calls c";
+        $dql = "SELECT p, c FROM ".DDC881PhoneNumber::class." p JOIN p.calls c";
         $numbers = $this->em->createQuery($dql)->getResult();
 
         self::assertCount(2, $numbers);

tests/Doctrine/Tests/ORM/Functional/Ticket/DDC960Test.php

Doc Comments +6 added lines

@@ -78,11 +78,17 @@
      */
     private $name;
 
+    /**
+     * @param string $name
+     */
     public function __construct($name)
     {
         $this->name = $name;
     }
 
+    /**
+     * @param string $name
+     */
     public function setName($name)
     {
         $this->name = $name;

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\Ticket;
 
@@ -19,7 +19,7 @@ 
                 $this->em->getClassMetadata(DDC960Child::class)
                 ]
             );
-        } catch(\Exception $e) {
+        } catch (\Exception $e) {
 
         }
     }

tests/Doctrine/Tests/ORM/Functional/Ticket/Ticket69.php

Doc Comments +1 added lines, -1 removed lines

@@ -240,7 +240,7 @@
 
     /**
      *
-     * @return Phrase
+     * @return Lemma
      */
     public function getParent() {
         return $this->parent;

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\Ticket;
 
@@ -90,7 +90,7 @@ 
         self::assertInstanceOf(Lemma::class, $lemma);
         $relations = $lemma->getRelations();
 
-        foreach($relations as $relation) {
+        foreach ($relations as $relation) {
             self::assertInstanceOf(Relation::class, $relation);
             self::assertTrue($relation->getType()->getType() != '');
         }

tests/Doctrine/Tests/ORM/Functional/Ticket/DDC1080Test.php

Doc Comments +2 added lines, -2 removed lines

@@ -127,7 +127,7 @@ 
     }
 
     /**
-     * @param field_type $fooTitle
+     * @param string $fooTitle
      */
     public function setFooTitle($fooTitle)
     {
@@ -205,7 +205,7 @@ 
     }
 
     /**
-     * @param field_type $barTitle
+     * @param string $barTitle
      */
     public function setBarTitle($barTitle)
     {

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\Ticket;
 

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

Doc Comments +6 added lines

@@ -148,6 +148,9 @@ 
         );
     }
 
+    /**
+     * @param string $dql
+     */
     protected function asyncDqlWithLock($dql, $params, $lockMode)
     {
         $this->startJob('dqlWithLock', [
@@ -168,6 +171,9 @@ 
         );
     }
 
+    /**
+     * @param string $fn
+     */
     protected function startJob($fn, $fixture)
     {
         $this->gearman->addTask($fn, serialize(

Spacing +6 added lines, -6 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\Tests\ORM\Functional\Locking;
 
@@ -19,7 +19,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.');
         }
 
@@ -135,11 +135,11 @@ 
         $this->gearman->runTasks();
 
         self::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.");
         self::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."
         );
     }
 

tools/sandbox/bootstrap.php

Spacing +4 added lines, -4 removed lines

@@ -5,8 +5,8 @@ 
 
 // Path to composer autoloader. You can use different provided by your favorite framework,
 // if you want to.
-$loaderPath = __DIR__ . '/../../vendor/autoload.php';
-if(!is_readable($loaderPath)){
+$loaderPath = __DIR__.'/../../vendor/autoload.php';
+if ( ! is_readable($loaderPath)) {
     throw new LogicException('Run php composer.phar install at first');
 }
 $loader = require $loaderPath;
@@ -19,7 +19,7 @@ 
 $config = new \Doctrine\ORM\Configuration();
 
 // Set up Metadata Drivers
-$driverImpl = $config->newDefaultAnnotationDriver([__DIR__ . "/Entities"]);
+$driverImpl = $config->newDefaultAnnotationDriver([__DIR__."/Entities"]);
 $config->setMetadataDriverImpl($driverImpl);
 
 // Set up caches, depending on $debug variable.
@@ -29,7 +29,7 @@ 
 $config->setQueryCacheImpl($cache);
 
 // Proxy configuration
-$config->setProxyDir(__DIR__ . '/Proxies');
+$config->setProxyDir(__DIR__.'/Proxies');
 $config->setProxyNamespace('Proxies');
 
 // Database connection information

lib/Doctrine/ORM/AbstractQuery.php

Doc Comments +1 added lines, -1 removed lines

@@ -1044,7 +1044,7 @@
      * Will return the configured id if it exists otherwise a hash will be
      * automatically generated for you.
      *
-     * @return array ($key, $hash)
+     * @return string[] ($key, $hash)
      */
     protected function getHydrationCacheId()
     {

Spacing +8 added lines, -8 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\ORM;
 
@@ -306,7 +306,7 @@ 
     public function getParameter($key)
     {
         $filteredParameters = $this->parameters->filter(
-            function ($parameter) use ($key)
+            function($parameter) use ($key)
             {
                 // Must not be identical because of string to integer conversion
                 return ($key == $parameter->getName());
@@ -355,14 +355,14 @@ 
     public function setParameter($key, $value, $type = null)
     {
         $filteredParameters = $this->parameters->filter(
-            function ($parameter) use ($key)
+            function($parameter) use ($key)
             {
                 // Must not be identical because of string to integer conversion
                 return ($key == $parameter->getName());
             }
         );
 
-        if (! $filteredParameters->isEmpty()) {
+        if ( ! $filteredParameters->isEmpty()) {
             $parameter = $filteredParameters->first();
             $parameter->setValue($value, $type);
 
@@ -451,7 +451,7 @@ 
      */
     private function translateNamespaces(Query\ResultSetMapping $rsm)
     {
-        $translate = function ($alias) {
+        $translate = function($alias) {
             return $this->em->getClassMetadata($alias)->getClassName();
         };
 
@@ -513,7 +513,7 @@ 
      */
     public function setResultCacheProfile(QueryCacheProfile $profile = null)
     {
-        if (! $profile->getResultCacheDriver()) {
+        if ( ! $profile->getResultCacheDriver()) {
             $resultCacheDriver = $this->em->getConfiguration()->getResultCacheImpl();
             $profile = $profile->setResultCacheDriver($resultCacheDriver);
         }
@@ -981,7 +981,7 @@ 
             $this->getTimestampKey()
         );
 
-        $result     = $queryCache->get($queryKey, $rsm, $this->hints);
+        $result = $queryCache->get($queryKey, $rsm, $this->hints);
 
         if ($result !== null) {
             if ($this->cacheLogger) {
@@ -1115,6 +1115,6 @@ 
 
         ksort($hints);
 
-        return sha1($query . '-' . serialize($params) . '-' . serialize($hints));
+        return sha1($query.'-'.serialize($params).'-'.serialize($hints));
     }
 }

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

Doc Comments +12 added lines

@@ -146,6 +146,12 @@ 
         return [$user1, $user2, $user3];
     }
 
+    /**
+     * @param string $name
+     * @param string $username
+     * @param string $status
+     * @param CmsAddress $address
+     */
     public function buildUser($name, $username, $status, $address)
     {
         $user = new CmsUser();
@@ -160,6 +166,12 @@ 
         return $user;
     }
 
+    /**
+     * @param string $country
+     * @param string $city
+     * @param string $street
+     * @param string $zip
+     */
     public function buildAddress($country, $city, $street, $zip)
     {
         $address = new CmsAddress();

Indentation +10 added lines, -10 removed lines

@@ -310,7 +310,7 @@ 
      */
     public function testExceptionIsThrownWhenCallingFindByWithoutParameter() {
         $this->em->getRepository(CmsUser::class)
-                  ->findByStatus();
+                    ->findByStatus();
     }
 
     /**
@@ -318,7 +318,7 @@ 
      */
     public function testExceptionIsThrownWhenUsingInvalidFieldName() {
         $this->em->getRepository(CmsUser::class)
-                  ->findByThisFieldDoesNotExist('testvalue');
+                    ->findByThisFieldDoesNotExist('testvalue');
     }
 
     /**
@@ -330,7 +330,7 @@ 
         $this->expectException(TransactionRequiredException::class);
 
         $this->em->getRepository(CmsUser::class)
-                  ->find(1, LockMode::PESSIMISTIC_READ);
+                    ->find(1, LockMode::PESSIMISTIC_READ);
     }
 
     /**
@@ -342,7 +342,7 @@ 
         $this->expectException(TransactionRequiredException::class);
 
         $this->em->getRepository(CmsUser::class)
-                  ->find(1, LockMode::PESSIMISTIC_WRITE);
+                    ->find(1, LockMode::PESSIMISTIC_WRITE);
     }
 
     /**
@@ -354,7 +354,7 @@ 
         $this->expectException(OptimisticLockException::class);
 
         $this->em->getRepository(CmsUser::class)
-                  ->find(1, LockMode::OPTIMISTIC);
+                    ->find(1, LockMode::OPTIMISTIC);
     }
 
     /**
@@ -435,13 +435,13 @@ 
      */
     public function testFindOneByOrderBy()
     {
-    	$this->loadFixture();
+        $this->loadFixture();
 
-    	$repos = $this->em->getRepository(CmsUser::class);
-    	$userAsc = $repos->findOneBy([], ["username" => "ASC"]);
-    	$userDesc = $repos->findOneBy([], ["username" => "DESC"]);
+        $repos = $this->em->getRepository(CmsUser::class);
+        $userAsc = $repos->findOneBy([], ["username" => "ASC"]);
+        $userDesc = $repos->findOneBy([], ["username" => "DESC"]);
 
-    	self::assertNotSame($userAsc, $userDesc);
+        self::assertNotSame($userAsc, $userDesc);
     }
 
     /**

Spacing +9 added lines, -9 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\Tests\ORM\Functional;
 
@@ -182,7 +182,7 @@ 
         $repos = $this->em->getRepository(CmsUser::class);
 
         $user = $repos->find($user1Id);
-        self::assertInstanceOf(CmsUser::class,$user);
+        self::assertInstanceOf(CmsUser::class, $user);
         self::assertEquals('Roman', $user->name);
         self::assertEquals('freak', $user->status);
     }
@@ -194,7 +194,7 @@ 
 
         $users = $repos->findBy(['status' => 'dev']);
         self::assertCount(2, $users);
-        self::assertInstanceOf(CmsUser::class,$users[0]);
+        self::assertInstanceOf(CmsUser::class, $users[0]);
         self::assertEquals('Guilherme', $users[0]->name);
         self::assertEquals('dev', $users[0]->status);
     }
@@ -220,7 +220,7 @@ 
         $addresses  = $repository->findBy(['user' => [$user1->getId(), $user2->getId()]]);
 
         self::assertCount(2, $addresses);
-        self::assertInstanceOf(CmsAddress::class,$addresses[0]);
+        self::assertInstanceOf(CmsAddress::class, $addresses[0]);
     }
 
     public function testFindByAssociationWithObjectAsParameter()
@@ -244,7 +244,7 @@ 
         $addresses  = $repository->findBy(['user' => [$user1, $user2]]);
 
         self::assertCount(2, $addresses);
-        self::assertInstanceOf(CmsAddress::class,$addresses[0]);
+        self::assertInstanceOf(CmsAddress::class, $addresses[0]);
     }
 
     public function testFindFieldByMagicCall()
@@ -254,7 +254,7 @@ 
 
         $users = $repos->findByStatus('dev');
         self::assertCount(2, $users);
-        self::assertInstanceOf(CmsUser::class,$users[0]);
+        self::assertInstanceOf(CmsUser::class, $users[0]);
         self::assertEquals('Guilherme', $users[0]->name);
         self::assertEquals('dev', $users[0]->status);
     }
@@ -594,7 +594,7 @@ 
         self::assertCount(2, $usersAsc);
         self::assertCount(2, $usersDesc);
 
-        self::assertInstanceOf(CmsUser::class,$usersAsc[0]);
+        self::assertInstanceOf(CmsUser::class, $usersAsc[0]);
         self::assertEquals('Alexander', $usersAsc[0]->name);
         self::assertEquals('dev', $usersAsc[0]->status);
 
@@ -675,7 +675,7 @@ 
     public function testCanRetrieveRepositoryFromClassNameWithLeadingBackslash()
     {
         self::assertSame(
-            $this->em->getRepository('\\' . CmsUser::class),
+            $this->em->getRepository('\\'.CmsUser::class),
             $this->em->getRepository(CmsUser::class)
         );
     }
@@ -713,7 +713,7 @@ 
         $data = $repo->findBy(['user' => [1, 2, 3]]);
 
         $query = array_pop($this->sqlLoggerStack->queries);
-        self::assertEquals([1,2,3], $query['params'][0]);
+        self::assertEquals([1, 2, 3], $query['params'][0]);
         self::assertEquals(Connection::PARAM_INT_ARRAY, $query['types'][0]);
     }