doctrine / orm

lib/Doctrine/ORM/Utility/PersisterHelper.php

Unused Use Statements -1 removed lines

@@ -29,7 +29,6 @@
 use Doctrine\ORM\Mapping\FieldMetadata;
 use Doctrine\ORM\Mapping\ManyToManyAssociationMetadata;
 use Doctrine\ORM\Mapping\ToOneAssociationMetadata;
-use Doctrine\ORM\Query\QueryException;
 
 /**
  * The PersisterHelper contains logic to infer binding types which is used in

Spacing +3 added lines, -3 removed lines

@@ -18,7 +18,7 @@ 
  * <http://www.doctrine-project.org>.
  */
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\ORM\Utility;
 
@@ -79,14 +79,14 @@ 
 
         // iterate over association mappings
         foreach ($class->getProperties() as $association) {
-            if (! ($association instanceof AssociationMetadata)) {
+            if ( ! ($association instanceof AssociationMetadata)) {
                 continue;
             }
 
             // resolve join columns over to-one or to-many
             $targetClass = $em->getClassMetadata($association->getTargetEntity());
 
-            if (! $association->isOwningSide()) {
+            if ( ! $association->isOwningSide()) {
                 $association = $targetClass->getProperty($association->getMappedBy());
                 $targetClass = $em->getClassMetadata($association->getTargetEntity());
             }

tests/Doctrine/Performance/Mock/NonProxyLoadingEntityManager.php

Unused Use Statements -1 removed lines

@@ -10,7 +10,6 @@
 use Doctrine\ORM\Proxy\Factory\DefaultProxyResolver;
 use Doctrine\ORM\Proxy\Factory\StaticProxyFactory;
 use Doctrine\ORM\Query\ResultSetMapping;
-use Doctrine\ORM\Utility\IdentifierFlattener;
 
 /**
  * An entity manager mock that prevents lazy-loading of proxies

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\Performance\Mock;
 

tests/Doctrine/Tests/Mocks/EntityPersisterMock.php

Unused Use Statements -2 removed lines

@@ -5,8 +5,6 @@
 namespace Doctrine\Tests\Mocks;
 
 use Doctrine\Common\Collections\Criteria;
-use Doctrine\ORM\Mapping\ClassMetadata;
-use Doctrine\ORM\Mapping\GeneratorType;
 use Doctrine\ORM\Persisters\Entity\BasicEntityPersister;
 
 /**

Spacing +1 added lines, -1 removed lines

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

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

Doc Comments +3 added lines, -3 removed lines

@@ -234,7 +234,7 @@ 
 
     /**
      *
-     * @return kateglo\application\helpers\collections\ArrayCollection
+     * @return ArrayCollection
      */
     public function getTypes()
     {
@@ -337,7 +337,7 @@ 
 
     /**
      *
-     * @param kateglo\application\models\Lemma $lemma
+     * @param Lemma $lemma
      * @return void
      */
     public function addLemma(Lemma $lemma)
@@ -350,7 +350,7 @@ 
 
     /**
      *
-     * @param kateglo\application\models\Lemma $lemma
+     * @param Lemma $lemma
      * @return void
      */
     public function removeLEmma(Lemma $lemma)

Spacing +4 added lines, -4 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\Tests\ORM\Functional;
 
@@ -206,7 +206,7 @@ 
      *
      * @return string
      */
-    public function getLemma(){
+    public function getLemma() {
         return $this->lemma;
     }
 
@@ -215,7 +215,7 @@ 
      */
     public function addType(Type $type)
     {
-        if (!$this->types->contains($type)) {
+        if ( ! $this->types->contains($type)) {
             $this->types[] = $type;
             $type->addLemma($this);
         }
@@ -342,7 +342,7 @@ 
      */
     public function addLemma(Lemma $lemma)
     {
-        if (!$this->lemmas->contains($lemma)) {
+        if ( ! $this->lemmas->contains($lemma)) {
             $this->lemmas[] = $lemma;
             $lemma->addType($this);
         }

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

Unused Use Statements -1 removed lines

@@ -6,7 +6,6 @@
 
 use Doctrine\DBAL\Logging\DebugStack;
 use Doctrine\ORM\EntityNotFoundException;
-use Doctrine\ORM\Mapping\ClassMetadata;
 use Doctrine\ORM\Mapping\FetchMode;
 use Doctrine\ORM\ORMInvalidArgumentException;
 use Doctrine\ORM\PersistentCollection;

Indentation +5 added lines, -5 removed lines

@@ -363,9 +363,9 @@ 
         self::assertEquals(2, count($user->phonenumbers));
         $dql = "SELECT u FROM Doctrine\Tests\Models\CMS\CmsUser u WHERE u.id = ?1";
         $user = $this->em->createQuery($dql)
-                          ->setParameter(1, $user->id)
-                          ->setHint(Query::HINT_REFRESH, true)
-                          ->getSingleResult();
+                            ->setParameter(1, $user->id)
+                            ->setHint(Query::HINT_REFRESH, true)
+                            ->getSingleResult();
 
         self::assertEquals(1, count($user->phonenumbers));
     }
@@ -400,8 +400,8 @@ 
 
         $dql = "SELECT u FROM Doctrine\Tests\Models\CMS\CmsUser u WHERE u.id = ?1";
         $user = $this->em->createQuery($dql)
-                          ->setParameter(1, $userId)
-                          ->getSingleResult();
+                            ->setParameter(1, $userId)
+                            ->getSingleResult();
 
         self::assertEquals(1, count($user->phonenumbers));
     }

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;
 
@@ -185,7 +185,7 @@ 
         $user->username = 'gblanco';
         $user->status = 'developer';
 
-        for ($i=0; $i<3; ++$i) {
+        for ($i = 0; $i < 3; ++$i) {
             $phone = new CmsPhonenumber;
             $phone->phonenumber = 100 + $i;
             $user->addPhonenumber($phone);
@@ -413,7 +413,7 @@ 
         $user->username = 'gblanco';
         $user->status = 'developer';
 
-        for ($i=0; $i<3; ++$i) {
+        for ($i = 0; $i < 3; ++$i) {
             $phone = new CmsPhonenumber;
             $phone->phonenumber = 100 + $i;
             $user->addPhonenumber($phone);
@@ -453,7 +453,7 @@ 
         $user->username = 'gblanco';
         $user->status = 'developer';
 
-        for ($i=0; $i<3; ++$i) {
+        for ($i = 0; $i < 3; ++$i) {
             $phone = new CmsPhonenumber;
             $phone->phonenumber = 100 + $i;
             $user->addPhonenumber($phone);
@@ -533,7 +533,7 @@ 
         $user->username = 'gblanco';
         $user->status = 'developer';
 
-        for ($i=0; $i<3; ++$i) {
+        for ($i = 0; $i < 3; ++$i) {
             $phone = new CmsPhonenumber;
             $phone->phonenumber = 100 + $i;
             $user->addPhonenumber($phone);
@@ -1003,7 +1003,7 @@ 
 
         self::assertInstanceOf(Proxy::class, $article->user, "It IS a proxy, ...");
         self::assertTrue($article->user->__isInitialized(), "...but its initialized!");
-        self::assertEquals($qc+2, $this->getCurrentQueryCount());
+        self::assertEquals($qc + 2, $this->getCurrentQueryCount());
     }
 
     /**
@@ -1064,7 +1064,7 @@ 
 
         $this->expectException(ORMInvalidArgumentException::class);
         $this->expectExceptionMessage(
-            'Expected value of type "Doctrine\Tests\Models\CMS\CmsAddress" for association field ' .
+            'Expected value of type "Doctrine\Tests\Models\CMS\CmsAddress" for association field '.
             '"Doctrine\Tests\Models\CMS\CmsUser#$address", got "Doctrine\Tests\Models\CMS\CmsUser" instead.'
         );
 

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

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 declare(strict_types=1);
 
 namespace Doctrine\Tests\ORM\Functional;
-use Doctrine\Common\Util\Debug;
 use Doctrine\ORM\ORMException;
 use Doctrine\ORM\Query\QueryException;
 use Doctrine\Tests\Models\Navigation\NavCountry;

Indentation +6 added lines, -6 removed lines

@@ -112,9 +112,9 @@ 
         $this->putTripAroundEurope();
 
         $dql = 'SELECT t, p, c '
-             . 'FROM Doctrine\Tests\Models\Navigation\NavTour t '
-             . 'INNER JOIN t.pois p '
-             . 'INNER JOIN p.country c'
+                . 'FROM Doctrine\Tests\Models\Navigation\NavTour t '
+                . 'INNER JOIN t.pois p '
+                . 'INNER JOIN p.country c'
         ;
 
         $tours = $this->em->createQuery($dql)->getResult();
@@ -134,9 +134,9 @@ 
         $this->putTripAroundEurope();
 
         $dql = 'SELECT t '
-             . 'FROM Doctrine\Tests\Models\Navigation\NavTour t '
-             . ', Doctrine\Tests\Models\Navigation\NavPointOfInterest p '
-             . 'WHERE p MEMBER OF t.pois'
+                . 'FROM Doctrine\Tests\Models\Navigation\NavTour t '
+                . ', Doctrine\Tests\Models\Navigation\NavPointOfInterest p '
+                . 'WHERE p MEMBER OF t.pois'
         ;
 
         $query = $this->em->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\Functional;
 use Doctrine\Common\Util\Debug;

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

Unused Use Statements -1 removed lines

@@ -6,7 +6,6 @@
 
 use Doctrine\DBAL\Exception\UniqueConstraintViolationException;
 use Doctrine\ORM\OptimisticLockException;
-use Doctrine\ORM\ORMException;
 use Doctrine\ORM\Proxy\Proxy;
 use Doctrine\Tests\Models\CMS\CmsUser;
 use Doctrine\Tests\Models\CMS\CmsPhonenumber;

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;
 
@@ -175,7 +175,7 @@ 
         $this->em->flush();
         $this->em->detach($user);
 
-        $dql = 'SELECT u FROM ' . CmsUser::class . ' u WHERE u.id = ?1';
+        $dql = 'SELECT u FROM '.CmsUser::class.' u WHERE u.id = ?1';
         $query = $this->em->createQuery($dql);
 
         $query->setParameter(1, $user);
@@ -219,7 +219,7 @@ 
 
         $this->em->detach($article);
 
-        $sql = 'UPDATE cms_articles SET version = version + 1 WHERE id = ' . $article->id;
+        $sql = 'UPDATE cms_articles SET version = version + 1 WHERE id = '.$article->id;
         $this->em->getConnection()->executeUpdate($sql);
 
         $this->expectException(OptimisticLockException::class);

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

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 namespace Doctrine\Tests\ORM\Functional;
 
-use Doctrine\ORM\Mapping\ClassMetadata;
 use Doctrine\ORM\Mapping\FetchMode;
 use Doctrine\Tests\Models\CMS\CmsArticle;
 use Doctrine\Tests\Models\CMS\CmsGroup;

Spacing +5 added lines, -5 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\Tests\ORM\Functional;
 
@@ -43,7 +43,7 @@ 
         $this->useModelSet('ddc2504');
         parent::setUp();
 
-        $class       = $this->em->getClassMetadata(CmsUser::class);
+        $class = $this->em->getClassMetadata(CmsUser::class);
 
         $class->getProperty('groups')->setFetchMode(FetchMode::EXTRA_LAZY);
         $class->getProperty('articles')->setFetchMode(FetchMode::EXTRA_LAZY);
@@ -304,7 +304,7 @@ 
 
         $queryCount = $this->getCurrentQueryCount();
         self::assertFalse($user->articles->contains($article));
-        self::assertEquals($queryCount+1, $this->getCurrentQueryCount(), "Checking for contains of persisted entity should cause one query to be executed.");
+        self::assertEquals($queryCount + 1, $this->getCurrentQueryCount(), "Checking for contains of persisted entity should cause one query to be executed.");
         self::assertFalse($user->articles->isInitialized(), "Post-Condition: Collection is not initialized.");
 
         // Test One to Many existence with state managed
@@ -471,7 +471,7 @@ 
 
         $queryCount = $this->getCurrentQueryCount();
         self::assertTrue($group->users->contains($user));
-        self::assertEquals($queryCount+1, $this->getCurrentQueryCount(), "Checking for contains of managed entity should cause one query to be executed.");
+        self::assertEquals($queryCount + 1, $this->getCurrentQueryCount(), "Checking for contains of managed entity should cause one query to be executed.");
         self::assertFalse($user->groups->isInitialized(), "Post-Condition: Collection is not initialized.");
 
         $newUser = new CmsUser();
@@ -1251,7 +1251,7 @@ 
         $user     = new User();
         $userList = new UserList();
 
-        $user->name     = 'ocramius';
+        $user->name = 'ocramius';
         $userList->listName = 'PHP Developers to follow closely';
 
         $user->addUserList($userList);

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

Unused Use Statements -2 removed lines

@@ -6,8 +6,6 @@
 
 use Doctrine\ORM\Mapping\FetchMode;
 use Doctrine\Tests\Models\ECommerce\ECommerceProduct;
-use Doctrine\ORM\Mapping\AssociationMapping;
-use Doctrine\ORM\Mapping\ClassMetadata;
 
 /**
  * Tests a self referential many-to-many association mapping (from a model to the same model, without inheritance).

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;