doctrine / orm

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

Doc Comments +6 added lines

@@ -151,6 +151,9 @@ 
         $this->books = new \Doctrine\Common\Collections\ArrayCollection;
     }
 
+    /**
+     * @param string $title
+     */
     public function addBook($title)
     {
         $book = new DDC618Book($title, $this);
@@ -175,6 +178,9 @@ 
     /** @ManyToOne(targetEntity="DDC618Author", inversedBy="books") */
     public $author;
 
+    /**
+     * @param DDC618Author $author
+     */
     function __construct($title, $author)
     {
         $this->title = $title;

Indentation +4 added lines, -4 removed lines

@@ -71,7 +71,7 @@ 
     public function testIndexByJoin()
     {
         $dql = 'SELECT A, B FROM Doctrine\Tests\ORM\Functional\Ticket\DDC618Author A '.
-               'INNER JOIN A.books B INDEX BY B.title ORDER BY A.name ASC';
+                'INNER JOIN A.books B INDEX BY B.title ORDER BY A.name ASC';
         $result = $this->_em->createQuery($dql)->getResult(\Doctrine\ORM\Query::HYDRATE_OBJECT);
 
         $this->assertEquals(3, count($result[0]->books)); // Alice, Joe doesn't appear because he has no books.
@@ -95,14 +95,14 @@ 
     public function testIndexByToOneJoinSilentlyIgnored()
     {
         $dql = 'SELECT B, A FROM Doctrine\Tests\ORM\Functional\Ticket\DDC618Book B '.
-               'INNER JOIN B.author A INDEX BY A.name ORDER BY A.name ASC';
+                'INNER JOIN B.author A INDEX BY A.name ORDER BY A.name ASC';
         $result = $this->_em->createQuery($dql)->getResult(\Doctrine\ORM\Query::HYDRATE_OBJECT);
 
         $this->assertInstanceOf(DDC618Book::class, $result[0]);
         $this->assertInstanceOf(DDC618Author::class, $result[0]->author);
 
         $dql = 'SELECT B, A FROM Doctrine\Tests\ORM\Functional\Ticket\DDC618Book B '.
-               'INNER JOIN B.author A INDEX BY A.name ORDER BY A.name ASC';
+                'INNER JOIN B.author A INDEX BY A.name ORDER BY A.name ASC';
         $result = $this->_em->createQuery($dql)->getResult(\Doctrine\ORM\Query::HYDRATE_ARRAY);
 
         $this->assertEquals("Alice", $result[0]['author']['name']);
@@ -114,7 +114,7 @@ 
     public function testCombineIndexBy()
     {
         $dql = 'SELECT A, B FROM Doctrine\Tests\ORM\Functional\Ticket\DDC618Author A INDEX BY A.id '.
-               'INNER JOIN A.books B INDEX BY B.title ORDER BY A.name ASC';
+                'INNER JOIN A.books B INDEX BY B.title ORDER BY A.name ASC';
         $result = $this->_em->createQuery($dql)->getResult(\Doctrine\ORM\Query::HYDRATE_OBJECT);
 
         $this->assertArrayHasKey(11, $result); // Alice

Spacing +10 added lines, -10 removed lines

@@ -36,7 +36,7 @@ 
 
             $this->_em->flush();
             $this->_em->clear();
-        } catch(\Exception $e) {
+        } catch (\Exception $e) {
 
         }
     }
@@ -76,17 +76,17 @@ 
 
         $this->assertEquals(3, count($result[0]->books)); // Alice, Joe doesn't appear because he has no books.
         $this->assertEquals('Alice', $result[0]->name);
-        $this->assertTrue( isset($result[0]->books["In Wonderland"] ), "Indexing by title should have books by title.");
-        $this->assertTrue( isset($result[0]->books["Reloaded"] ), "Indexing by title should have books by title.");
-        $this->assertTrue( isset($result[0]->books["Test"] ), "Indexing by title should have books by title.");
+        $this->assertTrue(isset($result[0]->books["In Wonderland"]), "Indexing by title should have books by title.");
+        $this->assertTrue(isset($result[0]->books["Reloaded"]), "Indexing by title should have books by title.");
+        $this->assertTrue(isset($result[0]->books["Test"]), "Indexing by title should have books by title.");
 
         $result = $this->_em->createQuery($dql)->getResult(\Doctrine\ORM\Query::HYDRATE_ARRAY);
 
         $this->assertEquals(3, count($result[0]['books'])); // Alice, Joe doesn't appear because he has no books.
         $this->assertEquals('Alice', $result[0]['name']);
-        $this->assertTrue( isset($result[0]['books']["In Wonderland"] ), "Indexing by title should have books by title.");
-        $this->assertTrue( isset($result[0]['books']["Reloaded"] ), "Indexing by title should have books by title.");
-        $this->assertTrue( isset($result[0]['books']["Test"] ), "Indexing by title should have books by title.");
+        $this->assertTrue(isset($result[0]['books']["In Wonderland"]), "Indexing by title should have books by title.");
+        $this->assertTrue(isset($result[0]['books']["Reloaded"]), "Indexing by title should have books by title.");
+        $this->assertTrue(isset($result[0]['books']["Test"]), "Indexing by title should have books by title.");
     }
 
     /**
@@ -121,9 +121,9 @@ 
 
         $this->assertEquals(3, count($result[11]->books)); // Alice, Joe doesn't appear because he has no books.
         $this->assertEquals('Alice', $result[11]->name);
-        $this->assertTrue( isset($result[11]->books["In Wonderland"] ), "Indexing by title should have books by title.");
-        $this->assertTrue( isset($result[11]->books["Reloaded"] ), "Indexing by title should have books by title.");
-        $this->assertTrue( isset($result[11]->books["Test"] ), "Indexing by title should have books by title.");
+        $this->assertTrue(isset($result[11]->books["In Wonderland"]), "Indexing by title should have books by title.");
+        $this->assertTrue(isset($result[11]->books["Reloaded"]), "Indexing by title should have books by title.");
+        $this->assertTrue(isset($result[11]->books["Test"]), "Indexing by title should have books by title.");
     }
 }
 

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 +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
                 $this->_em->getClassMetadata(DDC656Entity::class)
                 ]
             );
-        } catch(\Exception $e) {
+        } catch (\Exception $e) {
 
         }
     }

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

Doc Comments +11 added lines

@@ -137,6 +137,9 @@ 
      */
     public $version;
 
+    /**
+     * @param string $word
+     */
     public function __construct($word)
     {
         $this->word = $word;
@@ -166,6 +169,9 @@ 
      */
     public $version;
 
+    /**
+     * @param string $name
+     */
     public function __construct($name)
     {
         $this->name = $name;
@@ -183,6 +189,11 @@ 
     /** @Column(type="integer") */
     public $rgt;
 
+    /**
+     * @param string $name
+     * @param integer $lft
+     * @param integer $rgt
+     */
     public function __construct($name, $lft, $rgt)
     {
         $this->name = $name;

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
                     $this->_em->getClassMetadata(DDC832Like::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 +2 added lines, -2 removed lines

@@ -80,14 +80,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();
 
         $this->assertEquals(2, count($calls));
         $this->assertNotInstanceOf(Proxy::class, $calls[0]->getPhoneNumber());
         $this->assertNotInstanceOf(Proxy::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();
 
         $this->assertEquals(2, count($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 +1 added lines, -1 removed lines

@@ -16,7 +16,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 +1 added lines, -1 removed lines

@@ -86,7 +86,7 @@
         $this->assertInstanceOf(Lemma::class, $lemma);
         $relations = $lemma->getRelations();
 
-        foreach($relations as $relation) {
+        foreach ($relations as $relation) {
             $this->assertInstanceOf(Relation::class, $relation);
             $this->assertTrue($relation->getType()->getType() != '');
         }

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

Doc Comments +11 added lines

@@ -339,6 +339,9 @@ 
     /** @Embedded(class = "DDC93Timestamps") */
     public $timestamps;
 
+    /**
+     * @param string $name
+     */
     public function __construct($name = null, DDC93Address $address = null)
     {
         $this->name = $name;
@@ -403,6 +406,9 @@ 
      */
     public $name;
 
+    /**
+     * @param string $name
+     */
     public function __construct($name = null)
     {
         $this->name = $name;
@@ -431,6 +437,11 @@ 
     /** @Embedded(class = "DDC93Country") */
     public $country;
 
+    /**
+     * @param string $street
+     * @param string $zip
+     * @param string $city
+     */
     public function __construct($street = null, $zip = null, $city = null, DDC93Country $country = null)
     {
         $this->street = $street;

Spacing +15 added lines, -15 removed lines

@@ -27,7 +27,7 @@ 
                 $this->_em->getClassMetadata(DDC3027Dog::class),
                 ]
             );
-        } catch(\Exception $e) {
+        } catch (\Exception $e) {
         }
     }
 
@@ -82,7 +82,7 @@ 
         $this->assertEquals('United States of America', $person->address->country->name);
 
         // 4. check deleting works
-        $personId = $person->id;;
+        $personId = $person->id; ;
         $this->_em->remove($person);
         $this->_em->flush();
 
@@ -106,7 +106,7 @@ 
         $this->_em->flush();
         $this->_em->clear();
 
-        $dql = "SELECT p FROM " . __NAMESPACE__ . "\DDC93Person p";
+        $dql = "SELECT p FROM ".__NAMESPACE__."\DDC93Person p";
         $persons = $this->_em->createQuery($dql)->getResult();
 
         $this->assertCount(3, $persons);
@@ -119,7 +119,7 @@ 
             $this->assertEquals('United States of America', $person->address->country->name);
         }
 
-        $dql = "SELECT p FROM " . __NAMESPACE__ . "\DDC93Person p";
+        $dql = "SELECT p FROM ".__NAMESPACE__."\DDC93Person p";
         $persons = $this->_em->createQuery($dql)->getArrayResult();
 
         foreach ($persons as $person) {
@@ -144,7 +144,7 @@ 
         $this->_em->flush($person);
 
         // SELECT
-        $selectDql = "SELECT p FROM " . __NAMESPACE__ ."\\DDC93Person p WHERE p.address.city = :city AND p.address.country.name = :country";
+        $selectDql = "SELECT p FROM ".__NAMESPACE__."\\DDC93Person p WHERE p.address.city = :city AND p.address.country.name = :country";
         $loadedPerson = $this->_em->createQuery($selectDql)
             ->setParameter('city', 'Karlsruhe')
             ->setParameter('country', 'Germany')
@@ -159,7 +159,7 @@ 
         );
 
         // UPDATE
-        $updateDql = "UPDATE " . __NAMESPACE__ . "\\DDC93Person p SET p.address.street = :street, p.address.country.name = :country WHERE p.address.city = :city";
+        $updateDql = "UPDATE ".__NAMESPACE__."\\DDC93Person p SET p.address.street = :street, p.address.country.name = :country WHERE p.address.city = :city";
         $this->_em->createQuery($updateDql)
             ->setParameter('street', 'Boo')
             ->setParameter('country', 'DE')
@@ -171,7 +171,7 @@ 
         $this->assertEquals('DE', $person->address->country->name);
 
         // DELETE
-        $this->_em->createQuery("DELETE " . __NAMESPACE__ . "\\DDC93Person p WHERE p.address.city = :city AND p.address.country.name = :country")
+        $this->_em->createQuery("DELETE ".__NAMESPACE__."\\DDC93Person p WHERE p.address.city = :city AND p.address.country.name = :country")
             ->setParameter('city', 'Karlsruhe')
             ->setParameter('country', 'DE')
             ->execute();
@@ -188,7 +188,7 @@ 
         $this->_em->clear();
 
         // Prove that the entity was persisted correctly.
-        $dql = "SELECT p FROM " . __NAMESPACE__ ."\\DDC93Person p WHERE p.name = :name";
+        $dql = "SELECT p FROM ".__NAMESPACE__."\\DDC93Person p WHERE p.name = :name";
 
         $person = $this->_em->createQuery($dql)
             ->setParameter('name', 'Karl')
@@ -202,7 +202,7 @@ 
         // Clear the EM and prove that the embeddable can be the subject of a partial query.
         $this->_em->clear();
 
-        $dql = "SELECT PARTIAL p.{id,address.city} FROM " . __NAMESPACE__ ."\\DDC93Person p WHERE p.name = :name";
+        $dql = "SELECT PARTIAL p.{id,address.city} FROM ".__NAMESPACE__."\\DDC93Person p WHERE p.name = :name";
 
         $person = $this->_em->createQuery($dql)
             ->setParameter('name', 'Karl')
@@ -218,7 +218,7 @@ 
         // Clear the EM and prove that the embeddable can be the subject of a partial query regardless of attributes positions.
         $this->_em->clear();
 
-        $dql = "SELECT PARTIAL p.{address.city, id} FROM " . __NAMESPACE__ ."\\DDC93Person p WHERE p.name = :name";
+        $dql = "SELECT PARTIAL p.{address.city, id} FROM ".__NAMESPACE__."\\DDC93Person p WHERE p.name = :name";
 
         $person = $this->_em->createQuery($dql)
             ->setParameter('name', 'Karl')
@@ -237,7 +237,7 @@ 
         $this->expectException(QueryException::class);
         $this->expectExceptionMessage('no field or association named address.asdfasdf');
 
-        $this->_em->createQuery("SELECT p FROM " . __NAMESPACE__ . "\\DDC93Person p WHERE p.address.asdfasdf IS NULL")
+        $this->_em->createQuery("SELECT p FROM ".__NAMESPACE__."\\DDC93Person p WHERE p.address.asdfasdf IS NULL")
             ->execute();
     }
 
@@ -246,7 +246,7 @@ 
         $this->expectException(QueryException::class);
         $this->expectExceptionMessage("no mapped field named 'address.asdfasdf'");
 
-        $this->_em->createQuery("SELECT PARTIAL p.{id,address.asdfasdf} FROM " . __NAMESPACE__ . "\\DDC93Person p")
+        $this->_em->createQuery("SELECT PARTIAL p.{id,address.asdfasdf} FROM ".__NAMESPACE__."\\DDC93Person p")
             ->execute();
     }
 
@@ -308,15 +308,15 @@ 
         $this->expectException(MappingException::class);
         $this->expectExceptionMessage(
             sprintf(
-                'Infinite nesting detected for embedded property %s::nested. ' .
+                'Infinite nesting detected for embedded property %s::nested. '.
                 'You cannot embed an embeddable from the same type inside an embeddable.',
-                __NAMESPACE__ . '\\' . $declaredEmbeddableClassName
+                __NAMESPACE__.'\\'.$declaredEmbeddableClassName
             )
         );
 
         $this->_schemaTool->createSchema(
             [
-            $this->_em->getClassMetadata(__NAMESPACE__ . '\\' . $embeddableClassName),
+            $this->_em->getClassMetadata(__NAMESPACE__.'\\'.$embeddableClassName),
             ]
         );
     }

tests/Doctrine/Tests/ORM/Performance/SecondLevelCacheTest.php

Doc Comments +12 added lines

@@ -124,6 +124,9 @@ 
         $this->assertEquals(503, $this->countQuery($em));
     }
 
+    /**
+     * @param string $label
+     */
     private function queryEntity(EntityManagerInterface $em, $label)
     {
         $times        = 100;
@@ -155,6 +158,9 @@ 
         printf("\n%s\n", str_repeat('-', 50));
     }
 
+    /**
+     * @param string $label
+     */
     public function findEntityOneToMany(EntityManagerInterface $em, $label)
     {
         $times        = 50;
@@ -214,6 +220,9 @@ 
         printf("\n%s\n", str_repeat('-', 50));
     }
 
+    /**
+     * @param string $label
+     */
     private function findEntity(EntityManagerInterface $em, $label)
     {
         $times        = 10;
@@ -249,6 +258,9 @@ 
         printf("\n%s\n", str_repeat('-', 50));
     }
 
+    /**
+     * @param string $label
+     */
     private function findAllEntity(EntityManagerInterface $em, $label)
     {
         $times        = 100;

Spacing +8 added lines, -8 removed lines

@@ -130,7 +130,7 @@ 
         $size         = 500;
         $startPersist = microtime(true);
 
-        echo PHP_EOL . $label;
+        echo PHP_EOL.$label;
 
         for ($i = 0; $i < $size; $i++) {
             $em->persist(new Country("Country $i"));
@@ -164,7 +164,7 @@ 
         $startPersist = microtime(true);
         $country      = new Country("Country");
 
-        echo PHP_EOL . $label;
+        echo PHP_EOL.$label;
 
         $em->persist($country);
         $em->flush();
@@ -194,9 +194,9 @@ 
         $em->flush();
         $em->clear();
 
-        printf("\n[%s] persist %s states and %s cities", number_format( microtime(true) - $startPersist, 6), count($states), count($cities));
+        printf("\n[%s] persist %s states and %s cities", number_format(microtime(true) - $startPersist, 6), count($states), count($cities));
 
-        $startFind  = microtime(true);
+        $startFind = microtime(true);
 
         for ($i = 0; $i < $times; $i++) {
 
@@ -221,7 +221,7 @@ 
         $countries    = [];
         $startPersist = microtime(true);
 
-        echo PHP_EOL . $label;
+        echo PHP_EOL.$label;
 
         for ($i = 0; $i < $size; $i++) {
             $country = new Country("Country $i");
@@ -236,7 +236,7 @@ 
 
         printf("\n[%s] persist %s countries", number_format(microtime(true) - $startPersist, 6), $size);
 
-        $startFind  = microtime(true);
+        $startFind = microtime(true);
 
         for ($i = 0; $i <= $times; $i++) {
             foreach ($countries as $country) {
@@ -256,7 +256,7 @@ 
         $startPersist = microtime(true);
         $rep          = $em->getRepository(Country::class);
 
-        echo PHP_EOL . $label;
+        echo PHP_EOL.$label;
 
         for ($i = 0; $i < $size; $i++) {
             $em->persist(new Country("Country $i"));
@@ -267,7 +267,7 @@ 
 
         printf("\n[%s] persist %s countries", number_format(microtime(true) - $startPersist, 6), $size);
 
-        $startFind  = microtime(true);
+        $startFind = microtime(true);
 
         for ($i = 0; $i <= $times; $i++) {
             $list = $rep->findAll();

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

Doc Comments +4 added lines

@@ -43,6 +43,10 @@
         $this->_em = $this->_getTestEntityManager();
     }
 
+    /**
+     * @param string $dqlToBeTested
+     * @param string $sqlToBeConfirmed
+     */
     public function assertSqlGeneration($dqlToBeTested, $sqlToBeConfirmed)
     {
         try {